Merge pull request #5 from datatogether/sprint_prep

THAT'S A 🌶 📝

Author: b5

.circleci/config.yml

@@ -35,12 +35,15 @@ jobs:
           command: go-wrapper download && go-wrapper install && go get -v github.com/jstemmer/go-junit-report
       - run: 
           name: Run tests
-          command: go test -v -race ./... | tee /tmp/test-reports/datatogether/original.txt ; test ${PIPESTATUS[0]} -eq 0
+          command: go test -v -race -coverprofile=coverage.txt -covermode=atomic | tee /tmp/test-reports/datatogether/original.txt ; test ${PIPESTATUS[0]} -eq 0
       - run: 
           name: Convert test output to junit-style xml
           command: cat /tmp/test-reports/datatogether/original.txt | go-junit-report > /tmp/test-reports/datatogether/junit.xml
       - store_test_results:
           path: /tmp/test-reports/datatogether/junit.xml
+      - run:
+          name: Publish coverage info to codecov.io
+          command: bash <(curl -s https://codecov.io/bash)
       - setup_remote_docker
       - run:
           name: Install Docker client

.github/CONTRIBUTING.md

@@ -0,0 +1,3 @@
+# Contributing Guidelines
+
+We love improvements to our tools! Take a moment to check out our organization-wide [Contributing Guidelines](https://github.com/datatogether/datatogether/blob/master/CONTRIBUTING.md) and [Code of Conduct](https://github.com/datatogether/datatogether/blob/master/CONDUCT.md).

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,12 @@
+Hey there, thank you for submitting an issue!
+
+We are trying to keep issues for feature requests and bug reports. Please
+complete the following checklist before creating a new one:
+
+- [ ] Is this a **bug report** (if so, is it something you can **debug and fix**? 
+Send a pull request!)
+- [ ] feature request
+- [ ] support request => Please do not submit support requests here, ask your question 
+on [Slack](https://archivers-slack.herokuapp.com/).
+
+---

.gitignore

@@ -1,6 +1,5 @@
 .DS_Store
 gin-bin
-sentry
-coverage.out
+coverage.txt
 *.env
 identity

LICENSE

@@ -0,0 +1,32 @@
+# Identity
+
+[![GitHub](https://img.shields.io/badge/project-Data_Together-487b57.svg?style=flat-square)](http://github.com/datatogether)
+[![Slack](https://img.shields.io/badge/slack-Archivers-b44e88.svg?style=flat-square)](https://archivers-slack.herokuapp.com/)
+[![License](https://img.shields.io/github/license/mashape/apistatus.svg)](./LICENSE)
+[![Codecov](https://img.shields.io/codecov/c/github/datatogether/identity.svg?style=flat-square)](https://codecov.io/gh/datatogether/identity) 
+
+[1-3 sentence description of repository contents]
+
+## License & Copyright
+
+[Modelled on [project guidelines template](https://github.com/datatogether/roadmap/blob/master/PROJECT.md#license--copyright-readme-block) ]
+
+## Getting Involved
+
+We would love involvement from more people! If you notice any errors or would like to submit changes, please see our [Contributing Guidelines](./.github/CONTRIBUTING.md). 
+
+We use GitHub issues for [tracking bugs and feature requests](https://github.com/datatogether/REPONAME/issues) and Pull Requests (PRs) for [submitting changes](https://github.com/datatogether/REPONAME/pulls)
+
+## ...
+
+## [Optional section(s) on Installation (actually using the service!), Architecture, Dependencies, and Other Considerations]
+
+[fill  out this section if the repo contains deployable/installable code]
+
+## Development
+
+[Step-by-step instructions about how to set up a local dev environment and any dependencies]
+
+## Deployment
+
+[Optional section with deployment instructions]

README.md

@@ -93,6 +93,7 @@ func NewServerRoutes() *http.ServeMux {
 	m := http.NewServeMux()
 
 	m.Handle("/", middleware(NotFoundHandler))
+	m.Handle("/healthcheck", middleware(HealthCheckHandler))
 
 	m.Handle("/publickey", middleware(PublicKeyHandler))
 

server.go

@@ -0,0 +1,72 @@
+package main
+
+import (
+	"bytes"
+	"encoding/json"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/gorilla/sessions"
+)
+
+func init() {
+	cfg = &config{}
+	sessionStore = sessions.NewCookieStore([]byte(cfg.SessionSecret))
+}
+
+func TestServerRoutes(t *testing.T) {
+	cases := []struct {
+		method, endpoint string
+		expectBody       bool
+		body             []byte
+		resStatus        int
+	}{
+		{"GET", "/", false, nil, 404},
+		{"GET", "/healthcheck", false, nil, 200},
+	}
+
+	client := &http.Client{}
+	server := httptest.NewServer(NewServerRoutes())
+
+	for i, c := range cases {
+		req, err := http.NewRequest(c.method, server.URL+c.endpoint, bytes.NewReader(c.body))
+		if err != nil {
+			t.Errorf("case %d error creating request: %s", i, err.Error())
+			continue
+		}
+
+		res, err := client.Do(req)
+		if err != nil {
+			t.Errorf("case %d error performing request: %s", i, err.Error())
+			continue
+		}
+
+		if res.StatusCode != c.resStatus {
+			t.Errorf("case %d: %s - %s status code mismatch. expected: %d, got: %d", i, c.method, c.endpoint, c.resStatus, res.StatusCode)
+			continue
+		}
+
+		if c.expectBody {
+			env := &struct {
+				Meta       map[string]interface{}
+				Data       interface{}
+				Pagination map[string]interface{}
+			}{}
+
+			if err := json.NewDecoder(res.Body).Decode(env); err != nil {
+				t.Errorf("case %d: %s - %s error unmarshaling json envelope: %s", i, c.method, c.endpoint, err.Error())
+				continue
+			}
+
+			if env.Meta == nil {
+				t.Errorf("case %d: %s - %s doesn't have a meta field", i, c.method, c.endpoint)
+				continue
+			}
+			if env.Data == nil {
+				t.Errorf("case %d: %s - %s doesn't have a data field", i, c.method, c.endpoint)
+				continue
+			}
+		}
+	}
+}