From 1c224f495bd5a8f0fd55e8d1f68ca1c1f51582e6 Mon Sep 17 00:00:00 2001 From: Alex Flint Date: Thu, 27 Dec 2018 11:11:15 -0800 Subject: add integration test that fetches and compiles code using go 1.11 --- test/compile_with_go111.sh | 8 ++++++++ 1 file changed, 8 insertions(+) create mode 100644 test/compile_with_go111.sh (limited to 'test/compile_with_go111.sh') diff --git a/test/compile_with_go111.sh b/test/compile_with_go111.sh new file mode 100644 index 0000000..f0dc394 --- /dev/null +++ b/test/compile_with_go111.sh @@ -0,0 +1,8 @@ +#!/bin/bash + +docker run \ + --rm \ + -v $(pwd)/some-program:/src \ + -w /src \ + golang:1.11 \ + go build -- cgit v1.2.3 From 25cd36cac6373d182f4274bbbe685ea777d533d2 Mon Sep 17 00:00:00 2001 From: Alex Flint Date: Thu, 27 Dec 2018 11:17:24 -0800 Subject: add test that compiles under go 1.11 without the module system --- test/compile_with_go111.sh | 2 +- test/compile_with_go111_inside_gopath.sh | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) create mode 100644 test/compile_with_go111_inside_gopath.sh (limited to 'test/compile_with_go111.sh') diff --git a/test/compile_with_go111.sh b/test/compile_with_go111.sh index f0dc394..d5de179 100644 --- a/test/compile_with_go111.sh +++ b/test/compile_with_go111.sh @@ -5,4 +5,4 @@ docker run \ -v $(pwd)/some-program:/src \ -w /src \ golang:1.11 \ - go build + go build -o /dev/null diff --git a/test/compile_with_go111_inside_gopath.sh b/test/compile_with_go111_inside_gopath.sh new file mode 100644 index 0000000..b55a7d2 --- /dev/null +++ b/test/compile_with_go111_inside_gopath.sh @@ -0,0 +1,12 @@ +#!/bin/bash + +# Under go 1.11, modules are disabled by default when user code is located within the GOPATH. +# In this test, we check that we can correctly "go get" the go-arg package, and then compile +# some code that uses it. + +docker run \ + --rm \ + -v $(pwd)/some-program:/go/src/some-program \ + -w /go/src/some-program \ + golang:1.11 \ + bash -c "go get github.com/alexflint/go-arg && go build" -- cgit v1.2.3 From e5a1f3c9998f0debb175b9d44ebf97deb8c7ca36 Mon Sep 17 00:00:00 2001 From: Alex Flint Date: Thu, 27 Dec 2018 11:25:10 -0800 Subject: add script to compile under go 1.10 --- test/compile_with_go110.sh | 12 ++++++++++++ test/compile_with_go111.sh | 3 +++ test/compile_with_go111_inside_gopath.sh | 7 +++---- 3 files changed, 18 insertions(+), 4 deletions(-) create mode 100644 test/compile_with_go110.sh (limited to 'test/compile_with_go111.sh') diff --git a/test/compile_with_go110.sh b/test/compile_with_go110.sh new file mode 100644 index 0000000..2f0906d --- /dev/null +++ b/test/compile_with_go110.sh @@ -0,0 +1,12 @@ +#!/bin/bash + +# This test checks that we can correctly "go get" and then use the go-arg package +# under go 1.10, which was the last release before introduction of the new go +# module system. + +docker run \ + --rm \ + -v $(pwd)/some-program:/src \ + -w /src \ + golang:1.10 \ + bash -c "go get github.com/alexflint/go-arg && go build -o /dev/null" diff --git a/test/compile_with_go111.sh b/test/compile_with_go111.sh index d5de179..e11daaf 100644 --- a/test/compile_with_go111.sh +++ b/test/compile_with_go111.sh @@ -1,5 +1,8 @@ #!/bin/bash +# This test checks that we can compile some code that depends on go-arg when using go 1.11 +# with the new go module system active. + docker run \ --rm \ -v $(pwd)/some-program:/src \ diff --git a/test/compile_with_go111_inside_gopath.sh b/test/compile_with_go111_inside_gopath.sh index b55a7d2..714af47 100644 --- a/test/compile_with_go111_inside_gopath.sh +++ b/test/compile_with_go111_inside_gopath.sh @@ -1,12 +1,11 @@ #!/bin/bash -# Under go 1.11, modules are disabled by default when user code is located within the GOPATH. -# In this test, we check that we can correctly "go get" the go-arg package, and then compile -# some code that uses it. +# This test checks that we can correctly "go get" and then use the go-arg package using +# go 1.11 when the code is within the GOPATH (in which case modules are disabled by default). docker run \ --rm \ -v $(pwd)/some-program:/go/src/some-program \ -w /go/src/some-program \ golang:1.11 \ - bash -c "go get github.com/alexflint/go-arg && go build" + bash -c "go get github.com/alexflint/go-arg && go build -o /dev/null" -- cgit v1.2.3 From f8987d11059a6a868a8e179832e7dfaeaac854f0 Mon Sep 17 00:00:00 2001 From: Alex Flint Date: Thu, 27 Dec 2018 11:28:34 -0800 Subject: hook integration tests into travis --- .travis.yml | 3 +++ test/compile_with_go110.sh | 4 +++- test/compile_with_go111.sh | 4 +++- test/compile_with_go111_inside_gopath.sh | 4 +++- 4 files changed, 12 insertions(+), 3 deletions(-) (limited to 'test/compile_with_go111.sh') diff --git a/.travis.yml b/.travis.yml index 465683c..f953225 100644 --- a/.travis.yml +++ b/.travis.yml @@ -11,3 +11,6 @@ before_install: - if ! go get github.com/golang/tools/cmd/cover; then go get golang.org/x/tools/cmd/cover; fi script: - $HOME/gopath/bin/goveralls -service=travis-ci + - bash test/compile_with_go110.sh + - bash test/compile_with_go111.sh + - bash test/compile_with_go111_inside_gopath.sh diff --git a/test/compile_with_go110.sh b/test/compile_with_go110.sh index 2f0906d..bfa2c71 100644 --- a/test/compile_with_go110.sh +++ b/test/compile_with_go110.sh @@ -4,9 +4,11 @@ # under go 1.10, which was the last release before introduction of the new go # module system. +DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )" + docker run \ --rm \ - -v $(pwd)/some-program:/src \ + -v $DIR/some-program:/src \ -w /src \ golang:1.10 \ bash -c "go get github.com/alexflint/go-arg && go build -o /dev/null" diff --git a/test/compile_with_go111.sh b/test/compile_with_go111.sh index e11daaf..fd20339 100644 --- a/test/compile_with_go111.sh +++ b/test/compile_with_go111.sh @@ -3,9 +3,11 @@ # This test checks that we can compile some code that depends on go-arg when using go 1.11 # with the new go module system active. +DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )" + docker run \ --rm \ - -v $(pwd)/some-program:/src \ + -v $DIR/some-program:/src \ -w /src \ golang:1.11 \ go build -o /dev/null diff --git a/test/compile_with_go111_inside_gopath.sh b/test/compile_with_go111_inside_gopath.sh index 714af47..8755c0f 100644 --- a/test/compile_with_go111_inside_gopath.sh +++ b/test/compile_with_go111_inside_gopath.sh @@ -3,9 +3,11 @@ # This test checks that we can correctly "go get" and then use the go-arg package using # go 1.11 when the code is within the GOPATH (in which case modules are disabled by default). +DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )" + docker run \ --rm \ - -v $(pwd)/some-program:/go/src/some-program \ + -v $DIR/some-program:/go/src/some-program \ -w /go/src/some-program \ golang:1.11 \ bash -c "go get github.com/alexflint/go-arg && go build -o /dev/null" -- cgit v1.2.3