A code review is a quality assurance activity to ensure that check-ins
are reviewed by someone other than the author. If not pair-programming
when writing the code, mature development shops often practice this
activity as it’s an excellent way to catch errors early in the process.
| Automation Code Review Check List | ||
| #No | Item | Comments |
| 1 | Duplication | To avoid code redundancy and to save the development time |
| 2 | Exceptions Handling | All exceptions handled properly, even unexpected ones. |
| 3 | Framework Hacks | Test automation projects need a strong overall design strategy to make use of framework desired feature. |
| 4 | Bad Config Changes | configuration changes can have huge impacts |
| 5 | Poor Code | Organized structure is necessary for project scalability and teamwork |
| 6 | Documentation | Documentation is vital for good testing and good maintenance. When a test fails, the doc it provides significantly assist triage |
| 7 | Test Coverage | Keep the original test procedure handy, and watch out for missing coverage. |
| 8 | Hard-Coded Values | Hard-coded values often indicate hasty development |
| 9 | Typos and Bad Formatting | Typos and bad formatting reflect carelessness, cause frustration, and damage reputation. |
| 10 | Proof of Success | Tests need to run successfully in order to pass review, and proof of success (such as a log or a screen shot) must be attached to the review. |
No comments:
Post a Comment