Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(114)

Unified Diff: scripts/tools/unittests/mastermap_test.py

Issue 1688043003: Move master_port_alts from 40000 to 25000 so they don't overlap with ephemeral ports (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Fix mastermap.py Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/tools/unittests/mastermap_test.py
diff --git a/scripts/tools/unittests/mastermap_test.py b/scripts/tools/unittests/mastermap_test.py
index 38da7250250b9c8d64ea9e26707da4f0e31f28b2..ed58d9d442785668df3983a15d4a6b07c29923e3 100755
--- a/scripts/tools/unittests/mastermap_test.py
+++ b/scripts/tools/unittests/mastermap_test.py
@@ -105,7 +105,7 @@ class FindPortTest(unittest.TestCase):
self.assertEquals(output.data, [ {
u'master_base_class': u'Master1',
u'master_port': u'20105',
- u'master_port_alt': u'40105',
+ u'master_port_alt': u'25105',
u'slave_port': u'30105',
} ])
@@ -115,7 +115,7 @@ class FindPortTest(unittest.TestCase):
output = FakeOutput()
_real_blacklist = mastermap.PORT_BLACKLIST
try:
- mastermap.PORT_BLACKLIST = set(xrange(40000, 50000)) # All alt_ports
+ mastermap.PORT_BLACKLIST = set(xrange(25000, 30000)) # All alt_ports
self.assertRaises(RuntimeError, mastermap.find_port,
master_class_name, masters, output, FakeOpts())
finally:

Powered by Google App Engine
This is Rietveld 408576698