```
ContentTranslation (master)$ ack ".getCategories"
tests/qunit/tools/ext.cx.tools.categories.test.js
28: mw.cx.categoryTool.getCategories( 'source' )
73: mw.cx.categoryTool.getCategories( 'adapted' )
113: function testGetTargetCategories( title ) {
118: mw.cx.categoryTool.getCategories( 'target' )
127: testGetTargetCategories( 'Han Fei' )
136: testGetTargetCategories( 'Han' )
145: testGetTargetCategories( 'Shen Dao' )
modules/tools/ext.cx.tools.categories.js
607: CXCategoryTool.prototype.getCategories = function ( categorySet ) {
724: mw.cx.categoryTool.getCategories().done( function ( categories ) {
```
This shows that getCategories is always called without parameters from user interface.
But CXCategoryTool.prototype.getCategories expect category set and there are 4 cases for category set. Only the 'default' case is actually used in the user interface. Rest of them serves only unit test cases.
Please do not test the code that is never executed outside test and remove the unwanted code. Rewrite the tests accordingly.