Use IS_TEST in fides-js build script #6385
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes ENG-1035
Description Of Changes
A prior PR added this env parameter into the Docker build, but the way the fides-js build script works, it's specifying a specific set of env parameters. So we need to explicitly pass it, rather than relying on process.env.
Code Changes
Steps to Confirm
npm run build
on fides-js to see the testids removed.export IS_TEST=true
npm run build
again; the testids should now be present. (For example, in the Manage Preferences button).Pre-Merge Checklist
CHANGELOG.md
updatedmain
downgrade()
migration is correct and works