demandforce/CXTestFramework — pull request #32 by jarterberry (2012-06-15 03:11:49+00:00)
Title: 3rd Review: Refactored recommended changes to code for test class and page object

Review:
1. Parameterized both the test class and page object and optimized use of methods, for dynamic data.
2. Removed 80% of XPath usage to benefit from CSS selectors. Some XPath remain, but are required and are commented. These few can be removed after a refactor, if any remote issues should occur.
3. All dynamic data was removed from the page object and only static data remains.
4. Moved static parameters to the top of test class for easy review of test approach and test methods.
5. The test class was updated to support a more robust use of the data provider. Now, three tests are consolidated into one.
6. The test class and page object were both reformated to proper specifications.