From c28a8e4553fed920425c6c9cb32d20f2da2f7a9a Mon Sep 17 00:00:00 2001 From: Robert Griesemer Date: Mon, 23 Nov 2015 15:21:13 -0800 Subject: [PATCH] test: add test case for issue 13268 See https://go-review.googlesource.com/#/c/17047/ for the bug fix. Change-Id: Id5b0a37439d0f8f1e668f7c2ac84335b37f3a300 Reviewed-on: https://go-review.googlesource.com/17177 Reviewed-by: Brad Fitzpatrick Reviewed-by: Russ Cox Run-TryBot: Robert Griesemer TryBot-Result: Gobot Gobot --- test/fixedbugs/issue13268.go | 48 ++++++++++++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) create mode 100644 test/fixedbugs/issue13268.go diff --git a/test/fixedbugs/issue13268.go b/test/fixedbugs/issue13268.go new file mode 100644 index 0000000000..2a063fa60c --- /dev/null +++ b/test/fixedbugs/issue13268.go @@ -0,0 +1,48 @@ +// run + +// Copyright 2015 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +// Test error message when EOF is encountered in the +// middle of a BOM. +// +// Since the error requires an EOF, we cannot use the +// errorcheckoutput mechanism. + +package main + +import ( + "io/ioutil" + "log" + "os" + "os/exec" + "runtime" + "strings" +) + +func main() { + // cannot use temp file on nacl via child process + if runtime.GOOS == "nacl" { + return + } + + // create source + f, err := ioutil.TempFile("", "issue13268-") + if err != nil { + log.Fatalf("could not create source file: %v", err) + } + f.Write([]byte("package p\n\nfunc \xef\xef")) // if this fails, we will die later + f.Close() + defer os.Remove(f.Name()) + + // compile and test output + cmd := exec.Command("go", "tool", "compile", f.Name()) + out, err := cmd.CombinedOutput() + if err == nil { + log.Fatalf("expected cmd/compile to fail") + } + if strings.HasPrefix(string(out), "illegal UTF-8 sequence") { + log.Fatalf("error %q not found", out) + } +} -- 2.50.0