Skip to content

test: replace stale serve-static source-text assertion #754

Description

@flyingrobots

Problem

test/serve-static.bats greps for the obsolete literal .js: application/javascript mapping. The implementation now correctly returns application/javascript; charset=utf-8, so the source-text assertion fails even though both executable behavior lanes pass.

Evidence

  • test/serve-static-unit.bats passes the data-driven contentType() assertion.
  • test/serve-static-e2e.bats passes the HTTP response header assertion.
  • Full BATS_LIB_PATH=test/vendor bats -t test/*.bats is 110/111; this is the sole failure.

Acceptance

  • Replace or remove the brittle source-text grep in favor of observable behavior.
  • Keep unit and HTTP end-to-end coverage for the JavaScript content type.
  • Full repository Bats is green.
  • pnpm run preflight remains green.

Found during the five-slice documentation reconstruction drift check; not caused by #747.

Metadata

Metadata

Assignees

No one assigned

    Labels

    testsTest suite and coveragetriage:bad-codeDebt intake that still needs scheduling, closure, or a repo movework:integrityRemoves lies, drift, flake, or trust debt

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions