Skip to content

Commit f3a2eed

Browse files
RyanZimRyanZim
authored
Upgrade to standard v12 (#685)
1 parent fa661f3 commit f3a2eed

19 files changed

+47
-47
lines changed

‎lib/__tests__/promise.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ describe('promise support', () => {
2929
it('provides fse.promises API', () => {
3030
const desc = Object.getOwnPropertyDescriptor(fse, 'promises')
3131
assert.ok(desc)
32-
assert.equal(typeof desc.get, 'function')
32+
assert.strictEqual(typeof desc.get, 'function')
3333
})
3434
}
3535
})

‎lib/copy-sync/__tests__/broken-symlink.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ describe('copy-sync / broken symlink', () => {
2828
})
2929

3030
it('should throw an error when dereference=true', () => {
31-
assert.throws(() => copySync(src, out, {dereference: true}), err => err.code === 'ENOENT')
31+
assert.throws(() => copySync(src, out, { dereference: true }), err => err.code === 'ENOENT')
3232
})
3333
})
3434

‎lib/copy-sync/__tests__/copy-sync-file.test.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -136,15 +136,15 @@ describe('+ copySync() / file', () => {
136136
describe('> when destination file does NOT exist', () => {
137137
describe('> when overwrite is true', () => {
138138
it('should copy the file and not throw an error', () => {
139-
fs.copySync(src, dest, {overwrite: true})
139+
fs.copySync(src, dest, { overwrite: true })
140140
const destData = fs.readFileSync(dest, 'utf8')
141141
assert.strictEqual(srcData, destData)
142142
})
143143
})
144144

145145
describe('> when overwrite is false', () => {
146146
it('should copy the file and not throw an error', () => {
147-
fs.copySync(src, dest, {overwrite: false})
147+
fs.copySync(src, dest, { overwrite: false })
148148
const destData = fs.readFileSync(dest, 'utf8')
149149
assert.strictEqual(srcData, destData)
150150
})
@@ -161,22 +161,22 @@ describe('+ copySync() / file', () => {
161161

162162
describe('> when overwrite is true', () => {
163163
it('should copy the file and not throw an error', () => {
164-
fs.copySync(src, dest, {overwrite: true})
164+
fs.copySync(src, dest, { overwrite: true })
165165
destData = fs.readFileSync(dest, 'utf8')
166166
assert.strictEqual(srcData, destData)
167167
})
168168
})
169169

170170
describe('> when overwrite is false', () => {
171171
it('should not throw an error', () => {
172-
fs.copySync(src, dest, {overwrite: false})
172+
fs.copySync(src, dest, { overwrite: false })
173173

174174
// copy never happened
175175
const destDataNew = fs.readFileSync(dest, 'utf8')
176176
assert.strictEqual(destData, destDataNew)
177177
})
178178
it('should throw an error when errorOnExist is true', () => {
179-
assert.throws(() => fs.copySync(src, dest, {overwrite: false, errorOnExist: true}))
179+
assert.throws(() => fs.copySync(src, dest, { overwrite: false, errorOnExist: true }))
180180

181181
// copy never happened
182182
const destDataNew = fs.readFileSync(dest, 'utf8')
@@ -188,7 +188,7 @@ describe('+ copySync() / file', () => {
188188
it('should copy the file and not throw an error', () => {
189189
try {
190190
fs.chmodSync(dest, parseInt('444', 8))
191-
fs.copySync(src, dest, {overwrite: true})
191+
fs.copySync(src, dest, { overwrite: true })
192192
destData = fs.readFileSync(dest, 'utf8')
193193
assert.strictEqual(srcData, destData)
194194
} finally {
@@ -212,7 +212,7 @@ describe('+ copySync() / file', () => {
212212
})
213213

214214
it('is an alias for overwrite', () => {
215-
fs.copySync(src, dest, {clobber: false})
215+
fs.copySync(src, dest, { clobber: false })
216216

217217
// copy never happened
218218
const destDataNew = fs.readFileSync(dest, 'utf8')

‎lib/copy-sync/__tests__/copy-sync-preserve-timestamp.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ describeIfPractical('copySync() - preserveTimestamps option', () => {
3232

3333
describe('> when preserveTimestamps option is true', () => {
3434
it('should have the same timestamps on copy', () => {
35-
fs.copySync(SRC, DEST, {preserveTimestamps: true})
36-
FILES.forEach(testFile({preserveTimestamps: true}))
35+
fs.copySync(SRC, DEST, { preserveTimestamps: true })
36+
FILES.forEach(testFile({ preserveTimestamps: true }))
3737
})
3838
})
3939

‎lib/copy-sync/__tests__/symlink.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ describe('copy-sync / symlink', () => {
3535

3636
it('copies file contents when dereference=true', () => {
3737
try {
38-
copySync(src, out, {dereference: true})
38+
copySync(src, out, { dereference: true })
3939
} catch (err) {
4040
assert.ifError(err)
4141
}

‎lib/copy-sync/copy-sync.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const stat = require('../util/stat')
1010

1111
function copySync (src, dest, opts) {
1212
if (typeof opts === 'function') {
13-
opts = {filter: opts}
13+
opts = { filter: opts }
1414
}
1515

1616
opts = opts || {}

‎lib/copy/__tests__/copy-preserve-timestamp.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@ describeIfPractical('copy() - preserve timestamp', () => {
3333

3434
describe('> when timestamp option is true', () => {
3535
it('should have the same timestamps on copy', done => {
36-
copy(SRC, DEST, {preserveTimestamps: true}, () => {
37-
FILES.forEach(testFile({preserveTimestamps: true}))
36+
copy(SRC, DEST, { preserveTimestamps: true }, () => {
37+
FILES.forEach(testFile({ preserveTimestamps: true }))
3838
done()
3939
})
4040
})

‎lib/copy/__tests__/copy.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ describe('fs-extra', () => {
3838

3939
fse.ensureFileSync(src)
4040
fse.ensureFileSync(dest)
41-
fse.copy(src, dest, {overwrite: false, errorOnExist: true}, err => {
41+
fse.copy(src, dest, { overwrite: false, errorOnExist: true }, err => {
4242
assert(err)
4343
done()
4444
})
@@ -50,7 +50,7 @@ describe('fs-extra', () => {
5050

5151
fse.ensureFileSync(src)
5252
fse.ensureFileSync(dest)
53-
fse.copy(src, dest, {overwrite: false, errorOnExist: true}, err => {
53+
fse.copy(src, dest, { overwrite: false, errorOnExist: true }, err => {
5454
assert(err)
5555
done()
5656
})

‎lib/copy/__tests__/ncp/broken-symlink.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ describe('ncp broken symlink', () => {
3131
})
3232

3333
it('should return an error when dereference=true', done => {
34-
ncp(src, out, {dereference: true}, err => {
34+
ncp(src, out, { dereference: true }, err => {
3535
assert.strictEqual(err.code, 'ENOENT')
3636
done()
3737
})

‎lib/copy/__tests__/ncp/ncp.test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ describe('ncp', () => {
7272
})
7373

7474
it('the copy is complete after callback', done => {
75-
ncp(src, out, {overwrite: true}, err => {
75+
ncp(src, out, { overwrite: true }, err => {
7676
fs.createReadStream = () => done(new Error('createReadStream after callback'))
7777

7878
assert.ifError(err)
@@ -85,15 +85,15 @@ describe('ncp', () => {
8585
beforeEach(done => rimraf(out, done))
8686

8787
it('works', cb => {
88-
ncp(src, out, {overwrite: false}, err => {
88+
ncp(src, out, { overwrite: false }, err => {
8989
assert.ifError(err)
9090
cb()
9191
})
9292
})
9393

9494
it('should not error if files exist', cb => {
9595
ncp(src, out, () => {
96-
ncp(src, out, {overwrite: false}, err => {
96+
ncp(src, out, { overwrite: false }, err => {
9797
assert.ifError(err)
9898
cb()
9999
})

‎lib/copy/__tests__/ncp/symlink.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ describe('ncp / symlink', () => {
3535
})
3636

3737
it('copies file contents when dereference=true', done => {
38-
ncp(src, out, {dereference: true}, err => {
38+
ncp(src, out, { dereference: true }, err => {
3939
assert.ifError(err)
4040

4141
const fileSymlinkPath = path.join(out, 'file-symlink')

‎lib/copy/copy.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ function copy (src, dest, opts, cb) {
1414
cb = opts
1515
opts = {}
1616
} else if (typeof opts === 'function') {
17-
opts = {filter: opts}
17+
opts = { filter: opts }
1818
}
1919

2020
cb = cb || function () {}

‎lib/ensure/__tests__/symlink-paths.test.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -38,16 +38,16 @@ describe('symlink-type', () => {
3838
})
3939

4040
const tests = [
41-
[['foo.txt', 'symlink.txt'], {toCwd: 'foo.txt', toDst: 'foo.txt'}], // smart && nodestyle
42-
[['foo.txt', 'empty-dir/symlink.txt'], {toCwd: 'foo.txt', toDst: '../foo.txt'}], // smart
43-
[['../foo.txt', 'empty-dir/symlink.txt'], {toCwd: 'foo.txt', toDst: '../foo.txt'}], // nodestyle
44-
[['foo.txt', 'dir-bar/symlink.txt'], {toCwd: 'foo.txt', toDst: '../foo.txt'}], // smart
45-
[['../foo.txt', 'dir-bar/symlink.txt'], {toCwd: 'foo.txt', toDst: '../foo.txt'}], // nodestyle
41+
[['foo.txt', 'symlink.txt'], { toCwd: 'foo.txt', toDst: 'foo.txt' }], // smart && nodestyle
42+
[['foo.txt', 'empty-dir/symlink.txt'], { toCwd: 'foo.txt', toDst: '../foo.txt' }], // smart
43+
[['../foo.txt', 'empty-dir/symlink.txt'], { toCwd: 'foo.txt', toDst: '../foo.txt' }], // nodestyle
44+
[['foo.txt', 'dir-bar/symlink.txt'], { toCwd: 'foo.txt', toDst: '../foo.txt' }], // smart
45+
[['../foo.txt', 'dir-bar/symlink.txt'], { toCwd: 'foo.txt', toDst: '../foo.txt' }], // nodestyle
4646
// this is to preserve node's symlink capability these arguments say create
4747
// a link to 'dir-foo/foo.txt' this works because it exists this is unlike
4848
// the previous example with 'empty-dir' because 'empty-dir/foo.txt' does not exist.
49-
[['foo.txt', 'dir-foo/symlink.txt'], {toCwd: 'dir-foo/foo.txt', toDst: 'foo.txt'}], // nodestyle
50-
[['foo.txt', 'real-alpha/real-beta/real-gamma/symlink.txt'], {toCwd: 'foo.txt', toDst: '../../../foo.txt'}]
49+
[['foo.txt', 'dir-foo/symlink.txt'], { toCwd: 'dir-foo/foo.txt', toDst: 'foo.txt' }], // nodestyle
50+
[['foo.txt', 'real-alpha/real-beta/real-gamma/symlink.txt'], { toCwd: 'foo.txt', toDst: '../../../foo.txt' }]
5151
]
5252

5353
// formats paths to pass on multiple operating systems

‎lib/json/__tests__/output-json-sync.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ describe('json', () => {
2222
const file = path.join(TEST_DIR, 'this-dir', 'does-not', 'exist', 'file.json')
2323
assert(!fs.existsSync(file))
2424

25-
const data = {name: 'JP'}
25+
const data = { name: 'JP' }
2626
outputJsonSync(file, data)
2727

2828
assert(fs.existsSync(file))
@@ -37,7 +37,7 @@ describe('json', () => {
3737
assert(!fs.existsSync(file))
3838

3939
const replacer = (k, v) => v === 'JP' ? 'Jon Paul' : v
40-
const data = {name: 'JP'}
40+
const data = { name: 'JP' }
4141

4242
outputJsonSync(file, data, { replacer })
4343
const newData = JSON.parse(fs.readFileSync(file, 'utf8'))

‎lib/json/__tests__/output-json.test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ describe('json', () => {
2222
const file = path.join(TEST_DIR, 'this-dir', 'prob-does-not', 'exist', 'file.json')
2323
assert(!fs.existsSync(file))
2424

25-
const data = {name: 'JP'}
25+
const data = { name: 'JP' }
2626
outputJson(file, data, err => {
2727
if (err) return done(err)
2828

@@ -38,7 +38,7 @@ describe('json', () => {
3838
const file = path.join(TEST_DIR, 'this-dir', 'prob-does-not', 'exist', 'file.json')
3939
assert(!fs.existsSync(file))
4040

41-
const data = {name: 'JP'}
41+
const data = { name: 'JP' }
4242
return outputJson(file, data)
4343
})
4444

@@ -48,7 +48,7 @@ describe('json', () => {
4848
assert(!fs.existsSync(file))
4949

5050
const replacer = (k, v) => v === 'JP' ? 'Jon Paul' : v
51-
const data = {name: 'JP'}
51+
const data = { name: 'JP' }
5252

5353
outputJson(file, data, { replacer }, err => {
5454
assert.ifError(err)

‎lib/move-sync/__tests__/move-sync.test.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ describe('moveSync()', () => {
103103
assert(fs.existsSync(dest))
104104

105105
try {
106-
fse.moveSync(src, dest, {overwrite: false})
106+
fse.moveSync(src, dest, { overwrite: false })
107107
} catch (err) {
108108
assert.strictEqual(err.message, 'dest already exists.')
109109
}
@@ -116,7 +116,7 @@ describe('moveSync()', () => {
116116
// verify file exists already
117117
assert(fs.existsSync(dest))
118118

119-
fse.moveSync(src, dest, {overwrite: true})
119+
fse.moveSync(src, dest, { overwrite: true })
120120

121121
const contents = fs.readFileSync(dest, 'utf8')
122122
const expected = /^sonic the hedgehog\r?\n$/
@@ -137,7 +137,7 @@ describe('moveSync()', () => {
137137
assert(pathsBefore.indexOf('another-file') >= 0)
138138
assert(pathsBefore.indexOf('another-folder') >= 0)
139139

140-
fse.moveSync(src, dest, {overwrite: true})
140+
fse.moveSync(src, dest, { overwrite: true })
141141

142142
// verify dest does not have old stuff
143143
const pathsAfter = fs.readdirSync(dest)
@@ -198,7 +198,7 @@ describe('moveSync()', () => {
198198

199199
setUpMockFs('EXDEV')
200200

201-
fse.moveSync(src, dest, {overwrite: true})
201+
fse.moveSync(src, dest, { overwrite: true })
202202

203203
const contents = fs.readFileSync(dest + '/another-folder/file3', 'utf8')
204204
const expected = /^knuckles\r?\n$/
@@ -228,7 +228,7 @@ describe('moveSync()', () => {
228228
// verify file exists already
229229
assert(fs.existsSync(dest))
230230

231-
fse.moveSync(src, dest, {clobber: true})
231+
fse.moveSync(src, dest, { clobber: true })
232232

233233
const contents = fs.readFileSync(dest, 'utf8')
234234
const expected = /^sonic the hedgehog\r?\n$/

‎lib/move/__tests__/move.test.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ describe('+ move()', () => {
6060
// verify file exists already
6161
assert(fs.existsSync(dest))
6262

63-
fse.move(src, dest, {overwrite: true}, err => {
63+
fse.move(src, dest, { overwrite: true }, err => {
6464
assert.ifError(err)
6565
fs.readFile(dest, 'utf8', (err, contents) => {
6666
const expected = /^sonic the hedgehog\r?\n$/
@@ -85,7 +85,7 @@ describe('+ move()', () => {
8585
assert(paths.indexOf('another-file') >= 0)
8686
assert(paths.indexOf('another-folder') >= 0)
8787

88-
fse.move(src, dest, {overwrite: true}, err => {
88+
fse.move(src, dest, { overwrite: true }, err => {
8989
assert.ifError(err)
9090

9191
// verify dest does not have old stuff
@@ -109,7 +109,7 @@ describe('+ move()', () => {
109109

110110
setUpMockFs('EXDEV')
111111

112-
fse.move(src, dest, {overwrite: true}, err => {
112+
fse.move(src, dest, { overwrite: true }, err => {
113113
assert.ifError(err)
114114
assert.strictEqual(fs.rename.callCount, 1)
115115

@@ -190,7 +190,7 @@ describe('+ move()', () => {
190190
// verify file exists already
191191
assert(fs.existsSync(dest))
192192

193-
fse.move(src, dest, {overwrite: false}, err => {
193+
fse.move(src, dest, { overwrite: false }, err => {
194194
assert.strictEqual(err.message, 'dest already exists.')
195195
done()
196196
})
@@ -281,7 +281,7 @@ describe('+ move()', () => {
281281
// verify file exists already
282282
assert(fs.existsSync(dest))
283283

284-
fse.move(src, dest, {clobber: true}, err => {
284+
fse.move(src, dest, { clobber: true }, err => {
285285
assert.ifError(err)
286286
fs.readFile(dest, 'utf8', (err, contents) => {
287287
const expected = /^sonic the hedgehog\r?\n$/

‎lib/util/__tests__/utimes.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ describe('utimes', () => {
1919
fse.emptyDir(TEST_DIR, done)
2020
// reset stubs
2121
gracefulFsStub = {}
22-
utimes = proxyquire('../utimes', {'graceful-fs': gracefulFsStub})
22+
utimes = proxyquire('../utimes', { 'graceful-fs': gracefulFsStub })
2323
})
2424

2525
describe('hasMillisResSync()', () => {

‎package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
"proxyquire": "^2.0.1",
5151
"read-dir-files": "^0.1.1",
5252
"semver": "^5.3.0",
53-
"standard": "^11.0.1"
53+
"standard": "^12.0.1"
5454
},
5555
"main": "./lib/index.js",
5656
"scripts": {

0 commit comments

Comments
 (0)