diff --git a/pipeline/fiftyone_pipeline_core/src/fiftyone_pipeline_core/jsonbundler.py b/pipeline/fiftyone_pipeline_core/src/fiftyone_pipeline_core/jsonbundler.py index 030cba4a..6ddf50a3 100644 --- a/pipeline/fiftyone_pipeline_core/src/fiftyone_pipeline_core/jsonbundler.py +++ b/pipeline/fiftyone_pipeline_core/src/fiftyone_pipeline_core/jsonbundler.py @@ -108,9 +108,8 @@ def process_internal(self, flowdata): sequence = flowdata.evidence.get("query.sequence") if isinstance(sequence, bool) or not isinstance(sequence, int): raise Exception( - "The JSON Builder needs the sequence number that a Sequence " - "Element produces. A Sequence Element needs to be placed " - "before it in the Pipeline.") + "The JSON Builder needs a sequence number. Place a Sequence " + "Element before the JSON Builder in the Pipeline.") # Get every property on every FlowElement # Storing JavaScript properties in an extra section diff --git a/pipeline/fiftyone_pipeline_core/tests/test_jsonbundler_sequence_required.py b/pipeline/fiftyone_pipeline_core/tests/test_jsonbundler_sequence_required.py index 8cc8058b..45100d5f 100644 --- a/pipeline/fiftyone_pipeline_core/tests/test_jsonbundler_sequence_required.py +++ b/pipeline/fiftyone_pipeline_core/tests/test_jsonbundler_sequence_required.py @@ -50,7 +50,10 @@ def test_no_sequence_number_is_an_error_that_names_the_sequence_element( flowdata = pipeline.create_flowdata() with self.assertRaises(Exception) as caught: flowdata.process() - self.assertIn("Sequence Element", str(caught.exception)) + self.assertEqual( + "The JSON Builder needs a sequence number. Place a Sequence " + "Element before the JSON Builder in the Pipeline.", + str(caught.exception)) def test_no_json_is_set_when_the_error_is_suppressed(self): pipeline = bundler_only(suppress=True)