Skip to content

Commit 6a8ca11

Browse files
committed
test: avoid duplicate test data
1 parent 9543f86 commit 6a8ca11

File tree

1 file changed

+21
-29
lines changed

1 file changed

+21
-29
lines changed

packages/core/src/test/awsService/ec2/prompter.test.ts

Lines changed: 21 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import { Ec2InstanceNode } from '../../../awsService/ec2/explorer/ec2InstanceNod
1212
import { testClient, testInstance, testParentNode } from './explorer/ec2ParentNode.test'
1313
import { intoCollection } from '../../../shared/utilities/collectionUtils'
1414
import { AsyncCollection } from '../../../shared/utilities/asyncCollection'
15+
import { DataQuickPickItem } from '../../../shared/ui/pickerPrompter'
1516

1617
describe('Ec2Prompter', async function () {
1718
it('initializes properly', function () {
@@ -83,6 +84,24 @@ describe('Ec2Prompter', async function () {
8384
},
8485
],
8586
]
87+
88+
const defaultQuickPickItems: DataQuickPickItem<string>[] = [
89+
{
90+
label: Ec2Prompter.getLabel(defaultInstances[0][0]),
91+
detail: defaultInstances[0][0].InstanceId,
92+
data: defaultInstances[0][0].InstanceId,
93+
},
94+
{
95+
label: Ec2Prompter.getLabel(defaultInstances[0][1]),
96+
detail: defaultInstances[0][1].InstanceId,
97+
data: defaultInstances[0][1].InstanceId,
98+
},
99+
{
100+
label: Ec2Prompter.getLabel(defaultInstances[1][0]),
101+
detail: defaultInstances[1][0].InstanceId,
102+
data: defaultInstances[1][0].InstanceId,
103+
},
104+
]
86105
const defaultGetInstances: (regionCode: string) => AsyncCollection<PatchedEc2Instance[]> = (_) =>
87106
intoCollection(defaultInstances)
88107

@@ -98,23 +117,7 @@ describe('Ec2Prompter', async function () {
98117

99118
const itemsIterator = prompter.getInstancesAsQuickPickItems('test-region')
100119
const items = await extractItems(itemsIterator)
101-
assert.deepStrictEqual(items, [
102-
{
103-
label: Ec2Prompter.getLabel(defaultInstances[0][0]),
104-
detail: defaultInstances[0][0].InstanceId,
105-
data: defaultInstances[0][0].InstanceId,
106-
},
107-
{
108-
label: Ec2Prompter.getLabel(defaultInstances[0][1]),
109-
detail: defaultInstances[0][1].InstanceId,
110-
data: defaultInstances[0][1].InstanceId,
111-
},
112-
{
113-
label: Ec2Prompter.getLabel(defaultInstances[1][0]),
114-
detail: defaultInstances[1][0].InstanceId,
115-
data: defaultInstances[1][0].InstanceId,
116-
},
117-
])
120+
assert.deepStrictEqual(items, defaultQuickPickItems)
118121
})
119122

120123
it('applies filter when given', async function () {
@@ -126,18 +129,7 @@ describe('Ec2Prompter', async function () {
126129
const itemsIterator = prompter.getInstancesAsQuickPickItems('test-region')
127130
const items = await extractItems(itemsIterator)
128131

129-
assert.deepStrictEqual(items, [
130-
{
131-
label: Ec2Prompter.getLabel(defaultInstances[0][0]),
132-
detail: defaultInstances[0][0].InstanceId,
133-
data: defaultInstances[0][0].InstanceId,
134-
},
135-
{
136-
label: Ec2Prompter.getLabel(defaultInstances[1][0]),
137-
detail: defaultInstances[1][0].InstanceId,
138-
data: defaultInstances[1][0].InstanceId,
139-
},
140-
])
132+
assert.deepStrictEqual(items, [defaultQuickPickItems[0], defaultQuickPickItems[2]])
141133
})
142134
})
143135

0 commit comments

Comments
 (0)