fix: move e2e tests beside the page

[wyMinLwin]

Closes: #914

I just simply renamed e2e/ directory to tests/ and updated playwright.config.ts testDir accordingly

[changeset-bot]

🦋 Changeset detected

Latest commit: 1768b46

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
sv	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[jycouet]

I think that like this there is a confusion between files for Playwright and the one for Vitest.

[wyMinLwin]

so "Include it via the SvelteKit typescript config option" would be ok ?

 kit: {
    typescript: {
      config: (config) => {
        config.include = [...config.include, '../e2e/**/*.ts'];
      }
    }
  }

[wyMinLwin]

Hope you could review new approach

[jycouet]

I'll look Friday

[pkg-pr-new]

Open in StackBlitz

npx https://pkg.pr.new/sveltejs/cli/svelte-migrate@919

npx https://pkg.pr.new/sveltejs/cli/sv@919

npx https://pkg.pr.new/sveltejs/cli/@sveltejs/sv-utils@919

commit: 1768b46

[jycouet]

@wyMinLwin , what do you think about this?

[wyMinLwin]

@jycouet really great! I think it's cleaner and better

[jycouet]

FYI: we are thinking what is the best... And maybe not like this right now!
We still have the original issue in mind... Let's see how to fix it 👍

[svelte-docs-bot]

Preview: https://svelte-dev-git-preview-cli-919-svelte.vercel.app/

[sacrosanctic]

	files
unit	.{test}.{js,ts}
e2e	.e2e.test.{js,ts}

what about naming them all *.test.js

[jycouet]

files
unit .{test}.{js,ts}
e2e .e2e.test.{js,ts}
what about naming them all *.test.js

Like this, vitest will pick e2e tests and fail.