A tad bit of code cleanup

Author: Arnas Navašinskas

castai/resource_security_runtime_rule.go

@@ -338,21 +338,11 @@ func resourceSecurityRuntimeRuleDelete(ctx context.Context, d *schema.ResourceDa
 	isBuiltIn := ok && isBuiltInRaw.(bool)
 
 	if isBuiltIn {
-		// Built-in rule: disable instead of deleting
-		req := sdk.RuntimeV1ToggleRulesRequest{
-			Enabled: false,
-			Ids:     []string{ruleID},
-		}
-		resp, err := client.RuntimeSecurityAPIToggleRulesWithResponse(ctx, req)
-		if err := sdk.CheckOKResponse(resp, err); err != nil {
-			return diag.Errorf("disabling built-in runtime rule (instead of deleting, we can't delete built in rules): %v", err)
-		}
-
-		d.SetId("")
-		return nil
+		// Built-in rule: disable instead of deleting, we can't delete built in rules
+		return disableRule(ctx, d, ruleID, client)
 	}
 
-	// Normal rule: delete it
+	// not Build-in rule: delete it
 	delReq := sdk.RuntimeSecurityAPIDeleteRulesJSONRequestBody{
 		Ids: []string{ruleID},
 	}
@@ -365,6 +355,20 @@ func resourceSecurityRuntimeRuleDelete(ctx context.Context, d *schema.ResourceDa
 	return nil
 }
 
+func disableRule(ctx context.Context, d *schema.ResourceData, ruleID string, client sdk.ClientWithResponsesInterface) diag.Diagnostics {
+	req := sdk.RuntimeV1ToggleRulesRequest{
+		Enabled: false,
+		Ids:     []string{ruleID},
+	}
+	resp, err := client.RuntimeSecurityAPIToggleRulesWithResponse(ctx, req)
+	if err := sdk.CheckOKResponse(resp, err); err != nil {
+		return diag.Errorf("disabling built-in runtime rule (instead of deleting, we can't delete built in rules): %v", err)
+	}
+
+	d.SetId("")
+	return nil
+}
+
 // findRuntimeRuleByName pages through API results to find a rule by name.
 func findRuntimeRuleByName(ctx context.Context, client sdk.ClientWithResponsesInterface, name string) (*sdk.RuntimeV1Rule, error) {
 	var cursor *string