Include query parameters and headers in the generated Postman collection.

[MrMage]

Fixes #485.

[shalvah]

Thanks. Please update the tests.

[codecov-io]

Codecov Report

Merging #537 into master will increase coverage by 0.16%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #537      +/-   ##
============================================
+ Coverage     90.72%   90.88%   +0.16%     
- Complexity      289      290       +1     
============================================
  Files            14       14              
  Lines           679      691      +12     
============================================
+ Hits            616      628      +12     
  Misses           63       63

Impacted Files	Coverage Δ	Complexity Δ
src/Postman/CollectionWriter.php	93.18% <100%> (+2.55%)	19 <2> (+1)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d07f68a...f8430b6. Read the comment docs.

[MrMage]

@shalvah sorry, I had missed the tests. I have now:

• fixed the existing tests
• added new tests for query params and custom headers (added features in this test)
• added a test for body params (previously implemented but not tested)
• improved the formatting of some of the collection fixture JSON files, for easier updating of the test fixtures in the future
• improved diff output for failing tests by decoding the JSON fixtures as arrays instead of objects (this gives us more detailed info on exactly which properties are different).

[shalvah]

Nice work. A few minor changes to go...

[MrMage]

@shalvah applied your changes, please take another look.

I'd suggest using GitHub's "Squash and Merge" function when merging to reduce the number of commits in the main repo's history.

[shalvah]

Nice work!