Currently, when running "npm test", the following scores appear for controller.js:
| | % Statements | % Branch | % Functions | % Lines |
|:------------:|:------------:|:--------:|:-------------:|:---------:|
| Currently | 3.5 | 0 | 0.36 | 3.52 |
The test are help to keep the controller bug-free. It also gives confirmation to developers contributing to the code that the old functionality was not changed (unless this is the intented purpose). This could be useful when the controller is split up in modules (#1300).
In order to create better scores, the following controllers need additional testing:
I was lucky to get karma and test coverage into the code repository. Don't expect @zhukov to tackle your issue on himself. In his opinion UI related does not need to be automatically tested (IIRC).
However he's likely to accept your PR if you submit one. Currently I'm lacking time to wrap my head around the code base. Otherwise I'd have contributed more patches myself.
Let me see if I can give some help here.
Thanks! If you do, can you "claim" some controllers so that we don't do the work twice?
You can look at previous PR's and existing tests in order to create a uniform testing suite
Most helpful comment
Let me see if I can give some help here.