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.

Sign up for GitHub

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

Jump to bottom

[Tests-Only] Change issue number in expected failure file forlogin test #4678

Merged
merged 1 commit into from
Feb 2, 2021
Merged

[Tests-Only] Change issue number in expected failure file forlogin test #4678

merged 1 commit into from
Feb 2, 2021

Conversation

swoichha
Copy link
Contributor

@swoichha swoichha commented Feb 2, 2021

Description

This PR changes issue number tagged for this test scenario.

web/tests/acceptance/features/webUILogin/login.feature

Line 62 in fb92fed

Scenario: the user session of a deleted user is cleared properly

Related Issue

How Has This Been Tested?

  • CI

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

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

Open tasks:

  • ...

@swoichha swoichha self-assigned this Feb 2, 2021
@swoichha swoichha added QA:team Status:Needs-Review Needs review from a maintainer labels Feb 2, 2021
@haribhandari07 haribhandari07 merged commit e80e4d8 into master Feb 2, 2021
@delete-merged-branch delete-merged-branch bot deleted the test-user-session branch February 2, 2021 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phil-davis phil-davis phil-davis approved these changes

@jasson99 jasson99 Awaiting requested review from jasson99

@kiranparajuli589 kiranparajuli589 Awaiting requested review from kiranparajuli589

@haribhandari07 haribhandari07 Awaiting requested review from haribhandari07

@dpakach dpakach Awaiting requested review from dpakach

Assignees

@swoichha swoichha

Labels
QA:team Status:Needs-Review Needs review from a maintainer
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@swoichha @phil-davis @haribhandari07