Merge pull request #276 from AsherGlick/fixing_test_text

Fixing some minor issues in the integration tests harness
AsherGlick · Feb 28, 2024 · c9965fa · c9965fa
2 parents 7d7682f + 1869833
commit c9965fa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/xml_converter/integration_tests/run_tests.py b/xml_converter/integration_tests/run_tests.py
@@ -263,7 +263,7 @@ def diff_dirs(actual_output_dir: str, expected_output_dir: str) -> bool:
 
         if len_diff(diff) != 0:
             diff_found = True
-            print("XML output was incorrect for test")
+            print("Output was incorrect for test")
             for line in diff:
                 print(line)
 

diff --git a/xml_converter/integration_tests/src/proto_utils.py b/xml_converter/integration_tests/src/proto_utils.py
@@ -16,7 +16,7 @@ def compare_protos(
     expected_textproto = get_waypoint_textproto(expected_proto_path)
     actual_textproto = get_waypoint_textproto(actual_proto_path)
 
-    diff = list(difflib.unified_diff(expected_textproto.split("\n"), actual_textproto.split("\n"), fromfile=expected_proto_path, tofile=actual_proto_path, lineterm=""))
+    diff = list(difflib.unified_diff(actual_textproto.split("\n"), expected_textproto.split("\n"), fromfile=actual_proto_path, tofile=expected_proto_path, lineterm=""))
 
     if len(diff) == 0:
         diff = ["Something went wrong diffing {} and {}.".format(expected_proto_path, actual_proto_path)]