Merge pull request #3 from libp2p/kevina/travisci

Update Travis CI to use standard test script and go 1.9.x.

.travis.yml

@@ -2,19 +2,18 @@ os:
   - linux
   - osx
 
+sudo: flase
+
 language: go
 
 go:
-    - 1.7
-
-install: true
+    - 1.9.x
 
-before_install:
+install:
   - make deps
 
 script:
-  - go vet
-  - $GOPATH/bin/goveralls -service="travis-ci"
+  - bash <(curl -s https://raw.githubusercontent.com/ipfs/ci-helpers/master/travis-ci/run-standard-tests.sh)
 
 cache:
     directories:

Makefile

@@ -2,11 +2,7 @@ gx:
 	go get github.com/whyrusleeping/gx
 	go get github.com/whyrusleeping/gx-go
 
-covertools:
-	go get github.com/mattn/goveralls
-	go get golang.org/x/tools/cmd/cover
-
-deps: gx covertools
+deps: gx
 	gx --verbose install --global
 	gx-go rewrite
 

codecov.yml

+coverage:
+  range: "50...100"
+comment: off

nat.go

@@ -122,7 +122,7 @@ func (nat *NAT) NewMapping(maddr ma.Multiaddr) (Mapping, error) {
 
 	network, addr, err := manet.DialArgs(maddr)
 	if err != nil {
-		return nil, fmt.Errorf("DialArgs failed on addr:", maddr.String())
+		return nil, fmt.Errorf("DialArgs failed on addr: %s", maddr.String())
 	}
 
 	switch network {
@@ -235,10 +235,10 @@ func (nat *NAT) PortMapAddrs(addrs []ma.Multiaddr) {
 	for _, addr := range addrs {
 		// do all of them concurrently
 		wg.Add(1)
-		go func() {
+		go func(addr ma.Multiaddr) {
 			defer wg.Done()
 			nat.NewMapping(addr)
-		}()
+		}(addr)
 	}
 	wg.Wait()
 }