Skip to content

Commit a5b35fa

Browse files
committed
Add better messages to asserts
1 parent 37bae34 commit a5b35fa

File tree

2 files changed

+13
-8
lines changed

2 files changed

+13
-8
lines changed

.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ repos:
3131
rev: v1.16.0
3232
hooks:
3333
- id: mypy
34-
exclude: tests/integration/.*/.*|tests/integration/quick_check_numpy.py|tests/unit/test_wheel_abi.py|scripts/calculate_symbol_versions.py
34+
exclude: tests/integration/.*/.*|tests/integration/.*|tests/integration/quick_check_numpy.py|tests/unit/test_wheel_abi.py|scripts/calculate_symbol_versions.py
3535
args: ["--python-version=3.9"]
3636
additional_dependencies:
3737
- nox

tests/integration/test_manylinux.py

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -578,22 +578,27 @@ def test_numpy_sbom(
578578
component["purl"].split("@")[0] for component in sbom_components[1:]
579579
}
580580
# Package URL prefixes must match for a policy.
581-
print(policy, component_purls)
582581
if policy.startswith("musllinux"):
583-
assert all(purl.startswith("pkg:apk/alpine/") for purl in component_purls)
582+
assert all(
583+
purl.startswith("pkg:apk/alpine/") for purl in component_purls
584+
), str(component_purls)
584585
elif policy == "manylinux_2_34_x86_64":
585586
assert all(
586587
purl.startswith("pkg:rpm/almalinux/") for purl in component_purls
587-
)
588+
), str(component_purls)
588589
else:
589590
assert all(
590591
purl.startswith("pkg:rpm/almalinux/") for purl in component_purls
591-
)
592+
), str(component_purls)
592593

593594
# We expect libgfortran, libquadmath, and openblas* as dependencies.
594-
assert any("libgfortran" in purl for purl in component_purls)
595-
assert any("libquadmath" in purl for purl in component_purls)
596-
assert any("openblas" in purl for purl in component_purls)
595+
assert any("libgfortran" in purl for purl in component_purls), str(
596+
component_purls
597+
)
598+
assert any("libquadmath" in purl for purl in component_purls), str(
599+
component_purls
600+
)
601+
assert any("openblas" in purl for purl in component_purls), str(component_purls)
597602

598603
assert len(sbom_dependencies) == len(component_purls) + 1
599604

0 commit comments

Comments
 (0)