add tests

Author: mbfreder

packages/core/src/test/shared/sam/build.test.ts

@@ -17,6 +17,7 @@ import {
     getBuildFlags,
     ParamsSource,
     runBuild,
+    SamBuildResult,
 } from '../../../shared/sam/build'
 import { TreeNode } from '../../../shared/treeview/resourceTreeDataProvider'
 import { createWizardTester } from '../wizards/wizardTestUtils'
@@ -32,6 +33,7 @@ import { samconfigCompleteData, validTemplateData } from './samTestUtils'
 import { CloudFormationTemplateRegistry } from '../../../shared/fs/templateRegistry'
 import { getTestWindow } from '../vscode/window'
 import { CancellationError } from '../../../shared/utilities/timeoutUtils'
+import { SamAppLocation } from '../../../awsService/appBuilder/explorer/samProject'
 
 describe('SAM BuildWizard', async function () {
     const createTester = async (params?: Partial<BuildParams>, arg?: TreeNode | undefined) =>
@@ -388,22 +390,8 @@ describe('SAM runBuild', () => {
             verifyCorrectDependencyCall()
         })
 
-        it('[entry: appbuilder node] with default flags should instantiate correct process in terminal', async () => {
-            const prompterTester = PrompterTester.init()
-                .handleQuickPick('Specify parameter source for build', async (quickPick) => {
-                    // Need sometime to wait for the template to search for template file
-                    await quickPick.untilReady()
-
-                    assert.strictEqual(quickPick.items.length, 2)
-                    const items = quickPick.items
-                    assert.strictEqual(quickPick.items.length, 2)
-                    assert.deepStrictEqual(items[0], { data: ParamsSource.Specify, label: 'Specify build flags' })
-                    assert.deepStrictEqual(items[1].label, 'Use default values')
-                    quickPick.acceptItem(quickPick.items[1])
-                })
-                .build()
-
-            // Invoke sync command from command palette
+        it('[entry: appbuilder node] with default flags should instantiate correct process in terminal and show progress notification', async () => {
+            const prompterTester = getPrompterTester()
             const expectedSamAppLocation = {
                 workspaceFolder: workspaceFolder,
                 samTemplateUri: templateFile,
@@ -412,6 +400,10 @@ describe('SAM runBuild', () => {
 
             await runBuild(new AppNode(expectedSamAppLocation))
 
+            getTestWindow()
+                .getFirstMessage()
+                .assertProgress(`Building SAM template at ${expectedSamAppLocation.samTemplateUri.path}`)
+
             assert.deepEqual(mockChildProcessClass.getCall(0).args, [
                 'sam-cli-path',
                 [
@@ -437,6 +429,27 @@ describe('SAM runBuild', () => {
             prompterTester.assertCallAll()
         })
 
+        it('[entry: appbuilder node] should throw an error when running two build processes in parallel for the same template', async () => {
+            const prompterTester = getPrompterTester()
+            const expectedSamAppLocation = {
+                workspaceFolder: workspaceFolder,
+                samTemplateUri: templateFile,
+                projectRoot: projectRoot,
+            }
+            await assert.rejects(
+                async () => {
+                    await runInParallel(expectedSamAppLocation)
+                },
+                (e: any) => {
+                    assert.strictEqual(e instanceof ToolkitError, true)
+                    assert.strictEqual(e.message, 'Template is already being built')
+                    assert.strictEqual(e.code, 'BuildInProgress')
+                    return true
+                }
+            )
+            prompterTester.assertCallAll(undefined, 2)
+        })
+
         it('[entry: command palette] use samconfig should instantiate correct process in terminal', async () => {
             const samconfigFile = vscode.Uri.file(await testFolder.write('samconfig.toml', samconfigCompleteData))
 
@@ -551,3 +564,23 @@ describe('SAM runBuild', () => {
         })
     })
 })
+
+async function runInParallel(samLocation: SamAppLocation): Promise<[SamBuildResult, SamBuildResult]> {
+    return Promise.all([runBuild(new AppNode(samLocation)), runBuild(new AppNode(samLocation))])
+}
+
+function getPrompterTester() {
+    return PrompterTester.init()
+        .handleQuickPick('Specify parameter source for build', async (quickPick) => {
+            // Need sometime to wait for the template to search for template file
+            await quickPick.untilReady()
+
+            assert.strictEqual(quickPick.items.length, 2)
+            const items = quickPick.items
+            assert.strictEqual(quickPick.items.length, 2)
+            assert.deepStrictEqual(items[0], { data: ParamsSource.Specify, label: 'Specify build flags' })
+            assert.deepStrictEqual(items[1].label, 'Use default values')
+            quickPick.acceptItem(quickPick.items[1])
+        })
+        .build()
+}

packages/core/src/testInteg/sam.test.ts

@@ -700,7 +700,7 @@ describe('SAM Integration Tests', async function () {
     describe('SAM Build', async function () {
         let workspaceFolder: vscode.WorkspaceFolder
         // We're testing only one case (python 3.10 (ZIP)) on the integration tests. More niche cases are handled as unit tests.
-        const scenarioIndex = 2
+        const scenarioIndex = 3
         const scenario = scenarios[scenarioIndex]
         let testRoot: string
         let sandbox: sinon.SinonSandbox