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

QA: manual validation of legacy v2 sms routes #163

Open
9 tasks
cris-oddball opened this issue Feb 10, 2025 · 0 comments
Open
9 tasks

QA: manual validation of legacy v2 sms routes #163

cris-oddball opened this issue Feb 10, 2025 · 0 comments
Assignees
Labels
Dev Reviewed Reviewed by Tech Lead Notify Board trigger PM Reviewed Reviewed by Product Manager QA Reviewed Reviewed by Quality Assurance QA Issue requires QA collaboration

Comments

@cris-oddball
Copy link
Contributor

cris-oddball commented Feb 10, 2025

User Story - Business Need

Manual validation of Pydantic models and route validation for the POST /legacy/v2/notifications/sms and GET /legacy/v2/notifications/:notification_idroutes was deferred because not everything is connected at the time of this writing to properly validate the responses. This ticket aims to manually validate that the routes return the same responses as the notification-api routes.

  • Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)

As a Developer working on ENP
I want to ensure that the ENP legacy sms routes return the same responses as the notification-api
So that I can ensure parity between the two services.

Additional Info and Resources

QA-245 and QA-246depend on the manual validation that happens in this ticket.

Also note that GET route needs to be under automation, which can either be folded into the above tickets or a new ticket created if needed.

Acceptance Criteria

  • POST /v2/notifications/sms request model is exactly the same as it is in the notification-api
  • POST /v2/notifications/sms response model is exactly the same as it is in the notification-api
  • POST /legacy/v2/notifications/sms request model is exactly the same as it is in the notification-api
  • POST /legacy/v2/notifications/sms response model is exactly the same as it is in the notification-api
  • This work is added to the sprint review slide deck (key win bullet point and demo slide)

QA Considerations

  • Determine what, if anything, needs to be changed in the QA regression to be able to support this work
  • Update the QA tickets linked above with the relevant information.
  • Create a GET legacy/v2/notifications/:notification_id ticket for QA automation if needed

Potential Dependencies

#117 added the Pydantic models for the legacy v2 sms routes
#112 added route validation for the legacy v2 sms routes
#114 connects ENP to the notification-api database

All three tickets need to be complete before any work can begin on this ticket.

Out of Scope

  • Automation of the above, which is covered in a different ticket.
@cris-oddball cris-oddball added Notify Board trigger QA Issue requires QA collaboration QA Reviewed Reviewed by Quality Assurance labels Feb 10, 2025
@cris-oddball cris-oddball self-assigned this Feb 10, 2025
@k-macmillan k-macmillan added the Dev Reviewed Reviewed by Tech Lead label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dev Reviewed Reviewed by Tech Lead Notify Board trigger PM Reviewed Reviewed by Product Manager QA Reviewed Reviewed by Quality Assurance QA Issue requires QA collaboration
Projects
None yet
Development

No branches or pull requests

4 participants