Support initial_version for creating branch based modules (#1363)

Author: aaabdelgany

CHANGELOG.md

@@ -4,6 +4,7 @@ ENHANCEMENTS:
 * `r/tfe_oauth_client`: Add Bitbucket Data Center support with the `bitbucket_data_center` option for `service_provider` by @zainq11 [#1303](https://github.com/hashicorp/terraform-provider-tfe/pull/1304)
 * `r/tfe_workspace`: Add an `auto_destroy_at` attribute for scheduling an auto-destroy run in the future, by @notchairmk [1354](https://github.com/hashicorp/terraform-provider-tfe/pull/1354)
 * `d/tfe_workspace`: Add an `auto_destroy_at` attribute for reading a scheduled auto-destroy, by @notchairmk [1354](https://github.com/hashicorp/terraform-provider-tfe/pull/1354)
+* `r/tfe_registry_module`: Add `initial_version` support for Branch Based Modules by @aaabdelgany [#1363](https://github.com/hashicorp/terraform-provider-tfe/pull/1363)
 
 ## v0.55.0
 

internal/provider/resource_tfe_registry_module.go

@@ -134,6 +134,10 @@ func resourceTFERegistryModule() *schema.Resource {
 					},
 				},
 			},
+			"initial_version": {
+				Type:     schema.TypeString,
+				Optional: true,
+			},
 		},
 	}
 }
@@ -167,13 +171,17 @@ func resourceTFERegistryModuleCreateWithVCS(v interface{}, meta interface{}, d *
 
 	tags, tagsOk := vcsRepo["tags"].(bool)
 	branch, branchOk := vcsRepo["branch"].(string)
+	initialVersion, initialVersionOk := d.GetOk("initial_version")
 
 	if tagsOk && tags && branchOk && branch != "" {
 		return nil, fmt.Errorf("tags must be set to false when a branch is provided")
 	}
 
 	if branchOk && branch != "" {
 		options.VCSRepo.Branch = tfe.String(branch)
+		if initialVersionOk && initialVersion.(string) != "" {
+			options.InitialVersion = tfe.String(initialVersion.(string))
+		}
 	}
 
 	if vcsRepo["oauth_token_id"] != nil && vcsRepo["oauth_token_id"].(string) != "" {

internal/provider/resource_tfe_registry_module_test.go

@@ -403,7 +403,6 @@ func TestAccTFERegistryModuleImport_vcsPrivateRMRecommendedFormat(t *testing.T)
 }
 
 func TestAccTFERegistryModuleImport_vcsPublishingMechanismBranchToTagsToBranch(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -463,7 +462,6 @@ func TestAccTFERegistryModuleImport_vcsPublishingMechanismBranchToTagsToBranch(t
 }
 
 func TestAccTFERegistryModuleImport_vcsPublishingMechanismBranchToTagsToBranchWithTests(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -505,7 +503,6 @@ func TestAccTFERegistryModuleImport_vcsPublishingMechanismBranchToTagsToBranchWi
 }
 
 func TestAccTFERegistryModuleImport_vcsPublishingMechanismTagsToBranchToTags(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -556,7 +553,6 @@ func TestAccTFERegistryModuleImport_vcsPublishingMechanismTagsToBranchToTags(t *
 }
 
 func TestAccTFERegistryModule_invalidTestConfigOnCreate(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -575,7 +571,6 @@ func TestAccTFERegistryModule_invalidTestConfigOnCreate(t *testing.T) {
 }
 
 func TestAccTFERegistryModule_invalidTestConfigOnUpdate(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -603,7 +598,6 @@ func TestAccTFERegistryModule_invalidTestConfigOnUpdate(t *testing.T) {
 }
 
 func TestAccTFERegistryModule_branchAndInvalidTagsOnCreate(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -622,7 +616,6 @@ func TestAccTFERegistryModule_branchAndInvalidTagsOnCreate(t *testing.T) {
 }
 
 func TestAccTFERegistryModule_branchAndTagsEnabledOnCreate(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -641,7 +634,6 @@ func TestAccTFERegistryModule_branchAndTagsEnabledOnCreate(t *testing.T) {
 }
 
 func TestAccTFERegistryModule_branchAndTagsEnabledOnUpdate(t *testing.T) {
-	skipUnlessBeta(t)
 	rInt := rand.New(rand.NewSource(time.Now().UnixNano())).Int()
 
 	resource.Test(t, resource.TestCase{
@@ -1027,6 +1019,8 @@ resource "tfe_registry_module" "foobar" {
    tags				  = false
  }
 
+ initial_version = "1.0.0"
+
  test_config {
    tests_enabled = false
  }
@@ -1061,6 +1055,8 @@ resource "tfe_registry_module" "foobar" {
    tags				  = false
  }
 
+ initial_version = "1.0.0"
+
  test_config {
  }
 }`,
@@ -1094,6 +1090,8 @@ resource "tfe_registry_module" "foobar" {
    tags				  = false
  }
 
+ initial_version = "1.0.0"
+
  test_config {
    tests_enabled = true
  }
@@ -1129,6 +1127,8 @@ resource "tfe_registry_module" "foobar" {
    tags				  = false
  }
 
+ initial_version = "1.0.0"
+
  test_config {
  }
 }`,
@@ -1163,6 +1163,8 @@ resource "tfe_registry_module" "foobar" {
    tags				  = true
  }
 
+ initial_version = "1.0.0"
+
  test_config {
    tests_enabled = true
  }

website/docs/r/registry_module.html.markdown

@@ -154,6 +154,7 @@ The following arguments are supported:
 * `organization` - (Optional) The name of the organization associated with the registry module. It must be set if `module_provider` is used, or if `vcs_repo` is used via a GitHub App.
 * `namespace` - (Optional) The namespace of a public registry module. It can be used if `module_provider` is set and `registry_name` is public.
 * `registry_name` - (Optional) Whether the registry module is private or public. It can be used if `module_provider` is set.
+* `initial_version` - (Optional) This specifies the initial version for a branch based module. It can be used if `vcs_repo.branch` is set. If it is omitted, the initial modules version will default to `0.0.0`.
 
 The `test_config` block supports
 * `tests_enabled` - (Optional) Specifies whether tests run for the registry module. Tests are only supported for branch-based publishing.