test(client): add parser buffer reallocation case #703

dnlup · 2021-04-08T12:02:52Z

dnlup · 2021-04-08T12:06:06Z

test/client.js

@@ -1063,3 +1063,38 @@ test('emit disconnect after destory', t => {
 })
 })
 })
+
+test('parser dinamic allocation', { only: true }, t => {
+ t.plan(3)


Should we go deeper and actually check the parser bufferSize property?

No I don't think that is necessary. But, wouldn't hurt either.

I'll add it in #704

test(client): add parser buffer reallocation case

aefb245

ronag merged commit 84ae945 into main Apr 8, 2021

dnlup commented Apr 8, 2021

View reviewed changes

ronag deleted the more_parser_coverage branch April 8, 2021 12:09

crysmags pushed a commit to crysmags/undici that referenced this pull request Feb 27, 2024

test(client): add parser buffer reallocation case (nodejs#703)

07d1994

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test(client): add parser buffer reallocation case #703

test(client): add parser buffer reallocation case #703

dnlup commented Apr 8, 2021 •

edited

Loading

dnlup Apr 8, 2021

ronag Apr 8, 2021

dnlup Apr 8, 2021

test(client): add parser buffer reallocation case #703

test(client): add parser buffer reallocation case #703

Conversation

dnlup commented Apr 8, 2021 • edited Loading

dnlup Apr 8, 2021

Choose a reason for hiding this comment

ronag Apr 8, 2021

Choose a reason for hiding this comment

dnlup Apr 8, 2021

Choose a reason for hiding this comment

dnlup commented Apr 8, 2021 •

edited

Loading