compiler/natives/src/io: Skip TestMultiWriterSingleChainFlatten.

dmitshur · dmitshur · commit 12f65e290eec · 2018-02-16T18:04:23.000-05:00
TestMultiWriterSingleChainFlatten was added in Go 1.10, and it needs to be skipped (just like TestMultiReaderFlatten) because it relies on unsupported runtime features. Related to golang/go@060d1a5. Fixes: === RUN TestMultiWriterSingleChainFlatten --- FAIL: TestMultiWriterSingleChainFlatten (0.01s) test.558082205:38: multiWriter did not flatten chained multiWriters: expected writeDepth 12, got 4 FAIL io 0.888s
diff --git a/compiler/natives/src/io/io_test.go b/compiler/natives/src/io/io_test.go
@@ -11,7 +11,11 @@ func TestMultiWriter_WriteStringSingleAlloc(t *testing.T) {
 }
 
 func TestMultiReaderFlatten(t *testing.T) {
-	t.Skip()
+	t.Skip("test relies on runtime.Callers and runtime.CallersFrames, which GopherJS doesn't support")
+}
+
+func TestMultiWriterSingleChainFlatten(t *testing.T) {
+	t.Skip("test relies on runtime.Callers and runtime.CallersFrames, which GopherJS doesn't support")
 }
 
 func TestMultiReaderFreesExhaustedReaders(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,11 @@ func TestMultiWriter_WriteStringSingleAlloc(t *testing.T) {`
`11`	`11`	`}`
`12`	`12`
`13`	`13`	`func TestMultiReaderFlatten(t *testing.T) {`
`14`		`- t.Skip()`
	`14`	`+ t.Skip("test relies on runtime.Callers and runtime.CallersFrames, which GopherJS doesn't support")`
	`15`	`+}`
	`16`	`+`
	`17`	`+func TestMultiWriterSingleChainFlatten(t *testing.T) {`
	`18`	`+ t.Skip("test relies on runtime.Callers and runtime.CallersFrames, which GopherJS doesn't support")`
`15`	`19`	`}`
`16`	`20`
`17`	`21`	`func TestMultiReaderFreesExhaustedReaders(t *testing.T) {`