-
Notifications
You must be signed in to change notification settings - Fork 630
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement waiting room resource #1053
Conversation
waitingRoomID := d.Id() | ||
zoneID := d.Get("zone_id").(string) | ||
|
||
return resourceCloudflareWaitingRoomReadFromWaitingRoomId(d, meta, zoneID, waitingRoomID) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this functionality should be pulled into the Read
method
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Co-authored-by: Jacob Bednarz <[email protected]>
Co-authored-by: Jacob Bednarz <[email protected]>
thanks for the updates here; it looks like we're almost there but still missing tests for this new functionality. are you able to add those to ensure this functionality works as intended and we can catch any regressions going forward? |
Do you mean unit test or acceptance test? I did not find a lot of unit test. And I'm not sure how acceptance test works. Do you have a doc around them? |
integration tests for the actual resources. see the README (https://github.com/cloudflare/terraform-provider-cloudflare#developing-the-provider) and Terraform's own documentation for acceptance testing https://www.terraform.io/docs/extend/testing/acceptance-tests/index.html |
@bpaquet are you still interested in completing this one? |
Yes, but I cannt write acceptance test, I need a dedicated Cloudflare
account for this.
Bertrand
…On Mon, Jun 21, 2021 at 2:56 AM Jacob Bednarz ***@***.***> wrote:
@bpaquet <https://github.com/bpaquet> are you still interested in
completing this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1053 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACSJXITALOE65KQZWSH6WLTT2E4JANCNFSM44LC5ZUQ>
.
|
If you create a free zone, email me (or Cloudflare Support) the zone ID and I can take a look at adding the entitlements temporarily for you to add this. |
Tested with `TESTARGS='-run "^TestAccCloudflareWaitingRoom"' make testacc`
Nice, we're all green on the acceptance testing accounts too so this is good to merge.
|
Need cloudflare/cloudflare-go#627 to be merged and released before.