#3601 - Many automated tests failing

Identifier #3601
Issue type Feature request or suggestion
Title Many automated tests failing
Status Completed
Handling member Salman
Addon General / Uncategorised
Description The main issues are below. Finish the core work, then when done make sure all the tests pass. Obviously the tests themselves can be adjusted if needed.

http://localhost/composr/_tests/index.php?id=unit_tests/lang_no_unused&close_if_passed=1
Fail: lang_no_unused_test_set -> testNothingUnused -> MARK_TASK_DONE: cannot find usage of language string (Click to mark task done) at [/Library/WebServer/Documents/composr/_tests/tests/unit_tests/lang_no_unused.php line 227]

http://localhost/composr/_tests/index.php?id=unit_tests/microformats&close_if_passed=1
Lots like:
Exception: microformats_test_set -> testHCalendar -> Unexpected PHP error [file_get_contents(/Library/WebServer/Documents/composr/themes/_unnamed_/images/icons/sprite.svg) [<a href='http://php.net/manual/en/function.file-get-contents.php'>function.file-get-contents.php</a>]: failed to open stream: No such file or directory] severity [E_WARNING] in [/Library/WebServer/Documents/composr/sources/symbols.php line 6511]
This is probably an issue assuming that a theme image exists in any theme. Other tests are failing similarly.

http://localhost/composr/_tests/index.php?id=unit_tests/optimisations_fragile&close_if_passed=1
Lots like:
Fail: optimisations_fragile_test_set -> testSymbols2Optimisation -> Loaded symbols2.php in module galleries at [/Library/WebServer/Documents/composr/_tests/tests/unit_tests/optimisations_fragile.php line 53]
You need to make sure any commonly used symbols are in symbols.php and non-commonly used ones are in symbols2.php.

http://localhost/composr/_tests/index.php?id=unit_tests/basic_code_formatting&close_if_passed=1
Many errors, trailing white-space and incorrect number of blank lines at the end of files. I've fixed many issues like this before, but you need to maintain this yourself now.

http://localhost/composr/_tests/index.php?id=unit_tests/comment_encapsulation&close_if_passed=1
Many errors. I think this mostly stems from using a directive for a parameter of a symbol. That's not allowed in Tempcode. You should use the SET directive instead. Then the test will pass as it's not misunderstanding the use of the Tempcode syntax.

http://localhost/composr/_tests/index.php?id=unit_tests/web_resources&close_if_passed=1
The CSS/JS validation code needs updating if we are using things that are beyond the standards it recognises. E.g. SVG properties in CSS.
Or, it may be finding legitimate issues in the code.
Steps to reproduce

Funded? No
The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".

Rating

Unrated