change TagConfigFile variable setting #30
Conversation
| String sConfigFileFolder = System.getProperty("TagConfigFile", "config"); | ||
| LOG.debug("Loading properties from {}/{}", sConfigFileFolder, sConfigFile); | ||
| InputStream streamFromResource = Props.class.getClassLoader().getResourceAsStream(sConfigFileFolder + "/" + sConfigFile); | ||
| String configFileDefault = "config/application.properties"; |
There was a problem hiding this comment.
Предлагаю, тогда сделать возможность задавать и папку и файл, считаю что так код и логика будут более прозрачными.
String configFile = System.getProperty("TagConfigFile", "application.properties");
String configFileFolder = System.getProperty("TagConfigFileFolder", "config");
There was a problem hiding this comment.
необходимо будет обновить тогда документацию
https://docs.sbtqa.ru/?version=14.6.0#_Конфигурационный_файл
There was a problem hiding this comment.
Да, Вы правы, можете создать pr в page-factory-2.
Этот текст находится в файле https://github.com/sbtqa/page-factory-2/blob/master/page-factory-doc/src/main/asciidoc/common_properties.asciidoc
added TagConfigFileFolder variable for set path to config
| } | ||
| String configFile = System.getProperty("TagConfigFile", "application.properties"); | ||
| String configFileFolder = System.getProperty("TagConfigFileFolder", "config"); | ||
| LOG.warn("Loading properties from {}/{}", configFileFolder, configFile); |
There was a problem hiding this comment.
Почему теперь это сообщение на уровне warn?
2 participants
this is pr created because I have issue #29