Only run import-map within install if its a theme repo

Author: t-kelly

src/commands/theme/component/install.ts

@@ -10,6 +10,7 @@
 import Args from '../../../utilities/args.js'
 import BaseCommand from '../../../utilities/base-command.js'
 import Flags from '../../../utilities/flags.js'
+import {isThemeRepo} from '../../../utilities/validate.js'
 import GenerateImportMap from '../generate/import-map.js'
 import Clean from './clean.js'
 import Copy from './copy.js'
@@ -42,6 +43,9 @@ export default class Install extends BaseCommand {
     await Manifest.run([this.args[Args.DEST_DIR]!])
     await Copy.run([this.args[Args.DEST_DIR]!])
     await Clean.run([this.args[Args.DEST_DIR]!])
-    await GenerateImportMap.run([this.args[Args.DEST_DIR]!, '--quiet'])
+
+    if (isThemeRepo(this.args[Args.DEST_DIR])) {
+      await GenerateImportMap.run([this.args[Args.DEST_DIR]!, '--quiet'])
+    }
   }
 }

test/commands/theme/component/install.test.ts

@@ -15,6 +15,7 @@ const fixturesPath = path.join(__dirname, '../../../fixtures')
 const collectionPath = path.join(__dirname, '../../../fixtures/collection')
 const themePath = path.join(__dirname, '../../../fixtures/theme')
 const testCollectionPath = path.join(fixturesPath, 'test-collection')
+const testCollectionBPath = path.join(fixturesPath, 'test-collection-b')
 const testThemePath = path.join(fixturesPath, 'test-theme')
 
 describe('theme component install', () => {
@@ -32,13 +33,15 @@ describe('theme component install', () => {
     generateRunStub = sandbox.stub(GenerateImportMap.prototype, 'run').resolves()
 
     fs.cpSync(collectionPath, testCollectionPath, { recursive: true })
+    fs.cpSync(collectionPath, testCollectionBPath, { recursive: true })
     fs.cpSync(themePath, testThemePath, { recursive: true })
     process.chdir(testCollectionPath)
   })
 
   afterEach(() => {
     sandbox.restore()
     fs.rmSync(testCollectionPath, { force: true, recursive: true })
+    fs.rmSync(testCollectionBPath, { force: true, recursive: true })
     fs.rmSync(testThemePath, { force: true, recursive: true })
   })
 
@@ -57,11 +60,16 @@ describe('theme component install', () => {
     expect(cleanRunStub.calledOnce).to.be.true
   })
 
-  it('runs the theme component generate import map command', async () => {
+  it('runs the theme component generate import map command if the destination is a theme repo', async () => {
     await Install.run([testThemePath])
     expect(generateRunStub.calledOnce).to.be.true
   })
 
+  it('does not run the theme component generate import map command if the destination is not a theme repo', async () => {
+    await Install.run([testCollectionBPath])
+    expect(generateRunStub.calledOnce).to.be.false
+  })
+
   it('runs sub-commands in correct order', async () => {
     await Install.run([testThemePath])
     sinon.assert.callOrder(manifestRunStub, copyRunStub, cleanRunStub, generateRunStub)