e2e test user page: personal info and tokens #9975

archibald1418 · 2025-11-05T14:16:28Z

Motivation and context

Coverage for #9789 and #9829

Checklist

I submit my changes into the develop branch
I have created a changelog fragment
I have updated the documentation accordingly
I have added tests to cover my changes
I have linked related issues (see GitHub docs)

License

I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

codecov-commenter · 2025-11-06T20:38:25Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.73%. Comparing base (98c0a2a) to head (12d133b).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #9975      +/-   ##
===========================================
- Coverage    81.35%   73.73%   -7.63%     
===========================================
  Files          483      427      -56     
  Lines        49322    46243    -3079     
  Branches      4132     4132              
===========================================
- Hits         40124    34095    -6029     
- Misses        9198    12148    +2950

Components	Coverage Δ
cvat-ui	`77.50% <ø> (+0.58%)`	⬆️
cvat-server	`70.54% <ø> (-14.16%)`	⬇️

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

cvat-ui/src/components/profile-page/security-content/api-tokens-card.tsx

klakhov · 2025-11-07T10:37:13Z

cvat-ui/src/components/profile-page/security-content/api-tokens-card.tsx

                    <Button type='text' icon={<MoreOutlined />} />
                </Dropdown>
            ),
+            className: 'cvat-api-token-action',


Suggested change

className: 'cvat-api-token-action',

why? I didn't find a class name I could use here to press on actions

got it, it's an extra class, all passes

This classname is still not used anywhere. I suggest to remove it.

sorry, stashed it somewhere, now removed

klakhov · 2025-11-07T10:40:31Z

tests/cypress/support/commands.js

        req.continue((response) => {
            delete response.headers['set-cookie'];
-            expect(response.statusCode).to.eq(201, response.statusMessage);
+            expect(response.statusCode).to.eq(201, response.body.username); // contains msg


Suggested change

expect(response.statusCode).to.eq(201, response.body.username); // contains msg

expect(response.statusCode).to.eq(201, response.body.username);

done
on that note, that error message was hard to find.
Is there a reason that the error message is returned in the body's username instead of something like this?

{ body: { "statusCode": 409, "message": "This username is already present" } }

I believe thats because of django validation logic. If its an error in serializer validator it will print error in body for each errored field(it does it because there can be several fields with error)

tests/cypress/e2e/actions_users/registration_involved/case_2_user_profile_page.js

Co-authored-by: Kirill Lakhov <[email protected]>

Copilot

Pull Request Overview

This PR adds comprehensive end-to-end tests for user profile page functionality, specifically focusing on password changes and API token management. The old test file for password changes has been replaced with a more thorough test suite.

Adds new test file case_2_user_profile_page.js covering profile updates, password changes, and API token operations
Introduces utility functions and classes for test helpers (clipboard interaction, date manipulation, snake case conversion)
Adds cypress-repeat package as a dev dependency
Enhances UI components with CSS class names for better test targeting
Updates the date format for API token expiration to DD/MM/YYYY for consistency

Reviewed Changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
tests/cypress/e2e/actions_users/registration_involved/case_2_user_profile_page.js	New comprehensive test file replacing the old password change test, covering profile updates, password changes, and API token CRUD operations
tests/cypress/e2e/actions_users/registration_involved/case_2_register_user_change_pass.js	Removed old test file that is replaced by the more comprehensive case_2_user_profile_page.js
tests/cypress/support/utils.js	Added utility functions for date manipulation (aYearFrom, aMonthFrom, parseDatetime, format) and object transformation (toSnakeCase)
tests/cypress/support/const.js	Added ClipboardCtx class for managing clipboard interactions in tests and CLIPBOARD_ALIAS constant
tests/cypress/support/commands.js	Added new Cypress commands for user management (headlessDeleteUserByUsername, headlessGetSelfId, headlessGetUserId) and updated headlessCreateUser to use snake_case conversion
tests/cypress/e2e/actions_objects2/case_copy_frame_filename.js	Refactored to use new ClipboardCtx class for cleaner clipboard testing
tests/cypress/e2e/features/case_113_new_organization_pipeline.js	Simplified capitalizeEmail function to use Cypress._.capitalize utility
tests/package.json	Added cypress-repeat as a devDependency
tests/yarn.lock	Updated with dependencies for cypress-repeat and its transitive dependencies
cvat-ui/src/components/profile-page/security-content/api-tokens-card.tsx	Added CSS class names to table columns and modal for test targeting
cvat-ui/src/components/profile-page/security-content/api-token-form.tsx	Added DD/MM/YYYY format to date picker for consistency with token table display
cvat-ui/src/components/profile-page/security-content/api-token-created-modal.tsx	Added CSS class name to confirmation button for test targeting
cvat-ui/src/components/profile-page/profile-page.tsx	Added CSS class names to welcome message and menu items for test targeting
cvat-ui/src/reducers/notifications-reducer.ts	Added CSS class name to password change success notification for test targeting
changelog.d/20251104_154354_oleg.valiulin_test_e2e_user_page_tokens.md	Added changelog entry documenting the date format change

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

tests/cypress/support/utils.js

Co-authored-by: Copilot <[email protected]>

tests/cypress/support/utils.js

tests/cypress/support/commands.js

klakhov · 2025-11-10T10:32:27Z

cvat-ui/src/components/profile-page/security-content/api-tokens-card.tsx

                    <Button type='text' icon={<MoreOutlined />} />
                </Dropdown>
            ),
+            className: 'cvat-api-token-action',


This classname is still not used anywhere. I suggest to remove it.

klakhov

LGTM, just a couple minor comments

klakhov · 2025-11-12T07:21:20Z

tests/cypress/e2e/actions_users/registration_involved/case_2_user_profile_page.js

+        });
+
+        context('Security tab', () => {
+            before('open security tab', () => {


Suggested change

before('open security tab', () => {

before('Open security tab', () => {

klakhov · 2025-11-12T07:21:54Z

tests/cypress/e2e/actions_users/registration_involved/case_2_user_profile_page.js

+                    openProfileTab('Security');
+                });
+
+                it('Change password unsuccessful, error notif appears. Cancel button works', () => {


Suggested change

it('Change password unsuccessful, error notif appears. Cancel button works', () => {

it('Change password unsuccessful, error notification appears. Cancel button works', () => {

sonarqubecloud · 2025-11-12T13:00:04Z

Quality Gate failed

Failed conditions
4 Security Hotspots

See analysis details on SonarQube Cloud

Oleg Valiulin added 17 commits October 29, 2025 12:08

toSnakeCase util, selfId cypress command

63d8287

move user registration from the test case

99d57a6

test profile tab, change info, add classes

0b6184f

wip: create and copy token, refactor before hooks for isolation

9878f68

token is sent and copied case, apply clipboard ctx

c414b99

ensure modal is closed

da85d8e

add cypress-repeat plugin for debugging

d692343

rework password change flow, remove self delete user

36b66df

refactor

5a8c7b5

add first token test

91d6fd0

add class names to elements under test

c0abcf5

sync date formats between expiration date form and table view

174b313

test edit token and table view

7847032

add cy.clock for Date determinism

dc77d4a

add classes, refactor create, revoke every token

d765f58

move date utils to utils file, refactor

6635092

wip: test token requests

d2caa4f

archibald1418 requested review from SpecLad, azhavoro, bsekachev, nmanovic and zhiltsov-max as code owners November 5, 2025 14:16

Merge branch 'develop' into ov/test-e2e-user-page-tokens

d8fba7d

archibald1418 force-pushed the ov/test-e2e-user-page-tokens branch from c3d48f9 to d8fba7d Compare November 5, 2025 14:19

archibald1418 removed request for SpecLad, azhavoro, bsekachev, nmanovic and zhiltsov-max November 5, 2025 14:20

finish token tests

969d5a4

archibald1418 requested a review from klakhov November 6, 2025 20:23

klakhov reviewed Nov 7, 2025

View reviewed changes

archibald1418 and others added 5 commits November 7, 2025 12:11

update date entry class name

da9cb78

Co-authored-by: Kirill Lakhov <[email protected]>

update expiry date entry's class name

64492b1

Co-authored-by: Kirill Lakhov <[email protected]>

action -> actions in class name

d27ebd1

Co-authored-by: Kirill Lakhov <[email protected]>

remove comment

a140a88

Co-authored-by: Kirill Lakhov <[email protected]>

apply comments

008d6b6

archibald1418 requested a review from Copilot November 7, 2025 14:03

Copilot AI reviewed Nov 7, 2025

View reviewed changes

tests/cypress/support/utils.js Outdated Show resolved Hide resolved

Oleg Valiulin and others added 6 commits November 7, 2025 14:10

merge develop, fix minor conflicts with #9981

0203ea1

use snake copy instead of reassign

498b5ac

let the month increment wrap around the date

cdc29a0

Co-authored-by: Copilot <[email protected]>

add logs

62af64f

fix typing of personal info

3a8f7fb

lint

2654205

archibald1418 requested a review from klakhov November 7, 2025 15:45

klakhov reviewed Nov 10, 2025

View reviewed changes

Oleg Valiulin added 3 commits November 11, 2025 11:15

remove extra classname

2a20a7a

remove extra comment

a3854e0

Merge branch 'develop' into ov/test-e2e-user-page-tokens

3cd35b1

archibald1418 requested a review from klakhov November 11, 2025 11:40

remove deleted commands

eb423d3

klakhov approved these changes Nov 12, 2025

View reviewed changes

Oleg Valiulin added 3 commits November 12, 2025 12:51

fix titles

d1428c1

Merge branch 'develop' into ov/test-e2e-user-page-tokens

9dabe1d

remove skips

12d133b

archibald1418 merged commit 5d62475 into develop Nov 12, 2025
36 of 37 checks passed

archibald1418 deleted the ov/test-e2e-user-page-tokens branch November 12, 2025 13:23

	expect(response.statusCode).to.eq(201, response.body.username); // contains msg
	expect(response.statusCode).to.eq(201, response.body.username);

	before('open security tab', () => {
	before('Open security tab', () => {

	it('Change password unsuccessful, error notif appears. Cancel button works', () => {
	it('Change password unsuccessful, error notification appears. Cancel button works', () => {

e2e test user page: personal info and tokens #9975

e2e test user page: personal info and tokens #9975

Uh oh!

Conversation

archibald1418 commented Nov 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Motivation and context

Checklist

License

Uh oh!

codecov-commenter commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

archibald1418 Nov 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

archibald1418 Nov 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

klakhov left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Nov 12, 2025

Quality Gate failed

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

archibald1418 commented Nov 5, 2025 •

edited

Loading

codecov-commenter commented Nov 6, 2025 •

edited

Loading

archibald1418 Nov 7, 2025 •

edited

Loading

archibald1418 Nov 7, 2025 •

edited

Loading