[Pkg-javascript-devel] Bug#952178: node-express: FTBFS: dh_auto_test: error: /bin/sh -e debian/tests/pkg-js/test returned exit code 1

[Lucas Nussbaum]

Source: node-express
Version: 4.17.1-2
Severity: serious
Justification: FTBFS on amd64
Tags: bullseye sid ftbfs
Usertags: ftbfs-20200222 ftbfs-bullseye

Hi,

During a rebuild of all packages in sid, your package failed to build
on amd64.

Relevant part (hopefully):
> dpkg-buildpackage
> -----------------
> 
> Command: dpkg-buildpackage -us -uc -sa -rfakeroot
> dpkg-buildpackage: info: source package node-express
> dpkg-buildpackage: info: source version 4.17.1-2
> dpkg-buildpackage: info: source distribution unstable
> dpkg-buildpackage: info: source changed by Xavier Guimard <yadd at debian.org>
>  dpkg-source --before-build .
> dpkg-buildpackage: info: host architecture amd64
>  debian/rules clean
> dh clean --with nodejs
>    dh_auto_clean --buildsystem=nodejs
> 	rm -rf ./node_modules/.cache
>    dh_clean
>  dpkg-source -b .
> dpkg-source: info: using source format '3.0 (quilt)'
> dpkg-source: info: building node-express using existing ./node-express_4.17.1.orig.tar.gz
> dpkg-source: info: using patch list from debian/patches/series
> dpkg-source: info: building node-express in node-express_4.17.1-2.debian.tar.xz
> dpkg-source: info: building node-express in node-express_4.17.1-2.dsc
>  debian/rules binary
> dh binary --with nodejs
>    dh_update_autotools_config
>    dh_autoreconf
>    dh_auto_configure --buildsystem=nodejs
>    dh_auto_build --buildsystem=nodejs
> No build command found, searching known files
>    dh_auto_test --buildsystem=nodejs
> 	/bin/sh -e debian/tests/pkg-js/test
> 
> 
> (node:16816) DeprecationWarning: Configuration via mocha.opts is DEPRECATED and will be removed from a future version of Mocha. Use RC files or package.json instead.
>   Route
>     ✓ should work without handlers
>     .all
>       ✓ should add handler
>       ✓ should handle VERBS
>       ✓ should stack
>     .VERB
>       ✓ should support .get
>       ✓ should limit to just .VERB
>       ✓ should allow fallthrough
>     errors
>       ✓ should handle errors via arity 4 functions
>       ✓ should handle throw
>       ✓ should handle throwing inside error handlers
>       ✓ should handle throw in .all
>       ✓ should handle single error handler
> 
>   Router
>     ✓ should return a function with router methods
>     ✓ should support .use of other routers
>     ✓ should support dynamic routes
>     ✓ should handle blank URL
>     ✓ should handle missing URL
>     ✓ should not stack overflow with many registered routes (225ms)
>     .handle
>       ✓ should dispatch
>     .multiple callbacks
>       ✓ should throw if a callback is null
>       ✓ should throw if a callback is undefined
>       ✓ should throw if a callback is not a function
>       ✓ should not throw if all callbacks are functions
>     error
>       ✓ should skip non error middleware
>       ✓ should handle throwing inside routes with params
>       ✓ should handle throwing in handler after async param
>       ✓ should handle throwing inside error handlers
>     FQDN
>       ✓ should not obscure FQDNs
>       ✓ should ignore FQDN in search
>       ✓ should ignore FQDN in path
>       ✓ should adjust FQDN req.url
>       ✓ should adjust FQDN req.url with multiple handlers
>       ✓ should adjust FQDN req.url with multiple routed handlers
>     .all
>       ✓ should support using .all to capture all http verbs
>       ✓ should be called for any URL when "*"
>     .use
>       ✓ should require middleware
>       ✓ should reject string as middleware
>       ✓ should reject number as middleware
>       ✓ should reject null as middleware
>       ✓ should reject Date as middleware
>       ✓ should be called for any URL
>       ✓ should accept array of middleware
>     .param
>       ✓ should call param function when routing VERBS
>       ✓ should call param function when routing middleware
>       ✓ should only call once per request
>       ✓ should call when values differ
>     parallel requests
>       ✓ should not mix requests (51ms)
> 
>   app.all()
>     ✓ should add a router per method (23ms)
>     ✓ should run the callback for a method just once
> 
>   app.del()
>     ✓ should alias app.delete()
> 
>   app
>     .engine(ext, fn)
>       ✓ should map a template engine
>       ✓ should throw when the callback is missing
>       ✓ should work without leading "."
>       ✓ should work "view engine" setting
>       ✓ should work "view engine" with leading "."
> 
>   HEAD
>     ✓ should default to GET
>     ✓ should output the same headers as GET requests
> 
>   app.head()
>     ✓ should override
> 
>   app
>     ✓ should inherit from event emitter
>     ✓ should be callable
>     ✓ should 404 without routes
> 
>   app.parent
>     ✓ should return the parent when mounted
> 
>   app.mountpath
>     ✓ should return the mounted path
> 
>   app.router
>     ✓ should throw with notice
> 
>   app.path()
>     ✓ should return the canonical
> 
>   in development
>     ✓ should disable "view cache"
> 
>   in production
>     ✓ should enable "view cache"
> 
>   without NODE_ENV
>     ✓ should default to development
> 
>   app.listen()
>     1) should wrap with an HTTP server
> 
> 
>   68 passing (402ms)
>   1 failing
> 
>   1) app.listen()
>        should wrap with an HTTP server:
>      Uncaught Error: listen EADDRINUSE: address already in use :::9999
>       at Server.setupListenHandle [as _listen2] (net.js:1280:14)
>       at listenInCluster (net.js:1328:12)
>       at Server.listen (net.js:1415:7)
>       at Function.listen (lib/application.js:618:24)
>       at Context.<anonymous> (test/app.listen.js:12:22)
>       at callFnAsync (/usr/share/nodejs/mocha/lib/runnable.js:423:21)
>       at Test.Runnable.run (/usr/share/nodejs/mocha/lib/runnable.js:363:7)
>       at Runner.runTest (/usr/share/nodejs/mocha/lib/runner.js:541:10)
>       at /usr/share/nodejs/mocha/lib/runner.js:667:12
>       at next (/usr/share/nodejs/mocha/lib/runner.js:450:14)
>       at /usr/share/nodejs/mocha/lib/runner.js:460:7
>       at next (/usr/share/nodejs/mocha/lib/runner.js:362:14)
>       at Immediate.<anonymous> (/usr/share/nodejs/mocha/lib/runner.js:428:5)
> 
> 
> 
> dh_auto_test: error: /bin/sh -e debian/tests/pkg-js/test returned exit code 1

The full build log is available from:
   http://qa-logs.debian.net/2020/02/22/node-express_4.17.1-2_unstable.log

A list of current common problems and possible solutions is available at
http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!

About the archive rebuild: The rebuild was done on EC2 VM instances from
Amazon Web Services, using a clean, minimal and up-to-date chroot. Every
failed build was retried once to eliminate random failures.