Merge pull request #239 from codecov/fix-uuid-token

fix: Validate UUID tokens

Author: thomasrockhu

src/helpers/validate.js

@@ -6,7 +6,7 @@ const validator = require('validator')
  * @returns boolean
  */
 function validateToken(token) {
-  return validator.isAlphanumeric(token)
+  return validator.isAlphanumeric(token) || validator.isUUID(token)
 }
 
 function validateURL(url) {

test/helpers/validate.test.js

@@ -5,9 +5,13 @@ const realEnv = { ...process.env }
 
 describe('Input Validators', () => {
   describe('Tokens', () => {
-    it('Returns true with a valid token', () => {
+    it('Returns true with a valid alphanumeric token', () => {
       expect(validate.validateToken('1bc123')).toBe(true)
     })
+    it('Returns true with a valid UUID token', () => {
+      // Use a randomly generated UUIDv4
+      expect(validate.validateToken('5becd1a9-efa8-4bd8-8f94-e9f8613820c3')).toBe(true)
+    })
     it('Returns false with an invalid token', () => {
       expect(validate.validateToken('1bc1 23')).toBe(false)
     })

test/index.test.js

@@ -4,6 +4,9 @@ const { version } = require('../package.json')
 const nock = require('nock')
 const fs = require('fs')
 
+// Backup the env
+const realEnv = { ...process.env }
+
 describe('Uploader Core', () => {
   const env = process.env
 
@@ -68,16 +71,26 @@ describe('Uploader Core', () => {
     expect(log).toHaveBeenCalledWith(expect.stringMatching(/<<<<<< ENV/))
   })
 
-  it('Can upload without token', async () => {
-    jest.spyOn(process, 'exit').mockImplementation(() => {})
-    const log = jest.spyOn(console, 'log').mockImplementation(() => {})
-    await app.main({
-      name: 'customname',
-      url: 'https://codecov.io',
-      dryRun: true,
-      env: 'SOMETHING,ANOTHER',
+  describe('Token', () => {
+    beforeEach(() => {
+      delete process.env.CODECOV_TOKEN
+    })
+
+    afterEach(() => {
+      process.env = realEnv
+    })
+
+    it('Can upload without token', async () => {
+      jest.spyOn(process, 'exit').mockImplementation(() => {})
+      const log = jest.spyOn(console, 'log').mockImplementation(() => {})
+      await app.main({
+        name: 'customname',
+        url: 'https://codecov.io',
+        dryRun: true,
+        env: 'SOMETHING,ANOTHER',
+      })
+      expect(log).toHaveBeenCalledWith(expect.stringMatching('-> No token specified or token is empty'))
     })
-    expect(log).toHaveBeenCalledWith(expect.stringMatching('-> No token specified or token is empty'))
   })
 
   describe('Flags', () => {