@@ -24,21 +24,21 @@ const INITIALIZING_TEST_CASES = [
2424// please make sure this is sorted alphabetically when making changes.
2525const SKIPPED_TEST_SUITES = {
2626 'test/development/acceptance-app/ReactRefreshRegression.test.ts' : [
27- 'ReactRefreshRegression app can fast refresh a page with dynamic rendering' ,
2827 'ReactRefreshRegression app can fast refresh a page with config' ,
28+ 'ReactRefreshRegression app can fast refresh a page with dynamic rendering' ,
2929 ] ,
3030 'test/development/acceptance-app/ReactRefreshRequire.test.ts' : [
31- 'ReactRefreshRequire app re-runs accepted modules' ,
3231 'ReactRefreshRequire app propagates a hot update to closest accepted module' ,
3332 'ReactRefreshRequire app propagates hot update to all inverse dependencies' ,
33+ 'ReactRefreshRequire app re-runs accepted modules' ,
3434 ] ,
3535 'test/development/acceptance/ReactRefreshLogBox.test.ts' : [
3636 'ReactRefreshLogBox turbo conversion to class component (1)' ,
3737 ] ,
3838 'test/development/acceptance/ReactRefreshRequire.test.ts' : [
39- 'ReactRefreshRequire re-runs accepted modules' ,
4039 'ReactRefreshRequire propagates a hot update to closest accepted module' ,
4140 'ReactRefreshRequire propagates hot update to all inverse dependencies' ,
41+ 'ReactRefreshRequire re-runs accepted modules' ,
4242 ] ,
4343 'test/development/basic/hmr.test.ts' : [
4444 'basic HMR, basePath: "/docs" Error Recovery should show the error on all pages' ,
@@ -47,8 +47,16 @@ const SKIPPED_TEST_SUITES = {
4747 / s h o u l d a u t o m a t i c a l l y f a s t r e f r e s h c o n t e n t w h e n p a t h i s a d d e d w i t h o u t e r r o r / ,
4848 / s h o u l d r e c o v e r f r o m m o d u l e n o t f o u n d w h e n p a t h s i s u p d a t e d / ,
4949 ] ,
50+ 'test/development/middleware-errors/index.test.ts' : [
51+ 'middleware - development errors when there is a compilation error after boot logs the error correctly' ,
52+ 'middleware - development errors when there is a compilation error from boot logs the error correctly' ,
53+ ] ,
5054 'test/development/tsconfig-path-reloading/index.test.ts' : [
5155 / s h o u l d a u t o m a t i c a l l y f a s t r e f r e s h c o n t e n t w h e n p a t h i s a d d e d w i t h o u t e r r o r / ,
56+ 'tsconfig-path-reloading tsconfig added after starting dev should load with initial paths config correctly' ,
57+ ] ,
58+ 'test/e2e/app-dir/app-compilation/index.test.ts' : [
59+ 'app dir HMR should not cause error when removing loading.js' ,
5260 ] ,
5361 'test/e2e/app-dir/app-css/index.test.ts' : [
5462 'app dir - css css support server layouts should support external css imports' ,
@@ -68,6 +76,9 @@ const SKIPPED_TEST_SUITES = {
6876 'basePath should 404 when manually adding basePath with router.push' ,
6977 'basePath should 404 when manually adding basePath with router.replace' ,
7078 ] ,
79+ 'test/e2e/conflicting-app-page-error/index.test.ts' : [
80+ 'Conflict between app file and pages file should not show error overlay for non conflict pages under app or pages dir' ,
81+ ] ,
7182 'test/e2e/middleware-rewrites/test/index.test.ts' : [
7283 'Middleware Rewrite should have props for afterFiles rewrite to SSG page' ,
7384 ] ,
@@ -82,8 +93,8 @@ const SKIPPED_TEST_SUITES = {
8293 'Document and App Client side should detect the changes to pages/_document.js and display it' ,
8394 ] ,
8495 'test/integration/css/test/css-modules.test.js' : [
85- 'CSS Modules Composes Ordering Development Mode should have correct color on index page (on nav from other)' ,
8696 'CSS Modules Composes Ordering Development Mode should have correct color on index page (on nav from index)' ,
97+ 'CSS Modules Composes Ordering Development Mode should have correct color on index page (on nav from other)' ,
8798 ] ,
8899 'test/integration/custom-error/test/index.test.js' : [ / C u s t o m _ e r r o r / ] ,
89100 'test/integration/dynamic-routing/test/index.test.js' : [
@@ -96,13 +107,41 @@ const SKIPPED_TEST_SUITES = {
96107 'Dynamic Routing production mode should output a routes-manifest correctly' ,
97108 ] ,
98109 'test/integration/env-config/test/index.test.js' : [
99- 'Env Config dev mode with hot reload should provide env for SSG' ,
100- 'Env Config dev mode with hot reload should provide env correctly for SSR' ,
101110 'Env Config dev mode with hot reload should provide env correctly for API routes' ,
111+ 'Env Config dev mode with hot reload should provide env correctly for SSR' ,
112+ 'Env Config dev mode with hot reload should provide env for SSG' ,
102113 ] ,
103114 'test/integration/import-assertion/test/index.test.js' : [
104115 / s h o u l d h a n d l e j s o n a s s e r t i o n s / ,
105116 ] ,
117+ 'test/integration/next-image-legacy/unicode/test/index.test.ts' : [
118+ / I m a g e C o m p o n e n t U n i c o d e I m a g e U R L / ,
119+ ] ,
120+ 'test/integration/next-image-new/unicode/test/index.test.ts' : [
121+ / I m a g e C o m p o n e n t U n i c o d e I m a g e U R L / ,
122+ ] ,
123+ }
124+
125+ function checkSorted ( arr , name ) {
126+ const sorted = [ ...arr ] . sort ( )
127+ if ( JSON . stringify ( arr ) !== JSON . stringify ( sorted ) ) {
128+ console . log ( `Expected order of ${ name } :` )
129+ for ( let i = 0 ; i < arr . length ; i ++ ) {
130+ if ( arr [ i ] === sorted [ i ] ) {
131+ console . log ( ` ${ arr [ i ] } ` )
132+ } else {
133+ console . log ( bold ( ) . red ( `- ${ arr [ i ] } ` ) )
134+ console . log ( bold ( ) . green ( `+ ${ sorted [ i ] } ` ) )
135+ }
136+ }
137+ throw new Error ( `${ name } is not sorted` )
138+ }
139+ }
140+
141+ checkSorted ( Object . keys ( SKIPPED_TEST_SUITES ) , 'SKIPPED_TEST_SUITES' )
142+
143+ for ( const [ key , value ] of Object . entries ( SKIPPED_TEST_SUITES ) ) {
144+ checkSorted ( value , `SKIPPED_TEST_SUITES['${ key } ']` )
106145}
107146
108147/**
@@ -234,7 +273,7 @@ async function updatePassingTests() {
234273
235274 if ( skippedPassingNames . length > 0 ) {
236275 console . log (
237- `${ bold ( ) . red ( filepath ) } has ${
276+ `${ bold ( ) . yellow ( filepath ) } has ${
238277 skippedPassingNames . length
239278 } passing tests that are marked as skipped:\n${ skippedPassingNames
240279 . map ( ( name ) => ` - ${ name } ` )
0 commit comments