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

Documentation update #57

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Nikhilkapoor20
Copy link

Bypass the directive in unit test cases

chrisvfabio and others added 3 commits May 7, 2018 14:39
* refactor: migrate to angular package format v6

* ci(travis): add chrome headless configuration for production builds

* Added test:single script for travis; * Added development testing script with watch mode;

* ci(travis): use the latest stable nodejs engine v8.11.1

fix(npm): append test command
* Add missing tests

* Fix test coverage
* By pass the directive in unit test cases
@codecov-io
Copy link

codecov-io commented May 31, 2018

Codecov Report

Merging #57 into master will decrease coverage by 0.25%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
- Coverage   97.53%   97.28%   -0.26%     
==========================================
  Files          16       16              
  Lines         527      515      -12     
  Branches       96       96              
==========================================
- Hits          514      501      -13     
- Misses          1        2       +1     
  Partials       12       12
Impacted Files Coverage Δ
...permissions/src/lib/service/permissions.service.ts 100% <ø> (ø)
.../ngx-permissions/src/lib/model/permission.model.ts 100% <ø> (ø)
...issions/src/lib/directive/permissions.directive.ts 92.24% <ø> (ø)
projects/ngx-permissions/src/lib/utils/utils.ts 100% <ø> (ø)
...s/ngx-permissions/src/lib/service/roles.service.ts 100% <ø> (ø)
...ssions/src/lib/router/permissions-guard.service.ts 100% <ø> (ø)
...x-permissions/src/lib/store/configuration.store.ts 100% <ø> (ø)
...ssions/src/lib/enums/predefined-strategies.enum.ts 100% <ø> (ø)
...lib/testing/permissions-restrict.directive.stub.ts 100% <ø> (ø)
...rc/lib/testing/permissions-allow.directive.stub.ts 95.45% <ø> (ø)
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d33015...50193bf. Read the comment docs.

@AlexKhymenko
Copy link
Owner

Thank You for the pull request. But im planning to leave only wiki page.

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.

None yet

4 participants