Why did we create 2 takeScreenshot() methods for the project. Don't you think the takeScreenshot() in BasePage is not necessary because it is not part of the pages and can be called from the test classes when needed.
Instructor
Ashish Thakur Replied on 23/05/2019
You can also use One. That's not a issue.
One more question. Don't you think having just one Base class would have been better than having 2 different base classes (1 for pages and 1 for tests). This way we could share common features between pages and tests. For eg, we could call takeScreenshot method in pages as well as test classes without having to take a long approach. Let me know your thoughts
Instructor
Ashish Thakur Replied on 28/05/2019
Complexity would increase this way. If some changes have to be done. they may cause issues.