chore: remove unneeded otplease mocks

npm · Apr 4, 2022 · 31a0218 · 31a0218
1 parent e3da5df
commit 31a0218
Show file tree

Hide file tree

Showing 8 changed files with 0 additions and 25 deletions.
diff --git a/test/lib/auth/sso.js b/test/lib/auth/sso.js
@@ -29,16 +29,6 @@ const sso = t.mock('../../../lib/auth/sso.js', {
       )
     }
   },
-  '../../../lib/utils/otplease.js': (opts, fn) => {
-    if (opts) {
-      return fn({ ...opts, otp: '1234' })
-    } else {
-      throw Object.assign(
-        new Error('failed retrieving otp'),
-        { code: 'ERROR' }
-      )
-    }
-  },
 })
 
 const npm = {
@@ -75,7 +65,6 @@ t.test('simple login', async (t) => {
       opts,
       {
         creds: {},
-        otp: '1234',
         registry: 'https://registry.npmjs.org/',
         scope: '',
         ssoType: 'oauth',

diff --git a/test/lib/commands/deprecate.js b/test/lib/commands/deprecate.js
@@ -19,7 +19,6 @@ npmFetch.json = async (uri, opts) => {
 
 const Deprecate = t.mock('../../../lib/commands/deprecate.js', {
   '../../../lib/utils/get-identity.js': async () => getIdentityImpl(),
-  '../../../lib/utils/otplease.js': async (opts, fn) => fn(opts),
   libnpmaccess: {
     lsPackages: async () => ({ foo: 'write', bar: 'write', baz: 'write', buzz: 'read' }),
   },

diff --git a/test/lib/commands/hook.js b/test/lib/commands/hook.js
@@ -63,7 +63,6 @@ const libnpmhook = {
 }
 
 const Hook = t.mock('../../../lib/commands/hook.js', {
-  '../../../lib/utils/otplease.js': async (opts, fn) => fn(opts),
   libnpmhook,
 })
 const hook = new Hook(npm)

diff --git a/test/lib/commands/org.js b/test/lib/commands/org.js
@@ -43,7 +43,6 @@ const libnpmorg = {
 }
 
 const Org = t.mock('../../../lib/commands/org.js', {
-  '../../../lib/utils/otplease.js': async (opts, fn) => fn(opts),
   libnpmorg,
 })
 const org = new Org(npm)

diff --git a/test/lib/commands/owner.js b/test/lib/commands/owner.js
@@ -21,7 +21,6 @@ const mocks = {
   'proc-log': log,
   'npm-registry-fetch': npmFetch,
   pacote,
-  '../../../lib/utils/otplease.js': async (opts, fn) => fn({ otp: '123456', opts }),
   '../../../lib/utils/read-package-name.js': async (prefix) => {
     readPackageNamePrefix = prefix
     return readPackageNameResponse
@@ -193,7 +192,6 @@ t.test('owner add <user> <pkg>', async t => {
           _rev: '1-foobaaa1',
           maintainers: npmcliMaintainers,
         },
-        otp: '123456',
         spec: {
           name: '@npmcli/map-workspaces',
         },
@@ -507,7 +505,6 @@ t.test('owner rm <user> <pkg>', async t => {
         body: {
           _rev: '1-foobaaa1',
         },
-        otp: '123456',
         spec: {
           name: '@npmcli/map-workspaces',
         },

diff --git a/test/lib/commands/profile.js b/test/lib/commands/profile.js
@@ -41,10 +41,6 @@ const mocks = {
         .join('\n')
     }
   },
-  '../../../lib/utils/pulse-till-done.js': {
-    withPromise: async a => a,
-  },
-  '../../../lib/utils/otplease.js': async (opts, fn) => fn(opts),
   '../../../lib/utils/read-user-info.js': {
     async password () {},
     async otp () {},

diff --git a/test/lib/commands/team.js b/test/lib/commands/team.js
@@ -22,7 +22,6 @@ const npm = mockNpm({
 const mocks = {
   libnpmteam,
   'cli-columns': a => a.join(' '),
-  '../../../lib/utils/otplease.js': async (opts, fn) => fn(opts),
 }
 
 t.afterEach(() => {

diff --git a/test/lib/commands/token.js b/test/lib/commands/token.js
@@ -10,9 +10,6 @@ const npm = {
 }
 
 const mockToken = (otherMocks) => t.mock('../../../lib/commands/token.js', {
-  '../../../lib/utils/otplease.js': (opts, fn) => {
-    return Promise.resolve().then(() => fn(opts))
-  },
   '../../../lib/utils/read-user-info.js': mocks.readUserInfo,
   'npm-profile': mocks.profile,
   ...otherMocks,