search

Trax-air / swagger-tester

Automatically test your swagger API
MIT License
104 stars 36 forks source link

Fixes #51 #52

Closed ThomasHickman closed 6 years ago

ThomasHickman commented 6 years ago

Fixes #51, by only removing basePath at the end of urls

flavianh commented 6 years ago

@ThomasHickman thanks for your contribution! Could you write a test for this? One that fails with the previous behavior but which your PR allows to pass

cyprieng commented 6 years ago

Thanks for your contribution! Could you fix the flake 8 issue /tests/test_swagger-tester.py:72:1: E302 expected 2 blank lines, found 1 so we can merge ?

cyprieng commented 6 years ago

@flavianh any clue on the build error ERROR: InterpreterNotFound: pypy ?

flavianh commented 6 years ago

@cyprieng Maybe check this: https://github.com/travis-ci/travis-ci/issues/6865 Setting pypy-5.4.1 in the travis.yml file could fix the issue: https://github.com/Yelp/venv-update/blob/19c18924afb55d23333150bf3bdb66da995b046f/.travis.yml#L12

cyprieng commented 6 years ago

@flavianh thanks I have fixed the build on master :+1: @ThomasHickman Can you update your branch with master (to check that the test are green) ?