CLC update: tests and alias

[ack]

customer requested account alias be optionally available. updated tests as per previous PR discussion

[phinze]

Hi @ack - thanks for this!

I see builtin/providers/clc/resource_clc_load_balancer_test.go is going entirely away here - is that because it's exercised elsewhere?

If that's the case, I think we should keep a file there with a comment pointing to where it is tested, just because the "every resource has a test file sibling" is such a pattern for us.

[ack]

gotcha, resurrected an empty test

[phinze]

LGTM

[phinze]

Congrats on your first merge 😄

There's a user facing change here - so let's have you add a line to the changelog under IMPROVEMENTS about the fact that you can now set an account alias. You can follow the patterns elsewhere in the file and commit right to master. Most of us just use the GitHub edit button just after merging to keep the workflow lightweight.

[fatmcgav]

Cheers for the quick turnaround @ack...

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.