Skip to content

Fix project acceptance import state - #106

Merged
IlyaasK merged 1 commit into
mainfrom
hypeship/fix-project-import-acceptance
Jul 28, 2026
Merged

Fix project acceptance import state#106
IlyaasK merged 1 commit into
mainfrom
hypeship/fix-project-import-acceptance

Conversation

@IlyaasK

@IlyaasK IlyaasK commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Stop persisting command-import state over an already-managed project.
  • Keep strict imported-state verification in an isolated Terraform state.
  • Verify declarative import produces a no-op plan in a separate import-block
    step.

Why

The live project acceptance test failed before exercising provider import because
ImportStatePersist reused state that already contained kernel_project.test.
Terraform correctly rejected importing another object at the managed address.

The default command-import test uses isolated state and
ImportStateVerify compares the imported attributes with the prior resource
state. The import-block step independently removes the copied state entry and
requires the configured import plan to be a no-op.

Tests

  • gofmt -l internal/resources/project
  • go test -short -timeout=2m ./internal/resources/project
  • go vet ./internal/resources/project
  • git diff --check
  • Acceptance workflow:
    • Project acceptance passed in 25 seconds.
    • Browser pool acceptance passed in 35 seconds with
      KERNEL_ALT_PROJECT_ID configured.

Scope

No provider behavior, schema, API calls, SDK usage, or production import logic
changes.

@IlyaasK
IlyaasK marked this pull request as ready for review July 28, 2026 17:17
@IlyaasK
IlyaasK force-pushed the hypeship/fix-project-import-acceptance branch from 48a787f to ef4844e Compare July 28, 2026 17:26
@IlyaasK
IlyaasK requested a review from Sayan- July 28, 2026 18:55
@IlyaasK
IlyaasK merged commit e0184b5 into main Jul 28, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants