search

cdent / gabbi

Declarative HTTP Testing for Python and anything else
http://gabbi.readthedocs.org/
Other
148 stars 34 forks source link

gabbi-run: Fix usage of STDIN #193

Closed sileht closed 7 years ago

sileht commented 7 years ago

Recent change have break gabbi-run with STDIN. "verbosity" is passed the the "datadir" instead of "."

This fixes that.

Also, I don't get why the test suite doesn't fail.

I have discover the issue here: http://logs.openstack.org/09/405109/1/check/gate-telemetry-dsvm-integration-ceilometer-ubuntu-xenial/b3544ba/console.html#_2016-12-01_06_44_54_763450

cdent commented 7 years ago

Also, I don't get why the test suite doesn't fail.

Yeah, that's weird. I'll merge this, release a new version, and then see about making a test that gets things more proper.

Thanks for discovering and fixing this.

cdent commented 7 years ago

Ah, looks like there are no stdin-based tests that do data loading, that should be easy to fix.