Skip to content

Commit 8f7fbdb

Browse files
vladshcherbinvladshcherbin
committed
Fix eslint errors
1 parent e2c42fc commit 8f7fbdb

File tree

4 files changed

+56
-60
lines changed

4 files changed

+56
-60
lines changed

tests/fixtures/src/assets/asset-1.js renamed to tests/fixtures/src/assets/asset-1.ts

File renamed without changes.

tests/fixtures/src/assets/asset-2.js renamed to tests/fixtures/src/assets/asset-2.ts

File renamed without changes.

tests/fixtures/src/index.js renamed to tests/fixtures/src/index.ts

File renamed without changes.

‎tests/index.test.ts‎

Lines changed: 56 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import type Options from '../src/types.ts'
1313

1414
async function build(options?: Options) {
1515
await rollup({
16-
input: 'src/index.js',
16+
input: 'src/index.ts',
1717
plugins: [
1818
copy(options)
1919
]
@@ -24,10 +24,6 @@ before(() => {
2424
process.chdir(`${import.meta.dirname}/fixtures`)
2525
})
2626

27-
// function readFile(filePath) {
28-
// return fs.readFile(filePath, 'utf-8')
29-
// }
30-
3127
afterEach(async () => {
3228
await remove('build')
3329
await remove('dist')
@@ -37,30 +33,30 @@ await describe('Copy', async () => {
3733
await test('No config passed', async () => {
3834
await build()
3935

40-
assert.strictEqual(await pathExists('dist/asset-1.js'), false)
36+
assert.strictEqual(await pathExists('dist/asset-1.ts'), false)
4137
})
4238

4339
await test('Empty array as target', async () => {
4440
await build({
4541
targets: []
4642
})
4743

48-
assert.strictEqual(await pathExists('dist/asset-1.js'), false)
44+
assert.strictEqual(await pathExists('dist/asset-1.ts'), false)
4945
})
5046

5147
await test('Files', async () => {
5248
await build({
5349
targets: [{
5450
dest: 'dist',
5551
src: [
56-
'src/assets/asset-1.js',
57-
'src/assets/asset-2.js'
52+
'src/assets/asset-1.ts',
53+
'src/assets/asset-2.ts'
5854
]
5955
}]
6056
})
6157

62-
assert.strictEqual(await pathExists('dist/asset-1.js'), true)
63-
assert.strictEqual(await pathExists('dist/asset-2.js'), true)
58+
assert.strictEqual(await pathExists('dist/asset-1.ts'), true)
59+
assert.strictEqual(await pathExists('dist/asset-2.ts'), true)
6460
})
6561

6662
await test('Folders', async () => {
@@ -89,16 +85,16 @@ await describe('Copy', async () => {
8985
targets: [{
9086
dest: 'dist',
9187
src: [
92-
'src/assets/asset-{1,2}.js',
88+
'src/assets/asset-{1,2}.ts',
9389
'src/assets/css/*.css',
9490
'!**/css-1.css',
9591
'src/assets/scss/scss-?(1).scss'
9692
]
9793
}]
9894
})
9995

100-
assert.strictEqual(await pathExists('dist/asset-1.js'), true)
101-
assert.strictEqual(await pathExists('dist/asset-2.js'), true)
96+
assert.strictEqual(await pathExists('dist/asset-1.ts'), true)
97+
assert.strictEqual(await pathExists('dist/asset-2.ts'), true)
10298
assert.strictEqual(await pathExists('dist/css-1.css'), false)
10399
assert.strictEqual(await pathExists('dist/css-2.css'), true)
104100
assert.strictEqual(await pathExists('dist/scss-1.scss'), true)
@@ -113,8 +109,8 @@ await describe('Copy', async () => {
113109
]
114110
})
115111

116-
assert.strictEqual(await pathExists('dist/asset-1.js'), true)
117-
assert.strictEqual(await pathExists('dist/asset-2.js'), true)
112+
assert.strictEqual(await pathExists('dist/asset-1.ts'), true)
113+
assert.strictEqual(await pathExists('dist/asset-2.ts'), true)
118114
assert.strictEqual(await pathExists('dist/css'), true)
119115
assert.strictEqual(await pathExists('dist/css/css-1.css'), true)
120116
assert.strictEqual(await pathExists('dist/css/css-2.css'), true)
@@ -127,19 +123,19 @@ await describe('Copy', async () => {
127123
targets: [{
128124
dest: ['dist', 'build'],
129125
src: [
130-
'src/assets/asset-1.js',
126+
'src/assets/asset-1.ts',
131127
'src/assets/css',
132128
'src/assets/scss/scss-?(1).scss'
133129
]
134130
}]
135131
})
136132

137-
assert.strictEqual(await pathExists('dist/asset-1.js'), true)
133+
assert.strictEqual(await pathExists('dist/asset-1.ts'), true)
138134
assert.strictEqual(await pathExists('dist/css'), true)
139135
assert.strictEqual(await pathExists('dist/css/css-1.css'), true)
140136
assert.strictEqual(await pathExists('dist/css/css-2.css'), true)
141137
assert.strictEqual(await pathExists('dist/scss-1.scss'), true)
142-
assert.strictEqual(await pathExists('build/asset-1.js'), true)
138+
assert.strictEqual(await pathExists('build/asset-1.ts'), true)
143139
assert.strictEqual(await pathExists('build/css'), true)
144140
assert.strictEqual(await pathExists('build/css/css-1.css'), true)
145141
assert.strictEqual(await pathExists('build/css/css-2.css'), true)
@@ -151,25 +147,25 @@ await describe('Copy', async () => {
151147
targets: [
152148
{ dest: 'dist', src: 'src/assets/css' },
153149
{ dest: 'dist', src: 'src/assets/css' },
154-
{ dest: 'build', src: ['src/assets/asset-1.js', 'src/assets/asset-1.js'] }
150+
{ dest: 'build', src: ['src/assets/asset-1.ts', 'src/assets/asset-1.ts'] }
155151
]
156152
})
157153

158154
assert.strictEqual(await pathExists('dist/css'), true)
159155
assert.strictEqual(await pathExists('dist/css/css-1.css'), true)
160156
assert.strictEqual(await pathExists('dist/css/css-2.css'), true)
161-
assert.strictEqual(await pathExists('build/asset-1.js'), true)
157+
assert.strictEqual(await pathExists('build/asset-1.ts'), true)
162158
})
163159

164160
await test('Throw if target is not an object', async () => {
165161
await assert.rejects(
166162
build({
167163
targets: [
168164
// @ts-expect-error
169-
'src/assets/asset-1.js'
165+
'src/assets/asset-1.ts'
170166
]
171167
}),
172-
{ message: '\'src/assets/asset-1.js\' target must be an object' }
168+
{ message: '\'src/assets/asset-1.ts\' target must be an object' }
173169
)
174170
})
175171

@@ -178,10 +174,10 @@ await describe('Copy', async () => {
178174
build({
179175
targets: [
180176
// @ts-expect-error
181-
{ src: 'src/assets/asset-1.js' }
177+
{ src: 'src/assets/asset-1.ts' }
182178
]
183179
}),
184-
{ message: '{ src: \'src/assets/asset-1.js\' } target must have "src" and "dest" properties' }
180+
{ message: '{ src: \'src/assets/asset-1.ts\' } target must have "src" and "dest" properties' }
185181
)
186182
})
187183

@@ -191,21 +187,21 @@ await describe('Copy', async () => {
191187
targets: [
192188
// @ts-expect-error
193189

194-
{ src: 'src/assets/asset-1.js', dest: 'dist', rename: [] }
190+
{ src: 'src/assets/asset-1.ts', dest: 'dist', rename: [] }
195191
]
196192
}),
197-
{ message: '{ src: \'src/assets/asset-1.js\', dest: \'dist\', rename: [] } target\'s "rename" property must be a string or a function' }
193+
{ message: '{ src: \'src/assets/asset-1.ts\', dest: \'dist\', rename: [] } target\'s "rename" property must be a string or a function' }
198194
)
199195
})
200196

201197
await test('Rename target', async () => {
202198
await build({
203199
targets: [
204-
{ src: 'src/assets/asset-1.js', dest: 'dist', rename: 'asset-1-renamed.js' },
200+
{ src: 'src/assets/asset-1.ts', dest: 'dist', rename: 'asset-1-renamed.ts' },
205201
{ src: 'src/assets/css', dest: 'dist', rename: 'css-renamed' },
206202
{ src: 'src/assets/css/*', dest: 'dist/css-multiple', rename: 'css-1.css' },
207203
{
208-
src: 'src/assets/asset-2.js',
204+
src: 'src/assets/asset-2.ts',
209205
dest: 'dist',
210206
rename: (name, extension) => `${name}-renamed.${extension}`
211207
},
@@ -224,20 +220,20 @@ await describe('Copy', async () => {
224220
)
225221
},
226222
{
227-
src: 'src/assets/asset-1.js',
223+
src: 'src/assets/asset-1.ts',
228224
dest: 'dist',
229-
rename: (_, __, fullPath) => basename(fullPath).replace(1, 3)
225+
rename: (name, extension, fullPath) => basename(fullPath).replace('1', '3')
230226
}
231227
]
232228
})
233229

234-
assert.strictEqual(await pathExists('dist/asset-1-renamed.js'), true)
230+
assert.strictEqual(await pathExists('dist/asset-1-renamed.ts'), true)
235231
assert.strictEqual(await pathExists('dist/css-renamed'), true)
236232
assert.strictEqual(await pathExists('dist/css-renamed/css-1.css'), true)
237233
assert.strictEqual(await pathExists('dist/css-renamed/css-2.css'), true)
238234
assert.strictEqual(await pathExists('dist/css-multiple/css-1.css'), true)
239235
assert.strictEqual(await pathExists('dist/css-multiple/css-2.css'), false)
240-
assert.strictEqual(await pathExists('dist/asset-2-renamed.js'), true)
236+
assert.strictEqual(await pathExists('dist/asset-2-renamed.ts'), true)
241237
assert.strictEqual(await pathExists('dist/scss-renamed'), true)
242238
assert.strictEqual(await pathExists('dist/scss-renamed/scss-1.scss'), true)
243239
assert.strictEqual(await pathExists('dist/scss-renamed/scss-2.scss'), true)
@@ -247,7 +243,7 @@ await describe('Copy', async () => {
247243
assert.strictEqual(await pathExists('dist/scss-multiple/scss-2-renamed.scss'), true)
248244
assert.strictEqual(await pathExists('dist/scss-multiple/nested-renamed'), true)
249245
assert.strictEqual(await pathExists('dist/scss-multiple/nested-renamed/scss-3.scss'), true)
250-
assert.strictEqual(await pathExists('dist/asset-3.js'), true)
246+
assert.strictEqual(await pathExists('dist/asset-3.ts'), true)
251247
})
252248

253249
await test('Throw if transform target is not a file', async () => {
@@ -317,7 +313,7 @@ await describe('Options', async () => {
317313
targets: [{
318314
dest: 'dist',
319315
src: [
320-
'src/assets/asset-1.js',
316+
'src/assets/asset-1.ts',
321317
'src/assets/css/*',
322318
'src/assets/scss',
323319
'src/not-exist'
@@ -328,7 +324,7 @@ await describe('Options', async () => {
328324

329325
assert.strictEqual(log.mock.callCount(), 5)
330326
assert.strictEqual(log.mock.calls[0]!.arguments[0], styleText('green', 'copied:'))
331-
assert.strictEqual(log.mock.calls[1]!.arguments[0], styleText('green', ` ${styleText('bold', 'src/assets/asset-1.js')} → ${styleText('bold', 'dist/asset-1.js')}`))
327+
assert.strictEqual(log.mock.calls[1]!.arguments[0], styleText('green', ` ${styleText('bold', 'src/assets/asset-1.ts')} → ${styleText('bold', 'dist/asset-1.ts')}`))
332328
assert.strictEqual(log.mock.calls[2]!.arguments[0], styleText('green', ` ${styleText('bold', 'src/assets/scss')} → ${styleText('bold', 'dist/scss')}`))
333329
assert.strictEqual(log.mock.calls[3]!.arguments[0], styleText('green', ` ${styleText('bold', 'src/assets/css/css-1.css')} → ${styleText('bold', 'dist/css-1.css')}`))
334330
assert.strictEqual(log.mock.calls[4]!.arguments[0], styleText('green', ` ${styleText('bold', 'src/assets/css/css-2.css')} → ${styleText('bold', 'dist/css-2.css')}`))
@@ -350,7 +346,7 @@ await describe('Options', async () => {
350346
await build({
351347
targets: [
352348

353-
{ src: 'src/assets/asset-1.js', dest: 'dist', rename: 'asset-1-renamed.js' },
349+
{ src: 'src/assets/asset-1.ts', dest: 'dist', rename: 'asset-1-renamed.ts' },
354350
{
355351
dest: 'dist/scss-multiple',
356352
rename: (name, extension) => (
@@ -366,7 +362,7 @@ await describe('Options', async () => {
366362

367363
assert.strictEqual(log.mock.callCount(), 5)
368364
assert.strictEqual(log.mock.calls[0]!.arguments[0], styleText('green', 'copied:'))
369-
assert.strictEqual(log.mock.calls[1]!.arguments[0], `${styleText('green', ` ${styleText('bold', 'src/assets/asset-1.js')} → ${styleText('bold', 'dist/asset-1-renamed.js')}`)} ${styleText('yellow', '[R]')}`)
365+
assert.strictEqual(log.mock.calls[1]!.arguments[0], `${styleText('green', ` ${styleText('bold', 'src/assets/asset-1.ts')} → ${styleText('bold', 'dist/asset-1-renamed.ts')}`)} ${styleText('yellow', '[R]')}`)
370366
assert.strictEqual(log.mock.calls[2]!.arguments[0], `${styleText('green', ` ${styleText('bold', 'src/assets/scss/nested')} → ${styleText('bold', 'dist/scss-multiple/nested-renamed')}`)} ${styleText('yellow', '[R]')}`)
371367
assert.strictEqual(log.mock.calls[3]!.arguments[0], `${styleText('green', ` ${styleText('bold', 'src/assets/scss/scss-1.scss')} → ${styleText('bold', 'dist/scss-multiple/scss-1-renamed.scss')}`)} ${styleText('yellow', '[R]')}`)
372368
assert.strictEqual(log.mock.calls[4]!.arguments[0], `${styleText('green', ` ${styleText('bold', 'src/assets/scss/scss-2.scss')} → ${styleText('bold', 'dist/scss-multiple/scss-2-renamed.scss')}`)} ${styleText('yellow', '[R]')}`)
@@ -393,27 +389,27 @@ await describe('Options', async () => {
393389
hook: 'buildStart',
394390
targets: [{
395391
dest: 'dist',
396-
src: ['src/assets/asset-1.js', 'src/assets/css']
392+
src: ['src/assets/asset-1.ts', 'src/assets/css']
397393
}]
398394
})
399395

400-
assert.strictEqual(await pathExists('dist/asset-1.js'), true)
396+
assert.strictEqual(await pathExists('dist/asset-1.ts'), true)
401397
assert.strictEqual(await pathExists('dist/css'), true)
402398
assert.strictEqual(await pathExists('dist/css/css-1.css'), true)
403399
assert.strictEqual(await pathExists('dist/css/css-2.css'), true)
404400
})
405401

406402
await test('Copy once', async () => {
407403
const watcher = watch({
408-
input: 'src/index.js',
404+
input: 'src/index.ts',
409405
output: {
410406
dir: 'build',
411407
format: 'esm'
412408
},
413409
plugins: [
414410
copy({
415411
targets: [
416-
{ src: 'src/assets/asset-1.js', dest: 'dist' }
412+
{ src: 'src/assets/asset-1.ts', dest: 'dist' }
417413
],
418414
copyOnce: true
419415
})
@@ -422,26 +418,26 @@ await describe('Options', async () => {
422418

423419
await setTimeout(500)
424420

425-
assert.strictEqual(await pathExists('dist/asset-1.js'), true)
421+
assert.strictEqual(await pathExists('dist/asset-1.ts'), true)
426422

427423
await remove('dist')
428424

429-
assert.strictEqual(await pathExists('dist/asset-1.js'), false)
425+
assert.strictEqual(await pathExists('dist/asset-1.ts'), false)
430426

431427
await replaceInFile({
432-
files: 'src/index.js',
428+
files: 'src/index.ts',
433429
from: 'hey',
434430
to: 'ho'
435431
})
436432

437433
await setTimeout(500)
438434

439-
assert.strictEqual(await pathExists('dist/asset-1.js'), false)
435+
assert.strictEqual(await pathExists('dist/asset-1.ts'), false)
440436

441437
await watcher.close()
442438

443439
await replaceInFile({
444-
files: 'src/index.js',
440+
files: 'src/index.ts',
445441
from: 'ho',
446442
to: 'hey'
447443
})
@@ -453,23 +449,23 @@ await describe('Options', async () => {
453449
targets: [{
454450
dest: 'dist',
455451
src: [
456-
'src/assets/asset-1.js',
457-
'src/assets/asset-2.js'
452+
'src/assets/asset-1.ts',
453+
'src/assets/asset-2.ts'
458454
]
459455
}]
460456
})
461457

462-
assert.strictEqual(await pathExists('dist/asset-1.js'), true)
463-
assert.strictEqual(await pathExists('dist/asset-2.js'), true)
458+
assert.strictEqual(await pathExists('dist/asset-1.ts'), true)
459+
assert.strictEqual(await pathExists('dist/asset-2.ts'), true)
464460
})
465461

466462
await test('Flatten', async () => {
467463
await build({
468464
flatten: false,
469465
targets: [{
470466
src: [
471-
'src/assets/asset-1.js',
472-
'src/assets/asset-2.js'
467+
'src/assets/asset-1.ts',
468+
'src/assets/asset-2.ts'
473469
],
474470
dest: 'dist'
475471
},
@@ -484,8 +480,8 @@ await describe('Options', async () => {
484480
}]
485481
})
486482

487-
assert.strictEqual(await pathExists('dist/assets/asset-1.js'), true)
488-
assert.strictEqual(await pathExists('dist/assets/asset-2.js'), true)
483+
assert.strictEqual(await pathExists('dist/assets/asset-1.ts'), true)
484+
assert.strictEqual(await pathExists('dist/assets/asset-2.ts'), true)
489485
assert.strictEqual(await pathExists('dist/assets/css/css-1.css'), true)
490486
assert.strictEqual(await pathExists('dist/assets/css/css-2.css'), true)
491487
assert.strictEqual(await pathExists('dist/assets/scss/scss-1-renamed.scss'), true)
@@ -495,22 +491,22 @@ await describe('Options', async () => {
495491

496492
await test('Rest options', async () => {
497493
await build({
498-
ignore: ['**/asset-1.js'],
494+
ignore: ['**/asset-1.ts'],
499495
targets: [
500-
{ src: 'src/assets/asset-1.js', dest: 'dist' }
496+
{ src: 'src/assets/asset-1.ts', dest: 'dist' }
501497
]
502498
})
503499

504-
assert.strictEqual(await pathExists('dist/asset-1.js'), false)
500+
assert.strictEqual(await pathExists('dist/asset-1.ts'), false)
505501
})
506502

507503
await test('Rest target options', async () => {
508504
await build({
509505
targets: [
510-
{ src: 'src/assets/asset-1.js', dest: 'dist', ignore: ['**/asset-1.js'] }
506+
{ src: 'src/assets/asset-1.ts', dest: 'dist', ignore: ['**/asset-1.ts'] }
511507
]
512508
})
513509

514-
assert.strictEqual(await pathExists('dist/asset-1.js'), false)
510+
assert.strictEqual(await pathExists('dist/asset-1.ts'), false)
515511
})
516512
})

0 commit comments

Comments
 (0)