fix: move checkSlug fn to validate (#990)

Author: thomasrockhu-codecov

src/helpers/checkSlug.ts

@@ -46,3 +46,10 @@ export function validateSHA(
     commitSHA.length === requestedLength && validator.isAlphanumeric(commitSHA)
   )
 }
+
+export function checkSlug(slug: string): boolean {
+  if (slug !== '' && !slug.match(/\//)) {
+    return false
+  }
+  return true
+}

src/helpers/validate.ts

@@ -28,8 +28,7 @@ import { generateGcovCoverageFiles } from './helpers/gcov'
 import { generateSwiftCoverageFiles } from './helpers/swift'
 import { generateXcodeCoverageFiles } from './helpers/xcode'
 import { argAsArray } from './helpers/util'
-import { checkSlug } from './helpers/checkSlug'
-import { validateFlags } from './helpers/validate'
+import { checkSlug, validateFlags } from './helpers/validate'
 
 /**
  *

src/index.ts

@@ -1,9 +1,4 @@
 import * as validate from '../../src/helpers/validate'
-import { DEFAULT_UPLOAD_HOST } from '../../src/helpers/constansts'
-import Module from 'module'
-
-// Backup the env
-const realEnv = { ...process.env }
 
 describe('Input Validators', () => {
   describe('Tokens', () => {
@@ -76,4 +71,30 @@ describe('Input Validators', () => {
       ).toBe(true)
     })
   })
+
+  describe('checkSlug()', () => {
+      it('should return true for a slug with a forward slash', () => {
+          // arrange
+          const inputSlug = "testOrg/testRepo"
+          const expectedResult = true
+
+          // act
+          const result = validate.checkSlug(inputSlug)
+
+          // assert
+          expect(result).toEqual(expectedResult)
+      })
+
+      it('should return false for a slug without a forward slash', () => {
+          // arrange
+          const inputSlug = 'testRepo'
+          const expectedResult = false
+
+          // act
+          const result = validate.checkSlug(inputSlug)
+
+          // assert
+          expect(result).toEqual(expectedResult)
+      })
+  })
 })