Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: Add cross browsers smoke tests and add support for sendMessage rejected promise behavior #115

Merged
merged 3 commits into from Jun 2, 2018
Merged

test: Add cross browsers smoke tests and add support for sendMessage rejected promise behavior #115

merged 3 commits into from Jun 2, 2018

Commits on Jun 2, 2018

  1. fix: print a deprecation warning when sendResponse callback is used f… 

    …or the first time
    @rpl
    rpl committed Jun 2, 2018
    Configuration menu
    Copy the full SHA
    451af2d View commit details
    Browse the repository at this point in the history

  2. feat: Reject sendMessage returned promise when a onMessage listener r… 

    …eturns a rejected promise.
    @rpl
    rpl committed Jun 2, 2018
    Configuration menu
    Copy the full SHA
    4c4b635 View commit details
    Browse the repository at this point in the history

  3. test(browsers-smoketests): Run a set of smoke tests on both Chrome an… 

    …d Firefox

This commit introduces tape as the test framework used to define the tests in the
test extension contexts and send them to the nodejs script that orchestrate the
test run.

The nodejs script has also been migrated from mocha to tape, it uses the custom test
helpers provided to setup the test environment (e.g. create a temporary dir
for the test extension, copy the last polyfill build, bundle tape to be used
in the test extension, start the browser which run the test extension
and finally collect the results of the test extension) and then it merges all the
tap logs collected from every test extension into a single "per browser" test suite.

- updated travis nodejs environment to nodejs 8
- uses tape to collect test results from inside the test extension
- added test case to check polyfill 'existing browser API object' detection
- added test for expected rejection on tabs.sendMessage with an invalid tabId
- added test with multiple listeners which resolves to undefined and null
- optionally run chrome smoketests with --enable-features=NativeCrxBindings
    @rpl
    rpl committed Jun 2, 2018
    Configuration menu
    Copy the full SHA
    b16ee4f View commit details
    Browse the repository at this point in the history