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

B 20034 int #12649

Merged
merged 22 commits into from
May 15, 2024
Merged

B 20034 int #12649

merged 22 commits into from
May 15, 2024

Conversation

CoryKleinjanCACI
Copy link
Contributor

@CoryKleinjanCACI CoryKleinjanCACI commented May 11, 2024

20034

Review #1
Review #2

Summary

This PR is to fix some pre-test issues in the main review, that passed during the integration review(s). This review also includes a new UPDATE statement in the migrations. This is to automatically update the application_parameters table from supporting just validation_codes to supporting the new param_name and param_value structure. The INSERT for the validation_code values was initially done manually in testing and production environments.

How to test

Couple of setup things you need to do first:

  • make db_dev_migrate (this will create the application_parameters table
  • You'll need to fill in a row with a value in parameter_name, 'parameter_value' and id, updated_at and created_at will auto populate
  • In your .envrc, set export FEATURE_FLAG_VALIDATION_CODE_REQUIRED=true
  • Now fire up your client and server

To test db update of validation_code to param_name and param_value (this is meant to automatically update real environments to support the new table structure):

  1. add a new entry to your local DB application_parameters table with an ID column value of '30457d8e-b108-4173-a757-4acc17983904', and an entry into VALIDATION_CODE column with whatever value you want to use. I used test1234.

Sample SQL statement for insert:
INSERT INTO application_parameters(id, validation_code)
VALUES ('30457d8e-b108-4173-a757-4acc17983904', 'test1234');

  1. Run : update application_parameters set parameter_value = validation_code, parameter_name = 'validation_code' where id = '30457d8e-b108-4173-a757-4acc17983904'
  2. View the table to verify that the new columns param_name and param_value have been populated with param_name='validation_code' and param_value equal to the value you chose.

UI tests:
For the happiest path

  1. Access MM as a customer
  2. Login as a brand new customer
  3. You should now see the validation code screen
  4. Input the code you slapped in the application_parameters table
  5. It should now let you progress through the profile set up

For the saddest path

  1. Access MM as a customer
  2. Login as a brand new customer
  3. You should now see the validation code screen
  4. Input anything other than the code you put in the database
  5. You should see an error screen and not be able to progress
  6. You can have as many attempts as you want when inputting the code (this was a requirement from Kipp)

Screenshots

Screenshot 2024-04-12 at 11 07 01 AM

Mobile
Screenshot 2024-04-12 at 11 21 56 AM

Screenshot 2024-04-12 at 11 12 31 AM

Mobile
Screenshot 2024-04-12 at 11 21 38 AM

@CoryKleinjanCACI CoryKleinjanCACI requested a review from a team as a code owner May 11, 2024 04:20
@CoryKleinjanCACI CoryKleinjanCACI self-assigned this May 11, 2024
@CoryKleinjanCACI CoryKleinjanCACI added Scrummy Bears Scrum Team H INTEGRATION Slated for Integration Testing labels May 11, 2024
@ajlusk
Copy link
Contributor

ajlusk commented May 13, 2024

Functions as described both with correct and incorrect codes.

Testing instructions should probably include that the parameter_name needs to be validation_code.

One test is failing due to spectral issues. Will approve once that is passing.

Copy link
Contributor

@MInthavongsay MInthavongsay left a comment

Choose a reason for hiding this comment

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

It seems to work. One issue I have is that we had to manually add a record to the table. What happens when a developer decides to turn the FF to true like what we did for this PR? I don't see a specific migration to do the INSERT.

Copy link
Contributor

@ajlusk ajlusk left a comment

Choose a reason for hiding this comment

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

See a above comment.

@CoryKleinjanCACI CoryKleinjanCACI merged commit 854df2d into integrationTesting May 15, 2024
30 checks passed
@CoryKleinjanCACI CoryKleinjanCACI deleted the B-20034-INT branch May 15, 2024 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTEGRATION Slated for Integration Testing Scrummy Bears Scrum Team H
Development

Successfully merging this pull request may close these issues.

None yet

4 participants