Fix: format the file my last edit left unformatted; align the two black scopes

The AWS cleanup test assertions were widened by hand and not re-run
through black. Quick Checks caught it.

fast_ci checked black on clustrix/ tests/ while tests.yml checked
clustrix/ tests/ scripts/, so a file under scripts/ could pass one gate
and fail the other. Same scope now.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012gTBDPK16HUZ3kHQ2QyjuU

Author: jeremymanning

.github/workflows/fast_ci.yml

@@ -47,7 +47,9 @@ jobs:
           pip install -e ".[dev]"
       
       - name: Format check with Black
-        run: black --check clustrix/ tests/
+        # Same scope as tests.yml's black step. They covered different
+        # directories, so a file could pass one gate and fail the other.
+        run: black --check clustrix/ tests/ scripts/
 
       - name: Lint with Flake8
         run: flake8 clustrix/ --count --select=E9,F63,F7,F82 --show-source --statistics

tests/unit/test_aws_cleanup_scripts.py

@@ -135,7 +135,9 @@ def test_cleanup_dry_run_without_credentials_errors_clearly(self):
         # AWS SDK, so the SDK message is the one it hits; a developer machine
         # with boto3 installed hits the credential one.
         assert "credential" in output or "aws sdk" in output, output
-        assert "pip install boto3" in output or "aws_access_key_id" in output.lower(), output
+        assert (
+            "pip install boto3" in output or "aws_access_key_id" in output.lower()
+        ), output
 
     def test_cleanup_execute_without_credentials_also_errors_clearly(self):
         # --execute must not bypass the credential check either.
@@ -167,7 +169,9 @@ def test_cleanup_execute_without_credentials_also_errors_clearly(self):
         # AWS SDK, so the SDK message is the one it hits; a developer machine
         # with boto3 installed hits the credential one.
         assert "credential" in output or "aws sdk" in output, output
-        assert "pip install boto3" in output or "aws_access_key_id" in output.lower(), output
+        assert (
+            "pip install boto3" in output or "aws_access_key_id" in output.lower()
+        ), output
 
     def test_destroy_dry_run_without_credentials_errors_clearly(self):
         env, tmp_home = _clean_env_without_aws_credentials()
@@ -201,7 +205,9 @@ def test_destroy_dry_run_without_credentials_errors_clearly(self):
         # AWS SDK, so the SDK message is the one it hits; a developer machine
         # with boto3 installed hits the credential one.
         assert "credential" in output or "aws sdk" in output, output
-        assert "pip install boto3" in output or "aws_access_key_id" in output.lower(), output
+        assert (
+            "pip install boto3" in output or "aws_access_key_id" in output.lower()
+        ), output
 
     def test_destroy_execute_without_credentials_also_errors_clearly(self):
         env, tmp_home = _clean_env_without_aws_credentials()
@@ -233,7 +239,9 @@ def test_destroy_execute_without_credentials_also_errors_clearly(self):
         # AWS SDK, so the SDK message is the one it hits; a developer machine
         # with boto3 installed hits the credential one.
         assert "credential" in output or "aws sdk" in output, output
-        assert "pip install boto3" in output or "aws_access_key_id" in output.lower(), output
+        assert (
+            "pip install boto3" in output or "aws_access_key_id" in output.lower()
+        ), output
 
 
 class TestArgParsingRoundTrip: