collective / sphinxcontrib-httpexample

Adds example directive for sphinx-contrib httpdomain
23 stars 20 forks source link

Request builder creates platform dependent results #39

Open datakurre opened 5 years ago

datakurre commented 5 years ago

FYI, I couldn't get the snapshot tests to work on my mac. It looks like the output of this line may be slightly platform dependent?

I get almost the same results, but with some extra whitespace, as well as extra commas in data structures (e.g. {'k1': 'v1', 'k2': 'v2', } vs {'k1': 'v1', 'k2': 'v2'} in your snapshots).

To be clear, the resulting commands are equivalent, but the string representations are different, causing a bunch of failures.

Conversely, I had to manually edit the snapshot file instead of generating it automatically.

Originally posted by @ludaavics in https://github.com/collective/sphinxcontrib-httpexample/pull/38#issuecomment-428000011