TST: Don't hard code server port in tests #1690
Conversation
QuLogic
force-pushed
the
no-hardcode-ports
branch
from
2d070ab
to
c31af10
Compare
fsspec/tests/conftest.py
Outdated
| if "give_path" in self.headers: | ||
| return self._respond(200, data=json.dumps({"path": self.path}).encode()) | ||
| if "redirect" in self.headers and file_path != "/index/realfile": | ||
| new_url = f"http://127.0.0.1:{port}/index/realfile" | ||
| new_url = f"http://127.0.0.1:{self.server.server_port}/index/realfile" |
There was a problem hiding this comment.
self.server is the HTTPServer, not the server fixture, and doesn't have a realfile attribute. But this could probably use _make_realfile instead.
fsspec/tests/conftest.py
Outdated
| httpd = HTTPServer(server_address, HTTPTestHandler) | ||
| th = threading.Thread(target=httpd.serve_forever) | ||
| th.daemon = True | ||
| th.start() | ||
| try: | ||
| yield f"http://127.0.0.1:{port}" | ||
| yield f"http://127.0.0.1:{httpd.server_port}" |
There was a problem hiding this comment.
Since we are choosing to become port agnostic, this may be a good time to also become server address agnostic. httpd.server_name should be populated here, and might be other than the standard loopback, e.g., for systems that are IPv6 only.
QuLogic
force-pushed
the
no-hardcode-ports
branch
3 times, most recently
from
768c381
to
41053a2
Compare
|
Test failures with a bunch of "Cannot connect to host 0.0.0.0:50610" look real - but only on windows. |
|
Unfortunately, it looks like |
This allows running tests in parallel without running into "Port already in use" errors.
QuLogic
force-pushed
the
no-hardcode-ports
branch
from
41053a2
to
b52d782
Compare
|
Well done :) |
This allows running tests in parallel without running into "Port already in use" errors.