Merge pull request #145 from Sage-Bionetworks/etl-487

[ETL-487] Add new expected error for motion.json
Sage-Bionetworks · Jun 30, 2023 · 8056d75 · 8056d75
2 parents bc4543e + 8eadee1
commit 8056d75
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/src/glue/jobs/s3_to_json_s3.py b/src/glue/jobs/s3_to_json_s3.py
@@ -478,6 +478,7 @@ def remove_expected_validation_errors(validation_result, client_info):
                 "'rotationRate', 'userAcceleration']"
             ),
             "'stepPath' is a required property",
+            "None is not of type 'integer'"
         ]
         unexpected_motion_errors = [
             e for e in motion_errors if e not in allowed_motion_errors

diff --git a/tests/test_s3_to_json_s3.py b/tests/test_s3_to_json_s3.py
@@ -691,6 +691,7 @@ def test_remove_expected_validation_error(self):
                     "'rotationRate', 'userAcceleration']"
                 ),
                 "'stepPath' is a required property",
+                "None is not of type 'integer'"
             ]
         }
         unexpected_validation_errors = s3_to_json_s3.remove_expected_validation_errors(