From 2faa0229261950294821680de8fa9fe12d68588b Mon Sep 17 00:00:00 2001 From: JJ Kasper Date: Thu, 16 Jan 2020 19:12:20 -0600 Subject: [PATCH] Update to error instead of warn --- packages/next/export/worker.js | 2 +- .../next.config.js | 0 .../pages/amp.js | 0 .../pages/hello.js | 0 .../pages/ssg.js | 0 .../pages/ssr.js | 0 .../test/index.test.js | 0 7 files changed, 1 insertion(+), 1 deletion(-) rename test/integration/{auto-export-query-warn => auto-export-query-error}/next.config.js (100%) rename test/integration/{auto-export-query-warn => auto-export-query-error}/pages/amp.js (100%) rename test/integration/{auto-export-query-warn => auto-export-query-error}/pages/hello.js (100%) rename test/integration/{auto-export-query-warn => auto-export-query-error}/pages/ssg.js (100%) rename test/integration/{auto-export-query-warn => auto-export-query-error}/pages/ssr.js (100%) rename test/integration/{auto-export-query-warn => auto-export-query-error}/test/index.test.js (100%) diff --git a/packages/next/export/worker.js b/packages/next/export/worker.js index d86d626041967..e2be389d8e102 100644 --- a/packages/next/export/worker.js +++ b/packages/next/export/worker.js @@ -49,7 +49,7 @@ export default async function({ const hasOrigQueryValues = Object.keys(originalQuery).length > 0 const queryWithAutoExportWarn = () => { if (hasOrigQueryValues) { - console.warn( + throw new Error( `\nWarn: you provided query values for ${path} which is an auto-exported page. These can not be applied since the page can no longer be re-rendered on the server. To disable auto-export for this page add \`getInitialProps\`\n` ) } diff --git a/test/integration/auto-export-query-warn/next.config.js b/test/integration/auto-export-query-error/next.config.js similarity index 100% rename from test/integration/auto-export-query-warn/next.config.js rename to test/integration/auto-export-query-error/next.config.js diff --git a/test/integration/auto-export-query-warn/pages/amp.js b/test/integration/auto-export-query-error/pages/amp.js similarity index 100% rename from test/integration/auto-export-query-warn/pages/amp.js rename to test/integration/auto-export-query-error/pages/amp.js diff --git a/test/integration/auto-export-query-warn/pages/hello.js b/test/integration/auto-export-query-error/pages/hello.js similarity index 100% rename from test/integration/auto-export-query-warn/pages/hello.js rename to test/integration/auto-export-query-error/pages/hello.js diff --git a/test/integration/auto-export-query-warn/pages/ssg.js b/test/integration/auto-export-query-error/pages/ssg.js similarity index 100% rename from test/integration/auto-export-query-warn/pages/ssg.js rename to test/integration/auto-export-query-error/pages/ssg.js diff --git a/test/integration/auto-export-query-warn/pages/ssr.js b/test/integration/auto-export-query-error/pages/ssr.js similarity index 100% rename from test/integration/auto-export-query-warn/pages/ssr.js rename to test/integration/auto-export-query-error/pages/ssr.js diff --git a/test/integration/auto-export-query-warn/test/index.test.js b/test/integration/auto-export-query-error/test/index.test.js similarity index 100% rename from test/integration/auto-export-query-warn/test/index.test.js rename to test/integration/auto-export-query-error/test/index.test.js