Re-enabled tests and travis-build

Author: cmaglie

.travis.yml

@@ -0,0 +1,27 @@
+language: go
+
+go:
+  - 1.9.2
+
+install:
+  - go get github.com/go-errors/errors
+  - go get github.com/stretchr/testify
+  - go get github.com/jstemmer/go-junit-report
+  - go get golang.org/x/tools/cmd/cover
+  - go get github.com/mattn/goveralls
+  - go get github.com/wadey/gocovmerge
+  - go get github.com/arduino/go-properties-map
+  - go get github.com/arduino/go-timeutils
+
+script:
+  - go build -o $HOME/arduino-builder -v github.com/arduino/arduino-builder/arduino-builder
+  - export TEST_PACKAGES=`go list github.com/arduino/arduino-builder/...`
+  - RES=0; I=0; for PKG in $TEST_PACKAGES; do go test -v -timeout 30m -covermode=count -coverprofile=coverage.$I.out $PKG; ((RES=RES+$?)); ((I++)); done; ( exit $RES )
+
+after_success:
+  - bin/gocovmerge coverage.*.out > coverage.out
+  - bin/goveralls -coverprofile=coverage.out -service=travis-ci -repotoken $COVERALLS_TOKEN
+
+env:
+  global:
+    secure: mm7X+dFQD4tUYwXxeVOJzereC04KdpTe6vrho1g6zmWgnMEqBe/gnX16Hv1LeQRzTnMVUxpcTXMkh4Nj16WL2IFAnizWiiMfFb6wUBBOM0dFhYJR7EzSzKMu7/CuhxIqT5ps/Ls3UUnpzUoI9ImygiLJCaNJCSjJxQ+hwnxyyEZD0FHvlQQcv3kiKAd8sg9oWKqTSBGIwBnn9ZGT9G2uykv+XU2B5B+dW9IoXaDR3SfUGlANldorD4T6S9QNs+JdwhX1NDDd7rOqF92qu2RA0TFf2Ljy1STimLXvVhGgcfetsUhVgkmDGKxp1/iax4jlJ8wRU7faqeFAt1WvJpBaOnsK8OcsNqf7iqmYxWxHKavFHfGKgc7L+dMP9gWIqgtzmfqG94/49IN8lPMpB29J8sfZfrb4nKNu66S9GqQCr/jgGD1uTXzC6SggoL6ro2di0bAvAlb6zgpv5zChqKkTzhHE/6rVA6EZG/JIIx5lXaDvkD+yp3o2LNKNknpwRPEY0KpReRivvajucDwiPqQD79gkKWOGtoo5034SGSGCUmXvAydJNVPJt6c5ADKpVgyiT065Gja4/n+n4GhjT6QKc2WRGQRQZGtkYkNh95tKUvgZiZ1G7+EAuRCotoKiqLPkLgUpzCoD7kZpo/Ai5xU2aRcKpMsAHEHSRP9F06F+zp4=

README.md

@@ -55,31 +55,26 @@ See [Doing continuous integration with arduino builder](https://github.com/ardui
 
 ### Building from source
 
-You need [Go 1.6.3](https://golang.org/dl/#go1.6.3).
+You need [a recent version of Go (>=1.8.0)](https://golang.org/).
 
-To install `codereview/patch` you have to install [Mercurial](https://www.mercurial-scm.org/) first.
-
-To set things up, run the following inside the cloned arduino-builder
-directory:
+To build, run the following commands:
 
 ```
-export GOPATH=$(pwd)
 go get github.com/go-errors/errors
 go get github.com/stretchr/testify
 go get github.com/jstemmer/go-junit-report
-go build arduino.cc/arduino-builder
+go get github.com/arduino/go-properties-map
+go get github.com/arduino/go-timeutils
+go get github.com/arduino/arduino-builder
+go build github.com/arduino/arduino-builder/arduino-builder
 ```
 
-After installing dependencies, you do not need to run the `go get`
-commands again, but you will have to set `GOPATH` again for every shell
-you want to build in.
-
 ### TDD
 
 In order to run the tests, type:
 
 ```
-go test arduino.cc/...
+go test github.com/arduino/arduino-builder/arduino-builder/...
 ```
 
 This runs all tests, showing any failures and a summary at the end.
@@ -93,13 +88,13 @@ To run a single test, use the -run option, which accepts a regular
 expression (see also go help testflag).
 
 ```
-go test arduino.cc/... -run 'TestBuilderEmptySketch'
-go test arduino.cc/... -run 'TestPrototypesAdder.*'
+go test github.com/arduino/arduino-builder/arduino-builder/... -run 'TestBuilderEmptySketch'
+go test github.com/arduino/arduino-builder/arduino-builder/... -run 'TestPrototypesAdder.*'
 ```
 
 In jenkins, use
 ```
-go test -v arduino.cc/... | bin/go-junit-report > report.xml
+go test -v github.com/arduino/arduino-builder/arduino-builder/... | bin/go-junit-report > report.xml
 ```
 
 The first time you run the tests, some needed files (toolchains and
@@ -114,6 +109,6 @@ increasing the timeout might be needed as well.
 
 `arduino-builder` is licensed under General Public License version 2, as published by the Free Software Foundation. See [LICENSE.txt](LICENSE.txt).
 
-Copyright (C) 2015 Arduino LLC and contributors
+Copyright (C) 2017 Arduino AG and contributors
 
 See https://www.arduino.cc/ and https://github.com/arduino/arduino-builder/graphs/contributors

test/helper_tools_downloader.go

@@ -379,7 +379,7 @@ func coreAlreadyDownloadedAndUnpacked(targetPath string, core Core) (bool, error
 	if os.IsNotExist(err) {
 		return false, nil
 	}
-	platform, err := properties.Load(filepath.Join(corePath, "platform.txt"), i18n.HumanLogger{})
+	platform, err := properties.Load(filepath.Join(corePath, "platform.txt"))
 	if err != nil {
 		return false, i18n.WrapError(err)
 	}
@@ -435,7 +435,7 @@ func libraryAlreadyDownloadedAndUnpacked(targetPath string, library Library) boo
 		return false
 	}
 
-	libProps, err := properties.Load(filepath.Join(targetPath, strings.Replace(library.Name, " ", "_", -1), "library.properties"), i18n.HumanLogger{})
+	libProps, err := properties.Load(filepath.Join(targetPath, strings.Replace(library.Name, " ", "_", -1), "library.properties"))
 	if err != nil {
 		return false
 	}