fixup! fixup! fixup! fixup! stream: add iterator helper find

Linkgoron · Linkgoron · commit e0b110985619 · 2022-02-05T21:47:50.000+02:00
diff --git a/test/parallel/test-stream-filter.js b/test/parallel/test-stream-filter.js
@@ -100,7 +100,9 @@ const { setTimeout } = require('timers/promises');
 }
 {
   const stream = Readable.from([1, 2, 3, 4, 5]);
-  stream.map = common.mustNotCall(() => {});
+  Object.defineProperty(stream, 'map', {
+    value: common.mustNotCall(() => {}),
+  });
   // Check that map isn't getting called.
   stream.filter(() => true);
 }
diff --git a/test/parallel/test-stream-flatMap.js b/test/parallel/test-stream-flatMap.js
@@ -123,7 +123,9 @@ function oneTo5() {
 }
 {
   const stream = oneTo5();
-  stream.map = common.mustNotCall(() => {});
+  Object.defineProperty(stream, 'map', {
+    value: common.mustNotCall(() => {}),
+  });
   // Check that map isn't getting called.
   stream.flatMap(() => true);
 }
diff --git a/test/parallel/test-stream-forEach.js b/test/parallel/test-stream-forEach.js
@@ -86,7 +86,9 @@ const { setTimeout } = require('timers/promises');
 }
 {
   const stream = Readable.from([1, 2, 3, 4, 5]);
-  stream.map = common.mustNotCall(() => {});
+  Object.defineProperty(stream, 'map', {
+    value: common.mustNotCall(() => {}),
+  });
   // Check that map isn't getting called.
   stream.forEach(() => true);
 }
diff --git a/test/parallel/test-stream-some-find-every.mjs b/test/parallel/test-stream-some-find-every.mjs
@@ -165,7 +165,9 @@ function oneTo5Async() {
 {
   for (const op of ['some', 'every', 'find']) {
     const stream = oneTo5();
-    stream.map = common.mustNotCall(() => {});
+    Object.defineProperty(stream, 'map', {
+      value: common.mustNotCall(() => {}),
+    });
     // Check that map isn't getting called.
     stream[op](() => {});
   }

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,9 @@ const { setTimeout } = require('timers/promises');`
`100`	`100`	`}`
`101`	`101`	`{`
`102`	`102`	`const stream = Readable.from([1, 2, 3, 4, 5]);`
`103`		`- stream.map = common.mustNotCall(() => {});`
	`103`	`+ Object.defineProperty(stream, 'map', {`
	`104`	`+ value: common.mustNotCall(() => {}),`
	`105`	`+ });`
`104`	`106`	`// Check that map isn't getting called.`
`105`	`107`	`stream.filter(() => true);`
`106`	`108`	`}`
Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,9 @@ function oneTo5() {`
`123`	`123`	`}`
`124`	`124`	`{`
`125`	`125`	`const stream = oneTo5();`
`126`		`- stream.map = common.mustNotCall(() => {});`
	`126`	`+ Object.defineProperty(stream, 'map', {`
	`127`	`+ value: common.mustNotCall(() => {}),`
	`128`	`+ });`
`127`	`129`	`// Check that map isn't getting called.`
`128`	`130`	`stream.flatMap(() => true);`
`129`	`131`	`}`
Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,9 @@ const { setTimeout } = require('timers/promises');`
`86`	`86`	`}`
`87`	`87`	`{`
`88`	`88`	`const stream = Readable.from([1, 2, 3, 4, 5]);`
`89`		`- stream.map = common.mustNotCall(() => {});`
	`89`	`+ Object.defineProperty(stream, 'map', {`
	`90`	`+ value: common.mustNotCall(() => {}),`
	`91`	`+ });`
`90`	`92`	`// Check that map isn't getting called.`
`91`	`93`	`stream.forEach(() => true);`
`92`	`94`	`}`
Original file line number	Diff line number	Diff line change
`@@ -165,7 +165,9 @@ function oneTo5Async() {`
`165`	`165`	`{`
`166`	`166`	`for (const op of ['some', 'every', 'find']) {`
`167`	`167`	`const stream = oneTo5();`
`168`		`- stream.map = common.mustNotCall(() => {});`
	`168`	`+ Object.defineProperty(stream, 'map', {`
	`169`	`+ value: common.mustNotCall(() => {}),`
	`170`	`+ });`
`169`	`171`	`// Check that map isn't getting called.`
`170`	`172`	`stream[op](() => {});`
`171`	`173`	`}`