Skip to content

Commit 67c224f

Browse files
committed
Disable staging and bypass proxy tests on CI to reduce utilization
1 parent 5401d4a commit 67c224f

File tree

2 files changed

+53
-19
lines changed

2 files changed

+53
-19
lines changed

FirebaseAI/Tests/TestApp/Tests/Integration/GenerateContentIntegrationTests.swift

Lines changed: 21 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -50,15 +50,18 @@ struct GenerateContentIntegrationTests {
5050
@Test(arguments: [
5151
(InstanceConfig.vertexAI_v1beta, ModelNames.gemini2FlashLite),
5252
(InstanceConfig.vertexAI_v1beta_global, ModelNames.gemini2FlashLite),
53-
(InstanceConfig.vertexAI_v1beta_staging, ModelNames.gemini2FlashLite),
5453
(InstanceConfig.vertexAI_v1beta_global_appCheckLimitedUse, ModelNames.gemini2FlashLite),
5554
(InstanceConfig.googleAI_v1beta, ModelNames.gemini2FlashLite),
5655
(InstanceConfig.googleAI_v1beta_appCheckLimitedUse, ModelNames.gemini2FlashLite),
5756
(InstanceConfig.googleAI_v1beta, ModelNames.gemma3_4B),
58-
(InstanceConfig.googleAI_v1beta_staging, ModelNames.gemini2FlashLite),
59-
(InstanceConfig.googleAI_v1beta_staging, ModelNames.gemma3_4B),
60-
(InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemini2FlashLite),
61-
(InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemma3_4B),
57+
(InstanceConfig.googleAI_v1beta_freeTier, ModelNames.gemini2FlashLite),
58+
(InstanceConfig.googleAI_v1beta_freeTier, ModelNames.gemma3_4B),
59+
// Note: The following configs are commented out for easy one-off manual testing.
60+
// (InstanceConfig.googleAI_v1beta_staging, ModelNames.gemini2FlashLite),
61+
// (InstanceConfig.googleAI_v1beta_staging, ModelNames.gemma3_4B),
62+
// (InstanceConfig.vertexAI_v1beta_staging, ModelNames.gemini2FlashLite),
63+
// (InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemini2FlashLite),
64+
// (InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemma3_4B),
6265
])
6366
func generateContent(_ config: InstanceConfig, modelName: String) async throws {
6467
let model = FirebaseAI.componentInstance(config).generativeModel(
@@ -322,8 +325,9 @@ struct GenerateContentIntegrationTests {
322325
InstanceConfig.vertexAI_v1beta,
323326
InstanceConfig.vertexAI_v1beta_global,
324327
InstanceConfig.googleAI_v1beta,
325-
InstanceConfig.googleAI_v1beta_staging,
326-
InstanceConfig.googleAI_v1beta_freeTier_bypassProxy,
328+
// Note: The following configs are commented out for easy one-off manual testing.
329+
// InstanceConfig.googleAI_v1beta_staging,
330+
// InstanceConfig.googleAI_v1beta_freeTier_bypassProxy,
327331
])
328332
func generateImage(_ config: InstanceConfig) async throws {
329333
let generationConfig = GenerationConfig(
@@ -419,13 +423,18 @@ struct GenerateContentIntegrationTests {
419423
@Test(arguments: [
420424
(InstanceConfig.vertexAI_v1beta, ModelNames.gemini2FlashLite),
421425
(InstanceConfig.vertexAI_v1beta_global, ModelNames.gemini2FlashLite),
422-
(InstanceConfig.vertexAI_v1beta_staging, ModelNames.gemini2FlashLite),
426+
(InstanceConfig.vertexAI_v1beta_global_appCheckLimitedUse, ModelNames.gemini2FlashLite),
423427
(InstanceConfig.googleAI_v1beta, ModelNames.gemini2FlashLite),
428+
(InstanceConfig.googleAI_v1beta_appCheckLimitedUse, ModelNames.gemini2FlashLite),
424429
(InstanceConfig.googleAI_v1beta, ModelNames.gemma3_4B),
425-
(InstanceConfig.googleAI_v1beta_staging, ModelNames.gemini2FlashLite),
426-
(InstanceConfig.googleAI_v1beta_staging, ModelNames.gemma3_4B),
427-
(InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemini2FlashLite),
428-
(InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemma3_4B),
430+
(InstanceConfig.googleAI_v1beta_freeTier, ModelNames.gemini2FlashLite),
431+
(InstanceConfig.googleAI_v1beta_freeTier, ModelNames.gemma3_4B),
432+
// Note: The following configs are commented out for easy one-off manual testing.
433+
// (InstanceConfig.vertexAI_v1beta_staging, ModelNames.gemini2FlashLite),
434+
// (InstanceConfig.googleAI_v1beta_staging, ModelNames.gemini2FlashLite),
435+
// (InstanceConfig.googleAI_v1beta_staging, ModelNames.gemma3_4B),
436+
// (InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemini2FlashLite),
437+
// (InstanceConfig.googleAI_v1beta_freeTier_bypassProxy, ModelNames.gemma3_4B),
429438
])
430439
func generateContentStream(_ config: InstanceConfig, modelName: String) async throws {
431440
let expectedResponse = [

FirebaseAI/Tests/TestApp/Tests/Utilities/InstanceConfig.swift

Lines changed: 32 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -56,35 +56,56 @@ struct InstanceConfig: Equatable, Encodable {
5656
static let allConfigs = [
5757
vertexAI_v1beta,
5858
vertexAI_v1beta_global,
59-
vertexAI_v1beta_staging,
6059
vertexAI_v1beta_global_appCheckLimitedUse,
6160
googleAI_v1beta,
62-
googleAI_v1beta_staging,
63-
googleAI_v1beta_freeTier_bypassProxy,
6461
googleAI_v1beta_appCheckLimitedUse,
62+
googleAI_v1beta_freeTier,
63+
// Note: The following configs are commented out for easy one-off manual testing.
64+
// vertexAI_v1beta_staging,
65+
// googleAI_v1beta_staging,
66+
// googleAI_v1beta_freeTier_bypassProxy,
6567
]
6668

6769
static let vertexAI_v1beta_appCheckNotConfigured = InstanceConfig(
6870
appName: FirebaseAppNames.appCheckNotConfigured,
6971
apiConfig: APIConfig(service: .vertexAI(endpoint: .firebaseProxyProd), version: .v1beta)
7072
)
73+
static let vertexAI_v1beta_appCheckNotConfigured_limitedUseTokens = InstanceConfig(
74+
appName: FirebaseAppNames.appCheckNotConfigured,
75+
useLimitedUseAppCheckTokens: true,
76+
apiConfig: APIConfig(service: .vertexAI(endpoint: .firebaseProxyProd), version: .v1beta)
77+
)
7178
static let googleAI_v1beta_appCheckNotConfigured = InstanceConfig(
7279
appName: FirebaseAppNames.appCheckNotConfigured,
7380
apiConfig: APIConfig(service: .googleAI(endpoint: .firebaseProxyProd), version: .v1beta)
7481
)
82+
static let googleAI_v1beta_appCheckNotConfigured_limitedUseTokens = InstanceConfig(
83+
appName: FirebaseAppNames.appCheckNotConfigured,
84+
useLimitedUseAppCheckTokens: true,
85+
apiConfig: APIConfig(service: .googleAI(endpoint: .firebaseProxyProd), version: .v1beta)
86+
)
7587

7688
static let appCheckNotConfiguredConfigs = [
7789
vertexAI_v1beta_appCheckNotConfigured,
90+
vertexAI_v1beta_appCheckNotConfigured_limitedUseTokens,
7891
googleAI_v1beta_appCheckNotConfigured,
92+
googleAI_v1beta_appCheckNotConfigured_limitedUseTokens
7993
]
8094

8195
let appName: String?
8296
let location: String?
97+
let useLimitedUseAppCheckTokens: Bool
8398
let apiConfig: APIConfig
8499

85-
init(appName: String? = nil, location: String? = nil, apiConfig: APIConfig) {
100+
init(
101+
appName: String? = nil,
102+
location: String? = nil,
103+
useLimitedUseAppCheckTokens: Bool = false,
104+
apiConfig: APIConfig
105+
) {
86106
self.appName = appName
87107
self.location = location
108+
self.useLimitedUseAppCheckTokens = useLimitedUseAppCheckTokens
88109
self.apiConfig = apiConfig
89110
}
90111

@@ -118,8 +139,12 @@ extension InstanceConfig: CustomTestStringConvertible {
118139
" - Bypass Proxy"
119140
}
120141
let locationSuffix = location.map { " - \($0)" } ?? ""
142+
let appCheckLimitedUseDesignator = useLimitedUseAppCheckTokens ? " - FAC Limited-Use" : ""
121143

122-
return "\(serviceName) (\(versionName))\(freeTierDesignator)\(endpointSuffix)\(locationSuffix)"
144+
return """
145+
\(serviceName) (\(versionName))\(freeTierDesignator)\(endpointSuffix)\(locationSuffix)\
146+
\(appCheckLimitedUseDesignator)
147+
"""
123148
}
124149
}
125150

@@ -132,7 +157,7 @@ extension FirebaseAI {
132157
app: instanceConfig.app,
133158
location: location,
134159
apiConfig: instanceConfig.apiConfig,
135-
useLimitedUseAppCheckTokens: false
160+
useLimitedUseAppCheckTokens: instanceConfig.useLimitedUseAppCheckTokens
136161
)
137162
case .googleAI:
138163
assert(
@@ -143,7 +168,7 @@ extension FirebaseAI {
143168
app: instanceConfig.app,
144169
location: nil,
145170
apiConfig: instanceConfig.apiConfig,
146-
useLimitedUseAppCheckTokens: false
171+
useLimitedUseAppCheckTokens: instanceConfig.useLimitedUseAppCheckTokens
147172
)
148173
}
149174
}

0 commit comments

Comments
 (0)