feat: fix specs & skip copy PW config when using examples [kit-2292] (#1166)

Author: maxigimenez

examples/advanced-project-js/checkly.config.js

@@ -51,7 +51,6 @@ const config = defineConfig({
         //Use `testCommand: npx playwright test` to filter the tests you want to run
       }
     ],
-    ],
   },
   cli: {
     /* The default datacenter location to use when running npx checkly test */

…oject-js/src/playwright/homepage.spec.js …ed-project-js/src/tests/homepage.spec.jsexamples/advanced-project-js/src/playwright/homepage.spec.js renamed to examples/advanced-project-js/src/tests/homepage.spec.js examples/advanced-project-js/src/playwright/homepage.spec.js renamed to examples/advanced-project-js/src/tests/homepage.spec.js

@@ -35,9 +35,9 @@ const config = defineConfig({
     playwrightConfig: {
       timeout: 30000,
       use: {
-        baseURL: 'https://www.danube-web.shop',
+        baseURL: "https://www.checklyhq.com",
         viewport: { width: 1280, height: 720 },
-      }
+      },
     },
     browserChecks: {
       /* A glob pattern matches any Playwright .spec.ts files and automagically creates a Browser Check. This way, you

…-project-js/src/playwright/login.spec.js …anced-project-js/src/tests/login.spec.jsexamples/advanced-project-js/src/playwright/login.spec.js renamed to examples/advanced-project-js/src/tests/login.spec.js examples/advanced-project-js/src/playwright/login.spec.js renamed to examples/advanced-project-js/src/tests/login.spec.js

@@ -20,7 +20,6 @@ export default defineConfig({
     baseURL: "https://www.checklyhq.com",
     viewport: { width: 1280, height: 720 },
     trace: "on",
-
   },
   projects: [
     {

examples/advanced-project/checkly.config.ts

@@ -27,7 +27,12 @@ const config = defineConfig({
     /* A glob pattern that matches the Checks inside your repo, see https://www.checklyhq.com/docs/cli/using-check-test-match/ */
     checkMatch: '**/__checks__/**/*.check.js',
     /* Global configuration option for Playwright-powered checks. See https://www.checklyhq.com/docs/browser-checks/playwright-test/#global-configuration */
-    playwrightConfig: {},
+    playwrightConfig: {
+      use: {
+        baseURL: "https://www.checklyhq.com",
+        viewport: { width: 1280, height: 720 },
+      },
+    },
     browserChecks: {
       /* A glob pattern matches any Playwright .spec.js files and automagically creates a Browser Check. This way, you
       * can just write native Playwright code. See https://www.checklyhq.com/docs/cli/using-check-test-match/

examples/advanced-project/playwright.config.ts

@@ -27,7 +27,12 @@ const config = defineConfig({
     /* A glob pattern that matches the Checks inside your repo, see https://www.checklyhq.com/docs/cli/using-check-test-match/ */
     checkMatch: '**/__checks__/**/*.check.ts',
     /* Global configuration option for Playwright-powered checks. See https://www.checklyhq.com/docs/browser-checks/playwright-test/#global-configuration */
-    playwrightConfig: {},
+    playwrightConfig: {
+      use: {
+        baseURL: "https://www.checklyhq.com",
+        viewport: { width: 1280, height: 720 },
+      },
+    },
     browserChecks: {
       /* A glob pattern matches any Playwright .spec.ts files and automagically creates a Browser Check. This way, you
       * can just write native Playwright code. See https://www.checklyhq.com/docs/cli/using-check-test-match/

examples/boilerplate-project-js/checkly.config.js

@@ -103,8 +103,9 @@ export default class Bootstrap extends Command {
     }
 
     const projectDirectory = await getProjectDirectory({ onCancel })
+    const existingProject = hasPackageJsonFile(projectDirectory)
 
-    if (hasPackageJsonFile(projectDirectory)) {
+    if (existingProject) {
       // Init Checkly CLI for an existing project
       await installWithinProject({ projectDirectory, version, onCancel })
     } else {
@@ -115,7 +116,8 @@ export default class Bootstrap extends Command {
     await installDependenciesAndInitGit({ projectDirectory })
 
     const playwrightConfig = getPlaywrightConfig(projectDirectory)
-    if (playwrightConfig) {
+    // Only prompt playwright copy when not using exampels
+    if (playwrightConfig && existingProject) {
       await copyPlaywrightConfig({ projectDirectory, playwrightConfig })
     }