Merge pull request #223 from SkynetLabs/ivo/correction

Revert some changes

Author: ro-tex

Makefile

@@ -29,7 +29,7 @@ else
 	- DEL /F /Q cover output
 endif
 
-run = . 
+run = .
 
 # count says how many times to run the tests.
 count = 1
@@ -85,7 +85,7 @@ ifneq ("$(OS)","Windows_NT")
 endif
 
 # Define docker container name our test MongoDB instance.
-MONGO_TEST_CONTAINER_NAME=blocker-mongo-test-db
+MONGO_TEST_CONTAINER_NAME=accounts-mongo-test-db
 
 # start-mongo starts a local mongoDB container with no persistence.
 # We first prepare for the start of the container by making sure the test
@@ -136,11 +136,6 @@ test-long-ci:
 	@mkdir -p cover
 	GORACE='$(racevars)' go test -race --coverprofile='./cover/cover.out' -v -failfast -tags='testing debug netgo' -timeout=600s $(pkgs) -run=$(run) -count=$(count)
 
-# Cookie vars
-# TODO: Are these used?
-COOKIE_HASH_KEY="7eb32cfab5014d14394648dae1cf4e606727eee2267f6a50213cd842e61c5bce"
-COOKIE_ENC_KEY="65d31d12b80fc57df16d84c02a9bb62e2bc3b633388b05e49ef8abfdf0d35cf3"
-
 # docker-generate is a docker command for env var generation
 #
 # The sleep is to allow time for the docker container to start up after `docker

api/handlers_test.go

@@ -132,7 +132,7 @@ func TestUserLimitsGetFromTier(t *testing.T) {
 			}
 		}()
 		// The call that we expect to log a critical.
-		_ = userLimitsGetFromTier("", math.MaxInt64, false, true)
+		_ = userLimitsGetFromTier("", math.MaxInt, false, true)
 		return
 	}()
 	if err != nil {

build/debug_off.go

@@ -1,3 +1,4 @@
+//go:build !debug
 // +build !debug
 
 package build

build/debug_on.go

@@ -1,3 +1,4 @@
+//go:build debug
 // +build debug
 
 package build

build/release_dev.go

@@ -1,3 +1,4 @@
+//go:build dev
 // +build dev
 
 package build

build/release_standard.go

@@ -1,3 +1,4 @@
+//go:build !testing && !dev
 // +build !testing,!dev
 
 package build

build/release_testing.go

@@ -1,3 +1,4 @@
+//go:build testing
 // +build testing
 
 package build

types/email_test.go

@@ -12,7 +12,8 @@ import (
 func TestEmail_String(t *testing.T) {
 	s := "mIxEdCaSeStRiNg"
 	e := Email(s)
-	if !strings.EqualFold(e.String(), s) {
+	// We want to directly compare the strings in a case-sensitive way.
+	if e.String() != strings.ToLower(s) {
 		t.Fatalf("Expected '%s', got '%s'", strings.ToLower(s), e)
 	}
 }
@@ -44,7 +45,7 @@ func TestEmail_UnmarshalJSON(t *testing.T) {
 		t.Fatal(err)
 	}
 	// We expect the unmarshalled email to be lowercase only.
-	if !strings.EqualFold(string(e), string(b[1:len(b)-1])) {
+	if string(e) != strings.ToLower(string(b[1:len(b)-1])) {
 		t.Fatalf("Expected to get a lowercase version of '%s', i.e. '%s' but got '%s'", e, strings.ToLower(string(e)), e)
 	}
 }