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.

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

Update test coverage #33

Closed
wants to merge 8 commits into from
Closed

Conversation

ilyar
Copy link
Contributor

@ilyar ilyar commented May 30, 2017

Основная цель полное тестовое покрытие.
Не получилось получить отчет через nodeunit, поэтому мигрировал на jest. Плюс думал он поможет полефилы протетировать, сразу не получилось поэтому просто заигнорено, но решение позже нашел. Если это критично можно тоже покрыть, но наверное в этом мало смысла.

Отдельными коммитами фиксы 1, 3, 5 их #26

Отчет о покрытии:

 PASS  test/test.js
  Base
    ✓ Is Function (6ms)
  Instance
    ✓ Instance empty (2ms)
    ✓ Instance properties (3ms)
    ✓ Instance of (2ms)
    ✓ Instance of constructor result
    ✓ Instance of self (3ms)
  Inherit
    ✓ Self
    ✓ Base inherit (1ms)
    ✓ Inherit from plane function (1ms)
    ✓ Inherit and base call from plane function (1ms)
    ✓ Static inherit (1ms)
  Override
    ✓ Override method
    ✓ Override static method (1ms)
    ✓ Override base (1ms)
    ✓ Override static base
  Mixin
    ✓ Object mixin (1ms)
    ✓ Function mixin
    ✓ Function mixin static (1ms)
  Mocking
    ✓ Base mocking
  Use cases
    ✓ Example from docs (2ms)
    ✓ Inherited functions with private variables

Test Suites: 1 passed, 1 total
Tests:       21 passed, 21 total
Snapshots:   0 total
Time:        0.787s, estimated 1s
Ran all test suites.
-------------|----------|----------|----------|----------|----------------|
File         |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
-------------|----------|----------|----------|----------|----------------|
All files    |      100 |      100 |      100 |      100 |                |
 inherit     |      100 |      100 |      100 |      100 |                |
  index.js   |      100 |      100 |      100 |      100 |                |
 inherit/lib |      100 |      100 |      100 |      100 |                |
  inherit.js |      100 |      100 |      100 |      100 |                |
-------------|----------|----------|----------|----------|----------------|

Process finished with exit code 0

ilyar added 3 commits May 29, 2017 17:41
- ignoring tooling variable `emptyBase`
- ignoring polyfills
- ignoring wrappers for module system
- fix url homepage and repository
- add .npmignore
- auto reformat from npm
@ilyar ilyar force-pushed the update_test_coverage branch from 167cb77 to f222339 Compare May 30, 2017 05:01
@ilyar ilyar force-pushed the update_test_coverage branch from 58e7e24 to 2753e12 Compare May 30, 2017 05:43
- add package-lock.json
Copy link
Contributor Author

@ilyar ilyar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: ready

@ilyar
Copy link
Contributor Author

ilyar commented Jun 8, 2017

#37

@ilyar ilyar closed this Jun 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant