[test] refs fibercrypto#143 - Add Makefile command for test coverage …

…and add coverage files in gitignore.

Use make test-cover for test coverage.

.gitignore

@@ -59,3 +59,7 @@ rcc__*
 # Temporary build files
 rcc_cgo_*.go
 
+
+src/coin/skycoin/coverage.html
+
+src/coin/skycoin/coverage.out

Makefile

@@ -4,6 +4,10 @@
 UNAME_S = $(shell uname -s)
 DEFAULT_TARGET ?= desktop
 DEFAULT_ARCH ?= linux
+COIN = skycoin 					##In future use as a parameter tu make command.
+COVERAGEPATH = src/coin/$(COIN)
+COVERAGEFILE = $(COVERAGEPATH)/coverage.out
+COVERAGEHTML = $(COVERAGEPATH)/coverage.html
 
 deps: ## Add dependencies
 	dep ensure
@@ -81,8 +85,19 @@ clean: ## Clean project FiberCrypto Wallet.
 	@echo "Done."
 
 test-sky: ## Run Skycoin plugin test suite
-	go test -timeout 30s github.com/fibercrypto/FiberCryptoWallet/src/coin/skycoin
-	go test -timeout 30s github.com/fibercrypto/FiberCryptoWallet/src/coin/skycoin/models
+	go test -cover -timeout 30s github.com/fibercrypto/FiberCryptoWallet/src/coin/skycoin
+	go test -coverprofile=$(COVERAGEFILE) -timeout 30s github.com/fibercrypto/FiberCryptoWallet/src/coin/skycoin/models
+
+test-clean:
+	rm $(COVERAGEFILE)
+	rm $(COVERAGEHTML)
+
+test-sky-launch-html-cover:
+	go test -cover -timeout 30s github.com/fibercrypto/FiberCryptoWallet/src/coin/skycoin
+	go test -coverprofile=$(COVERAGEFILE) -timeout 30s github.com/fibercrypto/FiberCryptoWallet/src/coin/skycoin/models
+	go tool cover -html=$(COVERAGEFILE) -o $(COVERAGEHTML)
+
+test-cover: test-sky-launch-html-cover ## Show more details of test coverage
 
 test: test-sky ## Run project test suite
 

src/coin/skycoin/models/wallet_test.go

@@ -279,7 +279,8 @@ func TestRemoteWalletTransfer(t *testing.T) {
 		Type:      0,
 		InnerHash: hash,
 	}
-	crtTxn, _ := api.NewCreateTransactionResponse(&txn, nil)
+	crtTxn, err := api.NewCreateTransactionResponse(&txn, nil)
+	require.Nil(t, err)
 	crtTxn.Transaction.Fee = "500"
 
 	mockSkyApiWalletCreateTransaction(global_mock, &wreq, crtTxn)
@@ -387,7 +388,8 @@ func TestRemoteWalletSendFromAddress(t *testing.T) {
 		Type:      0,
 		InnerHash: hash,
 	}
-	crtTxn, _ := api.NewCreateTransactionResponse(&txn, nil)
+	crtTxn, err := api.NewCreateTransactionResponse(&txn, nil)
+	require.Nil(t, err)
 	crtTxn.Transaction.Fee = strconv.Itoa(sky)
 
 	mockSkyApiWalletCreateTransaction(global_mock, &wreq1, crtTxn)
@@ -473,7 +475,8 @@ func TestRemoteWalletSpend(t *testing.T) {
 		Type:      0,
 		InnerHash: hash,
 	}
-	crtTxn, _ := api.NewCreateTransactionResponse(&txn, nil)
+	crtTxn, err := api.NewCreateTransactionResponse(&txn, nil)
+	require.Nil(t, err)
 	crtTxn.Transaction.Fee = "500"
 
 	mockSkyApiWalletCreateTransaction(global_mock, &wreq, crtTxn)