Thanks for contribution!!! Please, do use make lint/vcbuild lint before pr.

@sreepurnajasti Thanks for the comments, I am pretty sure that I did run make lint, maybe I missed something 🤔 . I will run it again.

Do you think this would be ok ?

...
  const countdown = new Countdown(
    outCount * inCount,
    common.mustCall(() => server.close())
  );
  let onRequest = common.mustNotCall(); // Temporary
  const p = new Promise((resolve) => {
    onRequest = common.mustCall((res) => {
      if (countdown.dec() === 0) {
        resolve();
      }

      if (!shouldFail)
        res.resume();
    }, outCount * inCount);
  });
...

That should work. You can also run make -j4 test to make sure the test runs as expected. Or to run a single test, run tools/test.py --mode=release parallel/test-http-agent.

Thanks @apapirovski, I have already run all tests and linter 😄 after amending the last commit

@fedekau LGTM. Thanks!!

CI: https://ci.nodejs.org/job/node-test-pull-request/11991/

@apapirovski Do you think that this failure is related to my change? To me seems like it is not, or is it? 🤔

@fedekau Completely unrelated, this PR is all good :)

Landed in f373a1d

Thanks for the contribution @fedekau! Congrats on becoming a Contributor! 🎉

Thanks @apapirovski and @sreepurnajasti I hope this is the first of many PRs 💪