fix: Add some tests

codecov · Aug 3, 2021 · c7968b8 · c7968b8
1 parent fb89052
commit c7968b8
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/src/helpers/files.ts b/src/helpers/files.ts
@@ -314,12 +314,7 @@ export function readCoverageFile(
  * @returns boolean
  */
 export function fileExists(projectRoot: string, filePath: string): boolean {
-  try {
-    return fs.existsSync(getFilePath(projectRoot, filePath))
-  } catch (error) {
-    info(`File ${filePath} does not exist`)
-  }
-  return false
+  return fs.existsSync(getFilePath(projectRoot, filePath))
 }
 
 /**

diff --git a/test/helpers/files.test.ts b/test/helpers/files.test.ts
@@ -106,6 +106,17 @@ describe('File Helpers', () => {
         expect(fileHelpers.coverageFilePatterns()).toContain('jacoco*.xml')
       })
     })
+    describe('file exists', () => {
+      it('returns true if file exists', () => {
+        expect(fileHelpers.fileExists('', 'test/fixtures/coverage.txt'))
+      })
+      it('returns false if file doesnt exist', () => {
+        expect(fileHelpers.fileExists('', 'test/fake/path/to/file.txt'))
+      })
+      it('returns true if file exists in a directory', () => {
+        expect(fileHelpers.fileExists('test/fixtures', 'coverage.txt'))
+      })
+    })
     describe('getFilePath()', () => {
       it('should return path when file path has no starting slash', () => {
         expect(fileHelpers.getFilePath('/usr/', 'coverage.xml')).toEqual(