Skip to content

Commit

Permalink
tests: remove unneeded APP_HOME_URL setting
Browse files Browse the repository at this point in the history
These tests are for single-server setups that don't need this setting.
  • Loading branch information
jordigh committed Feb 7, 2025
1 parent 8be3dd1 commit 12f0f11
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
2 changes: 0 additions & 2 deletions test/gen-server/AuthCaching.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ describe('AuthCaching', function() {
await homeMS.run();
homeServer = homeMS.flexServer;
homeUrl = homeServer.getOwnUrl();
process.env.APP_HOME_URL = homeUrl;
const docsMS = await MergedServer.create(0, ['docs'],
{logToConsole: false, externalStorage: false});
await docsMS.run();
Expand All @@ -80,7 +79,6 @@ describe('AuthCaching', function() {

after(async function() {
delete process.env.GRIST_DATA_DIR;
delete process.env.APP_HOME_URL;
sandbox.restore();
await testUtils.captureLog('warn', async () => {
await docsServer.close();
Expand Down
1 change: 0 additions & 1 deletion test/nbrowser/testServer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,6 @@ export class TestServerMerged extends EventEmitter implements IMochaServer {
GRIST_SERVE_SAME_ORIGIN: 'true',
// Run with HOME_PORT, STATIC_PORT, DOC_PORT, DOC_WORKER_COUNT in the environment to override.
...(useSinglePort ? {
APP_HOME_URL: this.getHost(),
GRIST_SINGLE_PORT: 'true',
} : (isCore ? {
HOME_PORT: corePort,
Expand Down

0 comments on commit 12f0f11

Please sign in to comment.