Skip to content

fix: upgrade dependencies, make fb tests more reliable#424

Merged
AVVS merged 9 commits intomasterfrom
pajgo-fix/facebookTests
Sep 10, 2019
Merged

fix: upgrade dependencies, make fb tests more reliable#424
AVVS merged 9 commits intomasterfrom
pajgo-fix/facebookTests

Conversation

@AVVS
Copy link
Copy Markdown
Member

@AVVS AVVS commented Sep 10, 2019

No description provided.

pajgo and others added 9 commits September 2, 2019 16:45
* API update
* @hapi/bell error handling
* @hapi/boom error handling
* rfc update
* test cleanup
* test refactor - moved some functional to `helpers`
* delays in some operations
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 10, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@93b7371). Click here to learn what that means.
The diff coverage is 92.77%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #424   +/-   ##
=========================================
  Coverage          ?   94.52%           
=========================================
  Files             ?      130           
  Lines             ?     2374           
  Branches          ?        0           
=========================================
  Hits              ?     2244           
  Misses            ?      130           
  Partials          ?        0
Impacted Files Coverage Δ
src/actions/getInternalData.js 100% <ø> (ø)
src/utils/challenges/email/send.js 81.81% <0%> (ø)
src/actions/getMetadata.js 100% <100%> (ø)
src/actions/login.js 93.33% <100%> (ø)
src/actions/remove.js 95.65% <100%> (ø)
src/utils/jwt.js 92.42% <100%> (ø)
src/actions/list.js 89.47% <100%> (ø)
src/auth/strategy.bearer.js 86.66% <100%> (ø)
src/actions/organization/create.js 100% <100%> (ø)
src/accounts/init-admin.js 80.64% <100%> (ø)
... and 27 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 93b7371...530e224. Read the comment docs.

1 similar comment
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 10, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@93b7371). Click here to learn what that means.
The diff coverage is 92.77%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #424   +/-   ##
=========================================
  Coverage          ?   94.52%           
=========================================
  Files             ?      130           
  Lines             ?     2374           
  Branches          ?        0           
=========================================
  Hits              ?     2244           
  Misses            ?      130           
  Partials          ?        0
Impacted Files Coverage Δ
src/actions/getInternalData.js 100% <ø> (ø)
src/utils/challenges/email/send.js 81.81% <0%> (ø)
src/actions/getMetadata.js 100% <100%> (ø)
src/actions/login.js 93.33% <100%> (ø)
src/actions/remove.js 95.65% <100%> (ø)
src/utils/jwt.js 92.42% <100%> (ø)
src/actions/list.js 89.47% <100%> (ø)
src/auth/strategy.bearer.js 86.66% <100%> (ø)
src/actions/organization/create.js 100% <100%> (ø)
src/accounts/init-admin.js 80.64% <100%> (ø)
... and 27 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 93b7371...530e224. Read the comment docs.

@AVVS AVVS merged commit 0825a6d into master Sep 10, 2019
@AVVS AVVS deleted the pajgo-fix/facebookTests branch September 10, 2019 02:41
AVVS pushed a commit that referenced this pull request Sep 10, 2019
## [11.2.6](v11.2.5...v11.2.6) (2019-09-10)

### Bug Fixes

* upgrade dependencies, make fb tests more reliable ([#424](#424)) ([0825a6d](0825a6d))
@AVVS
Copy link
Copy Markdown
Member Author

AVVS commented Sep 10, 2019

🎉 This PR is included in version 11.2.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@AVVS AVVS added the released label Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants