Unify ApyProxy clients - Githubissues

Kuadrant / testsuite

3 stars 12 forks source link

Unify ApyProxy clients #483

Closed pehala closed 1 month ago

pehala commented 1 month ago

Client is required now
- Fixture will take care of client creation and cleaning up and initial setup (auth + verify)
Remove verify=false from Mockserver, we use HTTP anyways
Small updates to fixtures that use that
Tracing will get separate PR later with bigger changes

trepel commented 1 month ago

This does not work for me:

../../../.cache/pypoetry/virtualenvs/kuadrant-testsuite-npzTsQIr-py3.12/lib/python3.12/site-packages/apyproxy/__init__.py:78: in _
    url = os.path.join(self.__url.rstrip("/"), relpath)
E   AttributeError: 'URL' object has no attribute 'rstrip'

pehala commented 1 month ago

Thanks @trepel, great find, fixed.