Skip to content

Commit 9974b8b

Browse files
author
Kaushik Shetty
committed
feat: add support for GCP regions
1 parent 5b24db5 commit 9974b8b

File tree

3 files changed

+8
-11
lines changed

3 files changed

+8
-11
lines changed

__test__/utils.test.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ describe("formatAppRegion", () => {
1818
expect(formatAppRegion("AZURE_EU")).toBe(Region.AZURE_EU);
1919
});
2020

21+
it('should return GCP_NA for "GCP_NA"', () => {
22+
expect(formatAppRegion("GCP_NA")).toBe(Region.GCP_NA);
23+
});
24+
2125
it("should return unknown for any invalid region", () => {
2226
expect(formatAppRegion("invalid")).toBe(Region.UNKNOWN);
2327
});

src/types.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -229,4 +229,5 @@ export enum Region {
229229
EU = "EU",
230230
AZURE_NA = "AZURE_NA",
231231
AZURE_EU = "AZURE_EU",
232+
GCP_NA = "GCP_NA",
232233
}

src/utils/utils.ts

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,18 +12,10 @@ export function onError(error: Error) {
1212
}
1313

1414
export function formatAppRegion(region: string): Region {
15-
switch (region) {
16-
case "NA":
17-
return Region.NA;
18-
case "EU":
19-
return Region.EU;
20-
case "AZURE_NA":
21-
return Region.AZURE_NA;
22-
case "AZURE_EU":
23-
return Region.AZURE_EU;
24-
default:
25-
return Region.UNKNOWN;
15+
if (region && Object.values(Region).includes(region as Region)) {
16+
return region as Region;
2617
}
18+
return Region.UNKNOWN;
2719
}
2820

2921
export function getPreferredBodyElement(nodeCollection: HTMLCollection) {

0 commit comments

Comments
 (0)