From 7bcc4ccd8e83ca650ac9980110ba9db8058980af Mon Sep 17 00:00:00 2001 From: Joyee Cheung Date: Fri, 2 Nov 2018 18:04:39 +0800 Subject: [PATCH] doc: remove legacy WPT integration guide Point to the new guide in test/wpt/README.md instead. PR-URL: https://github.com/nodejs/node/pull/24035 Refs: https://github.com/nodejs/node/issues/23192 Reviewed-By: Daijiro Wachi --- doc/guides/writing-tests.md | 23 +---------------------- 1 file changed, 1 insertion(+), 22 deletions(-) diff --git a/doc/guides/writing-tests.md b/doc/guides/writing-tests.md index 4d0c0307bbd765..260dfa236fa615 100644 --- a/doc/guides/writing-tests.md +++ b/doc/guides/writing-tests.md @@ -310,27 +310,7 @@ functions worked correctly with the `beforeExit` event, then it might be named ### Web Platform Tests -Some of the tests for the WHATWG URL implementation (named -`test-whatwg-url-*.js`) are imported from the [Web Platform Tests Project][]. -These imported tests will be wrapped like this: - -```js -/* The following tests are copied from WPT. Modifications to them should be - upstreamed first. Refs: - https://github.com/w3c/web-platform-tests/blob/8791bed/url/urlsearchparams-stringifier.html - License: http://www.w3.org/Consortium/Legal/2008/04-testsuite-copyright.html -*/ -/* eslint-disable */ - -// Test code - -/* eslint-enable */ -``` - -To improve tests that have been imported this way, please send -a PR to the upstream project first. When the proposed change is merged in -the upstream project, send another PR here to update Node.js accordingly. -Be sure to update the hash in the URL following `WPT Refs:`. +See [`test/wpt`](../../test/wpt/README.md) for more information. ## C++ Unit test @@ -418,7 +398,6 @@ To generate a test coverage report, see the [ASCII]: http://man7.org/linux/man-pages/man7/ascii.7.html [Google Test]: https://github.com/google/googletest -[Web Platform Tests Project]: https://github.com/w3c/web-platform-tests/tree/master/url [`common` module]: https://github.com/nodejs/node/blob/master/test/common/README.md [all maintained branches]: https://github.com/nodejs/lts [node.green]: http://node.green/