fix comment and remove unnecessary else block

client.go

@@ -215,7 +215,7 @@ func (c *client) unassignRoles(ctx context.Context, roleIDs []string) error {
 
 	for _, id := range roleIDs {
 		if _, err := c.provider.DeleteRoleAssignmentByID(ctx, id); err != nil {
-			// If a role was deleted manually then Azure returns an error and status 204
+			// If a role was deleted out-of-band then Azure returns an error and status 204
 			respErr := new(azcore.ResponseError)
 			if errors.As(err, &respErr) && (respErr.StatusCode == http.StatusNoContent || respErr.StatusCode == http.StatusNotFound) {
 				continue

provider_mock_test.go

@@ -238,13 +238,13 @@ func (m *mockProvider) DeleteRoleAssignmentByID(_ context.Context, _ string) (ar
 			return armauthorization.RoleAssignmentsClientDeleteByIDResponse{}, &azcore.ResponseError{
 				ErrorCode: "mock: fail to delete role assignment",
 			}
-		} else {
-			// return empty response and with status code; will ignore error and assume role
-			// assignment was manually deleted based on status code
-			return armauthorization.RoleAssignmentsClientDeleteByIDResponse{}, &azcore.ResponseError{
-				StatusCode: m.unassignRolesFailureParams.statusCode,
-				ErrorCode:  "mock: fail to delete role assignment",
-			}
+		}
+
+		// return empty response and with status code; will ignore error and assume role
+		// assignment was manually deleted based on status code
+		return armauthorization.RoleAssignmentsClientDeleteByIDResponse{}, &azcore.ResponseError{
+			StatusCode: m.unassignRolesFailureParams.statusCode,
+			ErrorCode:  "mock: fail to delete role assignment",
 		}
 	}
 	return armauthorization.RoleAssignmentsClientDeleteByIDResponse{}, nil