samliew / chatexchange / 559

94%

push

travis-ci-com

Connecting leaveAllRooms to public API; Adding ability to ignore events by type (#91)

* 0.4.3

* exported ChatEventType as named export from main module

* made Client a named export too

* allowed Client to leave all rooms at once

* added unit test for leaving all rooms; improved fkey getter test

* added test case for failing to leave all rooms

* improved Browser tests

* better error reporting for fkey load failures

* moved utility tests to TypeScript

* fixed typo returning object instead of undefined

* reworked arrayToKvp to remove TS error suppresion;
fixed bug uncovered by removing suppression

* removed mutation from reputation parsing

* added test case for empty profile

* covered Browser.login LoginError branch

* improved fkey error reporting during login

* added test case for an InternalError during login

* covered case of successful Browser login

* switched Room id to shortcut notation

* configured ts-jest to use test-specific tsconfig

* improved Browser error logging

* extended Room class with methods for ignoring events

* added tests for ignore events methods

* forgot to reenable the full test suite

* finished grouping sendMessage tests

* removed unnecessary guards

* Added ScrapingError to avoid dumping HTML strings

* added unit test for the ScrapingError class

* switched from HTML string dumping to ScrapingError

* updated tests to use ScrapingError instead of InternalError

* updated documentation to reflect new modules and methods

* Revert back to not forcing users to use bash to run tests.

* Attempt to fix coveralls

* Revert "Attempt to fix coveralls"

This reverts commit cc2a4da41.

Co-authored-by: Daniel Beaupre <danbopes@gmail.com>

168 of 183 branches covered (91.8%)

Branch coverage included in aggregate %.

132 of 132 new or added lines in 6 files covered. (100.0%)

1429 of 1508 relevant lines covered (94.76%)

4.84 hits per line