Skip to content

Fixes tests for Search in Go TFE#1276

Merged
sowju-hashicorp merged 1 commit intomainfrom
sowju/TF-31829
Feb 11, 2026
Merged

Fixes tests for Search in Go TFE#1276
sowju-hashicorp merged 1 commit intomainfrom
sowju/TF-31829

Conversation

@sowju-hashicorp
Copy link
Copy Markdown
Contributor

@sowju-hashicorp sowju-hashicorp commented Jan 29, 2026

Description

This PR removes beta feature flags from QueryRuns integration tests, following the QueryRun API becoming generally available in HCP Terraform (as documented in CHANGELOG v1.96.0). QueryRuns is Terraform's query/search API for workspaces.

Changes:

Removed skipUnlessBeta(t) calls from all 7 QueryRuns integration test functions
Tests will now run in standard CI without requiring the ENABLE_BETA=1 environment variable

External links

Output from tests

Including output from tests may require access to a TFE instance. Ignore this section if you have no environment to test against.

$ TFE_ADDRESS="https://example" TFE_TOKEN="example" go test ./... -v -run TestFunctionsAffectedByChange

...

Rollback Plan

Changes to Security Controls

Copilot AI review requested due to automatic review settings January 29, 2026 13:27
@sowju-hashicorp sowju-hashicorp requested a review from a team as a code owner January 29, 2026 13:27
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes beta feature flags from QueryRuns integration tests, following the QueryRun API becoming generally available in HCP Terraform (as documented in CHANGELOG v1.96.0). QueryRuns is Terraform's query/search API for workspaces.

Changes:

  • Removed skipUnlessBeta(t) calls from all 7 QueryRuns integration test functions
  • Tests will now run in standard CI without requiring the ENABLE_BETA=1 environment variable

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@datadog-terraform-cloud-hashicorp
Copy link
Copy Markdown

datadog-terraform-cloud-hashicorp bot commented Jan 29, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

❄️ 2 New flaky tests detected

TestPlanExportsCreate_RunDependent from plan_export_integration_test.go (Datadog) (Fix with Cursor)
Run "run-wcW9ZGDonAupdomv" had status "plan_queued" at deadline
TestPlanExportsCreate_RunDependent from github.com/hashicorp/go-tfe (Datadog) (Fix with Cursor)
Failed

=== RUN   TestPlanExportsCreate_RunDependent
    helper_test.go:1583: Polling run "run-wcW9ZGDonAupdomv" for status included in ["cost_estimated" "planned" "post_plan_completed" "errored"] with deadline of 2026-02-11 17:49:43.204250389 +0000 UTC m=+125.839265237
    helper_test.go:1589: ...
    helper_test.go:1680: Reading run "run-wcW9ZGDonAupdomv"
    helper_test.go:1595: Run "run-wcW9ZGDonAupdomv" had status "plan_queued"
    helper_test.go:1589: ...
    helper_test.go:1680: Reading run "run-wcW9ZGDonAupdomv"
    helper_test.go:1595: Run "run-wcW9ZGDonAupdomv" had status "plan_queued"
...

ℹ️ Info

🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ca51e91 | Docs | Was this helpful? Give us feedback!

@sowju-hashicorp sowju-hashicorp added the no-changelog-needed check changelog entry github workflow label Jan 29, 2026
sebasslash
sebasslash previously approved these changes Feb 9, 2026
@sowju-hashicorp sowju-hashicorp merged commit da0deb6 into main Feb 11, 2026
15 checks passed
@sowju-hashicorp sowju-hashicorp deleted the sowju/TF-31829 branch February 11, 2026 21:50
@github-actions
Copy link
Copy Markdown

Reminder to the contributor that merged this PR: if your changes have added important functionality or fixed a relevant bug, open a follow-up PR to update CHANGELOG.md with a note on your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog-needed check changelog entry github workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants