[GitHub] [ignite] nizhikov commented on a change in pull request #9131: IGNITE-14786 Authentication ducktest should not use internal api

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org