-
Notifications
You must be signed in to change notification settings - Fork 106
Split webdrivercss into multiple packages #113
Comments
I'm very in favor of splitting this. I haven't thought about the assertion package, but it's probably a good idea. |
WebdriverIO has a screenshot command that could be improved to allow selectors to be passed in: http://webdriver.io/api/utility/saveScreenshot.html |
@klamping it's a good idea but it would require additional requirements ( |
👍 to the general idea of splitting the repo into separate packages. I only want to use a few of the functions the complete package provides. |
@christian-bromann Could you add your thoughts regarding splitting webdrivercss into separate modules? Especially with regard to interoperability with webdriverio v4.0? I'm very interested in this project, but wouldn't want all the functionality it provides. We're currently using Wraith, but we're very limited by not being able to run real interactions with the page before taking screenshots. |
Webdrivercss currently does a lot of different things. It takes screenshots, calculates diffs, uploads to aplitools / api, has testing helpers for images within tolerances etc.
I believe these are all distinct functions that people might want to swap with other options and should be packages layered on top of webdrivercss.
Here is my proposal:
By splitting this up into multiple packages people can replace or swap the functionality that they want.
For example, I want to take screenshots, but I don't want to generate any diffs locally and I need to upload to a different server with different endpoints than webdrivercss-adminpanel. I've had to clear the image folder every time so that webdrivercss doesn't try to generate any diffs and write my own upload at the end.
The text was updated successfully, but these errors were encountered: