<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">Le dim. 6 juil. 2025 à 08:13, Yadd <<a href="mailto:yadd@debian.org">yadd@debian.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 7/5/25 15:32, Paul Gevers wrote:<br>
> Source: node-superagent<br>
> Version: 9.0.1-1<br>
> Severity: serious<br>
> User: <a href="mailto:debian-ci@lists.debian.org" target="_blank">debian-ci@lists.debian.org</a><br>
> Usertags: regression<br>
> <br>
> Dear maintainer(s),<br>
> <br>
> Your package has an autopkgtest, great. However, it fails since June <br>
> 2025. Can you please investigate the situation and fix it? I copied some <br>
> of the output at the bottom of this report.<br>
> <br>
> The release team has announced [1] that failing autopkgtest on amd64 and <br>
> arm64 are considered RC in testing.<br>
> <br>
> More information about this bug and the reason for filing it can be <br>
> found on <a href="https://wiki.debian.org/ContinuousIntegration/" rel="noreferrer" target="_blank">https://wiki.debian.org/ContinuousIntegration/</a> <br>
> RegressionEmailInformation<br>
> <br>
> With my Release Team member hat on I allow you to tag this bug as <br>
> trixie-ignore if this is nearly completely a test issue as it's so late <br>
> in the freeze and it's not worth removing the package from trixie <br>
> because of a failing test at this moment unless it exposes a much bigger <br>
> issue. Having said that, if this is a test only issue and a fix is <br>
> possible without fully removing the autopkgtest and without making the <br>
> test superficial, it's still welcome, but it would need to happen soon.<br>
> <br>
> Paul<br>
> <br>
> [1] <a href="https://lists.debian.org/debian-devel-announce/2019/07/msg00002.html" rel="noreferrer" target="_blank">https://lists.debian.org/debian-devel-announce/2019/07/msg00002.html</a><br>
> <br>
> <a href="https://ci.debian.net/data/autopkgtest/testing/amd64/n/node-" rel="noreferrer" target="_blank">https://ci.debian.net/data/autopkgtest/testing/amd64/n/node-</a> <br>
> superagent/62133275/log.gz<br>
> <br>
> 105s   1) Agent<br>
> 105s        should remember defaults:<br>
> 105s<br>
> 105s       AssertionError [ERR_ASSERTION]: 0 == 1<br>
> 105s       + expected - actual<br>
> 105s<br>
> 105s       -0<br>
> 105s       +1<br>
> 105s<br>
> 105s       at Context.<anonymous> (test/agent-base.js:33:12)<br>
> 105s       at callFn (/usr/share/nodejs/mocha/lib/runnable.js:364:21)<br>
> 105s       at Runnable.run (/usr/share/nodejs/mocha/lib/runnable.js:352:5)<br>
> 105s       at Runner.runTest (/usr/share/nodejs/mocha/lib/runner.js:677:10)<br>
> 105s       at /usr/share/nodejs/mocha/lib/runner.js:800:12<br>
> 105s       at next (/usr/share/nodejs/mocha/lib/runner.js:592:14)<br>
> 105s       at /usr/share/nodejs/mocha/lib/runner.js:602:7<br>
> 105s       at next (/usr/share/nodejs/mocha/lib/runner.js:485:14)<br>
> 105s       at Immediate._onImmediate (/usr/share/nodejs/mocha/lib/ <br>
> runner.js:570:5)<br>
> 105s       at process.processImmediate (node:internal/timers:483:21)<br>
> 105s<br>
> 105s<br>
> 105s<br>
> 105s autopkgtest [13:05:26]: test pkg-js-autopkgtest: <br>
> -----------------------]<br>
Hi Jérémy,<br>
<br>
the broken test is test/agent-base.js, even with more recent <br>
node-superagent and a "npm instll", test still fail.<br>
<br>
The problem is that the initialization of "agent" return undefined when <br>
".query({ hello: 'world' })" is set. I can fix this with this:<br>
<br>
<br>
     const agent = request<br>
       .agent()<br>
       .accept('json')<br>
       .use(() => {<br>
         console.error('called_use');<br>
         called++;<br>
       })<br>
       .once('request', () => {<br>
         console.error('called_request');<br>
         event_called++;<br>
       })<br>
   //  DISABLED HERE<br>
   //  .query({ hello: 'world' })<br>
       .set('X-test', 'testing');<br>
<br>
     return agent<br>
       .get(`${base}/echo`)<br>
   //  ADDED HERE<br>
       .query({ hello: 'world' })<br>
       .then((res) => {<br>
         assert.equal(1, called);<br>
         assert.equal(1, event_called);<br>
         assert.equal('application/json', res.headers.accept);<br>
         assert.equal('testing', res.headers['x-test']);<br>
<br>
         return agent.get(`${base}/querystring`)<br>
   //  ADDED HERE ALSO<br>
           .query({hello:'world'});<br>
       })<br>
       .then((res) => {<br>
         assert.equal(2, called);<br>
         assert.equal(2, event_called);<br>
         assert.deepEqual({ hello: 'world' }, res.body);<br>
       });<br>
<br>
but then it proves the reverse than wanted: 'should remember defaults' <br>
becomes 'no rememebr default' or 'unable to set default'<br>
<br>
Is there a change in Nodejs "http" object ?<br></blockquote><div><br></div><div>Yes, there is. This returns a new QUERY method now.</div><div>I'm doing a patch right now.</div><div><br></div></div></div>