Skip to content

Commit

Permalink
Merge pull request #93 from phillc/cleanup
Browse files Browse the repository at this point in the history
Cleanup
  • Loading branch information
phillc authored Sep 30, 2019
2 parents be0c109 + ab94b7a commit 1b21bc4
Show file tree
Hide file tree
Showing 108 changed files with 491 additions and 464 deletions.
8 changes: 6 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ GOLANGCILINT_WARN_ARGS := run --no-config --issues-exit-code=0 --disable-all --e
test: build lint
@LINODE_FIXTURE_MODE="play" \
LINODE_TOKEN="awesometokenawesometokenawesometoken" \
LINODE_API_VERSION="v4beta" \
GO111MODULE="on" \
go test $(ARGS)

Expand All @@ -31,8 +32,11 @@ refresh-fixtures: clean-fixtures fixtures

run_fixtures:
@echo "* Running fixtures"
@LINODE_TOKEN=$(LINODE_TOKEN) \
LINODE_FIXTURE_MODE="record" go test $(ARGS)
@LINODE_FIXTURE_MODE="record" \
LINODE_TOKEN=$(LINODE_TOKEN) \
LINODE_API_VERSION="v4beta" \
GO111MODULE="on" \
go test $(ARGS)

sanitize:
@echo "* Santizing fixtures"
Expand Down
192 changes: 105 additions & 87 deletions client.go

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions example_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ func ExampleClient_GetType_missing() {
}

// Output:
// Request was: https://api.linode.com/v4/linode/types/missing-type
// Request was: https://api.linode.com/v4beta/linode/types/missing-type
// Response was: 404 NOT FOUND
// Error was: [404] Not found
}
Expand Down Expand Up @@ -184,7 +184,7 @@ func ExampleClient_GetImage_missing() {
}

// Output:
// Request was: https://api.linode.com/v4/images/not-found
// Request was: https://api.linode.com/v4beta/images/not-found
// Response was: 404 NOT FOUND
// Error was: [404] Not found
}
Expand Down
24 changes: 12 additions & 12 deletions fixtures/Example.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/1231
url: https://api.linode.com/v4beta/linode/instances/1231
method: GET
response:
body: '{"errors": [{"reason": "Not found"}]}'
Expand Down Expand Up @@ -69,7 +69,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances
url: https://api.linode.com/v4beta/linode/instances
method: GET
response:
body: '{"data": [{"region": "us-east", "alerts": {"network_in": 10, "network_out":
Expand Down Expand Up @@ -149,7 +149,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403
url: https://api.linode.com/v4beta/linode/instances/76859403
method: GET
response:
body: '{"image": "linode/ubuntu18.04", "group": "", "updated": "2018-01-02T03:04:05",
Expand Down Expand Up @@ -228,7 +228,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403/configs
url: https://api.linode.com/v4beta/linode/instances/76859403/configs
method: GET
response:
body: '{"results": 1, "data": [{"memory_limit": 0, "kernel": "linode/grub2", "label":
Expand Down Expand Up @@ -308,7 +308,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403/configs/9958462
url: https://api.linode.com/v4beta/linode/instances/76859403/configs/9958462
method: GET
response:
body: '{"initrd": null, "kernel": "linode/grub2", "devices": {"sdh": null, "sdf":
Expand Down Expand Up @@ -387,7 +387,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403/disks
url: https://api.linode.com/v4beta/linode/instances/76859403/disks
method: GET
response:
body: '{"pages": 1, "page": 1, "data": [{"updated": "2018-01-02T03:04:05", "label":
Expand Down Expand Up @@ -463,7 +463,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403/disks/18974977
url: https://api.linode.com/v4beta/linode/instances/76859403/disks/18974977
method: GET
response:
body: '{"created": "2018-01-02T03:04:05", "id": 18974977, "filesystem": "ext4",
Expand Down Expand Up @@ -537,7 +537,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403/backups
url: https://api.linode.com/v4beta/linode/instances/76859403/backups
method: GET
response:
body: '{"snapshot": {"in_progress": null, "current": {"configs": ["My Ubuntu 18.04
Expand Down Expand Up @@ -615,7 +615,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403/backups/96509901
url: https://api.linode.com/v4beta/linode/instances/76859403/backups/96509901
method: GET
response:
body: '{"label": "test-snapshot", "region": "us-east", "status": "successful",
Expand Down Expand Up @@ -691,7 +691,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/instances/76859403/volumes
url: https://api.linode.com/v4beta/linode/instances/76859403/volumes
method: GET
response:
body: '{"results": 1, "data": [{"created": "2018-01-02T03:04:05", "id": 6574839201,
Expand Down Expand Up @@ -766,7 +766,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/volumes/6574839201
url: https://api.linode.com/v4beta/volumes/6574839201
method: GET
response:
body: '{"filesystem_path": "/dev/disk/by-id/scsi-0Linode_Volume_test-volume",
Expand Down Expand Up @@ -843,7 +843,7 @@ interactions:
- linodego 0.0.1 https://github.com/linode/linodego
X-Filter:
- '{"mine":true}'
url: https://api.linode.com/v4/linode/stackscripts
url: https://api.linode.com/v4beta/linode/stackscripts
method: GET
response:
body: '{"pages": 1, "page": 1, "data": [{"script": "#!/bin/bash\n# <UDF name=\"gh_username\"
Expand Down
8 changes: 4 additions & 4 deletions fixtures/ExampleCreateNodeBalancer.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers
url: https://api.linode.com/v4beta/nodebalancers
method: POST
response:
body: '{"region": "us-east", "created": "2018-01-02T03:04:05", "updated": "2018-01-02T03:04:05",
Expand Down Expand Up @@ -86,7 +86,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36396
url: https://api.linode.com/v4beta/nodebalancers/36396
method: GET
response:
body: '{"ipv6": "1234::5678", "updated": "2018-01-02T03:04:05", "label":
Expand Down Expand Up @@ -161,7 +161,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36396
url: https://api.linode.com/v4beta/nodebalancers/36396
method: PUT
response:
body: '{"id": 36396, "label": "balancer36396_renamed", "ipv6": "1234::5678",
Expand Down Expand Up @@ -234,7 +234,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36396
url: https://api.linode.com/v4beta/nodebalancers/36396
method: DELETE
response:
body: '{}'
Expand Down
14 changes: 7 additions & 7 deletions fixtures/ExampleCreateNodeBalancerConfig.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers
url: https://api.linode.com/v4beta/nodebalancers
method: POST
response:
body: '{"ipv6": "1234::5678", "updated": "2018-01-02T03:04:05", "label":
Expand Down Expand Up @@ -86,7 +86,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36397/configs
url: https://api.linode.com/v4beta/nodebalancers/36397/configs
method: POST
response:
body: '{"id": 32600, "check_attempts": 3, "check_timeout": 30, "ssl_cert": null,
Expand Down Expand Up @@ -161,7 +161,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36397/configs/32600
url: https://api.linode.com/v4beta/nodebalancers/36397/configs/32600
method: PUT
response:
body: '{"check": "none", "protocol": "http", "check_interval": 0, "cipher_suite":
Expand Down Expand Up @@ -236,7 +236,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36397/configs
url: https://api.linode.com/v4beta/nodebalancers/36397/configs
method: GET
response:
body: '{"data": [{"algorithm": "roundrobin", "check_timeout": 30, "check_path":
Expand Down Expand Up @@ -313,7 +313,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36397/configs/32600
url: https://api.linode.com/v4beta/nodebalancers/36397/configs/32600
method: GET
response:
body: '{"id": 32600, "check_attempts": 3, "check_timeout": 30, "ssl_cert": null,
Expand Down Expand Up @@ -390,7 +390,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36397/configs/32600
url: https://api.linode.com/v4beta/nodebalancers/36397/configs/32600
method: DELETE
response:
body: '{}'
Expand Down Expand Up @@ -460,7 +460,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36397
url: https://api.linode.com/v4beta/nodebalancers/36397
method: DELETE
response:
body: '{}'
Expand Down
18 changes: 9 additions & 9 deletions fixtures/ExampleCreateNodeBalancerNode.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers
url: https://api.linode.com/v4beta/nodebalancers
method: POST
response:
body: '{"region": "us-east", "created": "2018-01-02T03:04:05", "updated": "2018-01-02T03:04:05",
Expand Down Expand Up @@ -86,7 +86,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398/configs
url: https://api.linode.com/v4beta/nodebalancers/36398/configs
method: POST
response:
body: '{"algorithm": "roundrobin", "check_timeout": 30, "check_path": "", "ssl_key":
Expand Down Expand Up @@ -161,7 +161,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398/configs/32601/nodes
url: https://api.linode.com/v4beta/nodebalancers/36398/configs/32601/nodes
method: POST
response:
body: '{"id": 313257, "address": "192.168.030.040:80", "mode": "accept", "nodebalancer_id":
Expand Down Expand Up @@ -232,7 +232,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398/configs/32601/nodes/313257
url: https://api.linode.com/v4beta/nodebalancers/36398/configs/32601/nodes/313257
method: PUT
response:
body: '{"mode": "accept", "status": "Unknown", "config_id": 32601, "id": 313257,
Expand Down Expand Up @@ -304,7 +304,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398/configs/32601/nodes
url: https://api.linode.com/v4beta/nodebalancers/36398/configs/32601/nodes
method: GET
response:
body: '{"data": [{"mode": "accept", "status": "Unknown", "config_id": 32601, "id":
Expand Down Expand Up @@ -378,7 +378,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398/configs/32601/nodes/313257
url: https://api.linode.com/v4beta/nodebalancers/36398/configs/32601/nodes/313257
method: GET
response:
body: '{"address": "192.168.030.040:8080", "nodebalancer_id": 36398, "label": "192.168.030.040-80",
Expand Down Expand Up @@ -452,7 +452,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398/configs/32601/nodes/313257
url: https://api.linode.com/v4beta/nodebalancers/36398/configs/32601/nodes/313257
method: DELETE
response:
body: '{}'
Expand Down Expand Up @@ -522,7 +522,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398/configs/32601
url: https://api.linode.com/v4beta/nodebalancers/36398/configs/32601
method: DELETE
response:
body: '{}'
Expand Down Expand Up @@ -592,7 +592,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/nodebalancers/36398
url: https://api.linode.com/v4beta/nodebalancers/36398
method: DELETE
response:
body: '{}'
Expand Down
10 changes: 5 additions & 5 deletions fixtures/ExampleCreateStackscript.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/stackscripts
url: https://api.linode.com/v4beta/linode/stackscripts
method: POST
response:
body: '{"user_defined_fields": [], "images": ["linode/debian9", "linode/ubuntu18.04"],
Expand Down Expand Up @@ -94,7 +94,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/stackscripts/324986
url: https://api.linode.com/v4beta/linode/stackscripts/324986
method: PUT
response:
body: '{"script": "#!/bin/bash\necho 2\n", "images": ["linode/debian9", "linode/ubuntu18.04"],
Expand Down Expand Up @@ -173,7 +173,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/stackscripts/324986
url: https://api.linode.com/v4beta/linode/stackscripts/324986
method: PUT
response:
body: '{"user_gravatar_id": "8293c9dd7d457c488afaee7fea921610", "id": 324986,
Expand Down Expand Up @@ -249,7 +249,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/stackscripts/324986
url: https://api.linode.com/v4beta/linode/stackscripts/324986
method: GET
response:
body: '{"user_defined_fields": [], "images": ["linode/debian9", "linode/ubuntu18.04"],
Expand Down Expand Up @@ -328,7 +328,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/linode/stackscripts/324986
url: https://api.linode.com/v4beta/linode/stackscripts/324986
method: DELETE
response:
body: '{}'
Expand Down
2 changes: 1 addition & 1 deletion fixtures/ExampleGetAccount.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/account
url: https://api.linode.com/v4beta/account
method: GET
response:
body: '{"phone": "555-555-5555", "email": "[email protected]", "address_2":
Expand Down
2 changes: 1 addition & 1 deletion fixtures/ExampleGetImage_missing.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ interactions:
- application/json
User-Agent:
- linodego 0.0.1 https://github.com/linode/linodego
url: https://api.linode.com/v4/images/not-found
url: https://api.linode.com/v4beta/images/not-found
method: GET
response:
body: '{"errors": [{"reason": "Not found"}]}'
Expand Down
Loading

0 comments on commit 1b21bc4

Please sign in to comment.