feat: Add rule_type parameter to merge_request_approval_rule

This is working in the API, and is necessary in order to enable
Eligible Approvers.

Closes: #904

Author: Petr Vála

docs/resources/project_approval_rule.md

@@ -54,6 +54,14 @@ resource "gitlab_project_approval_rule" "example-three" {
   approvals_required = 3
   user_ids           = [for user in data.gitlab_user.users : user.id]
 }
+
+# Example using `approval_rule`
+resource "gitlab_branch_protection" "any-approver" {
+  project            = 5
+  name               = "Any name"
+  rule_type          = "any_approver"
+  approvals_required = 1
+}
 ```
 
 <!-- schema generated by tfplugindocs -->
@@ -70,6 +78,7 @@ resource "gitlab_project_approval_rule" "example-three" {
 - **group_ids** (Set of Number) A list of group IDs whose members can approve of the merge request.
 - **id** (String) The ID of this resource.
 - **protected_branch_ids** (Set of Number) A list of protected branch IDs (not branch names) for which the rule applies.
+- **rule_type** (String) String, defaults to 'regular'. The type of rule. `any_approver` is a pre-configured default rule with `approvals_required` at `0`. Valid values are `regular`, `any_approver`.
 - **user_ids** (Set of Number) A list of specific User IDs to add to the list of approvers.
 
 ## Import

examples/resources/gitlab_project_approval_rule/resource.tf

@@ -36,3 +36,11 @@ resource "gitlab_project_approval_rule" "example-three" {
   approvals_required = 3
   user_ids           = [for user in data.gitlab_user.users : user.id]
 }
+
+# Example using `approval_rule`
+resource "gitlab_branch_protection" "any-approver" {
+  project            = 5
+  name               = "Any name"
+  rule_type          = "any_approver"
+  approvals_required = 1
+}

internal/provider/resource_gitlab_project_approval_rule.go

@@ -3,18 +3,24 @@ package provider
 import (
 	"context"
 	"errors"
+	"fmt"
 	"log"
 	"strconv"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
 	gitlab "github.com/xanzy/go-gitlab"
 )
 
 // https://docs.gitlab.com/ee/api/merge_request_approvals.html#create-project-level-rule
 var errApprovalRuleNotFound = errors.New("approval rule not found")
 
 var _ = registerResource("gitlab_project_approval_rule", func() *schema.Resource {
+	var validRuleTypeValues = []string{
+		"regular",
+		"any_approver",
+	}
 	return &schema.Resource{
 		Description: "This resource allows you to create and manage multiple approval rules for your GitLab projects. For further information on approval rules, consult the [gitlab documentation](https://docs.gitlab.com/ee/api/merge_request_approvals.html#project-level-mr-approvals).\n\n" +
 			"-> This feature requires GitLab Premium.",
@@ -43,6 +49,14 @@ var _ = registerResource("gitlab_project_approval_rule", func() *schema.Resource
 				Type:        schema.TypeInt,
 				Required:    true,
 			},
+			"rule_type": {
+				Description:      fmt.Sprintf("String, defaults to 'regular'. The type of rule. `any_approver` is a pre-configured default rule with `approvals_required` at `0`. Valid values are %s.", renderValueListForDocs(validRuleTypeValues)),
+				Type:             schema.TypeString,
+				ForceNew:         true,
+				Optional:         true,
+				Computed:         true,
+				ValidateDiagFunc: validation.ToDiagFunc(validation.StringInSlice(validRuleTypeValues, false)),
+			},
 			"user_ids": {
 				Description: "A list of specific User IDs to add to the list of approvers.",
 				Type:        schema.TypeSet,
@@ -77,6 +91,10 @@ func resourceGitlabProjectApprovalRuleCreate(ctx context.Context, d *schema.Reso
 		ProtectedBranchIDs: expandProtectedBranchIDs(d.Get("protected_branch_ids")),
 	}
 
+	if v, ok := d.GetOk("rule_type"); ok {
+		options.RuleType = gitlab.String(v.(string))
+	}
+
 	project := d.Get("project").(string)
 
 	log.Printf("[DEBUG] Project %s create gitlab project-level rule %+v", project, options)
@@ -115,6 +133,7 @@ func resourceGitlabProjectApprovalRuleRead(ctx context.Context, d *schema.Resour
 
 	d.Set("name", rule.Name)
 	d.Set("approvals_required", rule.ApprovalsRequired)
+	d.Set("rule_type", rule.RuleType)
 
 	if err := d.Set("group_ids", flattenApprovalRuleGroupIDs(rule.Groups)); err != nil {
 		return diag.FromErr(err)

internal/provider/resource_gitlab_project_approval_rule_test.go

@@ -12,7 +12,7 @@ import (
 	gitlab "github.com/xanzy/go-gitlab"
 )
 
-func TestAccGitLabProjectApprovalRule_basic(t *testing.T) {
+func TestAccGitLabProjectApprovalRule_Basic(t *testing.T) {
 	// Set up project, groups, users, and branches to use in the test.
 
 	testAccCheck(t)
@@ -43,10 +43,10 @@ func TestAccGitLabProjectApprovalRule_basic(t *testing.T) {
 		Steps: []resource.TestStep{
 			// Create rule
 			{
-				Config: testAccGitlabProjectApprovalRuleConfig(project.ID, 3, projectUsers[0].ID, groups[0].ID, branches[0].ID),
+				Config: testAccGitlabProjectApprovalRuleConfig_Basic(project.ID, 3, projectUsers[0].ID, groups[0].ID, branches[0].ID),
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckGitlabProjectApprovalRuleExists("gitlab_project_approval_rule.foo", &projectApprovalRule),
-					testAccCheckGitlabProjectApprovalRuleAttributes(&projectApprovalRule, &testAccGitlabProjectApprovalRuleExpectedAttributes{
+					testAccCheckGitlabProjectApprovalRuleAttributes_Basic(&projectApprovalRule, &testAccGitlabProjectApprovalRuleExpectedAttributes_Basic{
 						Name:                "foo",
 						ApprovalsRequired:   3,
 						EligibleApproverIDs: []int{currentUser.ID, projectUsers[0].ID, group0Users[0].ID},
@@ -57,10 +57,10 @@ func TestAccGitLabProjectApprovalRule_basic(t *testing.T) {
 			},
 			// Update rule
 			{
-				Config: testAccGitlabProjectApprovalRuleConfig(project.ID, 2, projectUsers[1].ID, groups[1].ID, branches[1].ID),
+				Config: testAccGitlabProjectApprovalRuleConfig_Basic(project.ID, 2, projectUsers[1].ID, groups[1].ID, branches[1].ID),
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckGitlabProjectApprovalRuleExists("gitlab_project_approval_rule.foo", &projectApprovalRule),
-					testAccCheckGitlabProjectApprovalRuleAttributes(&projectApprovalRule, &testAccGitlabProjectApprovalRuleExpectedAttributes{
+					testAccCheckGitlabProjectApprovalRuleAttributes_Basic(&projectApprovalRule, &testAccGitlabProjectApprovalRuleExpectedAttributes_Basic{
 						Name:                "foo",
 						ApprovalsRequired:   2,
 						EligibleApproverIDs: []int{currentUser.ID, projectUsers[1].ID, group1Users[0].ID},
@@ -79,15 +79,83 @@ func TestAccGitLabProjectApprovalRule_basic(t *testing.T) {
 	})
 }
 
-type testAccGitlabProjectApprovalRuleExpectedAttributes struct {
+func TestAccGitLabProjectApprovalRule_AnyApprover(t *testing.T) {
+	// Set up project, groups, users, and branches to use in the test.
+
+	testAccCheck(t)
+	testAccCheckEE(t)
+
+	project := testAccCreateProject(t)
+
+	// Terraform test starts here.
+
+	var projectApprovalRule gitlab.ProjectApprovalRule
+
+	resource.Test(t, resource.TestCase{
+		ProviderFactories: providerFactories,
+		CheckDestroy:      testAccCheckGitlabProjectApprovalRuleDestroy(project.ID),
+		Steps: []resource.TestStep{
+			// Create rule
+			{
+				Config: testAccGitlabProjectApprovalRuleConfig_AnyApprover(project.ID, 3, "any_approver"),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckGitlabProjectApprovalRuleExists("gitlab_project_approval_rule.bar", &projectApprovalRule),
+					testAccCheckGitlabProjectApprovalRuleAttributes_AnyApprover(&projectApprovalRule, &testAccGitlabProjectApprovalRuleExpectedAttributes_AnyApprover{
+						Name:              "bar",
+						ApprovalsRequired: 3,
+						RuleType:          "any_approver",
+					}),
+				),
+			},
+			// Update rule
+			{
+				Config: testAccGitlabProjectApprovalRuleConfig_AnyApprover(project.ID, 2, "any_approver"),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckGitlabProjectApprovalRuleExists("gitlab_project_approval_rule.bar", &projectApprovalRule),
+					testAccCheckGitlabProjectApprovalRuleAttributes_AnyApprover(&projectApprovalRule, &testAccGitlabProjectApprovalRuleExpectedAttributes_AnyApprover{
+						Name:              "bar",
+						ApprovalsRequired: 2,
+						RuleType:          "any_approver",
+					}),
+				),
+			},
+			// Re-create rule
+			{
+				Config: testAccGitlabProjectApprovalRuleConfig_AnyApprover(project.ID, 2, "regular"),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckGitlabProjectApprovalRuleExists("gitlab_project_approval_rule.bar", &projectApprovalRule),
+					testAccCheckGitlabProjectApprovalRuleAttributes_AnyApprover(&projectApprovalRule, &testAccGitlabProjectApprovalRuleExpectedAttributes_AnyApprover{
+						Name:              "bar",
+						ApprovalsRequired: 2,
+						RuleType:          "regular",
+					}),
+				),
+			},
+			// Verify import
+			{
+				ResourceName:      "gitlab_project_approval_rule.bar",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
+		},
+	})
+}
+
+type testAccGitlabProjectApprovalRuleExpectedAttributes_Basic struct {
 	Name                string
 	ApprovalsRequired   int
 	EligibleApproverIDs []int
 	GroupIDs            []int
 	ProtectedBranchIDs  []int
 }
 
-func testAccCheckGitlabProjectApprovalRuleAttributes(got *gitlab.ProjectApprovalRule, want *testAccGitlabProjectApprovalRuleExpectedAttributes) resource.TestCheckFunc {
+type testAccGitlabProjectApprovalRuleExpectedAttributes_AnyApprover struct {
+	Name              string
+	ApprovalsRequired int
+	RuleType          string
+}
+
+func testAccCheckGitlabProjectApprovalRuleAttributes_Basic(got *gitlab.ProjectApprovalRule, want *testAccGitlabProjectApprovalRuleExpectedAttributes_Basic) resource.TestCheckFunc {
 	return func(s *terraform.State) error {
 		return InterceptGomegaFailure(func() {
 			Expect(got.Name).To(Equal(want.Name), "name")
@@ -114,7 +182,17 @@ func testAccCheckGitlabProjectApprovalRuleAttributes(got *gitlab.ProjectApproval
 	}
 }
 
-func testAccGitlabProjectApprovalRuleConfig(project, approvals, userID, groupID, protectedBranchID int) string {
+func testAccCheckGitlabProjectApprovalRuleAttributes_AnyApprover(got *gitlab.ProjectApprovalRule, want *testAccGitlabProjectApprovalRuleExpectedAttributes_AnyApprover) resource.TestCheckFunc {
+	return func(s *terraform.State) error {
+		return InterceptGomegaFailure(func() {
+			Expect(got.Name).To(Equal(want.Name), "name")
+			Expect(got.ApprovalsRequired).To(Equal(want.ApprovalsRequired), "approvals_required")
+			Expect(got.RuleType).To(Equal(want.RuleType), "rule_type")
+		})
+	}
+}
+
+func testAccGitlabProjectApprovalRuleConfig_Basic(project, approvals, userID, groupID, protectedBranchID int) string {
 	return fmt.Sprintf(`
 resource "gitlab_project_approval_rule" "foo" {
   project              = %d
@@ -126,6 +204,16 @@ resource "gitlab_project_approval_rule" "foo" {
 }`, project, approvals, userID, groupID, protectedBranchID)
 }
 
+func testAccGitlabProjectApprovalRuleConfig_AnyApprover(project, approvals int, rule_type string) string {
+	return fmt.Sprintf(`
+resource "gitlab_project_approval_rule" "bar" {
+  project              = %d
+  name                 = "bar"
+  approvals_required   = %d
+  rule_type            = "%s"
+}`, project, approvals, rule_type)
+}
+
 func testAccCheckGitlabProjectApprovalRuleExists(n string, projectApprovalRule *gitlab.ProjectApprovalRule) resource.TestCheckFunc {
 	return func(s *terraform.State) error {
 		rs, ok := s.RootModule().Resources[n]