Support env.UPDATE_SNAPSHOTS=1 (#139)

l2beat · Nov 18, 2021 · d02b7d6 · d02b7d6
1 parent f06b941
commit d02b7d6
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/.changeset/wet-buttons-teach.md b/.changeset/wet-buttons-teach.md
@@ -0,0 +1,5 @@
+---
+'earljs': patch
+---
+
+process.env.UPDATE_SNAPSHOTS set to "1" is now treated the same as when set to "true"
diff --git a/packages/earljs/src/validators/snapshots/helpers.ts b/packages/earljs/src/validators/snapshots/helpers.ts
@@ -8,7 +8,7 @@ export type Env = typeof process.env
 
 export type ShouldUpdateSnapshots = () => boolean
 export function shouldUpdateSnapshots(env: Env): boolean {
-  return env.UPDATE_SNAPSHOTS === 'true'
+  return env.UPDATE_SNAPSHOTS === 'true' || env.UPDATE_SNAPSHOTS === '1'
 }
 
 const SNAPSHOTS_DIR = '__snapshots__'

diff --git a/packages/earljs/test/validators/snapshots/helpers.test.ts b/packages/earljs/test/validators/snapshots/helpers.test.ts
@@ -43,6 +43,7 @@ describe('getUpdateSnapshotMode', () => {
 
   it('should update all snapshots when requested', () => {
     expect(getUpdateSnapshotMode({ UPDATE_SNAPSHOTS: 'true' })).to.eq('all')
+    expect(getUpdateSnapshotMode({ UPDATE_SNAPSHOTS: '1' })).to.eq('all')
   })
 
   it('should update only new snapshots by default', () => {