* Added test for AppImPanelControllerSpec, cleaned up style of all tests
All tests give 0 errors when running gulp standard
All credits for those test still go to @Ryuno-Ki, I did not functionaly change the testing code
* add Test for DocumentModalController
* added tests for the EmbedModalController and VideoModalController
when now holding the Alt key while selecting a user from the mentions
autocomplete suggestions the text inserted into the composer will be
forced to be in the
@1234567 (firstname)
format (i.e. referring to the user by id and local firstname) instead of
@username
even if the selected user has a username set.
This allows to manually edit the display text ("firstname") of the
mention.
Fixed the errors that would definitly not change the functionality of the program
These include
* Unexpected && and || combinations
* variables with different values defined in same line of code
* Global variable never defined
* spacing/placing errors
These errors were not touched
* variables who are never used
* == and != errors
* Added test for ChangeLogModalController
This commit changes the test scores as follows
| | % Statements | % Branch | % Functions | % Lines |
|:---:|:------------:|:--------:|-------------|---------|
| Old | 3.5 | 0 | 0.36 | 3.52 |
| New | 4.22 | 0.36 | 1.08 | 4.24 |
* Added tests for AppFooterController
This commit changes the test scores as follows
| | % Statements | % Branch | % Functions | % Lines |
|:---:|:------------:|:--------:|-------------|---------|
| Old | 4.22 | 0.36 | 1.08 | 4.24 |
| New | 4.29 | 0.36 | 1.44 | 4.31 |
* Added tests for PeerSelectController
This commit changes the test scores as follows
| | % Statements | % Branch | % Functions | % Lines |
|:---:|:------------:|:--------:|-------------|---------|
| Old | 4.29 | 0.36 | 1.44 | 4.31 |
| New | 5.61 | 1.16 | 2.88 | 5.64 |
* Change Time-out time to variable
Instead of doing time-outs based on a constant, the time-out is based on a variable. This will make changing the time of time-out easier.
* Rename AppFooterController test, removed unnecessary test, fixed code style
Renamed test/unit/AppFooterController.js to test/unit/AppFooterControllerSpec.js to conform with other test files.
Removed a test in the renamed file. It only tested if the controller would compile given the test-environment, something that was implicitly tested in the other (now only) test.
The code style over all test files was not consistent (spacing), this is fixed.
* Changed style to StandardJS
When running "standard test/unit/..", there were some errors on the code-style of the tests
These are fixed
* Add karma, Jasmine, deps and karma.conf.js. Fixes#1291
* Improved service worker code
* npm start script executes gulp watch task.
* Written a gulp task to compile templates AOT. Get rid of loading deps in unit tests and a preprocessor.
* PhantomJS package is deprecated. Use phantomjs-prebuilt instead.
* Adjust `npm run test`.
John Yong