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

4.1.1+portage-4.2.3-uat-1 Manual Testing #950

Closed
aaronskiba opened this issue Nov 18, 2024 · 5 comments
Closed

4.1.1+portage-4.2.3-uat-1 Manual Testing #950

aaronskiba opened this issue Nov 18, 2024 · 5 comments

Comments

@aaronskiba
Copy link
Collaborator

aaronskiba commented Nov 18, 2024

What version of the DMPRoadmap code are you running? (e.g. v2.2.0)

  • 4.1.1+portage-4.2.3-uat-1

Expected behaviour:

  1. Add rake task for openid_connect / CILogon cleanup Add rake task for openid_connect / CILogon cleanup #944
    Changed

  2. Email Confirmation Changes Email Confirmation Changes #923

  3. Disable Updating of User Emails Disable Updating of User Emails #917

  4. Apply translation:sync to yaml Files and Remove Unused locale/ + locales/ Files Apply translation:sync to yaml Files and Remove Unused locale/ + locales/ Files #937

  5. Fix User Lookup Via SSO Email: Make Query Case-Insensitive Fix User Lookup Via SSO Email: Make Query Case-Insensitive #924

  6. Fix Paginating, Sorting, and Searching Issues Within "Research Outputs" Tab Fix Paginating, Sorting, and Searching Issues Within "Research Outputs" Tab  #938

@aaronskiba
Copy link
Collaborator Author

1.

The rake task mentioned in 1. also applies to 2. Here is the output after executing bundle exec rake dmp_assistant_upgrade:v4_2_3:

root@004a4add6d45:/usr/src/app# bundle exec rake dmp_assistant_upgrade:v4_2_3
Copying Bootstrap glyphicons to the public directory ...
Copying TinyMCE skins to the public directory ...
"------------------------------------------------------------------------"
"Executing upgrade tasks for DMP Assistant 4.1.1+portage-4.2.3"
"Beginning task: Handle email confirmations for existing users"
"------------------------------------------------------------------------"
"Updating :confirmable columns to nil for all users"
"(i.e. Setting confirmed_at, confirmation_token, and confirmation_sent_at to nil for all users)"
"------------------------------------------------------------------------"
":confirmable columns updated to nil for 16224 users"
"------------------------------------------------------------------------"
"Updating superusers so that they are not required to confirm their email addresses"
"(i.e. Setting `confirmed_at = Time.now.utc` for superusers)"
"------------------------------------------------------------------------"
"Updated confirmed_at = 2024-11-18 22:49:57 UTC for 3 superuser(s)"
"Task completed: Handle email confirmations for existing users"
"Beginning task: Update `openid_connect` IdentifierScheme and its identifers"
"------------------------------------------------------------------------"
"Updating identifier_prefix to '' for openid_connect IdentifierScheme"
"------------------------------------------------------------------------"
"identifier_prefix updated from http://cilogon.org/serverE/users/ to '' for openid_connect IdentifierScheme"
"Updating prefixed value for identifiers with multiple occurences of 'cilogon'"
"------------------------------------------------------------------------"
"(Found 0 such identifiers)"
"Updated prefixed value from http://cilogon.org/serverE/users/ to '' for 0 identifiers"
"Task completed: Update `openid_connect` IdentifierScheme and its identifers"
"All tasks completed successfully"

The task appears to have executed successfully. Note that with respect to the UAT db, no openid_connect-related identifiers have the multiple occurences of 'cilogon'.

@aaronskiba
Copy link
Collaborator Author

aaronskiba commented Nov 18, 2024

2.

Upon first sign-in attempt, the flash message indicating an email was autosent is rendered:
Screenshot from 2024-11-18 16-24-25

The autosent email was in fact received:
Screenshot from 2024-11-18 16-24-42

A second sign-in attempt renders the flash message with the embedded link to the email confirmation page:
Screenshot from 2024-11-18 16-25-52

The following confirmation flash message was rendered after requesting new confirmation instructions on the email confirmation page:
Screenshot from 2024-11-18 16-26-20

The following confirmation flash message was rendered after clicking the link included in the confirmation instructions email:
Screenshot from 2024-11-18 16-26-41

NOTE:

@aaronskiba
Copy link
Collaborator Author

3.

Email field is disabled within /users/edit and super_admin/users/:user_id/edit pages:

Screenshot from 2024-11-19 11-37-36

Screenshot from 2024-11-19 11-38-06

NOTE: Using Chrome Dev Tools as an example, the email can still be modified via the "elements" tab. If a modification is performed and then a user clicks save, the following popup is rendered:
Screenshot from 2024-11-19 11-45-12

Even if the password is entered correctly and continue is clicked, the email IS NOT changed. However, it might be best to disable the potential activation of this popup.

@aaronskiba
Copy link
Collaborator Author

6.

View All renders all of the plan's research outputs:
Screenshot from 2024-11-19 11-55-40

Pagination option successfully lists research output on subsequent page:
Screenshot from 2024-11-19 11-55-56

Search box successfully filters by research outputs title:
Screenshot from 2024-11-19 11-56-05

Sorting option works (picture shows research outputs sorted in reverse order by title):
Screenshot from 2024-11-19 11-56-18

@aaronskiba
Copy link
Collaborator Author

4.

New yaml-related translations are rendering correctly:
Screenshot from 2024-11-20 12-33-35
Screenshot from 2024-11-20 12-30-49

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

No branches or pull requests

1 participant