Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automation test for invalidate sessions. #2234

Merged
merged 5 commits into from
Jan 27, 2025

Conversation

labkey-sweta
Copy link
Contributor

@labkey-sweta labkey-sweta commented Jan 15, 2025

Rationale

Automation test for

Secure Issue 51523: Invalidate sessions on password change

Secure Issue 51637: Invalidate sessions when their API key becomes invalid

Secure Issue 31493: Test for session and cookie persistence through login and logout

Secure Issue 51187: Additional automation testing for group audit logs

Related Pull Requests

Changes

Copy link

ERROR: A pull request from 24.11_fb_sessionAndAPIKeyAutoTest is expected to target release24.11-SNAPSHOT, not develop

@labkey-sweta labkey-sweta changed the base branch from develop to release24.11-SNAPSHOT January 15, 2025 23:56
src/org/labkey/test/tests/ApiKeyTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/ApiKeyTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/ApiKeyTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/ApiKeyTest.java Outdated Show resolved Hide resolved
Copy link
Member

@labkey-tchad labkey-tchad left a comment

Choose a reason for hiding this comment

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

The test should not use WebTestHelper.getCookies.

src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
src/org/labkey/test/tests/InvalidateSessionTest.java Outdated Show resolved Hide resolved
@labkey-sweta labkey-sweta merged commit 2d7e93d into release24.11-SNAPSHOT Jan 27, 2025
3 checks passed
@labkey-sweta labkey-sweta deleted the 24.11_fb_sessionAndAPIKeyAutoTest branch January 27, 2025 08:05
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