Automated script review check points

Often we review the code,but will not be sure whether we are doing it right and doing it complete.So here is a checklist for automated script code review...

1. Automation coverage--Does the script automate each step of the test case?
2. Sequence--Does the script coded each step in the correct order as per the test case flow?
3. Validation--Does the script cover each and every validation as per manual test case?
4. Logic--Does the script logic call the correct functions? Are these functions called with the correct arguments?
5. Test data--Does the script take application input values from the correct input test data file(s)?
6. Object compatibility--Does the script execute user operations against the correct objects?
7. Synchronization--Does the script give the application sufficient time to transition to the correct test state e.g. the correct page/window/screen?
8. Output values--Does the script validate application output values/system generated data against the correct output test data file(s)?
9. Object validation--Does the script validate the state of the correct objects?
10. Object used--Does the script load only the necessary objects in memory?
11. Waits--Is the script free from unnecessary delays?
12. Error handling--Does the script handle application errors as designed?
13. Naming--Does the script have a unique identifier?
14. Unknown data--Is the script free of any magic numbers or magic strings?
15. Commenting--Is the script commented at each appropriate place in it? Is each comment correct?
16. Usage--Is the script free from unnecessary or extra code?
17. Reporting--Is the reporting correct? All the details are reported?
18. Documentation--All data like Developer,Date of scripting,Functions used, etc are
documented?

I think it helps...