Skip to content

Conversation

@DudaGod
Copy link
Member

@DudaGod DudaGod commented Nov 12, 2025

No description provided.

@DudaGod DudaGod force-pushed the TESTPLANE-790.run_vite_server_once branch 2 times, most recently from 9071824 to d9dfdec Compare November 12, 2025 21:09
@DudaGod DudaGod changed the title WIP fix(ct): run vite server once on init Nov 12, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 12, 2025

Open in StackBlitz

npm i https://pkg.pr.new/gemini-testing/testplane@1166

commit: f50861f

@DudaGod DudaGod force-pushed the TESTPLANE-790.run_vite_server_once branch from d9dfdec to f50861f Compare November 12, 2025 21:11

await this._server.listen();

this._useBaseUrlFromVite();
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moved here from src/runner/browser-env/index.ts (which is not needed anymore)

return this._server?.resolvedUrls!.local[0];
}

private _useBaseUrlFromVite(): void {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also moved from src/runner/browser-env/index.ts


const RunnerClass = isRunInNodeJsEnv(this._config)
? await import("./runner").then(m => m.MainRunner)
: await import("./runner/browser-env").then(m => m.MainRunner);
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

browser-env runner is not needed anymore, I removed it

@@ -1,51 +0,0 @@
import { ViteServer } from "./vite/server";
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is not needed anymore because now I run vite server on init

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants