gpaddmirrors: fix unit tests
The last commit removed the replication ports (replacing them with -1 in
the Python utilities), and those numbers were being checked as part of
this test. Comment the checks out and tag with FIXMEs.
Author: Asim R P
Author: Jacob Champion
@@ -104,8 +104,10 @@ class GpAddMirrorsTest(GpTestCase):
result = fp.readlines()
self.assertIn("41000", result[1])
- self.assertIn("42000", result[1])
- self.assertIn("43000", result[1])
+ # GPDB_SEGWALREP_FIXME: we have removed the replication port; what other
+ # fallout is there from that decision?
+ #self.assertIn("42000", result[1])
+ #self.assertIn("43000", result[1])
def test_generated_file_contains_port_offsets(self):
datadir_config = _write_datadir_config(self.mdd)
@@ -120,8 +122,10 @@ class GpAddMirrorsTest(GpTestCase):
result = fp.readlines()
self.assertIn("45000", result[1])
- self.assertIn("50000", result[1])
- self.assertIn("55000", result[1])
+ # GPDB_SEGWALREP_FIXME: we have removed the replication port; what other
+ # fallout is there from that decision?
+ #self.assertIn("50000", result[1])
+ #self.assertIn("55000", result[1])
def _createGpArrayWith2Primary2Mirrors(self):
self.master = Segment.initFromString(
