Bug#947118: [Pkg-javascript-devel] Bug#947118: node-on-headers: autopkgtest started failing and then times out

2019-12-21 Thread Xavier
Le 21/12/2019 à 12:04, Utkarsh Gupta a écrit :
> Hiya,
> 
> On Sat, Dec 21, 2019 at 3:51 PM Paul Gevers  > wrote:
> 
> Your package has an autopkgtest, great. However, I noticed that without
> any change from your side it started failing, and while doing so also
> doesn't finish, hence timing out on ci.debian.net
> . Obviously this isn't
> your fault, but please investigate, such that you can assign this bug to
> the right package. Please consider improving the test suite to not hang
> in case of failure. This is not the only node package that has this,
> e.g. check https://ci.debian.net/status/slow/ to see more. Don't
> hesitate to ask for help from the Debian CI team (in X-Debbugs-CC) if
> you need help solving this issue.
> 
> 
> I've just added a --timeout 1, which should be enough to fix this
> (and switched the package to use pkg-js-tools).
> 
> Could someone confirm if that indeed works?
> 
> Best,
> Utkarsh

No, this is a node-supertest problem. Fixed and uploaded (see the link,
test ≥ 2h)



Bug#947118: [Pkg-javascript-devel] Bug#947118: node-on-headers: autopkgtest started failing and then times out

2019-12-21 Thread Utkarsh Gupta
Hiya,

On Sat, Dec 21, 2019 at 3:51 PM Paul Gevers  wrote:

> Your package has an autopkgtest, great. However, I noticed that without
> any change from your side it started failing, and while doing so also
> doesn't finish, hence timing out on ci.debian.net. Obviously this isn't
> your fault, but please investigate, such that you can assign this bug to
> the right package. Please consider improving the test suite to not hang
> in case of failure. This is not the only node package that has this,
> e.g. check https://ci.debian.net/status/slow/ to see more. Don't
> hesitate to ask for help from the Debian CI team (in X-Debbugs-CC) if
> you need help solving this issue.
>

I've just added a --timeout 1, which should be enough to fix this (and
switched the package to use pkg-js-tools).

Could someone confirm if that indeed works?


Best,
Utkarsh

>


Bug#947118: node-on-headers: autopkgtest started failing and then times out

2019-12-21 Thread Xavier
Hi,

I updated node-supertest, this will fix most of node-* packages
referenced in https://ci.debian.net/status/slow/

Cheers,
Xavier



Bug#947118: node-on-headers: autopkgtest started failing and then times out

2019-12-21 Thread Paul Gevers
Source: node-on-headers
Version: 1.0.1-1
X-Debbugs-CC: debian...@lists.debian.org
Severity: serious
User: debian...@lists.debian.org
Usertags: timeout regression

Dear maintainers,

Your package has an autopkgtest, great. However, I noticed that without
any change from your side it started failing, and while doing so also
doesn't finish, hence timing out on ci.debian.net. Obviously this isn't
your fault, but please investigate, such that you can assign this bug to
the right package. Please consider improving the test suite to not hang
in case of failure. This is not the only node package that has this,
e.g. check https://ci.debian.net/status/slow/ to see more. Don't
hesitate to ask for help from the Debian CI team (in X-Debbugs-CC) if
you need help solving this issue.

To avoid wasting lots of time on the ci.debian.net infrastructure, I
consider adding your package to the blacklist. If that happens, please
ping me to try any uploads you make that should fix the issue if you are
unsure and don't want to close this bug until verified.

Paul

https://ci.debian.net/data/autopkgtest/testing/amd64/n/node-on-headers/3635022/log.gz

autopkgtest [01:16:29]: test testsuite: [---


  onHeaders(res, listener)
✓ should fire after setHeader
✓ should fire before write
✓ should fire with no headers
✓ should fire only once
✓ should fire in reverse order
arguments
  res
✓ should be required
  listener
1) should be required
2) should only accept function
setHeader
  ✓ should be available in listener
writeHead(status)
  ✓ should make status available in listener
  ✓ should allow manipulation of status in listener
  3) should pass-through core error
writeHead(status, reason)
  ✓ should be available in listener
writeHead(status, reason, obj)
  ✓ should be available in listener
writeHead(status, obj)
  ✓ should be available in listener
  ✓ should handle falsy keys
writeHead(status, arr)
  ✓ should be available in listener


  14 passing (76ms)
  3 failing

  1) onHeaders(res, listener)
   arguments
 listener
   should be required:
 Error: Internal Server Error
  at Test.Request.callback
(/usr/share/nodejs/superagent/lib/node/index.js:839:15)
  at IncomingMessage.parser
(/usr/share/nodejs/superagent/lib/node/index.js:1075:18)
  at endReadableNT (_stream_readable.js:1143:12)
  at process._tickCallback (internal/process/next_tick.js:63:19)

  2) onHeaders(res, listener)
   arguments
 listener
   should only accept function:
 Error: Internal Server Error
  at Test.Request.callback
(/usr/share/nodejs/superagent/lib/node/index.js:839:15)
  at IncomingMessage.parser
(/usr/share/nodejs/superagent/lib/node/index.js:1075:18)
  at endReadableNT (_stream_readable.js:1143:12)
  at process._tickCallback (internal/process/next_tick.js:63:19)

  3) onHeaders(res, listener)
   writeHead(status)
 should pass-through core error:
 Error: Internal Server Error
  at Test.Request.callback
(/usr/share/nodejs/superagent/lib/node/index.js:839:15)
  at IncomingMessage.parser
(/usr/share/nodejs/superagent/lib/node/index.js:1075:18)
  at endReadableNT (_stream_readable.js:1143:12)
  at process._tickCallback (internal/process/next_tick.js:63:19)




signature.asc
Description: OpenPGP digital signature