Browse Source

internal/driver: skip tests requiring tcp on js (#373)

This commit upstreams the changes made in golang.org/cl/110096. As the
change says, it is acceptable for the new js/wasm target to only support
what nacl supports right now.
Daniel Martí 7 years ago
parent
commit
943b86b5fa
2 changed files with 3 additions and 3 deletions
  1. 1
    1
      internal/driver/fetch_test.go
  2. 2
    2
      internal/driver/webui_test.go

+ 1
- 1
internal/driver/fetch_test.go View File

361
 }
361
 }
362
 
362
 
363
 func TestHttpsInsecure(t *testing.T) {
363
 func TestHttpsInsecure(t *testing.T) {
364
-	if runtime.GOOS == "nacl" {
364
+	if runtime.GOOS == "nacl" || runtime.GOOS == "js" {
365
 		t.Skip("test assumes tcp available")
365
 		t.Skip("test assumes tcp available")
366
 	}
366
 	}
367
 	saveHome := os.Getenv(homeEnv())
367
 	saveHome := os.Getenv(homeEnv())

+ 2
- 2
internal/driver/webui_test.go View File

33
 )
33
 )
34
 
34
 
35
 func TestWebInterface(t *testing.T) {
35
 func TestWebInterface(t *testing.T) {
36
-	if runtime.GOOS == "nacl" {
36
+	if runtime.GOOS == "nacl" || runtime.GOOS == "js" {
37
 		t.Skip("test assumes tcp available")
37
 		t.Skip("test assumes tcp available")
38
 	}
38
 	}
39
 
39
 
237
 }
237
 }
238
 
238
 
239
 func TestGetHostAndPort(t *testing.T) {
239
 func TestGetHostAndPort(t *testing.T) {
240
-	if runtime.GOOS == "nacl" {
240
+	if runtime.GOOS == "nacl" || runtime.GOOS == "js" {
241
 		t.Skip("test assumes tcp available")
241
 		t.Skip("test assumes tcp available")
242
 	}
242
 	}
243
 
243