Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Running unit tests replaces existing $/.bedrock/config.yaml file #53

Open
andrebriggs opened this issue May 19, 2020 · 0 comments
Open

Running unit tests replaces existing $/.bedrock/config.yaml file #53

andrebriggs opened this issue May 19, 2020 · 0 comments
Assignees
@andrebriggs
Labels
bug Something isn't working

Comments

@andrebriggs
Copy link
Member

As a Bedrock developer I am using and developing bedrock. After running yarn test on source code I find that my $/.bedrock/config.yaml file is replaced with this mock file

The offending test appears to config.test.ts which calls saveConfiguration to the default location ($/.bedrock/config.yaml) not a temp directory. See here

The expectations that "production" configuration doesn't get overwritten by tests. We should write to a temp directory.
@andrebriggs andrebriggs added the bug Something isn't working label May 19, 2020
@andrebriggs andrebriggs self-assigned this May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@andrebriggs andrebriggs

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andrebriggs