Added some tests and refactored the search command

Makefile

@@ -1,4 +1,5 @@
 SHELL = /bin/bash
+PACKAGES := ./commands
 
 mkfile_path := $(abspath $(lastword $(MAKEFILE_LIST)))
 current_dir := $(notdir $(patsubst %/,%,$(dir $(mkfile_path))))
@@ -9,6 +10,6 @@ $(app):
 	@go build
 
 test:
-	@echo NO TESTS!
+	@go test $(PACKAGES)
 
 .PHONY: clean $(app) run release test

commands/json_utils.go

@@ -40,7 +40,7 @@ func GetJson(url string) ([]byte, error) {
 func ParsePluginsList(body []byte) (Response, error) {
 	var r Response
 
-	json.Unmarshal(body, &r)
+	parseJsonError := json.Unmarshal(body, &r)
 
-	return r, nil
+	return r, parseJsonError
 }

commands/json_utils_test.go

@@ -0,0 +1,40 @@
+package commands_test
+
+import (
+	"testing"
+
+	"github.com/pepegar/vkg-go/commands"
+)
+
+func TestParsePluginsList(t *testing.T) {
+	validJson := []byte(`{
+	"total_results": 273,
+	"results_per_page": 20,
+	"total_pages": 14,
+	"plugins": [
+		{
+			"author": "Tim Pope",
+			"slug": "fugitive-vim",
+			"short_desc": "fugitive.vim: a Git wrapper so awesome, it should be illegal",
+			"github_url": "https://github.com/tpope/vim-fugitive"
+		}
+	]
+	}`)
+
+	response, parseError := commands.ParsePluginsList(validJson)
+
+	if parseError != nil {
+		t.Error("parsing a valid json throws an error")
+	}
+
+	if response.TotalResults != 273 {
+		t.Error(response.TotalResults)
+	}
+
+	invalidJson := []byte(`{asdf`)
+
+	_, parseErrorTwo := commands.ParsePluginsList(invalidJson)
+	if parseErrorTwo == nil {
+		t.Error("parsing invalid json doesn't throw an error")
+	}
+}

commands/search.go

@@ -16,13 +16,13 @@ var SearchCommand = Command{
 			json, jsonError := GetJson(url)
 
 			if nil != jsonError {
-				fmt.Println("error")
+				fmt.Println("invalid request")
 			}
 
-			response, parseError := ParsePluginsList(json)
+			response, parseJsonError := ParsePluginsList(json)
 
-			if nil != parseError {
-				fmt.Println("error")
+			if nil != parseJsonError {
+				fmt.Println("invalid json response")
 			}
 
 			for _, plugin := range response.Plugins {