Fix JsonHelper implementation and tests - Githubissues

keboola / php-component

General library for PHP applications running in Keboola Connection environment

MIT License

0 stars 1 forks source link

Fix JsonHelper implementation and tests #54

Closed tomasfejfar closed 5 years ago

tomasfejfar commented 5 years ago

without @ it won't proceed to exception, but will throw normal internal warning that is converted to error.
Exception was missing sprintf
test tests correct exception message
writing to root worked for me just fine - so the test failed (that's different in travis as it does not run as root)

Fixes: https://github.com/keboola/php-component/pull/52#issuecomment-449860813