Skip to content

feat: add-kv-ui-tests#14397

Open
hemantmm wants to merge 3 commits intokestra-io:developfrom
hemantmm:add-kv-ui-tests
Open

feat: add-kv-ui-tests#14397
hemantmm wants to merge 3 commits intokestra-io:developfrom
hemantmm:add-kv-ui-tests

Conversation

@hemantmm
Copy link
Contributor

@hemantmm hemantmm commented Feb 2, 2026

closes: #12767

✨ Description

What does this PR change?

The new tests include creating, saving, editing, and validating KV entries for all supported types of KV, to ensure that the values are properly restored when reopening a KV for editing.

The UI tests include the following:

  • STRING, NUMBER, BOOLEAN (true/false)
  • DATE, DATETIME, DURATION
  • JSON (object and array)
  • Description and TTL fields
  • Updating existing KV values
  • Validation errors (invalid JSON, duplicate keys)
  • Copy KV expression action

@github-project-automation github-project-automation bot moved this to To review in Pull Requests Feb 2, 2026
@MilosPaunovic MilosPaunovic added area/frontend Needs frontend code changes kind/external Pull requests raised by community contributors labels Feb 3, 2026
@brian-mulier-p
Copy link
Member

Hello @hemantmm, it looks AI generated or there is a missing piece as data-testid='add-kv-button' doesn't exist, please make it work and then I'll have a proper look.

Copy link
Member

@brian-mulier-p brian-mulier-p left a comment

Choose a reason for hiding this comment

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

AI Generated or missing pieces

@hemantmm
Copy link
Contributor Author

hemantmm commented Feb 5, 2026

I appreciate your feedback 👍.

I'll review the tests and UI selectors to make sure nothing is assumed or missing. Certain sections were swiftly scaffolded in and clearly require hardening to conform to the current user interface.

I'll fix it and soon update the PR.

@MilosPaunovic MilosPaunovic moved this from To review to On hold in Pull Requests Feb 5, 2026
hemantmm and others added 2 commits February 6, 2026 20:10
Signed-off-by: Hemant M Mehta <hemant29mehta@gmail.com>
@MilosPaunovic MilosPaunovic moved this from On hold to To review in Pull Requests Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/frontend Needs frontend code changes kind/external Pull requests raised by community contributors

Projects

Status: To review

Development

Successfully merging this pull request may close these issues.

Add some UI tests for KV

3 participants