Skip to content

Commit 1da1f29

Browse files
committed
fix: add vertex us and eu multi-region options
1 parent 04738ea commit 1da1f29

2 files changed

Lines changed: 17 additions & 37 deletions

File tree

packages/types/src/providers/vertex.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -599,6 +599,8 @@ export const VERTEX_1M_CONTEXT_MODEL_IDS = [
599599

600600
export const VERTEX_REGIONS = [
601601
{ value: "global", label: "global" },
602+
{ value: "us", label: "us" },
603+
{ value: "eu", label: "eu" },
602604
{ value: "us-central1", label: "us-central1" },
603605
{ value: "us-east1", label: "us-east1" },
604606
{ value: "us-east4", label: "us-east4" },

webview-ui/src/components/settings/providers/__tests__/Vertex.spec.tsx

Lines changed: 15 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -63,43 +63,21 @@ describe("Vertex", () => {
6363
expect(globalRegions).toHaveLength(1)
6464
})
6565

66-
it('should contain all expected regions including "global"', () => {
67-
// The expected list is the imported VERTEX_REGIONS itself
68-
expect(VERTEX_REGIONS).toEqual([
69-
{ value: "global", label: "global" },
70-
{ value: "us-central1", label: "us-central1" },
71-
{ value: "us-east1", label: "us-east1" },
72-
{ value: "us-east4", label: "us-east4" },
73-
{ value: "us-east5", label: "us-east5" },
74-
{ value: "us-south1", label: "us-south1" },
75-
{ value: "us-west1", label: "us-west1" },
76-
{ value: "us-west2", label: "us-west2" },
77-
{ value: "us-west3", label: "us-west3" },
78-
{ value: "us-west4", label: "us-west4" },
79-
{ value: "northamerica-northeast1", label: "northamerica-northeast1" },
80-
{ value: "northamerica-northeast2", label: "northamerica-northeast2" },
81-
{ value: "southamerica-east1", label: "southamerica-east1" },
82-
{ value: "europe-west1", label: "europe-west1" },
83-
{ value: "europe-west2", label: "europe-west2" },
84-
{ value: "europe-west3", label: "europe-west3" },
85-
{ value: "europe-west4", label: "europe-west4" },
86-
{ value: "europe-west6", label: "europe-west6" },
87-
{ value: "europe-central2", label: "europe-central2" },
88-
{ value: "asia-east1", label: "asia-east1" },
89-
{ value: "asia-east2", label: "asia-east2" },
90-
{ value: "asia-northeast1", label: "asia-northeast1" },
91-
{ value: "asia-northeast2", label: "asia-northeast2" },
92-
{ value: "asia-northeast3", label: "asia-northeast3" },
93-
{ value: "asia-south1", label: "asia-south1" },
94-
{ value: "asia-south2", label: "asia-south2" },
95-
{ value: "asia-southeast1", label: "asia-southeast1" },
96-
{ value: "asia-southeast2", label: "asia-southeast2" },
97-
{ value: "australia-southeast1", label: "australia-southeast1" },
98-
{ value: "australia-southeast2", label: "australia-southeast2" },
99-
{ value: "me-west1", label: "me-west1" },
100-
{ value: "me-central1", label: "me-central1" },
101-
{ value: "africa-south1", label: "africa-south1" },
102-
])
66+
it('should contain the "us" multi-region exactly once', () => {
67+
const usRegions = VERTEX_REGIONS.filter((r: { value: string; label: string }) => r.value === "us")
68+
expect(usRegions).toHaveLength(1)
69+
expect(usRegions[0]).toEqual({ value: "us", label: "us" })
70+
})
71+
72+
it('should contain the "eu" multi-region exactly once', () => {
73+
const euRegions = VERTEX_REGIONS.filter((r: { value: string; label: string }) => r.value === "eu")
74+
expect(euRegions).toHaveLength(1)
75+
expect(euRegions[0]).toEqual({ value: "eu", label: "eu" })
76+
})
77+
78+
it("should preserve existing regional endpoints", () => {
79+
expect(VERTEX_REGIONS).toContainEqual({ value: "us-east5", label: "us-east5" })
80+
expect(VERTEX_REGIONS).toContainEqual({ value: "europe-west1", label: "europe-west1" })
10381
})
10482

10583
it('should contain "asia-east1" region exactly once', () => {

0 commit comments

Comments
 (0)