Go report card fixes

Author: heraclmene

api_service.go

@@ -38,7 +38,7 @@ func (a seleniumAPIService) performRequest(url string, method string, body io.Re
 	client := http.Client{}
 	resp, err := client.Do(request)
 	if err != nil {
-		return nil, errors.New(fmt.Sprintf("%s: an unexpected communication failure occurred, error: %s", method, err.Error()))
+		return nil, fmt.Errorf("%s: an unexpected communication failure occurred, error: %s", method, err.Error())
 	}
 
 	defer resp.Body.Close()
@@ -54,10 +54,10 @@ func (a seleniumAPIService) performRequest(url string, method string, body io.Re
 		err := json.Unmarshal(r, &reqErr)
 		if err == nil {
 			return nil, &reqErr
-		} else {
-			errStr = fmt.Sprintf("Status code %v returned with no body", resp.StatusCode)
-			return nil, errors.New(errStr)
 		}
+
+		errStr = fmt.Sprintf("Status code %v returned with no body", resp.StatusCode)
+		return nil, errors.New(errStr)
 	}
 
 	return r, nil

doc.go

@@ -1,2 +1,2 @@
-// A Selenium web driver library written in Go.
+// Package goselenium is a Selenium web driver library written in Go.
 package goselenium

errors.go

@@ -1,9 +1,6 @@
 package goselenium
 
-import (
-	"errors"
-	"fmt"
-)
+import "fmt"
 
 // ErrorResponse is what is returned from the Selenium API when an error
 // occurs.
@@ -145,6 +142,3 @@ func IsInvalidURLError(err error) bool {
 func newInvalidURLError(url string) InvalidURLError {
 	return InvalidURLError(url)
 }
-
-// WaitTimeoutError is returned when a Wait call has timed out.
-var WaitTimeoutError error = errors.New("wait action timed out")

remote_driver_command.go

@@ -19,7 +19,7 @@ type WindowHandleResponse struct {
 // As per the W3C specification, it yields all of the available window handles
 // minus the active one that closes as a result of the CloseWindow() call.
 type CloseWindowResponse struct {
-	State   string   `json:state"`
+	State   string   `json:"state"`
 	Handles []string `json:"value"`
 }
 
@@ -31,7 +31,7 @@ type SwitchToWindowResponse struct {
 
 // WindowHandlesResponse is the response returned from the WindowHandles()
 // method. This is essentially an array of available window handlers that
-// aren't neccessarily active.
+// aren't necessarily active.
 type WindowHandlesResponse struct {
 	State   string   `json:"state"`
 	Handles []string `json:"value"`