Skip to content

test: increase extension-host coverage for provider wiring / call-site integration #166

Description

@eFAILution

Background

The unit test suite (now 176+ tests via the Mocha migration) covers extracted pure helpers well. However, two of the most impactful bugs in recent history were wiring bugs at the call site, not logic bugs in the helpers:

Both were invisible to the 176 green unit tests. The extension-host suite caught neither — it has ~10 tests, mostly activation and basic hover.

Proposed

Add extension-host tests covering the seams most likely to regress:

  • Cache → provider integration: detectIncludeComponent returns correct component for a branch ref on second hover (i.e. freshness logic wires end-to-end).
  • Catalog fetch: a project with one malformed template still returns the other valid components (the try/catch placement regression).
  • Validation fires correct diagnostics after a local include's spec.inputs changes (unsaved vs saved).
  • Completion offers only missing inputs after a multi-line array input.

Priority

Medium — blocking a regression requires a wiring bug to slip past code review. Current review discipline is the main guard; these tests would be the safety net.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions