Add prevent_forking_outside_group to group data source

Author: jtymes

docs/data-sources/group.md

@@ -45,6 +45,7 @@ data "gitlab_group" "foo" {
 - **name** (String) The name of this group.
 - **parent_id** (Number) Integer, ID of the parent group.
 - **path** (String) The path of the group.
+- **prevent_forking_outside_group** (Boolean) When enabled, users can not fork projects from this group to external namespaces
 - **request_access_enabled** (Boolean) Boolean, is request for access enabled to the group.
 - **runners_token** (String, Sensitive) The group level registration token to use during runner setup.
 - **visibility_level** (String) Visibility level of the group. Possible values are `private`, `internal`, `public`.

docs/resources/group.md

@@ -47,6 +47,7 @@ resource "gitlab_project" "example" {
 - **lfs_enabled** (Boolean) Boolean, defaults to true.  Whether to enable LFS
 - **mentions_disabled** (Boolean) Boolean, defaults to false.  Disable the capability
 - **parent_id** (Number) Integer, id of the parent group (creates a nested group).
+- **prevent_forking_outside_group** (Boolean) When enabled, users can not fork projects from this group to external namespaces
 - **project_creation_level** (String) , defaults to Maintainer.
 - **request_access_enabled** (Boolean) Boolean, defaults to false.  Whether to
 - **require_two_factor_authentication** (Boolean) Boolean, defaults to false.

internal/provider/data_source_gitlab_group.go

@@ -91,6 +91,11 @@ var _ = registerDataSource("gitlab_group", func() *schema.Resource {
 				Type:        schema.TypeInt,
 				Computed:    true,
 			},
+			"prevent_forking_outside_group": {
+				Description: "When enabled, users can not fork projects from this group to external namespaces",
+				Type:        schema.TypeBool,
+				Computed:    true,
+			},
 		},
 	}
 })
@@ -135,6 +140,7 @@ func dataSourceGitlabGroupRead(ctx context.Context, d *schema.ResourceData, meta
 	d.Set("parent_id", group.ParentID)
 	d.Set("runners_token", group.RunnersToken)
 	d.Set("default_branch_protection", group.DefaultBranchProtection)
+	d.Set("prevent_forking_outside_group", group.PreventForkingOutsideGroup)
 
 	d.SetId(fmt.Sprintf("%d", group.ID))
 

internal/provider/data_source_gitlab_group_test.go

@@ -56,6 +56,7 @@ func testAccDataSourceGitlabGroup(src, n string) resource.TestCheckFunc {
 			"visibility_level",
 			"parent_id",
 			"default_branch_protection",
+			"prevent_forking_outside_group",
 		}
 
 		for _, attribute := range testAttributes {

internal/provider/resource_gitlab_group_test.go

@@ -246,6 +246,33 @@ func TestAccGitlabGroup_disappears(t *testing.T) {
 	})
 }
 
+func TestAccGitlabGroup_PreventForkingOutsideGroup(t *testing.T) {
+	var group gitlab.Group
+	rInt := acctest.RandInt()
+
+	resource.Test(t, resource.TestCase{
+		PreCheck:          func() { testAccPreCheck(t) },
+		ProviderFactories: providerFactories,
+		CheckDestroy:      testAccCheckGitlabGroupDestroy,
+		Steps: []resource.TestStep{
+			{
+				SkipFunc: isRunningInCE,
+				Config:   testAccGitlabGroupPreventForkingOutsideGroupConfig(rInt),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckGitlabGroupExists("gitlab_group.foo", &group),
+					func(s *terraform.State) error {
+						if group.PreventForkingOutsideGroup != true {
+							return fmt.Errorf("expected forking outside the group to be disabled")
+						}
+
+						return nil
+					},
+				),
+			},
+		},
+	})
+}
+
 func testAccCheckGitlabGroupDisappears(group *gitlab.Group) resource.TestCheckFunc {
 	return func(s *terraform.State) error {
 		_, err := testGitlabClient.Groups.DeleteGroup(group.ID)
@@ -408,33 +435,6 @@ func testAccCheckGitlabGroupDestroy(s *terraform.State) error {
 	return nil
 }
 
-func TestAccGitlabGroup_PreventForkingOutsideGroup(t *testing.T) {
-	var group gitlab.Group
-	rInt := acctest.RandInt()
-
-	resource.Test(t, resource.TestCase{
-		PreCheck:          func() { testAccPreCheck(t) },
-		ProviderFactories: providerFactories,
-		CheckDestroy:      testAccCheckGitlabGroupDestroy,
-		Steps: []resource.TestStep{
-			{
-				SkipFunc: isRunningInCE,
-				Config:   testAccGitlabGroupPreventForkingOutsideGroupConfig(rInt),
-				Check: resource.ComposeTestCheckFunc(
-					testAccCheckGitlabGroupExists("gitlab_group.foo", &group),
-					func(s *terraform.State) error {
-						if group.PreventForkingOutsideGroup != true {
-							return fmt.Errorf("expected forking outside the group to be disabled")
-						}
-
-						return nil
-					},
-				),
-			},
-		},
-	})
-}
-
 func testAccGitlabGroupConfig(rInt int) string {
 	return fmt.Sprintf(`
 resource "gitlab_group" "foo" {