chore(cli): properly spell authentication in code (#78)

lirantal · Aug 8, 2020 · 9a1eb28 · 9a1eb28
1 parent 6db4833
commit 9a1eb28
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 16 deletions.
diff --git a/__tests__/Utils.test.js b/__tests__/Utils.test.js
@@ -84,37 +84,37 @@ describe('Utils', () => {
     expect(Utils.hasDevice(argv)).toEqual(true)
   })
 
-  test('hasAutentication & parseAutentication with none flag', async () => {
+  test('hasAuthentication & parseAuthentication with no flag', async () => {
     const argv = {}
-    expect(Utils.parseAutentication(argv)).toEqual({})
-    expect(Utils.hasAutentication(argv)).toEqual(false)
+    expect(Utils.parseAuthentication(argv)).toEqual({})
+    expect(Utils.hasAuthentication(argv)).toEqual(false)
   })
 
-  test('hasAutentication & parseAutentication with cookie flag', async () => {
+  test('hasAuthentication & parseAuthentication with cookie flag', async () => {
     const argv = { cookie: 'This is the COOKIE content' }
-    expect(Utils.parseAutentication(argv)).toEqual({
+    expect(Utils.parseAuthentication(argv)).toEqual({
       Cookie: 'This is the COOKIE content'
     })
-    expect(Utils.hasAutentication(argv)).toEqual(true)
+    expect(Utils.hasAuthentication(argv)).toEqual(true)
   })
 
-  test('hasAutentication & parseAutentication with token flag', async () => {
+  test('hasAuthentication & parseAuthentication with token flag', async () => {
     const argv = { token: 'This is the TOKEN content' }
-    expect(Utils.parseAutentication(argv)).toEqual({
+    expect(Utils.parseAuthentication(argv)).toEqual({
       Authorization: 'Bearer This is the TOKEN content'
     })
-    expect(Utils.hasAutentication(argv)).toEqual(true)
+    expect(Utils.hasAuthentication(argv)).toEqual(true)
   })
 
-  test('hasAutentication & parseAutentication with token and cookie flags', async () => {
+  test('hasAuthentication & parseAuthentication with token and cookie flags', async () => {
     const argv = {
       cookie: 'This is the COOKIE content',
       token: 'This is the TOKEN content'
     }
-    expect(Utils.parseAutentication(argv)).toEqual({
+    expect(Utils.parseAuthentication(argv)).toEqual({
       Cookie: 'This is the COOKIE content',
       Authorization: 'Bearer This is the TOKEN content'
     })
-    expect(Utils.hasAutentication(argv)).toEqual(true)
+    expect(Utils.hasAuthentication(argv)).toEqual(true)
   })
 })
diff --git a/bin/is-website-vulnerable.js b/bin/is-website-vulnerable.js
@@ -23,8 +23,8 @@ function getLighthouseOptions() {
     lighthouseOpts.emulatedFormFactor = Utils.parseDevice(argv)
   }
 
-  if (Utils.hasAutentication(argv)) {
-    lighthouseOpts.extraHeaders = Utils.parseAutentication(argv)
+  if (Utils.hasAuthentication(argv)) {
+    lighthouseOpts.extraHeaders = Utils.parseAuthentication(argv)
   }
 
   const { chromePath } = argv

diff --git a/src/Utils.js b/src/Utils.js
@@ -43,7 +43,7 @@ module.exports = {
   hasDevice: function(argv) {
     return argv.mobile || argv.desktop || argv.none || false
   },
-  parseAutentication: function(argv) {
+  parseAuthentication: function(argv) {
     const extraHeaders = {}
     if (argv.cookie) {
       extraHeaders.Cookie = argv.cookie
@@ -55,7 +55,7 @@ module.exports = {
 
     return extraHeaders
   },
-  hasAutentication: function(argv) {
+  hasAuthentication: function(argv) {
     return ['cookie', 'token'].some(prop => Object.hasOwnProperty.call(argv, prop))
   }
 }