Skip to content
This repository was archived by the owner on Dec 24, 2023. It is now read-only.

Conversation

@rundmd
Copy link
Contributor

@rundmd rundmd commented Jun 26, 2019

  • allow baselineFolder to be a string or function that is passed an
    options object
  • allow screenshotPath to be a string or function that is passed an
    options object

All of our tests aren't located in the same folder. The tests are bundled inside packages along with the source code for each of our components. Our use case is saving the screenshots under the same folder as our tests.

@wswebcreation
Copy link
Owner

Hi @rundmd

Thanks for this PR, can you provide some examples in the docs on how this can be used?

@rundmd rundmd force-pushed the set-baselinefolder-with-function-or-string branch 2 times, most recently from 8be67ea to 8bd7132 Compare June 26, 2019 18:05
- allow baselineFolder to be a string or function that is passed an
options object
- allow screenshotPath to be a string or function that is passed an
options object
@rundmd rundmd force-pushed the set-baselinefolder-with-function-or-string branch from 8bd7132 to f4b2c92 Compare June 26, 2019 23:37
@wswebcreation
Copy link
Owner

This looks great, tnx!

I'm going to merge it in a few days because I'm also busy with a fix so I can release them together

@wswebcreation wswebcreation self-requested a review June 29, 2019 11:20
Copy link
Owner

@wswebcreation wswebcreation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@rundmd
Copy link
Contributor Author

rundmd commented Jun 29, 2019

sounds good @wswebcreation

@wswebcreation wswebcreation merged commit 3febd27 into wswebcreation:master Jun 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants