fix: correct files entries array

Author: Ahmad Nassri

src/lib/files.js

@@ -10,7 +10,7 @@ import core from '@actions/core'
 
 export default async function (workspace, options) {
   // convert list of files to entries array
-  const filesEntries = options.files.map(item => typeof item === 'string' ? [[item], item] : Object.entries(item).pop())
+  const filesEntries = options.files.map(item => typeof item === 'string' ? [item, item] : Object.entries(item).pop())
 
   // convert entries array to an object
   const filesObject = Object.fromEntries(filesEntries)

src/test/config.js

@@ -42,7 +42,7 @@ test('config does not exist', assert => {
   sinon.stub(core, 'setFailed')
   sinon.stub(process, 'exit')
 
-  config({ workspace: __dirname, path: 'fixtures/invalid.yml' })
+  config({ workspace: __dirname, path: 'fixtures/configs/invalid.yml' })
 
   // debug
   assert.ok(core.setFailed.calledWith('failed to parse config'))
@@ -56,7 +56,7 @@ test('valid config', assert => {
   sinon.stub(core, 'setFailed')
   sinon.stub(process, 'exit')
 
-  const options = config({ workspace: __dirname, path: 'fixtures/valid.yml' })
+  const options = config({ workspace: __dirname, path: 'fixtures/configs/valid.yml' })
 
   // debug
   assert.ok(core.debug.calledWith(`config loaded: ${inspect(options)}`))

src/test/files.js

@@ -12,30 +12,43 @@ const __dirname = fileURLToPath(new URL('.', import.meta.url))
 sinon.stub(core, 'info')
 sinon.stub(core, 'debug')
 
-const workspace = join(__dirname, 'fixtures')
+const workspace = join(__dirname, 'fixtures/files')
 
-test('lists files', async assert => {
+test('lists all files', async assert => {
   assert.plan(3)
 
   const options = { files: [] }
 
   const contents = await files(workspace, options)
 
+  assert.same(core.info.lastCall.args, ['found 3 files available to sync'])
+  assert.same(core.debug.lastCall.args, [inspect(['file.1', 'file.2', 'file.3'])])
+
+  assert.equal(contents.size, 3)
+})
+
+test('lists specific files', async assert => {
+  assert.plan(3)
+
+  const options = { files: ['!file.3'] }
+
+  const contents = await files(workspace, options)
+
   assert.same(core.info.lastCall.args, ['found 2 files available to sync'])
-  assert.same(core.debug.lastCall.args, [inspect(['invalid.yml', 'valid.yml'])])
+  assert.same(core.debug.lastCall.args, [inspect(['file.1', 'file.2'])])
 
   assert.equal(contents.size, 2)
 })
 
 test('lists remapped files', async assert => {
   assert.plan(3)
 
-  const options = { files: [{ 'valid.yml': '/new/path/new.name' }] }
+  const options = { files: [{ 'file.1': '/new/path/new.name' }] }
 
   const contents = await files(workspace, options)
 
-  assert.same(core.info.lastCall.args, ['found 2 files available to sync'])
-  assert.same(core.debug.lastCall.args, [inspect(['/new/path/new.name', 'invalid.yml'])])
+  assert.same(core.info.lastCall.args, ['found 3 files available to sync'])
+  assert.same(core.debug.lastCall.args, [inspect(['/new/path/new.name', 'file.2', 'file.3'])])
 
-  assert.equal(contents.size, 2)
+  assert.equal(contents.size, 3)
 })