Skip to content

chore(auth_origin_pulls_cert): add test cases for migration#6880

Closed
ssicard wants to merge 1 commit intonextfrom
ssicard/stateupgrader/auth_origin_pulls_cert
Closed

chore(auth_origin_pulls_cert): add test cases for migration#6880
ssicard wants to merge 1 commit intonextfrom
ssicard/stateupgrader/auth_origin_pulls_cert

Conversation

@ssicard
Copy link
Copy Markdown
Contributor

@ssicard ssicard commented Mar 4, 2026

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

Add more test cases and get current migration tests to pass for auth_origin_pulls_certificate

Acceptance test run results

  • I have added or updated acceptance tests for my changes
  • I have run acceptance tests for my changes and included the results below

Steps to run acceptance tests

TF_ACC=1 TF_MIGRATE_BINARY_PATH=/Users/ssicard/workspace/terraform-devstack/tf-migrate/tf-migrate go test ./internal/services/authenticated_origin_pulls_certificate/migration/v500 -v

Test output

Resource Tests

=== RUN   TestAuthenticatedOriginPullsCertificateDataSourceModelSchemaParity
=== PAUSE TestAuthenticatedOriginPullsCertificateDataSourceModelSchemaParity
=== RUN   TestAuthenticatedOriginPullsCertificatesDataSourceModelSchemaParity
=== PAUSE TestAuthenticatedOriginPullsCertificatesDataSourceModelSchemaParity
=== RUN   TestAuthenticatedOriginPullsCertificateModelSchemaParity
=== PAUSE TestAuthenticatedOriginPullsCertificateModelSchemaParity
=== RUN   TestAccAuthenticatedOriginPullsCertificate_FullLifecycle
--- PASS: TestAccAuthenticatedOriginPullsCertificate_FullLifecycle (3.96s)
=== RUN   TestAccUpgradeAuthenticatedOriginPullsCertificate_FromPublishedV5
--- PASS: TestAccUpgradeAuthenticatedOriginPullsCertificate_FromPublishedV5 (20.68s)
=== CONT  TestAuthenticatedOriginPullsCertificateDataSourceModelSchemaParity
=== CONT  TestAuthenticatedOriginPullsCertificateModelSchemaParity
=== CONT  TestAuthenticatedOriginPullsCertificatesDataSourceModelSchemaParity
--- PASS: TestAuthenticatedOriginPullsCertificateModelSchemaParity (0.00s)
--- PASS: TestAuthenticatedOriginPullsCertificatesDataSourceModelSchemaParity (0.00s)
--- PASS: TestAuthenticatedOriginPullsCertificateDataSourceModelSchemaParity (0.00s)
PASS
ok  	github.com/cloudflare/terraform-provider-cloudflare/internal/services/authenticated_origin_pulls_certificate	26.226s

Migration Tests

=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone/from_v4_latest
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone/from_v5
--- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone (47.09s)
    --- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone/from_v4_latest (26.80s)
    --- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone/from_v5 (20.30s)
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_Minimal
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_Minimal/from_v4_latest_minimal
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_Minimal/from_v5_minimal
--- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_Minimal (40.38s)
    --- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_Minimal/from_v4_latest_minimal (24.17s)
    --- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_Minimal/from_v5_minimal (16.21s)
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_AllFields
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_AllFields/from_v4_latest_all_fields
=== RUN   TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_AllFields/from_v5_all_fields
--- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_AllFields (39.67s)
    --- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_AllFields/from_v4_latest_all_fields (21.77s)
    --- PASS: TestMigrateAuthenticatedOriginPullsCertificate_V4ToV5_PerZone_AllFields/from_v5_all_fields (17.89s)
PASS
ok  	github.com/cloudflare/terraform-provider-cloudflare/internal/services/authenticated_origin_pulls_certificate/migration/v500	128.611s

Additional context & links

@ssicard ssicard self-assigned this Mar 4, 2026
@ssicard ssicard requested a review from a team as a code owner March 4, 2026 21:36
@ssicard
Copy link
Copy Markdown
Contributor Author

ssicard commented Mar 9, 2026

Closing in favor of #6887, which does all aop resources at once.

@ssicard ssicard closed this Mar 9, 2026
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.

1 participant