Skip to content

bug fix #113

Merged
Merged

Conversation

sergeyuttsel
Copy link
Contributor

@sergeyuttsel sergeyuttsel commented Nov 12, 2018

Думаю нужно сначала вызвать метод loadValueFromFileOrPropertyOrVariableOrDefault, а потом уже полученный результат передать в метод resolveJsonVars. И актуально это только для BODY.

Нужно учесть, что если телом будет  json формат, то в нём используются фигурные скобки. Например в json'е {"name": "Alex", "object1": {"number" : 0.003}} регулярна будет считать значение "number" : 0.003 переменной. Так как такой переменной нигде не будет, то будет выброшено исключение. Самый простой способ это отказаться от исключения и просто писать это в лог.

@coveralls
Copy link

coveralls commented Nov 12, 2018

Coverage Status

Coverage decreased (-0.07%) to 77.312% when pulling d6dbeac on sergeyuttsel:create-request-bug into 51778ca on alfa-laboratory:master.

@GannaChernyshova GannaChernyshova merged commit 471d7d1 into alfa-laboratory:master Nov 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants