web
web copied to clipboard
feat(test-runner-commands): make snapshot file path configurable
What I did
- Added configuration option to control where snapshot files are saved.
I was not totally sure if the configuration of the plugin should happen somewhere other than TestRunnerConfig
, but could not figure a better way to configure it since the plugin seems to be added by default when the runner is started. If I understood correctly, if a user would add the snapshotPlugin(...)
in their own configuration file, that would not override the default one, right?
I've not added changesets or updated the docs yet, but can do that if the code changes otherwise look ok.
⚠️ No Changeset found
Latest commit: 9648e62c65d2076bb08e1cae3949f39d5dc27639
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
@koddsson @bashmish would you please comment on whether the behaviour described in #1521 is intended or, if not, what (apart from a changeset) would be needed to accept this pull request? Is the missing changeset the only reason this issue and pull request don't see any activity?
Recap of the situation: We currently have to commit an empty dist
directory with only a __snapshots__
subdirectory to our repository just because we're no longer able to configure the snapshot directory ever since moving away from karma.
Honestly it's been such a long time since I submitted this, that I'm a bit out of the loop regarding the codebase and have no idea how relevant the changes are anymore. The conflicts don't look too outlandish I suppose.
I've also since left the project where we were struggling with this issue, but can maybe give this a look at if this is deemed relevant in some way.