load-balancers: Add support for size_unit

[bbassingthwaite]

No description provided.

[bentranter]

Changes look great but could you update the tests as well for the new size_unit field? Thanks!

[bbassingthwaite]

@bentranter done! Also added a test for using just size_unit.

[andrewsomething]

    resource_digitalocean_loadbalancer_test.go:113: Step 2/2 error: Check failed: 1 error occurred:
                * Check 4/13 error: digitalocean_loadbalancer.foobar: Attribute 'size' not found
        
--- FAIL: TestAccDigitalOceanLoadbalancer_Updated (284.68s)

Looks like this one needs updating as well:

terraform-provider-digitalocean/digitalocean/resource_digitalocean_loadbalancer_test.go

Line 166 in 130b7a1

"digitalocean_loadbalancer.foobar", "size", "lb-small"),

(GitHub won't let you comment on an unchanged line.)

Also noticing this one failing:

    resource_digitalocean_loadbalancer_test.go:471: Step 2/2, expected an error but got none
--- FAIL: TestAccDigitalOceanLoadbalancer_resizeExpectedFailure (104.86s)

Does the policy around resizing change with this release?

terraform-provider-digitalocean/digitalocean/resource_digitalocean_loadbalancer_test.go

Lines 444 to 448 in 130b7a1

	// Load balancers can only be resized once an hour. The initial create counts
	// as a "resize" in this context. This test can not perform a resize, but it
	// does ensure that the the PUT includes the expected content by checking for
	// the failure.
	func TestAccDigitalOceanLoadbalancer_resizeExpectedFailure(t *testing.T) {

[bbassingthwaite]

TestAccDigitalOceanLoadbalancer_Updated

Fixed this one up!

TestAccDigitalOceanLoadbalancer_resizeExpectedFailure

We still have the same policy around resizing, but fixed the issue by moving the param to size_unit.

[andrewsomething]

👍 Looks good!

[bentranter]

LGTM as well!