arkinmodi / project-sayyara

Progressive Web Application for the Independent Automotive Repair Shop Industry
https://sayyara.vercel.app
MIT License
3 stars 2 forks source link

Verification and Validation Plan Peer Review #235

Closed arkinmodi closed 1 year ago

arkinmodi commented 1 year ago

DUE: November 4, 2022 + 2 Days of Grace Period

Team to Review: Team 7, 7eam Repository | Project Description

Details: Lecture 8 - Hazard Analysis (Slide 4)

arkinmodi commented 1 year ago
  1. Missing MG and MIS from Acronyms table
  2. Missing "Verification and Validation Plan Verification Plan" and "Reflections" section
  3. It is stated the both the SRS and Design plans will be verified "ad-hoc". This is not really a specific and clear plan.
  4. "Look and Feel T1" and other places mention the use of a "User". Who is this "user" and/or where do you plan to get them from.
  5. "Look and Feel T2", "Usability and Humanity T2" and "Performance T3" mention the use of a survey, but there is no survey attached in the document. This should be included to make the tests specific and clear.
  6. "API Auth T5" says "Output: Success response is returned (500)". A 500 is Internal Server Error. This is not the appropriate status response code for this scenario.
  7. Most, if not all, of your API tests state "Initial State: Not Applicable", however many of them depend on specific data being present in the database (i.e., the system being in a certain state).
arkinmodi commented 1 year ago

Peer Review Issues: