From ed19d9df42ce3d9f06ba3de63ef5462115323997 Mon Sep 17 00:00:00 2001 From: Pilar Martinez Date: Tue, 3 Feb 2026 15:13:09 -0300 Subject: [PATCH 1/5] feat: allow setting frequency on --create-check [red-121] --- packages/cli/src/commands/pw-test.ts | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/packages/cli/src/commands/pw-test.ts b/packages/cli/src/commands/pw-test.ts index 1d3aec4ad..3385c2071 100644 --- a/packages/cli/src/commands/pw-test.ts +++ b/packages/cli/src/commands/pw-test.ts @@ -89,6 +89,12 @@ export default class PwTestCommand extends AuthCommand { description: 'Create a Checkly check from the Playwright test.', default: false, }), + 'frequency': Flags.integer({ + char: 'f', + description: 'The frequency in minutes for the created check.', + default: 10, + options: ['1', '2', '5', '10', '15', '30', '60', '120', '180', '360', '720', '1440'], + }), 'stream-logs': Flags.boolean({ description: 'Stream logs from the test run to the console.', default: true, @@ -122,6 +128,7 @@ export default class PwTestCommand extends AuthCommand { 'create-check': createCheck, 'stream-logs': streamLogs, 'include': includeFlag, + 'frequency': frequency, } = flags const { configDirectory, configFilenames } = splitConfigFilePath(configFilename) const pwPathFlag = this.getConfigPath(playwrightFlags) @@ -148,6 +155,7 @@ export default class PwTestCommand extends AuthCommand { runLocation as keyof Region, privateRunLocation, dir, + frequency, ) if (createCheck) { this.style.actionStart('Adding check with specified options to the Checkly config file') @@ -335,6 +343,7 @@ export default class PwTestCommand extends AuthCommand { runLocation: keyof Region, privateRunLocation: string | undefined, dir: string, + frequency: number = 10, ): Promise { const parseArgs = args.map(arg => shellQuote(arg)) const input = parseArgs.join(' ') || '' @@ -351,7 +360,7 @@ export default class PwTestCommand extends AuthCommand { name: `Playwright Test: ${input}`, testCommand, ...locationConfig, - frequency: 10, + frequency, } } From a3b70ba728e03ab3288ff75e6dcd924b18d08e61 Mon Sep 17 00:00:00 2001 From: Pilar Martinez Date: Tue, 3 Feb 2026 15:39:21 -0300 Subject: [PATCH 2/5] fix: test [red-121] --- packages/cli/e2e/__tests__/pw-test.spec.ts | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/packages/cli/e2e/__tests__/pw-test.spec.ts b/packages/cli/e2e/__tests__/pw-test.spec.ts index fb7abad20..4f0c1ea34 100644 --- a/packages/cli/e2e/__tests__/pw-test.spec.ts +++ b/packages/cli/e2e/__tests__/pw-test.spec.ts @@ -54,5 +54,20 @@ describe('pw-test', { timeout: 45000 }, () => { expect(checklyConfig.config?.checks?.playwrightChecks.length).toBe(1) expect(checklyConfig.config?.checks?.playwrightChecks[0].name).toBe('Playwright Test: --grep @TAG-B') expect(checklyConfig.config?.checks?.playwrightChecks[0].testCommand).toBe('npx playwright test --grep @TAG-B') + expect(checklyConfig.config?.checks?.playwrightChecks[0].frequency).toBe(10) + }) + + it('Should add a Playwright test with custom frequency', async () => { + const result = await runChecklyCli({ + args: ['pw-test', '--create-check', '--frequency', '5', '--', `--grep`, '@TAG-B'], + apiKey: config.get('apiKey'), + accountId: config.get('accountId'), + directory: FIXTURE_TEST_PWT_NATIVE, + timeout: 120000, // 2 minutes + }) + expect(result.status).toBe(0) + const checklyConfig = await loadChecklyConfig(FIXTURE_TEST_PWT_NATIVE) + expect(checklyConfig.config?.checks?.playwrightChecks).toBeDefined() + expect(checklyConfig.config?.checks?.playwrightChecks[0].frequency).toBe(5) }) }) From 0f3a1b74a886898991dedf9c95a81bc0f1dcf1fb Mon Sep 17 00:00:00 2001 From: Pilar Martinez Date: Tue, 3 Feb 2026 15:46:19 -0300 Subject: [PATCH 3/5] fix: test [red-121] --- packages/cli/e2e/__tests__/pw-test.spec.ts | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/packages/cli/e2e/__tests__/pw-test.spec.ts b/packages/cli/e2e/__tests__/pw-test.spec.ts index 4f0c1ea34..fb7abad20 100644 --- a/packages/cli/e2e/__tests__/pw-test.spec.ts +++ b/packages/cli/e2e/__tests__/pw-test.spec.ts @@ -54,20 +54,5 @@ describe('pw-test', { timeout: 45000 }, () => { expect(checklyConfig.config?.checks?.playwrightChecks.length).toBe(1) expect(checklyConfig.config?.checks?.playwrightChecks[0].name).toBe('Playwright Test: --grep @TAG-B') expect(checklyConfig.config?.checks?.playwrightChecks[0].testCommand).toBe('npx playwright test --grep @TAG-B') - expect(checklyConfig.config?.checks?.playwrightChecks[0].frequency).toBe(10) - }) - - it('Should add a Playwright test with custom frequency', async () => { - const result = await runChecklyCli({ - args: ['pw-test', '--create-check', '--frequency', '5', '--', `--grep`, '@TAG-B'], - apiKey: config.get('apiKey'), - accountId: config.get('accountId'), - directory: FIXTURE_TEST_PWT_NATIVE, - timeout: 120000, // 2 minutes - }) - expect(result.status).toBe(0) - const checklyConfig = await loadChecklyConfig(FIXTURE_TEST_PWT_NATIVE) - expect(checklyConfig.config?.checks?.playwrightChecks).toBeDefined() - expect(checklyConfig.config?.checks?.playwrightChecks[0].frequency).toBe(5) }) }) From 039748f1bdb6500ab998fee0ea53f19a99ef204f Mon Sep 17 00:00:00 2001 From: Pilar Martinez Date: Wed, 4 Feb 2026 09:45:55 -0300 Subject: [PATCH 4/5] fix: tests again [red-121] --- packages/cli/e2e/__tests__/pw-test.spec.ts | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/packages/cli/e2e/__tests__/pw-test.spec.ts b/packages/cli/e2e/__tests__/pw-test.spec.ts index fb7abad20..4f0c1ea34 100644 --- a/packages/cli/e2e/__tests__/pw-test.spec.ts +++ b/packages/cli/e2e/__tests__/pw-test.spec.ts @@ -54,5 +54,20 @@ describe('pw-test', { timeout: 45000 }, () => { expect(checklyConfig.config?.checks?.playwrightChecks.length).toBe(1) expect(checklyConfig.config?.checks?.playwrightChecks[0].name).toBe('Playwright Test: --grep @TAG-B') expect(checklyConfig.config?.checks?.playwrightChecks[0].testCommand).toBe('npx playwright test --grep @TAG-B') + expect(checklyConfig.config?.checks?.playwrightChecks[0].frequency).toBe(10) + }) + + it('Should add a Playwright test with custom frequency', async () => { + const result = await runChecklyCli({ + args: ['pw-test', '--create-check', '--frequency', '5', '--', `--grep`, '@TAG-B'], + apiKey: config.get('apiKey'), + accountId: config.get('accountId'), + directory: FIXTURE_TEST_PWT_NATIVE, + timeout: 120000, // 2 minutes + }) + expect(result.status).toBe(0) + const checklyConfig = await loadChecklyConfig(FIXTURE_TEST_PWT_NATIVE) + expect(checklyConfig.config?.checks?.playwrightChecks).toBeDefined() + expect(checklyConfig.config?.checks?.playwrightChecks[0].frequency).toBe(5) }) }) From a0355944244b9e17905de321ed4e02a547f16617 Mon Sep 17 00:00:00 2001 From: Pilar Martinez Date: Thu, 5 Feb 2026 13:02:52 -0300 Subject: [PATCH 5/5] fix: cache [red-121] --- packages/cli/e2e/__tests__/pw-test.spec.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/cli/e2e/__tests__/pw-test.spec.ts b/packages/cli/e2e/__tests__/pw-test.spec.ts index 4f0c1ea34..8cf8f5f09 100644 --- a/packages/cli/e2e/__tests__/pw-test.spec.ts +++ b/packages/cli/e2e/__tests__/pw-test.spec.ts @@ -66,8 +66,8 @@ describe('pw-test', { timeout: 45000 }, () => { timeout: 120000, // 2 minutes }) expect(result.status).toBe(0) - const checklyConfig = await loadChecklyConfig(FIXTURE_TEST_PWT_NATIVE) - expect(checklyConfig.config?.checks?.playwrightChecks).toBeDefined() - expect(checklyConfig.config?.checks?.playwrightChecks[0].frequency).toBe(5) + const configContent = fs.readFileSync( + path.join(FIXTURE_TEST_PWT_NATIVE, 'checkly.config.ts'), 'utf-8') + expect(configContent).toContain('frequency: 5') }) })