Change orchestrator message

Author: lucia-sb

ddev/src/ddev/cli/validate/all/orchestrator.py

@@ -42,7 +42,7 @@ class ValidationConfig:
         description="Verify check versions match the Agent requirements file",
     ),
     "ci": ValidationConfig(
-        description="Validate CI configuration and Codecov settings",
+        description="Validate CI configuration and code coverage settings",
         repo_wide=True,
         fix_flag="--sync",
     ),

ddev/src/ddev/cli/validate/ci.py

@@ -248,7 +248,9 @@ def _validate_code_coverage(
     config_path = app.repo.path / config_filename
 
     if not config_path.is_file():
-        validation_tracker.error((repo_choice,), message=f'Unable to find the code coverage config file: {config_filename}')
+        validation_tracker.error(
+            (repo_choice,), message=f'Unable to find the code coverage config file: {config_filename}'
+        )
         validation_tracker.display()
         app.abort()
 
@@ -299,10 +301,12 @@ def _validate_code_coverage(
         if sync:
             fixed = True
             for check_name in missing_services:
-                existing_services.append({
-                    'id': check_name,
-                    'paths': get_coverage_sources(check_name, app),
-                })
+                existing_services.append(
+                    {
+                        'id': check_name,
+                        'paths': get_coverage_sources(check_name, app),
+                    }
+                )
                 app.display_success(f'Added service `{check_name}`\n')
         else:
             success = False
@@ -319,10 +323,12 @@ def _validate_code_coverage(
         # Ensure at least one gate exists
         gates = config.get('gates') or []
         if not gates:
-            gates.append({
-                'type': 'total_coverage_percentage',
-                'config': {'threshold': DEFAULT_COVERAGE_THRESHOLD},
-            })
+            gates.append(
+                {
+                    'type': 'total_coverage_percentage',
+                    'config': {'threshold': DEFAULT_COVERAGE_THRESHOLD},
+                }
+            )
             config['gates'] = gates
             app.display_success(f'Added default coverage gate with {DEFAULT_COVERAGE_THRESHOLD}% threshold\n')
 

ddev/tests/cli/validate/all/test_github.py

@@ -17,7 +17,7 @@
 from ddev.cli.validate.all.orchestrator import ValidationConfig, ValidationResult
 
 CONFIGS = {
-    "ci": ValidationConfig(description="Validate CI configuration and Codecov settings", repo_wide=True),
+    "ci": ValidationConfig(description="Validate CI configuration and code coverage settings", repo_wide=True),
     "config": ValidationConfig(description="Validate default configuration files against spec.yaml"),
     "metadata": ValidationConfig(description="Validate metadata.csv metric definitions"),
 }
@@ -134,7 +134,7 @@ def test_format_pr_comment_all_passed(helpers):
 
         | Validation | Description | Status |
         |---|---|---|
-        | `ci` | Validate CI configuration and Codecov settings | ✅ |
+        | `ci` | Validate CI configuration and code coverage settings | ✅ |
         | `config` | Validate default configuration files against spec.yaml | ✅ |
 
         </details>""")
@@ -160,7 +160,7 @@ def test_format_pr_comment_one_failure_with_target(helpers):
 
         | Validation | Description | Status |
         |---|---|---|
-        | `ci` | Validate CI configuration and Codecov settings | ✅ |
+        | `ci` | Validate CI configuration and code coverage settings | ✅ |
 
         </details>""")
     assert format_pr_comment(results, CONFIGS, "changed", list(results)) == expected
@@ -266,7 +266,7 @@ def test_format_step_summary_all_passed(helpers):
 
         | Validation | Description | Status |
         |---|---|---|
-        | `ci` | Validate CI configuration and Codecov settings | ✅ |
+        | `ci` | Validate CI configuration and code coverage settings | ✅ |
         | `config` | Validate default configuration files against spec.yaml | ✅ |""")
     assert format_step_summary(results, CONFIGS, "changed", list(results)) == expected
 
@@ -281,7 +281,7 @@ def test_format_step_summary_with_failures(helpers):
 
         | Validation | Description | Status |
         |---|---|---|
-        | `ci` | Validate CI configuration and Codecov settings | ✅ |
+        | `ci` | Validate CI configuration and code coverage settings | ✅ |
         | `config` | Validate default configuration files against spec.yaml | ❌ |
 
         Run `ddev validate all changed --fix` to attempt to auto-fix supported validations.""")