From e3a5bdf111323385fda3fdbed722c827305d3816 Mon Sep 17 00:00:00 2001 From: Tim Griesser Date: Tue, 29 Jun 2021 12:54:29 -0400 Subject: [PATCH] refactor: add @packages/resolve-dist to isolate client/server code (#17109) --- packages/desktop-gui/lib/.eslintrc.json | 5 ---- packages/desktop-gui/lib/gui.js | 11 -------- packages/desktop-gui/package.json | 1 - packages/driver/lib/resolve-dist.js | 13 ---------- packages/driver/package.json | 2 +- packages/proxy/lib/http/util/inject.ts | 4 +-- packages/proxy/package.json | 1 + packages/reporter/lib/.eslintrc.json | 5 ---- packages/reporter/lib/resolve-dist.js | 7 ------ packages/reporter/package.json | 5 ++-- packages/resolve-dist/index.js | 5 ++++ packages/resolve-dist/lib/index.ts | 23 +++++++++++++++++ packages/resolve-dist/package.json | 25 +++++++++++++++++++ packages/resolve-dist/tsconfig.json | 12 +++++++++ packages/runner-ct/index.ts | 1 - packages/runner-ct/package.json | 11 ++------ packages/runner/lib/resolve-dist.js | 22 ---------------- packages/runner/package.json | 3 +-- packages/server-ct/package.json | 1 + packages/server-ct/src/routes-ct.ts | 6 ++--- .../lib => server-ct/src}/runner-ct.ts | 18 +++---------- packages/server/lib/controllers/reporter.js | 7 ++++-- packages/server/lib/controllers/runner.js | 16 +++++------- packages/server/lib/controllers/static.js | 7 ++++-- packages/server/lib/gui/windows.ts | 4 +-- packages/server/package.json | 1 + .../test/integration/http_requests_spec.js | 15 ++++++----- packages/server/test/unit/gui/windows_spec.ts | 7 +++--- packages/static/lib/static.js | 16 ++---------- yarn.lock | 2 +- 30 files changed, 116 insertions(+), 140 deletions(-) delete mode 100644 packages/desktop-gui/lib/.eslintrc.json delete mode 100644 packages/desktop-gui/lib/gui.js delete mode 100644 packages/driver/lib/resolve-dist.js delete mode 100644 packages/reporter/lib/.eslintrc.json delete mode 100644 packages/reporter/lib/resolve-dist.js create mode 100644 packages/resolve-dist/index.js create mode 100644 packages/resolve-dist/lib/index.ts create mode 100644 packages/resolve-dist/package.json create mode 100644 packages/resolve-dist/tsconfig.json delete mode 100644 packages/runner-ct/index.ts delete mode 100644 packages/runner/lib/resolve-dist.js rename packages/{runner-ct/lib => server-ct/src}/runner-ct.ts (79%) diff --git a/packages/desktop-gui/lib/.eslintrc.json b/packages/desktop-gui/lib/.eslintrc.json deleted file mode 100644 index 10d2238837..0000000000 --- a/packages/desktop-gui/lib/.eslintrc.json +++ /dev/null @@ -1,5 +0,0 @@ -{ - "env": { - "node": true - } -} diff --git a/packages/desktop-gui/lib/gui.js b/packages/desktop-gui/lib/gui.js deleted file mode 100644 index 9175db3821..0000000000 --- a/packages/desktop-gui/lib/gui.js +++ /dev/null @@ -1,11 +0,0 @@ -const path = require('path') - -function file (name) { - return `file://${path.join(__dirname, '..', 'dist', name)}` -} - -module.exports = { - getPathToIndex () { - return file('index.html') - }, -} diff --git a/packages/desktop-gui/package.json b/packages/desktop-gui/package.json index 82830043f7..83a27dd462 100644 --- a/packages/desktop-gui/package.json +++ b/packages/desktop-gui/package.json @@ -57,7 +57,6 @@ }, "files": [ "dist", - "lib", "!lib/**/*_spec.jsx" ] } diff --git a/packages/driver/lib/resolve-dist.js b/packages/driver/lib/resolve-dist.js deleted file mode 100644 index 6e25f603a8..0000000000 --- a/packages/driver/lib/resolve-dist.js +++ /dev/null @@ -1,13 +0,0 @@ -const path = require('path') - -function dist (...args) { - const paths = [__dirname, '..', 'dist'].concat(args) - - return path.join(...paths) -} - -module.exports = { - getPathToDist (...args) { - return dist(...args) - }, -} diff --git a/packages/driver/package.json b/packages/driver/package.json index c254ef88cf..f06d71ff80 100644 --- a/packages/driver/package.json +++ b/packages/driver/package.json @@ -18,6 +18,7 @@ "@cypress/webpack-preprocessor": "0.0.0-development", "@cypress/what-is-circular": "1.0.1", "@packages/network": "0.0.0-development", + "@packages/resolve-dist": "0.0.0-development", "@packages/runner": "0.0.0-development", "@packages/server": "0.0.0-development", "@packages/ts": "0.0.0-development", @@ -78,7 +79,6 @@ "zone.js": "0.9.0" }, "files": [ - "lib", "patches" ], "workspaces": { diff --git a/packages/proxy/lib/http/util/inject.ts b/packages/proxy/lib/http/util/inject.ts index 837c6fcbe4..72bb703615 100644 --- a/packages/proxy/lib/http/util/inject.ts +++ b/packages/proxy/lib/http/util/inject.ts @@ -1,5 +1,5 @@ import { oneLine } from 'common-tags' -import runner from '@packages/runner/lib/resolve-dist' +import { getRunnerInjectionContents } from '@packages/resolve-dist' export function partial (domain) { return oneLine` @@ -10,7 +10,7 @@ export function partial (domain) { } export function full (domain) { - return runner.getInjectionContents().then((contents) => { + return getRunnerInjectionContents().then((contents) => { return oneLine`