Setup GH Actions [ch32361]

.github/workflows/test.yml

@@ -0,0 +1,77 @@
+name: Run specs and generate Code Climate report
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        go-versions: ['1.11', '1.12', '1.13', '1.14']
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Set up Go ${{ matrix.go-versions }}
+        uses: actions/setup-go@v2
+        with:
+          go-version: ${{ matrix.go-versions }}
+
+      - name: Run tests
+        run: make test
+
+  code-coverage:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Set up Go ${{ matrix.go-versions }}
+        uses: actions/setup-go@v2
+        with:
+          go-version: '1.14'
+
+      - name: Set ENV for codeclimate (pull_request)
+        run: |
+          git fetch --no-tags --prune --depth=1 origin +refs/heads/$GITHUB_HEAD_REF:refs/remotes/origin/$GITHUB_HEAD_REF
+          echo "GIT_BRANCH=$GITHUB_HEAD_REF" >> $GITHUB_ENV
+          echo "GIT_COMMIT_SHA=$(git rev-parse origin/$GITHUB_HEAD_REF)" >> $GITHUB_ENV
+        if: github.event_name == 'pull_request'
+
+      - name: Set ENV for codeclimate (push)
+        run: |
+          echo "GIT_BRANCH=${GITHUB_REF#refs/heads/}" >> $GITHUB_ENV
+          echo "GIT_COMMIT_SHA=$GITHUB_SHA" >> $GITHUB_ENV
+        if: github.event_name == 'push'
+
+      - name: Install Code Climate test report
+        run: |
+          curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
+          chmod +x ./cc-test-reporter
+          ./cc-test-reporter before-build
+
+      - name: Run test coverage
+        run: |
+            for pkg in $(go list ./... | grep -v vendor); do
+                go test -coverprofile=$(echo $pkg | tr / -).cover $pkg
+            done
+            echo "mode: set" > c.out
+            grep -h -v "^mode:" ./*.cover >> c.out
+            rm -f *.cover
+            sed -i 's#^github.com/chartmogul/chartmogul-go/##g' c.out
+
+      - name: Send Report to Code Climate
+        env:
+          CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
+        if: ${{ success() }}
+        run: ./cc-test-reporter after-build
+  linter:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v2
+        with:
+          version: 'v1.28.3'
+

.golangci.yml

@@ -0,0 +1,10 @@
+linters:
+  disable:
+    - gofumpt
+  presets:
+    - format
+    - unused
+    - performance
+    - complexity
+    - bugs
+

Makefile

@@ -23,7 +23,8 @@ pre-commit: .git/hooks/pre-commit lint
 	go test -timeout=20s --short ./...
 
 bin/golangci-lint:
-	wget -O - -q https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s v1.27.0 # Go 1.11+
+	curl -s -L https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s v1.28.3 # Go 1.11+
+
 
 .PHONY: lint
 lint: bin/golangci-lint

chartmogul_test.go

@@ -91,6 +91,9 @@ var matchersCases = map[string]struct {
 func TestErrorMatchers(t *testing.T) {
 	for testName, testCase := range matchersCases {
 		for fnName, expected := range testCase.expectations {
+			t := t
+			expected := expected
+
 			var fn func() bool
 			switch fnName {
 			case "IsAlreadyExists":

integration_tests/connect_subscription_test.go

@@ -136,7 +136,7 @@ func TestConnectSubscriptions(t *testing.T) {
 		t.Fatal(err)
 	}
 	//time.Sleep(time.Minute)
-	msubs, err := api.MetricsListSubscriptions(&cm.Cursor{PerPage: 200}, cus.UUID)
+	msubs, err := api.MetricsListCustomerSubscriptions(&cm.Cursor{PerPage: 200}, cus.UUID)
 	if err != nil {
 		t.Fatal(err)
 	}

integration_tests/create_invoice_test.go

@@ -86,9 +86,9 @@ func TestCreateInvoice(t *testing.T) {
 			},
 			{
 				Type:                      "subscription",
-				AmountInCents:         	   10000,
-				ExternalID:            	   "ext_line_item",
-				SubscriptionExternalID:	   "ext_subscription",
+				AmountInCents:             10000,
+				ExternalID:                "ext_line_item",
+				SubscriptionExternalID:    "ext_subscription",
 				SubscriptionSetExternalID: "ext_subscription_set",
 				PlanUUID:                  plan.UUID,
 				Quantity:                  10,

integration_tests/retrieve_invoice_test.go

@@ -68,7 +68,7 @@ func TestRetrieveInvoice(t *testing.T) {
 			},
 			{
 				Type:                      "subscription",
-				AmountInCents:         	   10000,
+				AmountInCents:             10000,
 				ExternalID:                "ext_line_item",
 				SubscriptionExternalID:    "ext_subscription",
 				SubscriptionSetExternalID: "ext_subscription_set",

metrics_customer_subscriptions.go

@@ -8,11 +8,11 @@ type MetricsCustomerSubscription struct {
 	ExternalID        string  `json:"external_id"`
 	Plan              string  `json:"plan"`
 	Quantity          uint32  `json:"quantity"`
+	BillingCycleCount uint32  `json:"billing-cycle-count"`
 	MRR               float64 `json:"mrr"`
 	ARR               float64 `json:"arr"`
 	Status            string  `json:"status"`
 	BillingCycle      string  `json:"billing-cycle"`
-	BillingCycleCount uint32  `json:"billing-cycle-count"`
 	StartDate         string  `json:"start-date"`
 	EndDate           string  `json:"end-date"`
 	Currency          string  `json:"currency"`
@@ -29,7 +29,7 @@ type MetricsCustomerSubscriptions struct {
 
 const metricsCustomerSubscriptionsEndpoint = "customers/:uuid/subscriptions"
 
-// MetricsListSubscriptions lists all subscriptions for cutomer of a given UUID.
+// MetricsListCustomerSubscriptions lists all subscriptions for cutomer of a given UUID.
 //
 // See https://dev.chartmogul.com/v1.0/reference#list-customer-subscriptions
 func (api API) MetricsListCustomerSubscriptions(cursor *Cursor, customerUUID string) (*MetricsCustomerSubscriptions, error) {

subscriptions_test.go

@@ -99,6 +99,7 @@ func TestCancelSubscriptionParams(t *testing.T) {
 	}
 
 	for name, tc := range testCases {
+		tc := tc
 		t.Run(name, func(t *testing.T) {
 			got, err := json.Marshal(tc.param)
 			if err != nil {