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: expand test coverage of events.js and fs.js #10947

Closed
wants to merge 2 commits into from
Closed

test: expand test coverage of events.js and fs.js #10947

wants to merge 2 commits into from

Commits on Jan 23, 2017

  1. test: expand test coverage of events.js 

    * test else path in emitMany function
* test calling removeAllListeners() in a event emitter instance
  with no events at all
* test calling removeListener() passing a event type that does
  not exist
* test calling eventNames() in a event emitter instance
  with no events at all

Refs: https://coverage.nodejs.org/coverage-ba776b3a56642d4c/root/events.js.html
    @vinimdocarmo
    vinimdocarmo committed Jan 23, 2017
    Configuration menu
    Copy the full SHA
    49b0d9f View commit details
    Browse the repository at this point in the history

  2. test: expand test coverage of fs.js 

    * test reading a file passing a not valid enconding

Refs: https://coverage.nodejs.org/coverage-067be658f966dafe/root/internal/fs.js.html
    @vinimdocarmo
    vinimdocarmo committed Jan 23, 2017
    Configuration menu
    Copy the full SHA
    f328546 View commit details
    Browse the repository at this point in the history