From 45a010390bcb5fdf0190ca83a91ce9c57a5a458b Mon Sep 17 00:00:00 2001 From: AleksandrMatsko Date: Wed, 13 Nov 2024 16:34:46 +0700 Subject: [PATCH] fix: add check for equality of fetched teams and renamed teams --- cmd/cli/teams_names.go | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/cmd/cli/teams_names.go b/cmd/cli/teams_names.go index 6592d4539..a0cb27a4e 100644 --- a/cmd/cli/teams_names.go +++ b/cmd/cli/teams_names.go @@ -3,6 +3,7 @@ package main import ( "context" "encoding/json" + "errors" "fmt" "strconv" "strings" @@ -17,6 +18,9 @@ const ( teamsByNamesKey = "moira-teams-by-names" ) +var errTeamsCountAndUniqueNamesCountMismatch = errors.New( + "count of teams does not match count of unique names after transformation") + // fillTeamNamesHash does the following // 1. Get all teams from DB. // 2. Group teams with same names. @@ -46,6 +50,10 @@ func fillTeamNamesHash(logger moira.Logger, database moira.Database) error { teamByUniqueName := transformTeamsByNameMap(teamsByNameMap) + if len(teamByUniqueName) != len(teamsMap) { + return errTeamsCountAndUniqueNamesCountMismatch + } + client := db.Client() ctx := db.Context()