fix build, improve error message

Author: kungasc

ydb/core/http_proxy/ut/datastreams_fixture.h

@@ -547,8 +547,8 @@ class THttpProxyTestMock : public NUnitTest::TBaseFixture {
                                                                           {"cloud_id", "cloud4"},
                                                                           {"database_id", "database4"}}));
 
-        client.CreateGroup("/Root", "Service1_id@as");
-        client.CreateGroup("/Root", "proxy_sa@as");
+        client.CreateUser("/Root", "Service1_id@as", "password1");
+        client.CreateUser("/Root", "proxy_sa@as", "password2");
 
         NACLib::TDiffACL acl;
         acl.AddAccess(NACLib::EAccessType::Allow, NACLib::GenericFull, "Service1_id@as");

ydb/core/tx/schemeshard/schemeshard__operation_modify_acl.cpp

@@ -56,15 +56,15 @@ class TModifyACL: public TSubOperationBase {
             for (const NACLibProto::TDiffACE& diffACE : diffACL.GetDiffACE()) {
                 if (static_cast<NACLib::EDiffType>(diffACE.GetDiffType()) == NACLib::EDiffType::Add) {
                     if (!context.SS->LoginProvider.Sids.contains(diffACE.GetACE().GetSID())) {
-                        result->SetError(NKikimrScheme::StatusPreconditionFailed, "Sid not found");
+                        result->SetError(NKikimrScheme::StatusPreconditionFailed, "SID not found");
                         return result;
                     }
                 } // remove diff type is allowed in any case
             }
         }
         if (owner) {
             if (!context.SS->LoginProvider.Sids.contains(owner)) {
-                result->SetError(NKikimrScheme::StatusPreconditionFailed, "Sid not found");
+                result->SetError(NKikimrScheme::StatusPreconditionFailed, "Owner SID not found");
                 return result;
             }
         }

ydb/core/tx/schemeshard/ut_login/ut_login.cpp

@@ -283,12 +283,12 @@ Y_UNIT_TEST_SUITE(TSchemeShardLoginTest) {
             NACLib::TDiffACL diffACL;
             diffACL.AddAccess(NACLib::EAccessType::Allow, NACLib::GenericUse, "user1");
             AsyncModifyACL(runtime, ++txId, "/MyRoot", "Dir1", diffACL.SerializeAsString(), "");
-            TestModificationResults(runtime, txId, {{NKikimrScheme::StatusPreconditionFailed, "Sid not found"}});
+            TestModificationResults(runtime, txId, {{NKikimrScheme::StatusPreconditionFailed, "SID not found"}});
         }
 
         {
             AsyncModifyACL(runtime, ++txId, "/MyRoot", "Dir1", NACLib::TDiffACL{}.SerializeAsString(), "user1");
-            TestModificationResults(runtime, txId, {{NKikimrScheme::StatusPreconditionFailed, "Sid not found"}});
+            TestModificationResults(runtime, txId, {{NKikimrScheme::StatusPreconditionFailed, "Owner SID not found"}});
         }
 
         CreateAlterLoginCreateUser(runtime, ++txId, "/MyRoot", "user1", "password1");