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

#3222 - Provide Ministry the ability to edit Basic BCeID Profile Info (e2e tests) #3794

Conversation

sh16011993
Copy link
Collaborator

@sh16011993 sh16011993 commented Oct 15, 2024

As a part of this PR, the following e2e tests were written:

StudentAESTController(e2e)-updateProfileInformation
√ Should allow the student profile update when the student is a Basic BCeID user. (5472 ms)
√ Should throw an HTTP Unprocessable Entity (422) error when the student is not a Basic BCeID user. (2619 ms)
√ Should throw an HTTP Forbidden (403) error when the ministry user performing the profile update does not have the associated role. (497 ms)
√ Should throw an HTTP Unprocessable Entity (422) error when the student is a Basic BCeID user but none of the user profile information that needs to be updated has changed. (3076 ms)
√ Should throw an HTTP Not Found (404) error when the student does not exist. (514 ms)
√ Should throw an HTTP Bad Request error when some mandatory profile update information is missing from the payload. (7 ms)
√ Should allow the student profile update when the student is a Basic BCeID user and the givenNames is not provided. (4945 ms)

Screenshot of the passed e2e tests:

image

@sh16011993 sh16011993 marked this pull request as draft October 15, 2024 22:30
@sh16011993 sh16011993 self-assigned this Oct 15, 2024
@sh16011993 sh16011993 marked this pull request as ready for review October 15, 2024 22:34
@sh16011993 sh16011993 changed the title #3222 - Provide Ministry the ability to edit basic bceid profile info… #3222 - Provide Ministry the ability to edit basic bceid profile info (e2e tests) Oct 15, 2024
@sh16011993 sh16011993 changed the title #3222 - Provide Ministry the ability to edit basic bceid profile info (e2e tests) #3222 - Provide Ministry the ability to edit Basic BCeID Profile Info (e2e tests) Oct 15, 2024
Copy link
Collaborator

@andrewsignori-aot andrewsignori-aot left a comment

Choose a reason for hiding this comment

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

Great work, please take a look at the comments.

@lewischen-aot lewischen-aot self-requested a review October 16, 2024 19:56
Copy link

Copy link

Backend Unit Tests Coverage Report

Totals Coverage
Statements: 22.6% ( 3617 / 16004 )
Methods: 10.08% ( 201 / 1995 )
Lines: 26.03% ( 3143 / 12076 )
Branches: 14.12% ( 273 / 1933 )

Copy link

E2E Workflow Workers Coverage Report

Totals Coverage
Statements: 58.64% ( 509 / 868 )
Methods: 52.88% ( 55 / 104 )
Lines: 62.27% ( 411 / 660 )
Branches: 41.35% ( 43 / 104 )

Copy link

E2E Queue Consumers Coverage Report

Totals Coverage
Statements: 83.88% ( 1046 / 1247 )
Methods: 83.59% ( 107 / 128 )
Lines: 84.91% ( 895 / 1054 )
Branches: 67.69% ( 44 / 65 )

Copy link

E2E SIMS API Coverage Report

Totals Coverage
Statements: 66.02% ( 5572 / 8440 )
Methods: 63.3% ( 683 / 1079 )
Lines: 70.13% ( 4402 / 6277 )
Branches: 44.93% ( 487 / 1084 )

Copy link
Collaborator

@lewischen-aot lewischen-aot left a comment

Choose a reason for hiding this comment

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

LGTM. Nice work @sh16011993

Copy link
Collaborator

@andrewsignori-aot andrewsignori-aot left a comment

Choose a reason for hiding this comment

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

Thanks for making the changes, looks good 👍

@sh16011993 sh16011993 added this pull request to the merge queue Oct 16, 2024
Merged via the queue into main with commit e79e7df Oct 16, 2024
20 checks passed
@sh16011993 sh16011993 deleted the 3222_provide_ministry_ability_to_edit_basic_bceid_profile_info_e2e_tests branch October 16, 2024 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants