Skip to content

Feature/new security e2e tests#2460

Draft
jvorcak wants to merge 4 commits into
masterfrom
feature/new-security-e2e-tests
Draft

Feature/new security e2e tests#2460
jvorcak wants to merge 4 commits into
masterfrom
feature/new-security-e2e-tests

Conversation

@jvorcak
Copy link
Copy Markdown
Collaborator

@jvorcak jvorcak commented May 21, 2026

No description provided.

jvorcak added 4 commits May 20, 2026 23:53
- Add enterprise E2E tests for new security users and roles pages
  (security-users.spec.ts, security-roles.spec.ts, utils/security-page-new.ts)
- Fix Combobox testid: use inputTestId instead of testId so the attribute
  lands on the <input> element, not the PopoverPrimitive.Root wrapper
- Fix Input testid in ChangePasswordModal: use testId prop instead of
  data-testid attribute to avoid being overridden by the component's spread
- Add save-password-button testid to the ChangePasswordModal submit button
…se scenarios

- Add featureFlags fixture to Kafka test variant
- Add security-kafka.spec.ts covering old/new layout tab states and ACL dialog
- Add security-users-advanced.spec.ts for role assignment, allow-all-operations,
  assign/remove role from user details, and permissions page integration
- Add testIds allow-all-operations-button and confirm-allow-all-button to AclsCard
- Fix assign-role-combobox to use inputTestId instead of testId on Combobox
- Add createUserWithRole, allowAllOperations, assignRoleFromDetails, removeRoleFromDetails
  helpers to SecurityPageNew POM
Enterprise variant enables schemaRegistryACLApi, producing 6 Allow cells
(not 4). Use nth(3) instead of toHaveCount(4) to assert at least 4 exist.
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.

1 participant