fix test

ydb-platform · Dec 12, 2023 · 7e84004 · 7e84004
1 parent 0f7c72b
commit 7e84004
Showing 1 changed file with 15 additions and 15 deletions.
diff --git a/ydb/library/login/login_ut.cpp b/ydb/library/login/login_ut.cpp
@@ -151,21 +151,6 @@ Y_UNIT_TEST_SUITE(Login) {
             UNIT_ASSERT(Count(groups, "group4") == 1);
             UNIT_ASSERT(Count(groups, "group5") == 1);
         }
-        {
-            auto response1 = provider.AddGroupMembership({.Group = "group2", .Members = {"group4"}});
-            UNIT_ASSERT(!response1.Error);
-            UNIT_ASSERT(response1.Notice == "Role \"group4\" is already a member of role \"group2\"");
-        }
-        {
-            auto response1 = provider.RemoveGroupMembership({.Group = "group2", .Members = {"group4"}});
-            UNIT_ASSERT(!response1.Error);
-            UNIT_ASSERT(!response1.Warning);
-        }
-        {
-            auto response1 = provider.RemoveGroupMembership({.Group = "group2", .Members = {"group4"}});
-            UNIT_ASSERT(!response1.Error);
-            UNIT_ASSERT(response1.Warning == "Role \"group4\" is not a member of role \"group2\"");
-        }
         {
             auto response1 = provider.RenameGroup({.Group = "group3", .NewName = "group33"});
             UNIT_ASSERT(!response1.Error);
@@ -201,6 +186,21 @@ Y_UNIT_TEST_SUITE(Login) {
             UNIT_ASSERT(Count(groups, "group2") == 1);
             UNIT_ASSERT(Count(groups, "group1") == 1);
         }
+        {
+            auto response1 = provider.AddGroupMembership({.Group = "group2", .Member = {"group4"}});
+            UNIT_ASSERT(!response1.Error);
+            UNIT_ASSERT(response1.Notice == "Role \"group4\" is already a member of role \"group2\"");
+        }
+        {
+            auto response1 = provider.RemoveGroupMembership({.Group = "group2", .Member = {"group4"}});
+            UNIT_ASSERT(!response1.Error);
+            UNIT_ASSERT(!response1.Warning);
+        }
+        {
+            auto response1 = provider.RemoveGroupMembership({.Group = "group2", .Member = {"group4"}});
+            UNIT_ASSERT(!response1.Error);
+            UNIT_ASSERT(response1.Warning == "Role \"group4\" is not a member of role \"group2\"");
+        }
         {
             auto groups = provider.GetGroupsMembership("user1");
             UNIT_ASSERT(groups.size() == 4);