Skip to content

Commit 9c070de

Browse files
committed
refractor prompters and tests
1 parent 46b966c commit 9c070de

27 files changed

+617
-235
lines changed

package-lock.json

Lines changed: 10 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/core/src/awsService/appBuilder/explorer/openTemplate.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { CloudFormationTemplateRegistry } from '../../../shared/fs/templateRegis
77
import { syncMementoRootKey } from '../../../shared/sam/sync'
88

99
import { createExitPrompter } from '../../../shared/ui/common/exitPrompter'
10-
import { createTemplatePrompter, TemplateItem } from '../../../shared/ui/common/samTemplate'
10+
import { createTemplatePrompter, TemplateItem } from '../../../shared/ui/sam/samTemplate'
1111
import { Wizard } from '../../../shared/wizards/wizard'
1212

1313
export interface OpenTemplateParams {

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
*/
55

66
import * as vscode from 'vscode'
7-
import { TemplateItem, createTemplatePrompter } from '../ui/common/samTemplate'
7+
import { TemplateItem, createTemplatePrompter } from '../ui/sam/samTemplate'
88
import { ChildProcess } from '../utilities/processUtils'
99
import { addTelemetryEnvVar } from './cli/samCliInvokerUtils'
1010
import { Wizard } from '../wizards/wizard'
@@ -59,7 +59,7 @@ export function createParamsSourcePrompter(existValidSamconfig: boolean) {
5959
)
6060

6161
return createQuickPick(items, {
62-
title: 'Specify parameters for build',
62+
title: 'Specify parameter source for build',
6363
placeholder: 'Select configuration options for sam build',
6464
buttons: createCommonButtons(samBuildUrl),
6565
})

packages/core/src/shared/sam/deploy.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,11 @@ import { CancellationError } from '../utilities/timeoutUtils'
2222
import { Wizard } from '../wizards/wizard'
2323
import { addTelemetryEnvVar } from './cli/samCliInvokerUtils'
2424
import { validateSamDeployConfig, SamConfig, writeSamconfigGlobal } from './config'
25-
import { BucketSource, createBucketSourcePrompter, createBucketPrompter } from '../ui/common/bucket'
26-
import { createStackPrompter } from '../ui/common/stack'
27-
import { TemplateItem, createTemplatePrompter } from '../ui/common/samTemplate'
25+
import { BucketSource, createBucketSourcePrompter, createBucketPrompter } from '../ui/sam/bucket'
26+
import { createStackPrompter } from '../ui/sam/stack'
27+
import { TemplateItem, createTemplatePrompter } from '../ui/sam/samTemplate'
2828
import { getProjectRoot, getRecentResponse, getSamCliPathAndVersion, getSource, updateRecentResponse } from './utils'
29-
import { createDeployParamsSourcePrompter, ParamsSource } from '../ui/common/paramsSource'
29+
import { createDeployParamsSourcePrompter, ParamsSource } from '../ui/sam/paramsSource'
3030
import { runInTerminal } from './processTerminal'
3131

3232
export interface DeployParams {

packages/core/src/shared/sam/process.ts

Lines changed: 0 additions & 93 deletions
This file was deleted.

packages/core/src/shared/sam/sync.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,11 @@ import {
4545
getSource,
4646
updateRecentResponse,
4747
} from './utils'
48-
import { TemplateItem, createTemplatePrompter } from '../ui/common/samTemplate'
49-
import { createStackPrompter } from '../ui/common/stack'
50-
import { ParamsSource, createSyncParamsSourcePrompter } from '../ui/common/paramsSource'
51-
import { createEcrPrompter } from '../ui/common/ecr'
52-
import { BucketSource, createBucketPrompter } from '../ui/common/bucket'
48+
import { TemplateItem, createTemplatePrompter } from '../ui/sam/samTemplate'
49+
import { createStackPrompter } from '../ui/sam/stack'
50+
import { ParamsSource, createSyncParamsSourcePrompter } from '../ui/sam/paramsSource'
51+
import { createEcrPrompter } from '../ui/sam/ecr'
52+
import { BucketSource, createBucketPrompter } from '../ui/sam/bucket'
5353
import { runInTerminal } from './processTerminal'
5454

5555
export interface SyncParams {
@@ -67,8 +67,6 @@ export interface SyncParams {
6767
readonly syncFlags?: string
6868
}
6969

70-
export const prefixNewBucketName = (name: string) => `newbucket:${name}`
71-
export const prefixNewRepoName = (name: string) => `newrepo:${name}`
7270
export const syncMementoRootKey = 'samcli.sync.params'
7371

7472
// TODO: hook this up so it prompts the user when more than 1 environment is present in `samconfig.toml`

packages/core/src/shared/sam/utils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import * as path from 'path'
88
import { AWSTreeNodeBase } from '../treeview/nodes/awsTreeNodeBase'
99
import { TreeNode, isTreeNode } from '../treeview/resourceTreeDataProvider'
1010
import * as CloudFormation from '../cloudformation/cloudformation'
11-
import { TemplateItem } from '../ui/common/samTemplate'
11+
import { TemplateItem } from '../ui/sam/samTemplate'
1212
import { RuntimeFamily, getFamily } from '../../lambda/models/samLambdaRuntime'
1313
import { SamCliSettings } from './cli/samCliSettings'
1414
import { ToolkitError } from '../errors'

packages/core/src/shared/ui/common/bucket.ts renamed to packages/core/src/shared/ui/sam/bucket.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,12 @@ import { DefaultS3Client } from '../../clients/s3Client'
66
import { samDeployUrl, samSyncUrl } from '../../constants'
77
import { createCommonButtons } from '../buttons'
88
import { createQuickPick, DataQuickPickItem } from '../pickerPrompter'
9-
import { SyncParams, prefixNewBucketName } from '../../sam/sync'
9+
import type { SyncParams } from '../../sam/sync'
1010
import * as nls from 'vscode-nls'
1111
import { getRecentResponse } from '../../sam/utils'
1212

1313
const localize = nls.loadMessageBundle()
14+
export const prefixNewBucketName = (name: string) => `newbucket:${name}`
1415

1516
export enum BucketSource {
1617
SamCliManaged,

packages/core/src/shared/ui/common/ecr.ts renamed to packages/core/src/shared/ui/sam/ecr.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,12 @@ import { DefaultEcrClient } from '../../clients/ecrClient'
77
import { samSyncUrl } from '../../constants'
88
import { createCommonButtons } from '../buttons'
99
import { createQuickPick } from '../pickerPrompter'
10-
import { prefixNewRepoName } from '../../sam/sync'
10+
1111
import * as nls from 'vscode-nls'
1212
import { getRecentResponse } from '../../sam/utils'
1313

1414
export const localize = nls.loadMessageBundle()
15+
export const prefixNewRepoName = (name: string) => `newrepo:${name}`
1516

1617
export function createEcrPrompter(client: DefaultEcrClient, mementoRootKey: string) {
1718
const recentEcrRepo = getRecentResponse(mementoRootKey, client.regionCode, 'ecrRepoUri')

packages/core/src/shared/ui/common/paramsSource.ts renamed to packages/core/src/shared/ui/sam/paramsSource.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ export function createDeployParamsSourcePrompter(existValidSamconfig: boolean |
3838
const items = loadParamsSourcePrompterItems(existValidSamconfig)
3939

4040
return createQuickPick(items, {
41-
title: 'Specify parameters for deploy',
41+
title: 'Specify parameter source for deploy',
4242
placeholder: 'Press enter to proceed with highlighted option',
4343
buttons: createCommonButtons(samDeployUrl),
4444
})
@@ -48,7 +48,7 @@ export function createSyncParamsSourcePrompter(existValidSamconfig: boolean | un
4848
const items = loadParamsSourcePrompterItems(existValidSamconfig)
4949

5050
return createQuickPick(items, {
51-
title: 'Specify parameters for sync',
51+
title: 'Specify parameter source for sync',
5252
placeholder: 'Press enter to proceed with highlighted option',
5353
buttons: createCommonButtons(samSyncUrl),
5454
})

0 commit comments

Comments
 (0)