Re: [streams-tests] WritableStream: Add tests for re-entrant calls inside strategySize (#4830)

# Firefox (nightly channel) #
Testing web-platform-tests at revision e59734f63d91f0aef5b3c718cd53349b23c0296f
Using browser at version BuildID 20170123125947; SourceStamp 36486fdc3813ef7943ae5b07b4128866d1938a6c
Starting 10 test iterations
All results were stable
## All results ##
<details>
<summary>8 tests ran</summary>
<details>
<summary><a href="http://w3c-test.org/submissions/4830/streams/writable-streams/reentrant-strategy.sharedworker.html">/streams/writable-streams/reentrant-strategy.sharedworker.html</a></summary>


|                                     Subtest                                      | Results |
|----------------------------------------------------------------------------------|---------|
|                                                                                  | OK      |
| `original reader should error when new reader is created within strategy.size()` | FAIL    |
| `controller.error() should work when called from within strategy.size()`         | FAIL    |
| `releaseLock() should abort the write() when called within strategy.size()`      | FAIL    |
| `close() should work when called from within strategy.size()`                    | FAIL    |
| `writes should be written in the standard order`                                 | FAIL    |
| `writer.write() promises should resolve in the standard order`                   | FAIL    |
| `abort() should work when called from within strategy.size()`                    | FAIL    |

</details>
<details>
<summary><a href="http://w3c-test.org/submissions/4830/streams/writable-streams/bad-strategies.html">/streams/writable-streams/bad-strategies.html</a></summary>


|                          Subtest                          | Results |
|-----------------------------------------------------------|---------|
|                                                           | OK      |
| `reject any non-function value for strategy.size`         | FAIL    |
| `Writable stream: invalid strategy.size return value`     | FAIL    |
| `Writable stream: invalid strategy.highWaterMark`         | FAIL    |
| `Writable stream: throwing strategy.highWaterMark getter` | FAIL    |
| `Writable stream: throwing strategy.size method`          | FAIL    |
| `Writable stream: throwing strategy.size getter`          | FAIL    |

</details>
<details>
<summary><a href="http://w3c-test.org/submissions/4830/streams/writable-streams/reentrant-strategy.html">/streams/writable-streams/reentrant-strategy.html</a></summary>


|                                     Subtest                                      | Results |
|----------------------------------------------------------------------------------|---------|
|                                                                                  | OK      |
| `original reader should error when new reader is created within strategy.size()` | FAIL    |
| `controller.error() should work when called from within strategy.size()`         | FAIL    |
| `releaseLock() should abort the write() when called within strategy.size()`      | FAIL    |
| `close() should work when called from within strategy.size()`                    | FAIL    |
| `writes should be written in the standard order`                                 | FAIL    |
| `writer.write() promises should resolve in the standard order`                   | FAIL    |
| `abort() should work when called from within strategy.size()`                    | FAIL    |

</details>
<details>
<summary><a href="http://w3c-test.org/submissions/4830/streams/writable-streams/bad-strategies.sharedworker.html">/streams/writable-streams/bad-strategies.sharedworker.html</a></summary>


|                          Subtest                          | Results |
|-----------------------------------------------------------|---------|
|                                                           | OK      |
| `reject any non-function value for strategy.size`         | FAIL    |
| `Writable stream: invalid strategy.size return value`     | FAIL    |
| `Writable stream: invalid strategy.highWaterMark`         | FAIL    |
| `Writable stream: throwing strategy.highWaterMark getter` | FAIL    |
| `Writable stream: throwing strategy.size method`          | FAIL    |
| `Writable stream: throwing strategy.size getter`          | FAIL    |

</details>
<details>
<summary><a href="https://w3c-test.org/submissions/4830/streams/writable-streams/bad-strategies.serviceworker.https.html">/streams/writable-streams/bad-strategies.serviceworker.https.html</a></summary>


|                          Subtest                          | Results |
|-----------------------------------------------------------|---------|
|                                                           | OK      |
| `reject any non-function value for strategy.size`         | FAIL    |
| `Writable stream: invalid strategy.size return value`     | FAIL    |
| `Writable stream: invalid strategy.highWaterMark`         | FAIL    |
| `Writable stream: throwing strategy.highWaterMark getter` | FAIL    |
| `Writable stream: throwing strategy.size method`          | FAIL    |
| `Service worker test setup`                               | PASS    |
| `Writable stream: throwing strategy.size getter`          | FAIL    |

</details>
<details>
<summary><a href="http://w3c-test.org/submissions/4830/streams/writable-streams/reentrant-strategy.dedicatedworker.html">/streams/writable-streams/reentrant-strategy.dedicatedworker.html</a></summary>


|                                     Subtest                                      | Results |
|----------------------------------------------------------------------------------|---------|
|                                                                                  | OK      |
| `original reader should error when new reader is created within strategy.size()` | FAIL    |
| `controller.error() should work when called from within strategy.size()`         | FAIL    |
| `releaseLock() should abort the write() when called within strategy.size()`      | FAIL    |
| `close() should work when called from within strategy.size()`                    | FAIL    |
| `writes should be written in the standard order`                                 | FAIL    |
| `writer.write() promises should resolve in the standard order`                   | FAIL    |
| `abort() should work when called from within strategy.size()`                    | FAIL    |

</details>
<details>
<summary><a href="https://w3c-test.org/submissions/4830/streams/writable-streams/reentrant-strategy.serviceworker.https.html">/streams/writable-streams/reentrant-strategy.serviceworker.https.html</a></summary>


|                                     Subtest                                      | Results |
|----------------------------------------------------------------------------------|---------|
|                                                                                  | OK      |
| `original reader should error when new reader is created within strategy.size()` | FAIL    |
| `controller.error() should work when called from within strategy.size()`         | FAIL    |
| `releaseLock() should abort the write() when called within strategy.size()`      | FAIL    |
| `close() should work when called from within strategy.size()`                    | FAIL    |
| `Service worker test setup`                                                      | PASS    |
| `writes should be written in the standard order`                                 | FAIL    |
| `writer.write() promises should resolve in the standard order`                   | FAIL    |
| `abort() should work when called from within strategy.size()`                    | FAIL    |

</details>
<details>
<summary><a href="http://w3c-test.org/submissions/4830/streams/writable-streams/bad-strategies.dedicatedworker.html">/streams/writable-streams/bad-strategies.dedicatedworker.html</a></summary>


|                          Subtest                          | Results |
|-----------------------------------------------------------|---------|
|                                                           | OK      |
| `reject any non-function value for strategy.size`         | FAIL    |
| `Writable stream: invalid strategy.size return value`     | FAIL    |
| `Writable stream: invalid strategy.highWaterMark`         | FAIL    |
| `Writable stream: throwing strategy.highWaterMark getter` | FAIL    |
| `Writable stream: throwing strategy.size method`          | FAIL    |
| `Writable stream: throwing strategy.size getter`          | FAIL    |

</details>
</details>

View on GitHub: https://github.com/w3c/web-platform-tests/pull/4830#issuecomment-279894709

Received on Wednesday, 15 February 2017 01:52:38 UTC