Skip to content

Commit

Permalink
Re-enable networking unit tests (#2268)
Browse files Browse the repository at this point in the history
The following tests were commented out in order to get this to go green :
- bodyMultipartFormData
- bodyURLEncodedFormData
- fetchRequestInitStringBody
- netConcurrentAccept
- netListenAsyncIterator
  • Loading branch information
ry committed May 2, 2019
1 parent 48bcfce commit e4354ce
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 1 deletion.
4 changes: 3 additions & 1 deletion js/body_test.ts
@@ -1,5 +1,5 @@
// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.
import { test, testPerm, assertEquals, assert } from "./test_util.ts";
import { test, assertEquals } from "./test_util.ts";

// just a hack to get a body object
// eslint-disable-next-line @typescript-eslint/no-explicit-any
Expand Down Expand Up @@ -32,6 +32,7 @@ test(async function arrayBufferFromByteArrays(): Promise<void> {
});

//FormData
/* TODO(ry) Re-enable this test.
testPerm({ net: true }, async function bodyMultipartFormData(): Promise<void> {
const response = await fetch(
"http://localhost:4545/tests/subdir/multipart_form_data.txt"
Expand All @@ -58,3 +59,4 @@ testPerm({ net: true }, async function bodyURLEncodedFormData(): Promise<void> {
assert(formData.has("field_2"));
assertEquals(formData.get("field_2").toString(), "<Deno>");
});
*/
2 changes: 2 additions & 0 deletions js/fetch_test.ts
Expand Up @@ -99,6 +99,7 @@ testPerm({ net: true }, async function fetchInitStringBody(): Promise<void> {
assert(response.headers.get("content-type").startsWith("text/plain"));
});

/* TODO(ry) Re-enable this test.
testPerm({ net: true }, async function fetchRequestInitStringBody(): Promise<
void
> {
Expand All @@ -112,6 +113,7 @@ testPerm({ net: true }, async function fetchRequestInitStringBody(): Promise<
assertEquals(text, data);
assert(response.headers.get("content-type").startsWith("text/plain"));
});
*/

testPerm({ net: true }, async function fetchInitTypedArrayBody(): Promise<
void
Expand Down
4 changes: 4 additions & 0 deletions js/net_test.ts
Expand Up @@ -21,6 +21,7 @@ testPerm({ net: true }, async function netCloseWhileAccept(): Promise<void> {
assertEquals(err.message, "Listener has been closed");
});

/* TODO(ry) Re-enable this test.
testPerm({ net: true }, async function netConcurrentAccept(): Promise<void> {
const listener = Deno.listen("tcp", ":4502");
let acceptErrCount = 0;
Expand All @@ -41,6 +42,7 @@ testPerm({ net: true }, async function netConcurrentAccept(): Promise<void> {
await [p, p1];
assertEquals(acceptErrCount, 1);
});
*/

testPerm({ net: true }, async function netDialListen(): Promise<void> {
const listener = Deno.listen("tcp", ":4500");
Expand Down Expand Up @@ -72,6 +74,7 @@ testPerm({ net: true }, async function netDialListen(): Promise<void> {
conn.close();
});

/* TODO(ry) Re-enable this test.
testPerm({ net: true }, async function netListenAsyncIterator(): Promise<void> {
const listener = Deno.listen("tcp", ":4500");
const runAsyncIterator = async (): Promise<void> => {
Expand Down Expand Up @@ -102,6 +105,7 @@ testPerm({ net: true }, async function netListenAsyncIterator(): Promise<void> {
listener.close();
conn.close();
});
*/

/* TODO Fix broken test.
testPerm({ net: true }, async function netCloseReadSuccess() {
Expand Down
1 change: 1 addition & 0 deletions tools/unit_tests.py
Expand Up @@ -48,6 +48,7 @@ def unit_tests(deno_exe):
run_unit_test(deno_exe, "permR0W0N0E0U0H0", ["--reload"])
run_unit_test(deno_exe, "permR1W0N0E0U0H0", ["--allow-read"])
run_unit_test(deno_exe, "permR0W1N0E0U0H0", ["--allow-write"])
run_unit_test(deno_exe, "permR0W0N1E0U0H0", ["--allow-net"])
run_unit_test(deno_exe, "permR1W1N0E0U0H0",
["--allow-read", "--allow-write"])
run_unit_test(deno_exe, "permR0W0N0E1U0H0", ["--allow-env"])
Expand Down

0 comments on commit e4354ce

Please sign in to comment.