Feature/subtask2b consul tests

[DeekshithaTimmareddy]

Description

Adds integration tests for the ConsulSource introduced in #898. Tests run against a real in-process Consul agent using consul/sdk/testutil — no mocks, no stubs.

The unit test file (consul_source_test.go) is removed; all coverage is now through integration tests that exercise the actual HTTP path.

Changes

• consul_source_integration_test.go — 9 subtests covering:
  • Fetching typed variables (number, string, bool, object) from a KV path
  • Keys not in pack schema are ignored
  • Empty value for non-string variable returns an error
  • Object variable with optional field missing converts correctly via ConstraintType
  • Bool variable decoded from JSON
  • Malformed JSON for non-string variable returns an error
  • Empty string kept for string variable
  • Path with no keys returns empty result

Reminders

• Add CHANGELOG.md entry

• If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

[tgross]

PR is targeting the wrong branch. LLM-generated PR description. Tests don't pass.