Skip to content

test(e2e): cover decrypt_fields consume policy lifecycle - #2194

Open
rspurgeon wants to merge 2 commits into
mainfrom
gh-2168
Open

rspurgeon wants to merge 2 commits into
mainfrom
gh-2168

Conversation

@rspurgeon

Copy link
Copy Markdown
Collaborator

The consume-policy E2E scenario did not exercise decrypt_fields.
Add a schema-validation parent and static key, then verify field-level
decryption creation, the exact parent link and returned config, no-op
replanning, sync deletion, and prerequisite cleanup.

Use Konnect's canonical bracket notation for the field path so the fixture
round-trips without normalization drift.

Closes #2168.

Validation:

  • Go modernization, formatting, and build passed.
  • Lint passed with zero issues (--allow-serial-runners to wait for another
    local lint process).
  • Unit, integration, installer, and E2E tooling tests passed.
  • make test-e2e-scenarios SCENARIO=event-gateway/consume-policy passed.
  • Local E2E artifacts:
    /home/rspurgeon/go/e2e-artifacts/20260915-225157.

@rspurgeon
rspurgeon requested review from a team as code owners September 16, 2026 03:55
@kong-api-ops
kong-api-ops Bot requested a review from a team September 16, 2026 03:55
@kong-api-ops

kong-api-ops Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

KSAI Review: Finished

  1. 03:59 UTC Auditing declarative planner behavior against test scenarios

Run report (federated) · 1 paid run · $0.0353 total
# Engine Result Model Turns Cost
1 opencode success zai-org/GLM-5.3-Flash/high 23 $0.0352

Reviewing cost $0.0352; deciding how to run it cost $0.0001

@kong-api-ops kong-api-ops Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check Result
Scope Two new declarative overlays and E2E scenario steps 013-016 covering a decrypt_fields child consume policy (create, verify, idempotency, sync-delete)
Mandate default-code-reviewer
Findings 2 Low
Findings audit Completed

The new steps match proven patterns in this scenario: delete plans key on resource name, nested empty lists signal sync deletes, and static-key plaintext drift does not break idempotency re-plans. No additional concerns found in the secrets, security, or leak dimensions.

Comment thread test/e2e/scenarios/event-gateway/consume-policy/scenario.yaml Outdated
@kong-api-ops
kong-api-ops Bot removed the request for review from a team September 16, 2026 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[e2e-scan] Test: Improve e2e coverage for consume policy decrypt_fields variant (expand scenario)

1 participant