Merge pull request #796 from timofurrer/feature/issue-mr-templates

Support `issues_template` and `merge_requests_template` in project resource

Author: nagyv

docs/resources/project.md

@@ -109,6 +109,10 @@ consult the [gitlab documentation](https://docs.gitlab.com/ee/user/project/repos
   Valid values are `disabled`, `private`, `enabled`, `public`.
   `private` is the default.
 
+* `issues_template` - (Optional) Sets the template for new issues in the project.
+
+* `merge_requests_template` - (Optional) Sets the template for new merge requests in the project.
+
 * `build_coverage_regex` - (Optional) Test coverage parsing for the project.
 
 * `ci_config_path` - (Optional) Custom Path to CI config file.

gitlab/resource_gitlab_project.go

@@ -290,6 +290,14 @@ var resourceGitLabProjectSchema = map[string]*schema.Schema{
 		Type:     schema.TypeString,
 		Optional: true,
 	},
+	"issues_template": {
+		Type:     schema.TypeString,
+		Optional: true,
+	},
+	"merge_requests_template": {
+		Type:     schema.TypeString,
+		Optional: true,
+	},
 	"ci_config_path": {
 		Type:     schema.TypeString,
 		Optional: true,
@@ -349,6 +357,8 @@ func resourceGitlabProjectSetToState(d *schema.ResourceData, project *gitlab.Pro
 	d.Set("mirror_overwrites_diverged_branches", project.MirrorOverwritesDivergedBranches)
 	d.Set("only_mirror_protected_branches", project.OnlyMirrorProtectedBranches)
 	d.Set("build_coverage_regex", project.BuildCoverageRegex)
+	d.Set("issues_template", project.IssuesTemplate)
+	d.Set("merge_requests_template", project.MergeRequestsTemplate)
 	d.Set("ci_config_path", project.CIConfigPath)
 	return nil
 }
@@ -551,6 +561,14 @@ func resourceGitlabProjectCreate(ctx context.Context, d *schema.ResourceData, me
 		editProjectOptions.ImportURL = gitlab.String(d.Get("import_url").(string))
 	}
 
+	if v, ok := d.GetOk("issues_template"); ok {
+		editProjectOptions.IssuesTemplate = gitlab.String(v.(string))
+	}
+
+	if v, ok := d.GetOk("merge_requests_template"); ok {
+		editProjectOptions.MergeRequestsTemplate = gitlab.String(v.(string))
+	}
+
 	if (editProjectOptions != gitlab.EditProjectOptions{}) {
 		if _, _, err := client.Projects.EditProject(d.Id(), &editProjectOptions, gitlab.WithContext(ctx)); err != nil {
 			return diag.Errorf("Could not update project %q: %s", d.Id(), err)
@@ -724,6 +742,14 @@ func resourceGitlabProjectUpdate(ctx context.Context, d *schema.ResourceData, me
 		options.BuildCoverageRegex = gitlab.String(d.Get("build_coverage_regex").(string))
 	}
 
+	if d.HasChange("issues_template") {
+		options.IssuesTemplate = gitlab.String(d.Get("issues_template").(string))
+	}
+
+	if d.HasChange("merge_requests_template") {
+		options.MergeRequestsTemplate = gitlab.String(d.Get("merge_requests_template").(string))
+	}
+
 	if d.HasChange("ci_config_path") {
 		options.CIConfigPath = gitlab.String(d.Get("ci_config_path").(string))
 	}

gitlab/resource_gitlab_project_test.go

@@ -50,6 +50,8 @@ func TestAccGitlabProject_basic(t *testing.T) {
 		PackagesEnabled:             true,
 		PagesAccessLevel:            gitlab.PublicAccessControl,
 		BuildCoverageRegex:          "foo",
+		IssuesTemplate:              "",
+		MergeRequestsTemplate:       "",
 		CIConfigPath:                ".gitlab-ci.yml@mynamespace/myproject",
 	}
 
@@ -381,6 +383,37 @@ func TestAccGitlabProject_initializeWithoutReadme(t *testing.T) {
 	})
 }
 
+func TestAccGitlabProject_IssueMergeRequestTemplates(t *testing.T) {
+	var project gitlab.Project
+	rInt := acctest.RandInt()
+
+	resource.Test(t, resource.TestCase{
+		PreCheck:     func() { testAccPreCheck(t) },
+		Providers:    testAccProviders,
+		CheckDestroy: testAccCheckGitlabProjectDestroy,
+		Steps: []resource.TestStep{
+			{
+				SkipFunc: isRunningInCE,
+				Config:   testAccGitlabProjectConfigIssueMergeRequestTemplates(rInt),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckGitlabProjectExists("gitlab_project.foo", &project),
+					func(s *terraform.State) error {
+						if project.IssuesTemplate != "foo" {
+							return fmt.Errorf("expected issues template to be 'foo'; got '%s'", project.IssuesTemplate)
+						}
+
+						if project.MergeRequestsTemplate != "bar" {
+							return fmt.Errorf("expected merge requests template to be 'bar'; got '%s'", project.MergeRequestsTemplate)
+						}
+
+						return nil
+					},
+				),
+			},
+		},
+	})
+}
+
 func TestAccGitlabProject_willError(t *testing.T) {
 	var received, defaults gitlab.Project
 	rInt := acctest.RandInt()
@@ -1298,3 +1331,15 @@ resource "gitlab_project" "template-mutual-exclusive" {
 }
 	`, rInt, rInt)
 }
+
+func testAccGitlabProjectConfigIssueMergeRequestTemplates(rInt int) string {
+	return fmt.Sprintf(`
+resource "gitlab_project" "foo" {
+  name = "foo-%d"
+  path = "foo.%d"
+  description = "Terraform acceptance tests"
+  issues_template = "foo"
+  merge_requests_template = "bar"
+}
+	`, rInt, rInt)
+}