Skip to content

Unit test for the issue that crashed when some fields were not retrieved by default#27

Merged
wheresvic merged 2 commits intowheresvic:masterfrom
eyp:master
Oct 25, 2018
Merged

Unit test for the issue that crashed when some fields were not retrieved by default#27
wheresvic merged 2 commits intowheresvic:masterfrom
eyp:master

Conversation

@eyp
Copy link
Copy Markdown
Contributor

@eyp eyp commented Oct 24, 2018

No description provided.

eyp added 2 commits October 23, 2018 09:23
Sometimes the field could be marked to not be retrieved, in that case, the decrypt function will fail because because the field is undefined.
@coveralls
Copy link
Copy Markdown

coveralls commented Oct 24, 2018

Pull Request Test Coverage Report for Build 97

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.5%) to 96.703%

Files with Coverage Reduction New Missed Lines %
lib/mongoose-field-encryption.js 2 96.7%
Totals Coverage Status
Change from base Build 96: 0.5%
Covered Lines: 130
Relevant Lines: 134

💛 - Coveralls

@wheresvic wheresvic merged commit efcf2e0 into wheresvic:master Oct 25, 2018
@wheresvic
Copy link
Copy Markdown
Owner

Awesome thanks!

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.

3 participants