提交 5cc2ddd0 编写于 作者: A Asim R P 提交者: Xin Zhang

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 <apraveen@pivotal.io>
Author: Jacob Champion <pchampion@pivotal.io>
上级 cadf63a8
......@@ -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(
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册