Adds new test that verifies that subdirectories are ignored when list…

…ing files
Sleavely · May 4, 2020 · 1e0e9f7 · 1e0e9f7
1 parent aee9fd3
commit 1e0e9f7
Showing 1 changed file with 14 additions and 5 deletions.
diff --git a/src/utils/fs.spec.js b/src/utils/fs.spec.js
@@ -154,17 +154,26 @@ describe('listDirectoryFiles()', () => {
   it('lists files in the supplied directory', async () => {
     const { listDirectoryFiles } = jest.requireActual('./fs')
     const { Dirent, constants } = jest.requireActual('fs')
-
-    const { UV_DIRENT_DIR, UV_DIRENT_FILE } = constants
+    const { UV_DIRENT_FILE } = constants
 
     const file = new Dirent('file', UV_DIRENT_FILE)
-    const directory = new Dirent('directory', UV_DIRENT_DIR)
 
-    fs.readdir.mockResolvedValue([ file, directory ])
+    fs.readdir.mockResolvedValue([ file ])
 
     const files = await listDirectoryFiles()
 
     expect(files).toMatchObject([ file.name ])
   })
-  it.todo('???')
+  it('ignores subdirectories in the supplied directory', async () => {
+    const { listDirectoryFiles } = jest.requireActual('./fs')
+    const { Dirent, constants } = jest.requireActual('fs')
+    const { UV_DIRENT_DIR } = constants
+
+    const directory = new Dirent('directory', UV_DIRENT_DIR)
+    fs.readdir.mockResolvedValue([ directory ])
+
+    const files = await listDirectoryFiles()
+
+    expect(files).toMatchObject([])
+  })
 })