Skip to content

Conversation

scottinet
Copy link
Contributor

Fixes #95 + added unit tests to catch this kind of bug

@codecov-io
Copy link

Current coverage is 100%

Merging #96 into develop will not change coverage

@@           develop    #96   diff @@
=====================================
  Files           11     11          
  Lines         1504   1506     +2   
  Methods        236    236          
  Messages         0      0          
  Branches       383    384     +1   
=====================================
+ Hits          1504   1506     +2   
  Misses           0      0          
  Partials         0      0          

Powered by Codecov. Last updated by 61f1960...d7047e4

@benoitvidis
Copy link
Contributor

👍

1 similar comment
@dbengsch
Copy link
Contributor

dbengsch commented Jul 6, 2016

+1

@dbengsch dbengsch merged commit 38eb62e into develop Jul 6, 2016
@dbengsch dbengsch deleted the fix-develop-95-empty-mapping branch July 6, 2016 12:24
scottinet added a commit that referenced this pull request Jul 6, 2016
* Reinforce unit tests on the `security.getProfile` method #94
* Remove jshint as it overlapped with eslint #93
* Fix #95: handle empty mappings #96
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.

4 participants