wehub-resource-sync
127 lines
4.5 KiB
Python
127 lines
4.5 KiB
Python
"""Contract tests for the bundle manifest schema (bundle.yml).
|
|
|
|
Mirrors contracts/bundle-manifest.schema.md: required identity/metadata fields,
|
|
semver pinning of components, preset priority+strategy, integration optionality.
|
|
"""
|
|
from __future__ import annotations
|
|
|
|
import pytest
|
|
|
|
from specify_cli.bundler import BundlerError
|
|
from specify_cli.bundler.models.manifest import BundleManifest
|
|
from tests.bundler_helpers import valid_manifest_dict
|
|
|
|
|
|
def test_valid_manifest_has_no_structural_errors():
|
|
manifest = BundleManifest.from_dict(valid_manifest_dict())
|
|
assert manifest.structural_errors() == []
|
|
assert manifest.bundle.id == "demo-bundle"
|
|
assert manifest.is_agnostic() is True
|
|
|
|
|
|
def test_missing_required_field_is_reported_by_name():
|
|
data = valid_manifest_dict()
|
|
del data["bundle"]["license"]
|
|
errors = BundleManifest.from_dict(data).structural_errors()
|
|
assert any("bundle.license" in e for e in errors)
|
|
|
|
|
|
def test_unsupported_schema_version_is_rejected():
|
|
data = valid_manifest_dict(schema_version="9.9")
|
|
errors = BundleManifest.from_dict(data).structural_errors()
|
|
assert any("schema_version" in e for e in errors)
|
|
|
|
|
|
def test_non_semver_bundle_version_is_rejected():
|
|
data = valid_manifest_dict()
|
|
data["bundle"]["version"] = "not-a-version"
|
|
errors = BundleManifest.from_dict(data).structural_errors()
|
|
assert any("semver" in e for e in errors)
|
|
|
|
|
|
def test_preset_requires_priority_and_strategy():
|
|
data = valid_manifest_dict()
|
|
data["provides"]["presets"] = [{"id": "p", "version": "1.0.0"}]
|
|
errors = BundleManifest.from_dict(data).structural_errors()
|
|
assert any("priority" in e for e in errors)
|
|
assert any("strategy" in e for e in errors)
|
|
|
|
|
|
def test_invalid_preset_strategy_is_rejected():
|
|
data = valid_manifest_dict()
|
|
data["provides"]["presets"][0]["strategy"] = "merge"
|
|
errors = BundleManifest.from_dict(data).structural_errors()
|
|
assert any("strategy" in e for e in errors)
|
|
|
|
|
|
def test_non_integer_priority_raises_actionable_error():
|
|
data = valid_manifest_dict()
|
|
data["provides"]["presets"][0]["priority"] = "high"
|
|
with pytest.raises(BundlerError, match="priority must be an integer"):
|
|
BundleManifest.from_dict(data)
|
|
|
|
|
|
def test_non_step_components_must_be_pinned():
|
|
data = valid_manifest_dict()
|
|
data["provides"]["extensions"] = [{"id": "ext-unpinned"}]
|
|
errors = BundleManifest.from_dict(data).structural_errors()
|
|
assert any("must be pinned" in e for e in errors)
|
|
|
|
|
|
def test_steps_may_be_unpinned():
|
|
data = valid_manifest_dict()
|
|
data["provides"]["steps"] = [{"id": "step-x"}]
|
|
manifest = BundleManifest.from_dict(data)
|
|
assert manifest.structural_errors() == []
|
|
|
|
|
|
def test_integration_makes_bundle_non_agnostic():
|
|
data = valid_manifest_dict(integration={"id": "copilot"})
|
|
manifest = BundleManifest.from_dict(data)
|
|
assert manifest.is_agnostic() is False
|
|
assert manifest.integration.id == "copilot"
|
|
|
|
|
|
def test_components_property_orders_by_kind():
|
|
manifest = BundleManifest.from_dict(valid_manifest_dict())
|
|
kinds = [c.kind for c in manifest.components]
|
|
assert kinds == ["extensions", "presets", "steps", "workflows"]
|
|
|
|
|
|
def test_string_tags_rejected_not_split_per_character():
|
|
# A bare string would otherwise be iterated character-by-character; the
|
|
# schema requires a list of strings.
|
|
data = valid_manifest_dict()
|
|
data["tags"] = "security"
|
|
with pytest.raises(BundlerError, match="'tags' must be a list of strings"):
|
|
BundleManifest.from_dict(data)
|
|
|
|
|
|
def test_unsafe_bundle_id_flagged_by_structural_validation():
|
|
data = valid_manifest_dict()
|
|
data["bundle"]["id"] = "../evil"
|
|
manifest = BundleManifest.from_dict(data)
|
|
errors = manifest.structural_errors()
|
|
assert any("bundle.id" in e and "slug" in e for e in errors)
|
|
|
|
|
|
def test_valid_slug_bundle_id_passes():
|
|
data = valid_manifest_dict()
|
|
data["bundle"]["id"] = "team-a.bundle_1"
|
|
manifest = BundleManifest.from_dict(data)
|
|
assert not any("bundle.id" in e for e in manifest.structural_errors())
|
|
|
|
|
|
def test_string_tools_rejected_not_split_per_character():
|
|
data = valid_manifest_dict()
|
|
data["requires"]["tools"] = "docker"
|
|
with pytest.raises(BundlerError, match="'requires.tools' must be a list of strings"):
|
|
BundleManifest.from_dict(data)
|
|
|
|
|
|
def test_string_mcp_rejected_not_split_per_character():
|
|
data = valid_manifest_dict()
|
|
data["requires"]["mcp"] = "github"
|
|
with pytest.raises(BundlerError, match="'requires.mcp' must be a list of strings"):
|
|
BundleManifest.from_dict(data)
|