Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Set __NEXT_NEW_LINK_BEHAVIOR in Jest tests when newNextLinkBehavior is true #40702

Merged
merged 5 commits into from Sep 22, 2022
Merged

Set __NEXT_NEW_LINK_BEHAVIOR in Jest tests when newNextLinkBehavior is true #40702

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
db37904
fix: set __NEXT_NEW_LINK_BEHAVIOR in jest tests when newNextLinkBehav…
TomerAberbach Sep 20, 2022
dd03e11
test: jest with and without newNextLinkBehavior
TomerAberbach Sep 21, 2022
1873956
update test
ijjk Sep 22, 2022
7e40103
Merge branch 'canary' into TomerAberbach/canary
ijjk Sep 22, 2022
7733523
revert extra change
ijjk Sep 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
13 changes: 12 additions & 1 deletion packages/next/build/jest/jest.ts
View file
Open in desktop
@@ -1,6 +1,7 @@
import { loadEnvConfig } from '@next/env'
import { resolve, join } from 'path'
import loadConfig from '../../server/config'
import { NextConfigComplete } from '../../server/config-shared'
import { PHASE_TEST } from '../../shared/lib/constants'
import loadJsConfig from '../load-jsconfig'
import * as Log from '../output/log'
Expand All @@ -27,6 +28,16 @@ function loadClosestPackageJson(dir: string, attempts = 1): any {
}
}

/** Loads dotenv files and sets environment variables based on next config. */
function setUpEnv(dir: string, nextConfig: NextConfigComplete) {
const dev = false
loadEnvConfig(dir, dev, Log)

if (nextConfig.experimental.newNextLinkBehavior) {
process.env.__NEXT_NEW_LINK_BEHAVIOR = 'true'
}
}

/*
// Usage in jest.config.js
const nextJest = require('next/jest');
Expand Down Expand Up @@ -61,7 +72,7 @@ export default function nextJest(options: { dir?: string } = {}) {
isEsmProject = packageConfig.type === 'module'

nextConfig = await getConfig(resolvedDir)
loadEnvConfig(resolvedDir, false, Log)
setUpEnv(resolvedDir, nextConfig)
// TODO: revisit when bug in SWC is fixed that strips `.css`
const result = await loadJsConfig(resolvedDir, nextConfig)
jsConfig = result.jsConfig
Expand Down