Skip to content

Commit 9d4f08e

Browse files
markmcdJon Wayne Parrott
authored and
Jon Wayne Parrott
committed
Uses a test port that is less likely to be taken (#12)
After running this in a different CI environment, I had issues with the test being unable to open port 8080 as it was already in use. This doesn't fix the problem perfectly but is less likely to encounter a collision.
1 parent a320e58 commit 9d4f08e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

tests/test_flow.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
# limitations under the License.
1414

1515
import concurrent.futures
16+
from functools import partial
1617
import json
1718
import os
1819

@@ -165,11 +166,12 @@ def test_run_console(self, input_mock, instance, mock_fetch_token):
165166
def test_run_local_server(
166167
self, webbrowser_mock, instance, mock_fetch_token):
167168
auth_redirect_url = urllib.parse.urljoin(
168-
'http://localhost:8080',
169+
'http://localhost:60452',
169170
self.REDIRECT_REQUEST_PATH)
170171

171172
with concurrent.futures.ThreadPoolExecutor(max_workers=1) as pool:
172-
future = pool.submit(instance.run_local_server)
173+
future = pool.submit(partial(
174+
instance.run_local_server, port=60452))
173175

174176
while not future.done():
175177
try:

0 commit comments

Comments
 (0)