fix(copy-filter): add package manager lockfiles to default ignore (#1182

)

Co-authored-by: Eddy Brown <[email protected]>
Co-authored-by: Mark Lee <[email protected]>

.gitignore

@@ -7,6 +7,7 @@ package-lock.json
 yarn.lock
 test/fixtures/basic/main-link.js
 !test/fixtures/prune-is-module/node_modules
+!test/fixtures/ignore-package-lock/*
 test/work
 typedoc
 .DS_Store

src/copy-filter.js

@@ -8,6 +8,8 @@ const prune = require('./prune')
 const targets = require('./targets')
 
 const DEFAULT_IGNORES = [
+  '/package-lock\\.json$',
+  '/yarn\\.lock$',
   '/\\.git($|/)',
   '/node_modules/\\.bin($|/)',
   '\\.o(bj)?$'

test/copy-filter.js

@@ -78,6 +78,12 @@ test('generateIgnoredOutDirs ignores all possible platform/arch permutations', (
 
 test('ignore default: .o files', util.testSinglePlatform(ignoreTest, null, 'ignore.o'))
 test('ignore default: .obj files', util.testSinglePlatform(ignoreTest, null, 'ignore.obj'))
+test('ignore default: package manager lock files', util.testSinglePlatform(async (t, opts) => {
+  opts.dir = util.fixtureSubdir('ignore-package-lock')
+  const targetDir = await copyDirToTempDirWithIgnores(t, opts)
+  await assertFileIgnored(t, targetDir, 'package-lock.json')
+  await assertFileIgnored(t, targetDir, 'yarn.lock')
+}))
 test('ignore: string in array', util.testSinglePlatform(ignoreTest, ['ignorethis'], 'ignorethis.txt'))
 test('ignore: string', util.testSinglePlatform(ignoreTest, 'ignorethis', 'ignorethis.txt'))
 test('ignore: RegExp', util.testSinglePlatform(ignoreTest, /ignorethis/, 'ignorethis.txt'))