Upgrade @ampproject/toolbox-optimizer to 2.7.0-alpha.4 #18248

omBratteng · 2020-10-26T11:27:36Z

This version has uses node-fetch v2.6.1
Fixes #18044
Related to #17416

This version has uses node-fetch v2.6.1 Related to #17416

ijjk · 2020-10-26T11:33:36Z

Failing test suites

Commit: 1c583b6

test/integration/amp-export-validation/test/index.test.js

AMP Validation on Export > should export AMP pages

Expand output

● AMP Validation on Export › should export AMP pages

expect(received).toBe(expected) // Object.is equality

Expected: "PASS"
Received: "FAIL"

  16 |     }
  17 |   }
> 18 |   expect(result.status).toBe('PASS')
     |                         ^
  19 | }
  20 | 

  at validateAMP (lib/amp-test-utils.js:18:25)
  at integration/amp-export-validation/test/index.test.js:37:9
      at async Promise.all (index 1)
  at Object.<anonymous> (integration/amp-export-validation/test/index.test.js:34:5)

test/integration/amphtml-custom-optimizer/test/index.test.js

AMP Custom Optimizer > should build and start for static page
AMP Custom Optimizer > should build and start for dynamic page

Expand output

● AMP Custom Optimizer › should build and start for static page

expect(received).toContain(expected) // indexOf

Expected substring: "i-amphtml-version=\"001515617716922\""
Received string:    "<!DOCTYPE html><html amp i-amphtml-layout i-amphtml-no-boilerplate transformed=\"self;v=1\"><head><style amp-custom></style></head><body><!-- __NEXT_DATA__ --><amp-twitter width=\"500\" height=\"500\" layout=\"responsive\" data-tweetid=\"1159145442896166912\" class=\"i-amphtml-layout-responsive i-amphtml-layout-size-defined\" i-amphtml-layout=\"responsive\"><i-amphtml-sizer style=\"display:block;padding-top:100%\"></i-amphtml-sizer></amp-twitter></body></html>"

  30 |       'amp-twitter width="500" height="500" layout="responsive" data-tweetid="1159145442896166912"'
  31 |     )
> 32 |     expect(html).toContain('i-amphtml-version="001515617716922"')
     |                  ^
  33 |     expect(html).toContain(
  34 |       'script async src="https://cdn.ampproject.org/rtv/001515617716922/v0.js"'
  35 |     )

  at Object.<anonymous> (integration/amphtml-custom-optimizer/test/index.test.js:32:18)

● AMP Custom Optimizer › should build and start for dynamic page

expect(received).toContain(expected) // indexOf

Expected substring: "i-amphtml-version=\"001515617716922\""
Received string:    "<!DOCTYPE html><html amp i-amphtml-layout i-amphtml-no-boilerplate transformed=\"self;v=1\"><head><style amp-custom></style></head><body><!-- __NEXT_DATA__ --><amp-img width=\"500\" height=\"500\" layout=\"responsive\" src=\"https://amp.dev/static/samples/img/story_dog2_portrait.jpg\" i-amphtml-ssr data-hero class=\"i-amphtml-layout-responsive i-amphtml-layout-size-defined\" i-amphtml-layout=\"responsive\"><i-amphtml-sizer style=\"display:block;padding-top:100%\"></i-amphtml-sizer><img class=\"i-amphtml-fill-content i-amphtml-replaced-content\" decoding=\"async\" src=\"https://amp.dev/static/samples/img/story_dog2_portrait.jpg\"></amp-img></body></html>"

  49 |       'amp-img width="500" height="500" layout="responsive" src="https://amp.dev/static/samples/img/story_dog2_portrait.jpg"'
  50 |     )
> 51 |     expect(html).toContain('i-amphtml-version="001515617716922"')
     |                  ^
  52 |     expect(html).toContain(
  53 |       'script async src="https://cdn.ampproject.org/rtv/001515617716922/v0.js"'
  54 |     )

  at Object.<anonymous> (integration/amphtml-custom-optimizer/test/index.test.js:51:18)

test/integration/amphtml-custom-validator/test/index.test.js

AMP Custom Validator > should build and start successfully
AMP Custom Validator > should run in dev mode successfully

Expand output

● AMP Custom Validator › should build and start successfully

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● AMP Custom Validator › should run in dev mode successfully

expect(received).not.toContain(expected) // indexOf

Expected substring: not "error"
Received string:        "warn  - You have enabled experimental feature(s).
warn  - Experimental features are not covered by semver, and may cause unexpected or broken application behavior. Use them at your own risk.

warn  - Amp Validation

/  error  The mandatory tag 'link rel=canonical' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
   error  The mandatory tag 'meta charset=utf-8' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
   error  The mandatory tag 'meta name=viewport' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
   error  The mandatory tag 'style[amp-runtime]' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#stylesheets
   error  The mandatory tag 'amphtml engine v0.js script' is missing or incorrect.  https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup

"

  44 |     await killApp(app)
  45 | 
> 46 |     expect(stderr).not.toContain('error')
     |                        ^
  47 |     expect(html).toContain('Hello from AMP')
  48 |   })
  49 | })

  at Object.<anonymous> (integration/amphtml-custom-validator/test/index.test.js:46:24)

test/integration/auto-export-query-error/test/index.test.js

Auto Export > server mode > should show warning for query provided for auto exported page correctly
Auto Export > serverless mode > should show warning for query provided for auto exported page correctly

Expand output

● Auto Export › server mode › should show warning for query provided for auto exported page correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● Auto Export › serverless mode › should show warning for query provided for auto exported page correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

test/integration/build-output/test/index.test.js

Build Output > With AMP Output > should not include custom error

Expand output

● Build Output › With AMP Output › should not include custom error

expect(received).toMatch(expected)

Expected pattern: /\/ [ 0-9.]* B [ 0-9.]* kB/
Received string:  "warn  - No build cache found. Please configure build caching for faster rebuilds. Read more: https://err.sh/next.js/no-cache
info  - Creating an optimized production build...
info  - Using external babel configuration from /home/runner/work/next.js/next.js/test/.babelrc
info  - Compiled successfully
info  - Collecting page data...
info  - Generating static pages (0/4)
info  - Generating static pages (1/4)
info  - Generating static pages (2/4)
info  - Generating static pages (3/4)
info  - Generating static pages (4/4)
Amp Validation·
/amp           error  The mandatory tag 'link rel=canonical' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
               error  The mandatory tag 'meta charset=utf-8' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
               error  The mandatory tag 'meta name=viewport' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
               error  The mandatory tag 'style[amp-runtime]' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#stylesheets
               error  The mandatory tag 'amphtml engine v0.js script' is missing or incorrect.  https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup·
/hybrid?amp=1  error  The mandatory tag 'link rel=canonical' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
               error  The mandatory tag 'meta charset=utf-8' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
               error  The mandatory tag 'meta name=viewport' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup
               error  The mandatory tag 'style[amp-runtime]' is missing or incorrect.           https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#stylesheets
               error  The mandatory tag 'amphtml engine v0.js script' is missing or incorrect.  https://amp.dev/documentation/guides-and-tutorials/learn/spec/amphtml/#required-markup·
"

  214 |       })
  215 | 
> 216 |       expect(stdout).toMatch(/\/ [ 0-9.]* B [ 0-9.]* kB/)
      |                      ^
  217 |       expect(stdout).toMatch(/\/amp .* AMP/)
  218 |       expect(stdout).toMatch(/\/hybrid [ 0-9.]* B/)
  219 |       expect(stdout).toMatch(/\+ First Load JS shared by all [ 0-9.]* kB/)

  at Object.<anonymous> (integration/build-output/test/index.test.js:216:22)

test/integration/basepath/test/index.test.js

basePath production > should add basePath to routes-manifest
basePath production > should prefetch pages correctly when manually called
basePath production > should prefetch pages correctly in viewport with
basePath production > should 404 for public file without basePath
basePath production > should serve public file with basePath correctly
basePath production > should rewrite with basePath by default
basePath production > should not rewrite without basePath without disabling
basePath production > should not rewrite with basePath when set to false
basePath production > should rewrite without basePath when set to false
basePath production > should redirect with basePath by default
basePath production > should not redirect without basePath without disabling
basePath production > should not redirect with basePath when set to false
basePath production > should redirect without basePath when set to false
basePath production > should add header with basePath by default
basePath production > should not add header without basePath without disabling
basePath production > should not add header with basePath when set to false
basePath production > should add header without basePath when set to false
basePath production > should not update URL for a 404
basePath production > should handle 404 urls that start with basePath
basePath production > should navigate back to a non-basepath 404 that starts with basepath
basePath production > should update dynamic params after mount correctly
basePath production > should navigate to index page with getStaticProps
basePath production > should navigate to nested index page with getStaticProps
basePath production > should work with nested folder with same name as basePath
basePath production > should work with normal dynamic page
basePath production > should work with hash links
basePath production > should work with catch-all page
basePath production > should redirect trailing slash correctly
basePath production > should redirect trailing slash on root correctly
basePath production > should navigate an absolute url
basePath production > should navigate an absolute local url with basePath
basePath production > should navigate an absolute local url without basePath
basePath production > should 404 when manually adding basePath with
basePath production > should 404 when manually adding basePath with router.push
basePath production > should 404 when manually adding basePath with router.replace
basePath production > should show the hello page under the /docs prefix
basePath production > should have correct router paths on first load of /
basePath production > should have correct router paths on first load of /hello
basePath production > should fetch data for getStaticProps without reloading
basePath production > should fetch data for getServerSideProps without reloading
basePath production > should have correct href for a link
basePath production > should have correct href for a link to /
basePath production > should show 404 for page not under the /docs prefix
basePath production > should show the other-page page under the /docs prefix
basePath production > should have basePath field on Router
basePath production > should navigate to the page without refresh
basePath production > should use urls with basepath in router events
basePath production > should use urls with basepath in router events for hash changes
basePath production > should use urls with basepath in router events for cancelled routes
basePath production > should use urls with basepath in router events for failed route change
basePath production > should allow URL query strings without refresh
basePath production > should correctly replace state when same asPath but different url
basePath production > should respect basePath in amphtml link rel
basePath serverless > should add basePath to routes-manifest
basePath serverless > should prefetch pages correctly when manually called
basePath serverless > should prefetch pages correctly in viewport with
basePath serverless > should 404 for public file without basePath
basePath serverless > should serve public file with basePath correctly
basePath serverless > should rewrite with basePath by default
basePath serverless > should not rewrite without basePath without disabling
basePath serverless > should not rewrite with basePath when set to false
basePath serverless > should rewrite without basePath when set to false
basePath serverless > should redirect with basePath by default
basePath serverless > should not redirect without basePath without disabling
basePath serverless > should not redirect with basePath when set to false
basePath serverless > should redirect without basePath when set to false
basePath serverless > should add header with basePath by default
basePath serverless > should not add header without basePath without disabling
basePath serverless > should not add header with basePath when set to false
basePath serverless > should add header without basePath when set to false
basePath serverless > should not update URL for a 404
basePath serverless > should handle 404 urls that start with basePath
basePath serverless > should navigate back to a non-basepath 404 that starts with basepath
basePath serverless > should update dynamic params after mount correctly
basePath serverless > should navigate to index page with getStaticProps
basePath serverless > should navigate to nested index page with getStaticProps
basePath serverless > should work with nested folder with same name as basePath
basePath serverless > should work with normal dynamic page
basePath serverless > should work with hash links
basePath serverless > should work with catch-all page
basePath serverless > should redirect trailing slash correctly
basePath serverless > should redirect trailing slash on root correctly
basePath serverless > should navigate an absolute url
basePath serverless > should navigate an absolute local url with basePath
basePath serverless > should navigate an absolute local url without basePath
basePath serverless > should 404 when manually adding basePath with
basePath serverless > should 404 when manually adding basePath with router.push
basePath serverless > should 404 when manually adding basePath with router.replace
basePath serverless > should show the hello page under the /docs prefix
basePath serverless > should have correct router paths on first load of /
basePath serverless > should have correct router paths on first load of /hello
basePath serverless > should fetch data for getStaticProps without reloading
basePath serverless > should fetch data for getServerSideProps without reloading
basePath serverless > should have correct href for a link
basePath serverless > should have correct href for a link to /
basePath serverless > should show 404 for page not under the /docs prefix
basePath serverless > should show the other-page page under the /docs prefix
basePath serverless > should have basePath field on Router
basePath serverless > should navigate to the page without refresh
basePath serverless > should use urls with basepath in router events
basePath serverless > should use urls with basepath in router events for hash changes
basePath serverless > should use urls with basepath in router events for cancelled routes
basePath serverless > should use urls with basepath in router events for failed route change
basePath serverless > should allow URL query strings without refresh
basePath serverless > should correctly replace state when same asPath but different url
basePath serverless > should always strip basePath in serverless-loader

Expand output

● basePath production › should add basePath to routes-manifest

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should prefetch pages correctly when manually called

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should prefetch pages correctly in viewport with

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should 404 for public file without basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should serve public file with basePath correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should rewrite with basePath by default

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should not rewrite without basePath without disabling

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should not rewrite with basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should rewrite without basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should redirect with basePath by default

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should not redirect without basePath without disabling

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should not redirect with basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should redirect without basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should add header with basePath by default

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should not add header without basePath without disabling

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should not add header with basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should add header without basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should not update URL for a 404

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should handle 404 urls that start with basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should navigate back to a non-basepath 404 that starts with basepath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should update dynamic params after mount correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should navigate to index page with getStaticProps

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should navigate to nested index page with getStaticProps

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should work with nested folder with same name as basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should work with normal dynamic page

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should work with hash links

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should work with catch-all page

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should redirect trailing slash correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should redirect trailing slash on root correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should navigate an absolute url

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should navigate an absolute local url with basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should navigate an absolute local url without basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should 404 when manually adding basePath with

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should 404 when manually adding basePath with router.push

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should 404 when manually adding basePath with router.replace

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should show the hello page under the /docs prefix

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should have correct router paths on first load of /

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should have correct router paths on first load of /hello

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should fetch data for getStaticProps without reloading

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should fetch data for getServerSideProps without reloading

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should have correct href for a link

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should have correct href for a link to /

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should show 404 for page not under the /docs prefix

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should show the other-page page under the /docs prefix

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should have basePath field on Router

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should navigate to the page without refresh

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should use urls with basepath in router events

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should use urls with basepath in router events for hash changes

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should use urls with basepath in router events for cancelled routes

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should use urls with basepath in router events for failed route change

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should allow URL query strings without refresh

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should correctly replace state when same asPath but different url

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath production › should respect basePath in amphtml link rel

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should add basePath to routes-manifest

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should prefetch pages correctly when manually called

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should prefetch pages correctly in viewport with

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should 404 for public file without basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should serve public file with basePath correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should rewrite with basePath by default

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should not rewrite without basePath without disabling

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should not rewrite with basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should rewrite without basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should redirect with basePath by default

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should not redirect without basePath without disabling

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should not redirect with basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should redirect without basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should add header with basePath by default

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should not add header without basePath without disabling

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should not add header with basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should add header without basePath when set to false

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should not update URL for a 404

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should handle 404 urls that start with basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should navigate back to a non-basepath 404 that starts with basepath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should update dynamic params after mount correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should navigate to index page with getStaticProps

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should navigate to nested index page with getStaticProps

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should work with nested folder with same name as basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should work with normal dynamic page

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should work with hash links

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should work with catch-all page

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should redirect trailing slash correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should redirect trailing slash on root correctly

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should navigate an absolute url

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should navigate an absolute local url with basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should navigate an absolute local url without basePath

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should 404 when manually adding basePath with

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should 404 when manually adding basePath with router.push

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should 404 when manually adding basePath with router.replace

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should show the hello page under the /docs prefix

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should have correct router paths on first load of /

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should have correct router paths on first load of /hello

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should fetch data for getStaticProps without reloading

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should fetch data for getServerSideProps without reloading

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should have correct href for a link

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should have correct href for a link to /

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should show 404 for page not under the /docs prefix

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should show the other-page page under the /docs prefix

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should have basePath field on Router

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should navigate to the page without refresh

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should use urls with basepath in router events

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should use urls with basepath in router events for hash changes

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should use urls with basepath in router events for cancelled routes

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should use urls with basepath in router events for failed route change

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should allow URL query strings without refresh

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should correctly replace state when same asPath but different url

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● basePath serverless › should always strip basePath in serverless-loader

command failed with code 1

  132 |         code !== 0
  133 |       ) {
> 134 |         return reject(new Error(`command failed with code ${code}`))
      |                       ^
  135 |       }
  136 | 
  137 |       resolve({

  at ChildProcess.<anonymous> (lib/next-test-utils.js:134:23)

● Test suite failed to run

TypeError: Cannot read property '__app' of undefined

  309 | 
  310 | export async function stopApp(server) {
> 311 |   if (server.__app) {
      |              ^
  312 |     await server.__app.close()
  313 |   }
  314 |   await promiseCall(server, 'close')

  at stopApp (lib/next-test-utils.js:311:14)
  at integration/basepath/test/index.test.js:1071:18

● Test suite failed to run

TypeError: Cannot read property 'pid' of undefined

  275 | export async function killApp(instance) {
  276 |   await new Promise((resolve, reject) => {
> 277 |     treeKill(instance.pid, (err) => {
      |                       ^
  278 |       if (err) {
  279 |         if (
  280 |           process.platform === 'win32' &&

  at lib/next-test-utils.js:277:23
  at killApp (lib/next-test-utils.js:276:9)
  at integration/basepath/test/index.test.js:1097:11

ijjk · 2020-10-26T11:36:33Z

Stats from current PR

Default Server Mode (Decrease detected ✓)

General Overall increase ⚠️

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
buildDuration	10.4s	11.7s	⚠️ +1.3s
nodeModulesSize	91.3 MB	91.3 MB	⚠️ +37.8 kB

Page Load Tests Overall decrease ⚠️

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
/ failed reqs	0	0	✓
/ total time (seconds)	1.956	2.091	⚠️ +0.14
/ avg req/sec	1277.82	1195.75	⚠️ -82.07
/error-in-render failed reqs	0	0	✓
/error-in-render total time (seconds)	1.08	1.151	⚠️ +0.07
/error-in-render avg req/sec	2315.1	2172.57	⚠️ -142.53

Client Bundles (main, webpack, commons)

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
677f882d2ed8..574c.js gzip	11.2 kB	11.2 kB	✓
framework.HASH.js gzip	39 kB	39 kB	✓
main-3dd37a4..54de.js gzip	7.34 kB	7.34 kB	✓
webpack-e067..f178.js gzip	751 B	751 B	✓
Overall change	58.3 kB	58.3 kB	✓

Client Bundles (main, webpack, commons) Modern

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
677f882d2ed8..dule.js gzip	7.02 kB	7.02 kB	✓
framework.HA..dule.js gzip	39 kB	39 kB	✓
main-082aff9..dule.js gzip	6.32 kB	6.32 kB	✓
webpack-07c5..dule.js gzip	751 B	751 B	✓
Overall change	53.1 kB	53.1 kB	✓

Legacy Client Bundles (polyfills)

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
polyfills-4b..e242.js gzip	31 kB	31 kB	✓
Overall change	31 kB	31 kB	✓

Client Pages

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
_app-9a0b9e1..b37e.js gzip	1.28 kB	1.28 kB	✓
_error-ed1b0..8fbd.js gzip	3.44 kB	3.44 kB	✓
hooks-89731c..c609.js gzip	887 B	887 B	✓
index-17468f..5d83.js gzip	227 B	227 B	✓
link-41ceea0..6b62.js gzip	1.35 kB	1.35 kB	✓
routerDirect..924c.js gzip	284 B	284 B	✓
withRouter-7..c13d.js gzip	284 B	284 B	✓
Overall change	7.75 kB	7.75 kB	✓

Client Pages Modern

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
_app-75d3a82..dule.js gzip	625 B	625 B	✓
_error-4469a..dule.js gzip	2.29 kB	2.29 kB	✓
hooks-cbf13f..dule.js gzip	387 B	387 B	✓
index-b9a643..dule.js gzip	226 B	226 B	✓
link-2d41a89..dule.js gzip	1.31 kB	1.31 kB	✓
routerDirect..dule.js gzip	284 B	284 B	✓
withRouter-f..dule.js gzip	282 B	282 B	✓
Overall change	5.4 kB	5.4 kB	✓

Client Build Manifests

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
_buildManifest.js gzip	323 B	323 B	✓
_buildManife..dule.js gzip	329 B	329 B	✓
Overall change	652 B	652 B	✓

Rendered Page Sizes

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
index.html gzip	1 kB	1 kB	✓
link.html gzip	1.01 kB	1.01 kB	✓
withRouter.html gzip	994 B	994 B	✓
Overall change	3 kB	3 kB	✓

Serverless Mode

General Overall increase ⚠️

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
buildDuration	12.8s	12.9s	⚠️ +11ms
nodeModulesSize	91.3 MB	91.3 MB	⚠️ +37.8 kB

Client Bundles (main, webpack, commons)

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
677f882d2ed8..574c.js gzip	11.2 kB	11.2 kB	✓
framework.HASH.js gzip	39 kB	39 kB	✓
main-3dd37a4..54de.js gzip	7.34 kB	7.34 kB	✓
webpack-e067..f178.js gzip	751 B	751 B	✓
Overall change	58.3 kB	58.3 kB	✓

Client Bundles (main, webpack, commons) Modern

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
677f882d2ed8..dule.js gzip	7.02 kB	7.02 kB	✓
framework.HA..dule.js gzip	39 kB	39 kB	✓
main-082aff9..dule.js gzip	6.32 kB	6.32 kB	✓
webpack-07c5..dule.js gzip	751 B	751 B	✓
Overall change	53.1 kB	53.1 kB	✓

Legacy Client Bundles (polyfills)

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
polyfills-4b..e242.js gzip	31 kB	31 kB	✓
Overall change	31 kB	31 kB	✓

Client Pages

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
_app-9a0b9e1..b37e.js gzip	1.28 kB	1.28 kB	✓
_error-ed1b0..8fbd.js gzip	3.44 kB	3.44 kB	✓
hooks-89731c..c609.js gzip	887 B	887 B	✓
index-17468f..5d83.js gzip	227 B	227 B	✓
link-41ceea0..6b62.js gzip	1.35 kB	1.35 kB	✓
routerDirect..924c.js gzip	284 B	284 B	✓
withRouter-7..c13d.js gzip	284 B	284 B	✓
Overall change	7.75 kB	7.75 kB	✓

Client Pages Modern

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
_app-75d3a82..dule.js gzip	625 B	625 B	✓
_error-4469a..dule.js gzip	2.29 kB	2.29 kB	✓
hooks-cbf13f..dule.js gzip	387 B	387 B	✓
index-b9a643..dule.js gzip	226 B	226 B	✓
link-2d41a89..dule.js gzip	1.31 kB	1.31 kB	✓
routerDirect..dule.js gzip	284 B	284 B	✓
withRouter-f..dule.js gzip	282 B	282 B	✓
Overall change	5.4 kB	5.4 kB	✓

Client Build Manifests

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
_buildManifest.js gzip	323 B	323 B	✓
_buildManife..dule.js gzip	329 B	329 B	✓
Overall change	652 B	652 B	✓

Serverless bundles

	vercel/next.js canary	omBratteng/next.js upgrade-amp-toolbox-optimizer	Change
_error.js	1.06 MB	1.06 MB	✓
404.html	4.73 kB	4.73 kB	✓
hooks.html	3.92 kB	3.92 kB	✓
index.js	1.06 MB	1.06 MB	✓
link.js	1.1 MB	1.1 MB	✓
routerDirect.js	1.1 MB	1.1 MB	✓
withRouter.js	1.1 MB	1.1 MB	✓
Overall change	5.42 MB	5.42 MB	✓

Commit: 1c583b6

Upgrade @ampproject/toolbox-optimizer to 2.7.0-alpha.4

1c583b6

This version has uses node-fetch v2.6.1 Related to #17416

omBratteng requested review from ijjk, lfades, Timer and timneutkens as code owners October 26, 2020 11:27

ijjk added the type: next label Oct 26, 2020

omBratteng closed this Oct 26, 2020

vercel locked as resolved and limited conversation to collaborators Jan 29, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Upgrade @ampproject/toolbox-optimizer to 2.7.0-alpha.4 #18248

Upgrade @ampproject/toolbox-optimizer to 2.7.0-alpha.4 #18248

omBratteng commented Oct 26, 2020

ijjk commented Oct 26, 2020 •

edited

Loading

ijjk commented Oct 26, 2020

Upgrade @ampproject/toolbox-optimizer to 2.7.0-alpha.4 #18248

Upgrade @ampproject/toolbox-optimizer to 2.7.0-alpha.4 #18248

Conversation

omBratteng commented Oct 26, 2020

ijjk commented Oct 26, 2020 • edited Loading

Failing test suites

ijjk commented Oct 26, 2020

Stats from current PR

ijjk commented Oct 26, 2020 •

edited

Loading