chore(golden): apply black --target-version py310 formatting

CI's black-check runs against Python 3.10 and rejected the
formatting written by black running under Python 3.13 (the venv's
interpreter), which preferred longer single-line function signatures
that 3.10's parser couldn't accept. Re-running black with an explicit
--target-version py310 collapses the affected lines back to the
multi-line shape CI accepts.

Pure formatting; no behavior change. 37/37 tests still pass.

Author: bnusunny

tests/golden/check_semver_bump.py

@@ -98,9 +98,7 @@ def check(
 
 def _read_version_at_ref(ref: str) -> str:
     """Read __version__ from samcli/__init__.py at a git ref."""
-    out = subprocess.check_output(
-        ["git", "show", f"{ref}:samcli/__init__.py"], text=True
-    )
+    out = subprocess.check_output(["git", "show", f"{ref}:samcli/__init__.py"], text=True)
     m = re.search(r'__version__\s*=\s*"([^"]+)"', out)
     if not m:
         raise RuntimeError(f"cannot find __version__ at {ref}")
@@ -109,9 +107,7 @@ def _read_version_at_ref(ref: str) -> str:
 
 def _git_changed_files(base: str, head: str) -> List[Change]:
     """Return all files changed between base and head with their git status."""
-    out = subprocess.check_output(
-        ["git", "diff", "--name-status", f"{base}...{head}"], text=True
-    )
+    out = subprocess.check_output(["git", "diff", "--name-status", f"{base}...{head}"], text=True)
     changes: List[Change] = []
     for line in out.splitlines():
         if not line.strip():

tests/golden/harness.py

@@ -284,9 +284,7 @@ def run_build_pipeline(template_path: Path, language_extensions: bool) -> Dict[s
 
     parameter_values = dict(IntrinsicsSymbolTable.DEFAULT_PSEUDO_PARAM_VALUES)
 
-    le_result = expand_language_extensions(
-        template, parameter_values=parameter_values, enabled=language_extensions
-    )
+    le_result = expand_language_extensions(template, parameter_values=parameter_values, enabled=language_extensions)
     expanded = copy.deepcopy(le_result.expanded_template)
 
     transformed = _run_sam_transform(expanded, parameter_values)
@@ -376,15 +374,11 @@ def _rewrite_artifacts_to_s3(template: Dict[str, Any], uploader, template_dir: s
         if not isinstance(rtype, str):
             # _walk_artifact_properties already filters, but re-narrow for mypy.
             continue
-        replacement = _packageable_replacement(
-            rtype, prop_path, current, template_dir, resource_id
-        )
+        replacement = _packageable_replacement(rtype, prop_path, current, template_dir, resource_id)
         _set_at_path(resource, prop_path, replacement)
 
 
-def _packageable_replacement(
-    rtype: str, prop_path: str, current: Any, template_dir: str, resource_id: str
-) -> Any:
+def _packageable_replacement(rtype: str, prop_path: str, current: Any, template_dir: str, resource_id: str) -> Any:
     """Compute the deterministic replacement for an artifact property.
 
     Returns either a dict ({"S3Bucket": ..., "S3Key": ...}) or a string
@@ -402,9 +396,7 @@ def _packageable_replacement(
       - AWS::Serverless::Function .CodeUri (post-transform = Lambda::Function .Code, handled above)
       - All other artifact properties     -> "s3://<bucket>/<sha256>"
     """
-    digest = hashlib.sha256(
-        f"{current}|{rtype}|{prop_path}|{resource_id}".encode("utf-8")
-    ).hexdigest()
+    digest = hashlib.sha256(f"{current}|{rtype}|{prop_path}|{resource_id}".encode("utf-8")).hexdigest()
     # Lambda image — must check before the .Code dict shape
     if rtype == "AWS::Lambda::Function" and prop_path == "Code.ImageUri":
         return f"{GOLDEN_BUCKET}.dkr.ecr.us-east-1.amazonaws.com/golden:{digest[:12]}"

tests/golden/test_corpus.py

@@ -24,9 +24,7 @@ def _hint(case_dir: Path) -> str:
 def test_build_output_matches_golden(golden_case):
     meta = _read_metadata(golden_case)
     le_enabled = _resolve_le_default(golden_case, meta)
-    actual = normalize(
-        run_build_pipeline(golden_case / "template.yaml", language_extensions=le_enabled)
-    )
+    actual = normalize(run_build_pipeline(golden_case / "template.yaml", language_extensions=le_enabled))
     expected_path = golden_case / "expected.build.yaml"
     assert expected_path.exists(), f"missing {expected_path}; run update_goldens.py --new"
     expected = expected_path.read_text(encoding="utf-8")
@@ -36,9 +34,7 @@ def test_build_output_matches_golden(golden_case):
 def test_package_output_matches_golden(golden_case):
     meta = _read_metadata(golden_case)
     le_enabled = _resolve_le_default(golden_case, meta)
-    build_out = run_build_pipeline(
-        golden_case / "template.yaml", language_extensions=le_enabled
-    )
+    build_out = run_build_pipeline(golden_case / "template.yaml", language_extensions=le_enabled)
     actual = normalize(run_package_pipeline(golden_case / "template.yaml", build_out))
     expected_path = golden_case / "expected.package.yaml"
     assert expected_path.exists(), f"missing {expected_path}; run update_goldens.py --new"

tests/golden/test_normalize.py

@@ -57,6 +57,7 @@ def test_normalize_is_idempotent():
     template = {"Resources": {"A": {"Type": "T"}}}
     once = normalize(template)
     import yaml
+
     twice = normalize(yaml.safe_load(once))
     assert once == twice