Commit Graph

7 Commits

Author SHA1 Message Date
Guy Sartorelli
5e71547035
Merge pull request #111 from creative-commoners/pulls/3.1/remove-todo
MNT Remove TODO comments
2024-01-05 00:14:48 +13:00
Sabina Talipova
8797f1f41e
ENH Restrict access to getJobStatus execution (#113) 2023-11-09 10:06:58 +13:00
Sabina Talipova
966cdb0689 MNT Remove TODO comments 2023-10-12 08:49:02 +13:00
Steve Boyd
1b66be22cd API Stop using deprecated API 2022-11-28 17:40:24 +13:00
Robbie Averill
17604a5b42 FIX Replace Convert JSON methods with json_* methods, deprecated from SilverStripe 4.4 2018-10-28 21:36:50 +00:00
Robbie Averill
44fbc027f3 NEW Add loading animation to Create Report button, fix bug in CurlLinkChecker 2017-11-29 12:14:39 +13:00
Dylan Wagstaff
98dac9b314 Update unit tests to be rid of Phockito
This odd dependency is causing inclusion issues to do with namespacing and
the ability to work well with the framework (according to the dependency's
readme). The usage of the tool in this case adds no value, as it's
performing purely stub type work, rather than mock work. So instead we can
use an actual stub (simply return test values depending on input) and have
everything work well.

After removing the Phockito dependency it was found that the tests that
would skip when it isn't present were actually in a failing state. So they
had to be repaired too, mostly through the use of fixing the poor code
that was causing them to fail (as opposed to being bad tests). This is
both configuration and proceedural work.
2017-11-27 15:14:16 +13:00