Skip to content

Conversation

@anon-pradip
Copy link
Contributor

@anon-pradip anon-pradip commented Jan 12, 2026

Description

Migrates groups.feature end-to-end tests from Cucumber/Gherkin to Playwright test framework.
Separated getDisplayedGroupsIds (for UUIDs) and getGroupsDisplayName (for display names) to avoid UI hangs during dynamic updates.

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • Locally and CI

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests
  • Documentation
  • Maintenance (e.g. dependency updates or tooling)

Open tasks:

@anon-pradip anon-pradip self-assigned this Jan 12, 2026
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch from 8a79e35 to 20fe508 Compare January 12, 2026 10:53
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch 3 times, most recently from 25fc463 to 8198f8e Compare January 13, 2026 06:50
),
page.locator(compareDialogConfirm).click()
])
await objects.a11y.Accessibility.assertNoSevereA11yViolations(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Toast doesn't appear hereafter so not checking the whole body

@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch 3 times, most recently from ade95bc to d42ee93 Compare January 13, 2026 07:23
@anon-pradip anon-pradip marked this pull request as ready for review January 13, 2026 08:05
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch from d42ee93 to 1e9fde6 Compare January 13, 2026 12:25
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch 3 times, most recently from c497f12 to d1b1b8e Compare January 15, 2026 08:44
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch 2 times, most recently from 3e2fd08 to 28c94de Compare January 15, 2026 09:00
await objects.a11y.Accessibility.assertNoSevereA11yViolations(
page,
['editPanel'],
`Edit group sidebar panel while changing ${attribute}`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Toast doesn't appear hereafter so not checking the whole body

@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch 2 times, most recently from 865ad09 to eb421bd Compare January 26, 2026 08:50
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch from c1f2d72 to 0e26b45 Compare January 26, 2026 11:24
Comment on lines +13 to +16
test.afterEach(async ({ usersEnvironment }) => {
const admin = usersEnvironment.getUser({ key: 'Admin' })
await cleanUpGroup(admin)
})
Copy link
Member

Choose a reason for hiding this comment

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

👍 we can add user deletion here later on.

@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch from 0e26b45 to 21009a0 Compare January 26, 2026 11:49
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch 3 times, most recently from aaa5601 to 10b2842 Compare January 27, 2026 04:12
@anon-pradip anon-pradip force-pushed the test/e2e-playwright-groups.feature branch from 10b2842 to 3a22853 Compare January 27, 2026 04:59
@anon-pradip anon-pradip enabled auto-merge January 27, 2026 05:00
@sonarqubecloud
Copy link

@anon-pradip anon-pradip merged commit a1ebc68 into master Jan 27, 2026
4 checks passed
@anon-pradip anon-pradip deleted the test/e2e-playwright-groups.feature branch January 27, 2026 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants