search

Codeception / module-rest

REST module for Codeception
MIT License
54 stars 27 forks source link

Calling JSONPath::data() is deprecated, please use JSONPath::getData() instead. #38

Closed ghost closed 3 years ago

ghost commented 4 years ago

After updating to 1.2.6, using grabDataFromResponseByJsonPath(..) results in

Calling JSONPath::data() is deprecated, please use JSONPath::getData() instead.

Which naturally fails the test unless most errors are ignored (which would be a very bad practice)

HaruAtari commented 4 years ago

+1 It throws in https://github.com/Codeception/module-rest/blob/master/src/Codeception/Util/JsonArray.php#L90

ghost commented 4 years ago

+1 It throws in https://github.com/Codeception/module-rest/blob/master/src/Codeception/Util/JsonArray.php#L90

Yea, got too late yesterday, but this should be a trivial fix. Don't know yet if there are other issues, but will try our full test suite with the fix in #39 .

ghost commented 4 years ago

Ah. #37 was already done.. too slow this morning, might need more coffee

Naktibalda commented 3 years ago

Fixed by #37