Skip to content

feat(tests): add plugin expressions tests and update related files - #2284

Merged
VarunAthreya merged 2 commits into
mainfrom
feat/support-plugin-expressions
Oct 14, 2026
Merged

VarunAthreya merged 2 commits into
mainfrom
feat/support-plugin-expressions

Conversation

@VarunAthreya

Copy link
Copy Markdown
Member

Summary

Integration-test coverage for plugin expressions in deck gateway sync/apply/dump. The reconciliation logic itself lives in
go-database-reconciler; this PR only adds the end-to-end tests and pins the
upstream releases that carry the support.

Depends on:

Fiixes: #2279

Changes

  • tests/integration/sync_test.go: sync + resync-no-diff, and removing
    expressions from the file clears them in Kong (enterprise >= 3.16.0 and
    Konnect variants).
  • tests/integration/apply_test.go: apply changes an expression and the state
    matches (enterprise + Konnect variants).
  • tests/integration/dump_test.go: dump emits the expressions block and the
    dumped file diffs clean (enterprise + Konnect variants).
  • New testdata under tests/integration/testdata/ and
    tests/integration/testdata/sync/003-create-a-plugin/.
  • go.mod/go.sum: bump go-kong and go-database-reconciler versions.

@codecov-commenter

codecov-commenter commented Oct 9, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 36.73%. Comparing base (4a272a1) to head (5ebdbba).

Files with missing lines Patch % Lines
tests/integration/test_utils.go 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2284      +/-   ##
==========================================
+ Coverage   35.92%   36.73%   +0.80%     
==========================================
  Files          82       82              
  Lines        9867     7943    -1924     
==========================================
- Hits         3545     2918     -627     
+ Misses       6056     4759    -1297     
  Partials      266      266              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

bcollard added a commit to bcollard/kong-identity-workflow-builder that referenced this pull request Oct 11, 2026
…ecK; syntax highlighting

- e2e/results/<payload>.json: runs of the generated files on a live gateway (changes made, checks, hashes of the applied
  tabs). The server injects them; the badge says 'Tested end to end' only while the hashes match this version's output,
  'Tested with an earlier version' otherwise, and the old lab match is shown as 'identity flow lab-tested, file not applied'
- Four scenarios recorded: AI Gateway 2.x employees (Keycloak) and apps (Kong Identity), API Gateway self-managed with
  Consumers, API Gateway on Konnect with principals, allow_when and metadata expressions (44/44 checks)
- decK can't sync plugin expressions yet (Kong/deck#2284): metadata plans move to the Konnect API tab (after decK) and
  real Terraform resources; How to apply says decK AND Konnect API
- Konnect API script captures the Kong Identity client and prints the issuer (no <client_id> placeholder)
- Syntax highlighting for YAML, HCL and shell (built in; the CSP allows no external scripts); How to apply padding

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@VarunAthreya
VarunAthreya force-pushed the feat/support-plugin-expressions branch from 2062fcb to 13a379f Compare October 12, 2026 13:10
@VarunAthreya
VarunAthreya force-pushed the feat/support-plugin-expressions branch from 156ae4c to 5ebdbba Compare October 14, 2026 05:38
@VarunAthreya
VarunAthreya merged commit c5bc488 into main Oct 14, 2026
56 checks passed
@VarunAthreya
VarunAthreya deleted the feat/support-plugin-expressions branch October 14, 2026 06:49
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.

3 participants