From 7ddcf420be7bf8caa9616a1932513e91c9dbd984 Mon Sep 17 00:00:00 2001 From: Andrew Montanez Date: Fri, 2 Aug 2024 14:22:15 -0500 Subject: [PATCH] Improving test descriptions --- tests/unit/multi_table/test_base.py | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/tests/unit/multi_table/test_base.py b/tests/unit/multi_table/test_base.py index e7ad74e4e..c1fb04aac 100644 --- a/tests/unit/multi_table/test_base.py +++ b/tests/unit/multi_table/test_base.py @@ -910,7 +910,12 @@ def test_preprocess_warning(self, mock_warnings): ) def test_preprocess_single_table_preprocess_raises_error_0_int_regex(self): - """Test that if the single table synthesizer raises a specific error, it is reformatted.""" + """Test that if the single table synthesizer raises a specific error, it is reformatted. + + If a single table synthesizer raises an error about the primary key being an integer + with a regex that can start with zero, the error should be reformatted to include the + table name. + """ # Setup metadata = get_multi_table_metadata() instance = BaseMultiTableSynthesizer(metadata) @@ -945,7 +950,11 @@ def test_preprocess_single_table_preprocess_raises_error_0_int_regex(self): instance.preprocess(data) def test_preprocess_single_table_preprocess_raises_error(self): - """Test that if the single table synthesizer raises any other error, it is raised.""" + """Test that if the single table synthesizer raises any other error, it is raised. + + If a single table synthesizer raises an error besides the one concerning int primary keys + starting with 0 and having a regex, then the error should be raised as is. + """ # Setup metadata = get_multi_table_metadata() instance = BaseMultiTableSynthesizer(metadata)