beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dhalp...@apache.org
Subject [1/2] beam git commit: Fix test_pipeline_test That Broke PostCommit
Date Fri, 13 Jan 2017 07:40:38 GMT
Repository: beam
Updated Branches:
  refs/heads/python-sdk e23c3cab3 -> 8c2251305


Fix test_pipeline_test That Broke PostCommit


Project: http://git-wip-us.apache.org/repos/asf/beam/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/59828df7
Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/59828df7
Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/59828df7

Branch: refs/heads/python-sdk
Commit: 59828df761bb25d67b69cd1615500c19382ab070
Parents: e23c3ca
Author: Mark Liu <markliu@google.com>
Authored: Thu Jan 12 16:21:39 2017 -0800
Committer: Mark Liu <markliu@google.com>
Committed: Thu Jan 12 16:21:39 2017 -0800

----------------------------------------------------------------------
 sdks/python/apache_beam/test_pipeline_test.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/beam/blob/59828df7/sdks/python/apache_beam/test_pipeline_test.py
----------------------------------------------------------------------
diff --git a/sdks/python/apache_beam/test_pipeline_test.py b/sdks/python/apache_beam/test_pipeline_test.py
index 526a8a2..c6bdba3 100644
--- a/sdks/python/apache_beam/test_pipeline_test.py
+++ b/sdks/python/apache_beam/test_pipeline_test.py
@@ -55,8 +55,8 @@ class TestPipelineTest(unittest.TestCase):
   def test_option_args_parsing(self):
     test_pipeline = TestPipeline(argv=self.TEST_CASE['options'])
     self.assertListEqual(
-        test_pipeline.get_full_options_as_args(),
-        self.TEST_CASE['expected_list'])
+        sorted(test_pipeline.get_full_options_as_args()),
+        sorted(self.TEST_CASE['expected_list']))
 
   def test_empty_option_args_parsing(self):
     test_pipeline = TestPipeline()
@@ -84,7 +84,7 @@ class TestPipelineTest(unittest.TestCase):
     test_pipeline = TestPipeline()
     for case in self.EXTRA_OPT_CASES:
       opt_list = test_pipeline.get_full_options_as_args(**case['options'])
-      self.assertListEqual(opt_list, case['expected'])
+      self.assertListEqual(sorted(opt_list), sorted(case['expected']))
 
   def test_append_verifier_in_extra_opt(self):
     extra_opt = {'matcher': SimpleMatcher()}


Mime
View raw message