Merge pull request #239 from adobe/esm-open-fix

Fix require() of ES Module error

Author: riddhi2910

src/commands/console/open.js

@@ -11,7 +11,6 @@ governing permissions and limitations under the License.
 const aioLogger = require('@adobe/aio-lib-core-logging')('@adobe/aio-cli-plugin-console:open', { provider: 'debug' })
 const { getCliEnv } = require('@adobe/aio-lib-env')
 const { OPEN_URLS } = require('../../config')
-const open = require('open')
 const ConsoleCommand = require('./index')
 class OpenCommand extends ConsoleCommand {
   async run () {
@@ -32,6 +31,8 @@ class OpenCommand extends ConsoleCommand {
       }
     }
     aioLogger.debug(`opening url ${url}`)
+    // eslint-disable-next-line node/no-unsupported-features/es-syntax
+    const { default: open } = await import('open')
     open(url)
   }
 }

test/commands/console/open.test.js

@@ -13,9 +13,11 @@ const TestCommand = require('../../../src/commands/console/open')
 const config = require('@adobe/aio-lib-core-config')
 const { STAGE_ENV } = require('@adobe/aio-lib-env')
 
-jest.mock('open', () => jest.fn())
+const mockOpen = jest.fn()
+jest.unstable_mockModule('open', () => ({
+  default: mockOpen
+}))
 const { Command } = require('@oclif/core')
-const open = require('open')
 
 let command
 let ORIGINAL_AIO_CLI_ENV
@@ -24,6 +26,7 @@ beforeAll(() => {
 })
 beforeEach(() => {
   config.get.mockReset()
+  mockOpen.mockReset()
   command = new TestCommand([])
 })
 afterEach(() => {
@@ -56,19 +59,19 @@ describe('console:open', () => {
 
   test('should open a browser', async () => {
     await expect(command.run()).resolves.not.toThrow()
-    expect(open).toHaveBeenCalledWith('https://developer.adobe.com/console/projects')
+    expect(mockOpen).toHaveBeenCalledWith('https://developer.adobe.com/console/projects')
   })
 
   test('should open a browser (stage_env)', async () => {
     process.env.AIO_CLI_ENV = STAGE_ENV
     await expect(command.run()).resolves.not.toThrow()
-    expect(open).toHaveBeenCalledWith('https://developer-stage.adobe.com/console/projects')
+    expect(mockOpen).toHaveBeenCalledWith('https://developer-stage.adobe.com/console/projects')
   })
 
   test('should open a browser with default view if no project/workspace selected', async () => {
     config.get.mockReturnValue(null)
     await expect(command.run()).resolves.not.toThrow()
-    expect(open).toHaveBeenLastCalledWith('https://developer.adobe.com/console/projects')
+    expect(mockOpen).toHaveBeenLastCalledWith('https://developer.adobe.com/console/projects')
   })
 
   test('should open a browser with project overview', async () => {
@@ -82,7 +85,7 @@ describe('console:open', () => {
       }
     })
     await expect(command.run()).resolves.not.toThrow()
-    expect(open).toHaveBeenLastCalledWith('https://developer.adobe.com/console/projects/53444/4566206088344853970/overview')
+    expect(mockOpen).toHaveBeenLastCalledWith('https://developer.adobe.com/console/projects/53444/4566206088344853970/overview')
   })
 
   test('should open a browser with project workspace', async () => {
@@ -97,6 +100,6 @@ describe('console:open', () => {
       workspace: { id: '4566206088344859372', name: 'Stage' }
     })
     await expect(command.run()).resolves.not.toThrow()
-    expect(open).toHaveBeenLastCalledWith('https://developer.adobe.com/console/projects/53444/4566206088344853970/workspaces/4566206088344859372/details')
+    expect(mockOpen).toHaveBeenLastCalledWith('https://developer.adobe.com/console/projects/53444/4566206088344853970/workspaces/4566206088344859372/details')
   })
 })