]> Cypherpunks repositories - gostls13.git/commitdiff
Revert "misc/cgo/testcshared: temporarily skip testing on android"
authorElias Naur <elias.naur@gmail.com>
Sat, 19 Aug 2017 11:21:55 +0000 (11:21 +0000)
committerElias Naur <elias.naur@gmail.com>
Sat, 19 Aug 2017 12:56:24 +0000 (12:56 +0000)
This reverts commit a6ffab6b6743a490151aadfd2f2a79244a84e03a.

Reason for revert: with CL 57290 the tests run on Android again.

Change-Id: Ifeb29762a4cd0178463acfeeb3696884d99d2993
Reviewed-on: https://go-review.googlesource.com/57310
Reviewed-by: Alex Brainman <alex.brainman@gmail.com>
misc/cgo/testcshared/cshared_test.go

index edd2dd9f9c338489a8d20896ff7bc132b4f61fc5..45ec1207be0d83a4d0c04cc2dcc11b3c55602326 100644 (file)
@@ -259,9 +259,6 @@ func cleanupAndroid(t *testing.T) {
 
 // test0: exported symbols in shared lib are accessible.
 func TestExportedSymbols(t *testing.T) {
-       if GOOS == "android" {
-               t.Skip("Skipping failing test, see issue 21513 for details")
-       }
        cmd := "testp"
        bin := cmdToRun(cmd)
 
@@ -284,9 +281,6 @@ func TestExportedSymbols(t *testing.T) {
 
 // test1: shared library can be dynamically loaded and exported symbols are accessible.
 func TestExportedSymbolsWithDynamicLoad(t *testing.T) {
-       if GOOS == "android" {
-               t.Skip("Skipping failing test, see issue 21513 for details")
-       }
        cmd := "testp"
        bin := cmdToRun(cmd)
 
@@ -309,9 +303,6 @@ func TestExportedSymbolsWithDynamicLoad(t *testing.T) {
 
 // test2: tests libgo2 which does not export any functions.
 func TestUnexportedSymbols(t *testing.T) {
-       if GOOS == "android" {
-               t.Skip("Skipping failing test, see issue 21513 for details")
-       }
        cmd := "testp2"
        libname := "libgo2." + libSuffix
        bin := cmdToRun(cmd)
@@ -351,9 +342,6 @@ func TestUnexportedSymbols(t *testing.T) {
 
 // test3: tests main.main is exported on android.
 func TestMainExportedOnAndroid(t *testing.T) {
-       if GOOS == "android" {
-               t.Skip("Skipping failing test, see issue 21513 for details")
-       }
        if GOOS != "android" {
                return
        }
@@ -380,9 +368,6 @@ func TestMainExportedOnAndroid(t *testing.T) {
 
 // test4: test signal handlers
 func TestSignalHandlers(t *testing.T) {
-       if GOOS == "android" {
-               t.Skip("Skipping failing test, see issue 21513 for details")
-       }
        cmd := "testp4"
        libname := "libgo4." + libSuffix
        bin := cmdToRun(cmd)
@@ -416,9 +401,6 @@ func TestSignalHandlers(t *testing.T) {
 
 // test5: test signal handlers with os/signal.Notify
 func TestSignalHandlersWithNotify(t *testing.T) {
-       if GOOS == "android" {
-               t.Skip("Skipping failing test, see issue 21513 for details")
-       }
        cmd := "testp5"
        libname := "libgo5." + libSuffix
        bin := cmdToRun(cmd)
@@ -451,9 +433,6 @@ func TestSignalHandlersWithNotify(t *testing.T) {
 }
 
 func TestPIE(t *testing.T) {
-       if GOOS == "android" {
-               t.Skip("Skipping failing test, see issue 21513 for details")
-       }
        switch GOOS {
        case "linux", "android":
                break