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][full-ci] Added API tests for searching users by normal user #8241

Merged
merged 1 commit into from
Jan 24, 2024
Merged

[tests-only][full-ci] Added API tests for searching users by normal user #8241

merged 1 commit into from
Jan 24, 2024

Conversation

nirajacharya2
Copy link
Contributor

@nirajacharya2 nirajacharya2 commented Jan 18, 2024 •
edited
Loading

Description

Adding test to get user data as a normal user using search queries

Scenario Outline: non-admin user searches other users by e-mail
Scenario Outline: non-admin user searches for a disabled users
Scenario Outline: non-admin user searches for multiple users having same displayname

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:local
  • test case 2: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 only (no source changes)

Checklist:

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

@nirajacharya2 nirajacharya2 self-assigned this Jan 18, 2024
@nirajacharya2 nirajacharya2 mentioned this pull request Jan 18, 2024
Closed
10 tasks
@saw-jan saw-jan changed the title [tests-only][full-ci] Added API tests for searching users by normal user [tests-only][full-ci] Added API tests for searching users by normal user using email Jan 18, 2024
@saw-jan saw-jan changed the title [tests-only][full-ci] Added API tests for searching users by normal user using email [tests-only][full-ci] Added API tests for searching users by normal user Jan 18, 2024
@nirajacharya2 nirajacharya2 force-pushed the user-as-normal-user-1-stable branch 2 times, most recently from ad14781 to 174d4b8 Compare January 19, 2024 11:07
@nirajacharya2 nirajacharya2 changed the base branch from stable-5.0 to master January 19, 2024 11:09
@nirajacharya2 nirajacharya2 changed the base branch from master to stable-5.0 January 19, 2024 11:12
@nirajacharya2 nirajacharya2 changed the base branch from stable-5.0 to master January 19, 2024 11:15
@nirajacharya2 nirajacharya2 force-pushed the user-as-normal-user-1-stable branch from 174d4b8 to 2ef7403 Compare January 22, 2024 03:06
@nirajacharya2 nirajacharya2 force-pushed the user-as-normal-user-1-stable branch from 2ef7403 to ad14781 Compare January 22, 2024 03:09
@nirajacharya2 nirajacharya2 reopened this Jan 22, 2024
@nirajacharya2 nirajacharya2 force-pushed the user-as-normal-user-1-stable branch 2 times, most recently from c508f82 to 36079dc Compare January 22, 2024 05:45
@nirajacharya2 nirajacharya2 marked this pull request as ready for review January 22, 2024 08:55
@nirajacharya2 nirajacharya2 force-pushed the user-as-normal-user-1-stable branch from 36079dc to 08ae081 Compare January 23, 2024 03:19
@nirajacharya2 nirajacharya2 force-pushed the user-as-normal-user-1-stable branch from 08ae081 to 41ab107 Compare January 23, 2024 04:44
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

saw-jan
saw-jan approved these changes Jan 23, 2024
View reviewed changes
Copy link
Member

@saw-jan saw-jan left a comment

Choose a reason for hiding this comment

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

LGTM

@saw-jan saw-jan merged commit 7d6cdd6 into master Jan 24, 2024
4 checks passed
@delete-merged-branch delete-merged-branch bot deleted the user-as-normal-user-1-stable branch January 24, 2024 04:12
ownclouders pushed a commit that referenced this pull request Jan 24, 2024
@nirajacharya2
added tests for searching users by normal user (#8241)
7163427
PrajwolAmatya pushed a commit that referenced this pull request Jan 24, 2024
@nirajacharya2 @PrajwolAmatya
added tests for searching users by normal user (#8241)
e77915b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ScharfViktor ScharfViktor ScharfViktor approved these changes

@saw-jan saw-jan saw-jan approved these changes

@grgprarup grgprarup Awaiting requested review from grgprarup

@SagarGi SagarGi Awaiting requested review from SagarGi

@nabim777 nabim777 Awaiting requested review from nabim777

@S-Panta S-Panta Awaiting requested review from S-Panta

@PrajwolAmatya PrajwolAmatya Awaiting requested review from PrajwolAmatya

Assignees

@nirajacharya2 nirajacharya2

Labels
QA:team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nirajacharya2 @saw-jan @ScharfViktor