Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Commit

Permalink
Merge pull request #222 from suzuki-shunsuke/test/refactor-test
Browse files Browse the repository at this point in the history
test: refactor test
  • Loading branch information
suzuki-shunsuke committed Jan 13, 2020
2 parents d2504e3 + f5f9096 commit f2da7c6
Show file tree
Hide file tree
Showing 3 changed files with 264 additions and 269 deletions.
139 changes: 18 additions & 121 deletions terraform/graylog/resource_dashboard_test.go
Original file line number Diff line number Diff line change
@@ -1,138 +1,35 @@
package graylog

import (
"io/ioutil"
"net/http"
"strings"
"testing"

"github.com/hashicorp/terraform/helper/resource"
"github.com/stretchr/testify/require"
"github.com/suzuki-shunsuke/flute/flute"

"github.com/suzuki-shunsuke/go-graylog/v9/testdata"
)

func TestAccDashboard(t *testing.T) {
setEnv()

createRespBody, err := ioutil.ReadFile("../../testdata/dashboard/create_dashboard_response.json")
require.Nil(t, err)

getBody, err := ioutil.ReadFile("../../testdata/dashboard/dashboard.json")
require.Nil(t, err)

updatedGetBody, err := ioutil.ReadFile("../../testdata/dashboard/updated_dashboard.json")
require.Nil(t, err)

createTF, err := ioutil.ReadFile("../../testdata/dashboard/dashboard.tf")
require.Nil(t, err)

updateTF, err := ioutil.ReadFile("../../testdata/dashboard/update_dashboard.tf")
require.Nil(t, err)

store := newBodyStore("")

ds := testdata.Dashboard()

dsPath := "/api/dashboards/" + ds.ID
tc := &testCase{
t: t,
Name: "dashboard",
CreatePath: "/api/dashboards",
GetPath: "/api/dashboards/" + ds.ID,

defaultTransport := http.DefaultClient.Transport
defer func() {
http.DefaultClient.Transport = defaultTransport
}()
http.DefaultClient.Transport = &flute.Transport{
T: t,
Services: []flute.Service{
{
Endpoint: "http://example.com",
Routes: []flute.Route{
{
Name: "get a dashboard",
Matcher: &flute.Matcher{
Method: "GET",
},
Tester: &flute.Tester{
Path: dsPath,
PartOfHeader: getTestHeader(),
},
Response: &flute.Response{
Response: func(req *http.Request) (*http.Response, error) {
return &http.Response{
StatusCode: 200,
Body: ioutil.NopCloser(strings.NewReader(store.Get())),
}, nil
},
},
},
{
Name: "create a dashboard",
Matcher: &flute.Matcher{
Method: "POST",
},
Tester: &flute.Tester{
Path: "/api/dashboards",
PartOfHeader: getTestHeader(),
Test: genTestBody(map[string]interface{}{
"title": "test",
"description": "test",
}, string(getBody), store),
},
Response: &flute.Response{
Base: http.Response{
StatusCode: 201,
},
BodyString: string(createRespBody),
},
},
{
Name: "update a dashboard",
Matcher: &flute.Matcher{
Method: "PUT",
},
Tester: &flute.Tester{
Path: dsPath,
PartOfHeader: getTestHeader(),
Test: genTestBody(map[string]interface{}{
"title": "updated title",
"description": "updated description",
}, string(updatedGetBody), store),
},
Response: &flute.Response{
Base: http.Response{
StatusCode: 204,
},
},
},
{
Name: "delete a dashboard",
Matcher: &flute.Matcher{
Method: "DELETE",
},
Tester: &flute.Tester{
Path: dsPath,
PartOfHeader: getTestHeader(),
},
Response: &flute.Response{
Base: http.Response{
StatusCode: 204,
},
},
},
},
},
CreateReqBodyMap: map[string]interface{}{
"title": "test",
"description": "test",
},
}

resource.Test(t, resource.TestCase{
Providers: getTestProviders(),
Steps: []resource.TestStep{
{
Config: string(createTF),
},
{
Config: string(updateTF),
},
UpdateReqBodyMap: map[string]interface{}{
"title": "updated title",
"description": "updated description",
},
})
CreatedDataPath: "dashboard/dashboard.json",
UpdatedDataPath: "dashboard/updated_dashboard.json",
CreateRespBodyPath: "dashboard/create_dashboard_response.json",
CreateTFPath: "dashboard/dashboard.tf",
UpdateTFPath: "dashboard/update_dashboard.tf",
}
tc.Test()
}
204 changes: 60 additions & 144 deletions terraform/graylog/resource_role_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,160 +2,76 @@ package graylog

import (
"encoding/json"
"io/ioutil"
"net/http"
"strings"
"errors"
"io"
"testing"

"github.com/hashicorp/terraform/helper/resource"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"github.com/suzuki-shunsuke/flute/flute"
"github.com/suzuki-shunsuke/go-set/v6"
)

func genTestRoleBody(role map[string]interface{}, bodyString string, store *bodyStore) func(t *testing.T, req *http.Request, svc *flute.Service, route *flute.Route) {
return func(t *testing.T, req *http.Request, svc *flute.Service, route *flute.Route) {
body := map[string]interface{}{}
require.Nil(t, json.NewDecoder(req.Body).Decode(&body))

perms := set.NewStrSet()
for _, p := range body["permissions"].([]interface{}) {
perms.Add(p.(string))
}
body["permissions"] = perms
assert.Equal(t, role, body)
store.Set(bodyString)
}
}

func TestAccRole(t *testing.T) {
setEnv()

roleBody, err := ioutil.ReadFile("../../testdata/role/role.json")
require.Nil(t, err)

updateRoleBody, err := ioutil.ReadFile("../../testdata/role/update_role.json")
require.Nil(t, err)

roleTF, err := ioutil.ReadFile("../../testdata/role/role.tf")
require.Nil(t, err)
tc := &testCase{
t: t,
Name: "role",
CreatePath: "/api/roles",
GetPath: "/api/roles/foo",

updateRoleTF, err := ioutil.ReadFile("../../testdata/role/update_role.tf")
require.Nil(t, err)

store := newBodyStore("")

defaultTransport := http.DefaultClient.Transport
defer func() {
http.DefaultClient.Transport = defaultTransport
}()
http.DefaultClient.Transport = &flute.Transport{
T: t,
Services: []flute.Service{
{
Endpoint: "http://example.com",
Routes: []flute.Route{
{
Matcher: &flute.Matcher{
Method: "GET",
},
Tester: &flute.Tester{
Path: "/api/roles/foo",
PartOfHeader: getTestHeader(),
},
Response: &flute.Response{
Response: func(req *http.Request) (*http.Response, error) {
return &http.Response{
StatusCode: 200,
Body: ioutil.NopCloser(strings.NewReader(store.Get())),
}, nil
},
},
},
{
Matcher: &flute.Matcher{
Method: "POST",
},
Tester: &flute.Tester{
Path: "/api/roles",
PartOfHeader: getTestHeader(),
Test: genTestRoleBody(map[string]interface{}{
"name": "foo",
"description": "Allows reading and writing all views and extended searches (built-in)",
"permissions": set.NewStrSet(
"extendedsearch:create",
"extendedsearch:use",
"view:create",
"view:edit",
"view:read",
"view:use",
),
"read_only": true,
}, string(roleBody), store),
},
Response: &flute.Response{
Base: http.Response{
StatusCode: 201,
},
BodyString: string(roleBody),
},
},
{
Matcher: &flute.Matcher{
Method: "PUT",
},
Tester: &flute.Tester{
Path: "/api/roles/foo",
PartOfHeader: getTestHeader(),
Test: genTestRoleBody(map[string]interface{}{
"name": "foo",
"description": "updated description",
"permissions": set.NewStrSet(
"extendedsearch:create",
"extendedsearch:use",
"view:edit",
"view:read",
"view:use",
),
"read_only": false,
}, string(updateRoleBody), store),
},
Response: &flute.Response{
Base: http.Response{
StatusCode: 201,
},
BodyString: string(roleBody),
},
},
{
Matcher: &flute.Matcher{
Method: "DELETE",
},
Tester: &flute.Tester{
Path: "/api/roles/foo",
PartOfHeader: getTestHeader(),
},
Response: &flute.Response{
Base: http.Response{
StatusCode: 204,
},
},
},
},
},
ConvertReqBody: func(body io.Reader) (map[string]interface{}, error) {
m := map[string]interface{}{}
if err := json.NewDecoder(body).Decode(&m); err != nil {
return nil, err
}
if perms, ok := m["permissions"]; ok {
arr, ok := perms.([]interface{})
if !ok {
return nil, errors.New("permissions should be list")
}
perms := make([]string, len(arr))
for i, p := range arr {
a, ok := p.(string)
if !ok {
return nil, errors.New("permissions should be list of string")
}
perms[i] = a
}
m["permissions"] = set.NewStrSet(perms...)
}
return m, nil
},
}

resource.Test(t, resource.TestCase{
Providers: getTestProviders(),
Steps: []resource.TestStep{
{
Config: string(roleTF),
},
{
Config: string(updateRoleTF),
},
CreateReqBodyMap: map[string]interface{}{
"name": "foo",
"description": "Allows reading and writing all views and extended searches (built-in)",
"permissions": set.NewStrSet(
"extendedsearch:create",
"extendedsearch:use",
"view:create",
"view:edit",
"view:read",
"view:use",
),
"read_only": true,
},
})
UpdateReqBodyMap: map[string]interface{}{
"name": "foo",
"description": "updated description",
"permissions": set.NewStrSet(
"extendedsearch:create",
"extendedsearch:use",
"view:edit",
"view:read",
"view:use",
),
"read_only": false,
},
CreatedDataPath: "role/role.json",
UpdatedDataPath: "role/update_role.json",
CreateRespBodyPath: "role/role.json",
UpdateRespBodyPath: "role/role.json",
CreateTFPath: "role/role.tf",
UpdateTFPath: "role/update_role.tf",
}
tc.Test()
}

0 comments on commit f2da7c6

Please sign in to comment.