Skip to content

Conversation

zypA13510
Copy link
Member

@coveralls
Copy link

coveralls commented Jul 26, 2018

Pull Request Test Coverage Report for Build 10

  • 6 of 8 (75.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.6%) to 95.974%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/middleware/authenticate.js 6 8 75.0%
Totals Coverage Status
Change from base Build 9: -0.6%
Covered Lines: 353
Relevant Lines: 359

💛 - Coveralls

@rwky
Copy link

rwky commented Jul 26, 2018

Note to @passport-next/developers add tests.

@rwky
Copy link

rwky commented Aug 13, 2018

@guyellis just wondering if you have time to look at this? If not I'll sort it.

@guyellis
Copy link

@rwky probably not this week. I'll see how my time availability pans out later.

@rwky rwky merged commit 1db59e7 into passport-next:master Sep 11, 2018
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