Skip to content

Commit 51e0fdf

Browse files
dante01yoonampagent
andcommitted
test(billing): remove retired flag from new fixtures
Amp-Thread-ID: https://ampcode.com/threads/T-019fc8cd-b288-7161-a590-cd76d51212bf Co-authored-by: Amp <amp@ampcode.com>
1 parent 2db14d8 commit 51e0fdf

2 files changed

Lines changed: 1 addition & 9 deletions

File tree

browser_tests/fixtures/utils/cloudAppSetup.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,6 @@ import { mockWorkspace } from '@e2e/fixtures/utils/workspaceMocks'
1414
export const APP_URL =
1515
process.env.PLAYWRIGHT_TEST_URL || 'http://localhost:8188'
1616

17-
// consolidated_billing_enabled routes personal workspaces to the unified
18-
// billing surfaces the cloud specs assert; without it they fall back to the
19-
// legacy variants.
20-
const DEFAULT_FEATURES = {
21-
consolidated_billing_enabled: true
22-
} satisfies RemoteConfig
23-
2417
// Disable the experimental Asset API: with it on (cloud default) the unmocked
2518
// asset endpoints 403 and workflow restore throws uncaught, aborting the
2619
// GraphCanvas onMounted chain before the URL action loaders.
@@ -63,7 +56,7 @@ export async function setupCloudApp(
6356
{ workspace, members = [], features }: CloudAppSetupOptions
6457
) {
6558
await mockCloudBoot(page, {
66-
features: { ...DEFAULT_FEATURES, ...features },
59+
features: features ?? {},
6760
settings: DEFAULT_SETTINGS
6861
})
6962
await mockGraphBootExtras(page)

browser_tests/tests/dialogs/creditsTile.spec.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,6 @@ async function mockCloudBoot(page: Page, billingControlEnabled = true) {
7171
await page.route('**/api/features', (r) =>
7272
r.fulfill(
7373
jsonRoute({
74-
consolidated_billing_enabled: true,
7574
billing_control_enabled: billingControlEnabled
7675
} satisfies RemoteConfig)
7776
)

0 commit comments

Comments
 (0)