nizhikov commented on a change in pull request #9131:
URL: https://github.com/apache/ignite/pull/9131#discussion_r640452900



##########
File path: modules/ducktests/tests/ignitetest/tests/auth_test.py
##########
@@ -24,12 +24,13 @@
 from ignitetest.services.utils.ignite_configuration import 
IgniteConfiguration, DataStorageConfiguration
 from ignitetest.services.utils.ignite_configuration.data_storage import 
DataRegionConfiguration
 from ignitetest.utils import ignite_versions, cluster
-from ignitetest.services.utils.ignite_configuration.discovery import 
from_ignite_cluster
+from ignitetest.services.utils.ignite_configuration import 
IgniteThinClientConfiguration
+from ignitetest.services.utils.ssl.client_connector_configuration import 
ClientConnectorConfiguration
 from ignitetest.utils.ignite_test import IgniteTest
 from ignitetest.utils.version import DEV_BRANCH, LATEST, IgniteVersion
 
 WRONG_PASSWORD = "wrong_password"
-TEST_USERNAME = "admin"
+TEST_USERNAME = "ADMIN"

Review comment:
       `TEST_USERNAME` implicitly connected with `UserModifyingApplication` - 
username should be in uppercase to work correctly.
   
   It seems wrong connection. 
   Please, fix the app and add quotes to queries to make queries not related to 
parameter case.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to