Skip to content

Commit 6786c9c

Browse files
committed
Update var name
1 parent 8b8d9fa commit 6786c9c

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

test/ingress/tryConfigureIngressUsingDockerfile.test.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import * as path from "path";
99
import { IngressContext, PortRange, getDockerfileExposePorts, tryConfigureIngressUsingDockerfile } from "../../extension.bundle";
1010
import type { MockIngressContext } from "./MockIngressContext";
1111

12-
const dockerfileSamplePorts: PortRange[][] = [
12+
const expectedSamplePorts: PortRange[][] = [
1313
[new PortRange(443), new PortRange(80)],
1414
[new PortRange(80), new PortRange(443)],
1515
[new PortRange(80), new PortRange(8080, 8090)],
@@ -25,12 +25,12 @@ suite('tryConfigureIngressUsingDockerfile', async () => {
2525
const dockerfileSamples = await AzExtFsExtra.readDirectory(dockerfileSamplesPath);
2626

2727
const expectedResult: MockIngressContext[] = [
28-
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: dockerfileSamplePorts[0], targetPort: 443 },
28+
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: expectedSamplePorts[0], targetPort: 443 },
2929
{ enableIngress: undefined, enableExternal: undefined, dockerfileExposePorts: undefined, targetPort: undefined }, // no dockerfilePath
30-
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: dockerfileSamplePorts[2], targetPort: 80 },
31-
{ enableIngress: undefined, enableExternal: undefined, dockerfileExposePorts: dockerfileSamplePorts[3], targetPort: undefined }, // alwaysPromptIngress
32-
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: dockerfileSamplePorts[4], targetPort: 443 },
33-
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: dockerfileSamplePorts[5], targetPort: 80 },
30+
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: expectedSamplePorts[2], targetPort: 80 },
31+
{ enableIngress: undefined, enableExternal: undefined, dockerfileExposePorts: expectedSamplePorts[3], targetPort: undefined }, // alwaysPromptIngress
32+
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: expectedSamplePorts[4], targetPort: 443 },
33+
{ enableIngress: true, enableExternal: true, dockerfileExposePorts: expectedSamplePorts[5], targetPort: 80 },
3434
{ enableIngress: false, enableExternal: false, dockerfileExposePorts: undefined, targetPort: undefined }, // no expose
3535
];
3636

@@ -64,8 +64,8 @@ suite('tryConfigureIngressUsingDockerfile', async () => {
6464
const portRange: PortRange[] = await getDockerfileExposePorts(ds.fsPath) ?? [];
6565

6666
for (const [j, pr] of portRange.entries()) {
67-
assert.equal(pr.start, dockerfileSamplePorts[i][j].start);
68-
assert.equal(pr.end, dockerfileSamplePorts[i][j].end);
67+
assert.equal(pr.start, expectedSamplePorts[i][j].start);
68+
assert.equal(pr.end, expectedSamplePorts[i][j].end);
6969
}
7070
}
7171
});

0 commit comments

Comments
 (0)