* added tests for filter.js, Added structure to test-directory
In the gulpfile I changed some of the karma execution to ensure templates.js exists before the tests
Also removed some StandardJS errors from filters.js
* added tests to create at least 50% coverage for filters.js
* Added test for full coverage of filters.js
* fixed 2 side-cases
* Removed duplicate test, improved test-titles and added outcome clarification for dateOrTimeFilter
* Added test-init
While running ``gulp test`` there were a lot of warnings concerning i18n: key not found
This was because init.js was not run.
Because some parts of init.js were not relevant, I created test-init.js, only containing the part concerning localization.
Now the log is a lot cleaner when running ``gulp test``
* added 'use strict' to testing files
* changed local vars to this.
* improved clarity of mistery numbers
* improved in-test structure
* Improved shortUrl tests
* removed unnecessary code
* added cleanup to single test
* 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
Bart