Changes between Version 2 and Version 3 of Ticket #6884


Ignore:
Timestamp:
08/25/2018 11:22:32 PM (15 months ago)
Author:
sebastian
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #6884 – Description

    v2 v3  
    44There are following issues with `nodeunit`: 
    55 
    6 * `nodeunit`is deprecated and no longer maintained. 
     6* `nodeunit` is deprecated and no longer maintained. 
    77* If a test failed, the process appears to be terminated before completing pending asynchronous tasks, which calls for obscure workarounds (if available) when dealing with resources that need to be cleaned up (e.g. browser processes managed through WebDriver). 
    88* There is no concept of shared resources, `setUp` and `tearDown` are always called for each test, making it difficult to share resources (that require cleanup) across tests. 
     
    1010The latter two make it in particular impossible to extend the test suite (e.g. by running against our [https://testpages.adblockplus.org test pages]), without resulting into an inferior test architecture and slower performance. 
    1111 
    12 Mocha doesn't seem to have any of this issue and appears to have become the defacto standard testing framework for Node.js in modern days. 
     12Mocha doesn't seem to have any of these issues and appears to have become the defacto standard testing framework for Node.js in modern days. 
    1313 
    1414=== What to change === 
    15 Replace `nodeunit`with the Mocha test framework in `adblockpluchrome`. While on it, leverage the `before` and `after` hooks (which wasn't possible before and will help to add more tests). 
     15Replace `nodeunit` with the Mocha test framework in `adblockpluchrome`. While on it, leverage the `before` and `after` hooks (which wasn't possible before and will help to add more tests).