refactor: rename "Grouping" to "Group"

Author: georgeRobertson

src/dve/reporting/excel_report.py

@@ -445,7 +445,7 @@ def _text_length(value):
     def _format_headings(headings: list[str]) -> list[str]:
         # TODO - ideally this would be config driven to allow customisation.
         _renames = {
-            "Table": "Grouping",
+            "Table": "Group",
             "Data Item": "Data Item Submission Name",
             "Error": "Errors and Warnings",
         }

tests/test_error_reporting/test_excel_report.py

@@ -116,7 +116,7 @@ def test_excel_report(report_dfs):
     column_headings = [cell.value for cell in aggs["1"]]
     assert column_headings == [
         "Type",
-        "Grouping",
+        "Group",
         "Data Item Submission Name",
         "Category",
         "Error Code",
@@ -126,7 +126,7 @@ def test_excel_report(report_dfs):
     details = workbook["Error Data"]
     column_headings = [cell.value for cell in details["1"]]
     assert column_headings == [
-        "Grouping",
+        "Group",
         "Type",
         "Error Code",
         "Data Item Submission Name",

tests/test_pipeline/test_spark_pipeline.py

@@ -469,7 +469,7 @@ def test_error_report_where_report_is_expected(  # pylint: disable=redefined-out
             OrderedDict(
                 **{
                     "Type": "Submission Failure",
-                    "Grouping": "planets",
+                    "Group": "planets",
                     "Data Item Submission Name": "orbitalPeriod",
                     "Category": "Bad value",
                     "Error Code": "LONG_ORBIT",
@@ -479,7 +479,7 @@ def test_error_report_where_report_is_expected(  # pylint: disable=redefined-out
             OrderedDict(
                 **{
                     "Type": "Submission Failure",
-                    "Grouping": "planets",
+                    "Group": "planets",
                     "Data Item Submission Name": "gravity",
                     "Category": "Bad value",
                     "Error Code": "STRONG_GRAVITY",
@@ -497,7 +497,7 @@ def test_error_report_where_report_is_expected(  # pylint: disable=redefined-out
     assert error_data_records == [
         OrderedDict(
             **{
-                "Grouping": "planets",
+                "Group": "planets",
                 "Type": "Submission Failure",
                 "Error Code": "LONG_ORBIT",
                 "Data Item Submission Name": "orbitalPeriod",
@@ -509,7 +509,7 @@ def test_error_report_where_report_is_expected(  # pylint: disable=redefined-out
         ),
         OrderedDict(
             **{
-                "Grouping": "planets",
+                "Group": "planets",
                 "Type": "Submission Failure",
                 "Error Code": "STRONG_GRAVITY",
                 "Data Item Submission Name": "gravity",