Merge pull request #5 from hellofresh/feature/test-ci

Test

README.md

@@ -8,7 +8,7 @@
 
 This is a lightweight, API Gateway and Management Platform enables you to control who accesses your API, 
 when they access it and how they access it. API Gateway will also record detailed analytics on how your 
-users are interacting with your API and when things go wrong.
+users are interacting with your API and when things go wrong. 
 
 ## What is an API Gateway?
 
@@ -18,7 +18,7 @@ services instead of implementing management infrastructure. For example if you h
 web service that provides geolocation data for all the cats in NYC, and you want to make it public, 
 integrating an API gateway is a faster, more secure route that writing your own authorisation middleware.
 
-## Key Features 
+## Key Features
 
 This API Gateway offers powerful, yet lightweight features that allow fine gained control over your API ecosystem.
 

ci/pipeline.yml

@@ -150,12 +150,6 @@ jobs:
         context: unit-tests
         status: pending
 
-  - put: prequest
-    params:
-        path: prequest
-        context: api-tests
-        status: pending
-
   - aggregate:
     - task: pullrequest unit test
       config:
@@ -167,7 +161,7 @@ jobs:
               username: {{quay_username}}
               password: {{quay_password}}
         run:
-          path: ci/tasks/unit-tests.sh
+          path: ./ci/tasks/unit-tests.sh
           dir: prequest
         inputs:
         - name: prequest
@@ -184,4 +178,4 @@ jobs:
               context: unit-tests
               status: success
       params:
-        PROJECT_SRC: prequest
+        PROJECT_SRC: /go/src/github.com/hellofresh/api-gateway

ci/tasks/unit-tests.sh

@@ -1,4 +1,5 @@
 #!/bin/sh
 cd ${PROJECT_SRC}
 
+pwd
 make test

main.go

@@ -16,7 +16,6 @@ var config Specification
 //loadConfigEnv loads environment variables
 func loadConfigEnv() Specification {
 	err := envconfig.Process("", &config)
-	log.Info(config)
 	if err != nil {
 		log.Fatal(err.Error())
 	}
@@ -72,6 +71,7 @@ func main() {
 
 	accessor := initializeDatabase()
 	defer accessor.Close()
+
 	database := Database{accessor}
 	router.Use(database.Middleware())
 

middleware_rate_limit.go

@@ -6,9 +6,10 @@ import (
 	"strconv"
 	"time"
 
+	"net/http"
+
 	"github.com/etcinit/speedbump"
 	"github.com/gin-gonic/gin"
-	"net/http"
 )
 
 type RateLimitMiddleware struct {
@@ -46,7 +47,7 @@ func (m RateLimitMiddleware) ProcessRequest(req *http.Request, c *gin.Context) (
 	if !ok {
 		m.Logger.Debug("Rate limit exceeded.")
 		return errors.New("Rate limit exceeded. Try again in " + nextTime.String()), http.StatusTooManyRequests
-	} else {
-		return nil, http.StatusOK
 	}
+
+	return nil, http.StatusOK
 }