-
Notifications
You must be signed in to change notification settings - Fork 6
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
webmockr in tests without installing all adapters #45
Comments
And it seems the other failure we're getting in a test after this one https://travis-ci.org/lockedata/HIBPwned/jobs/447952505#L1753 is also due to |
I thought a quick fix would be for me to add jumpingrivers/HIBPwned@367a216...2b00dae which means But it wasn't enough https://travis-ci.org/lockedata/HIBPwned/builds/448222145, same failures as before. Therefore, for now I put |
Not weird at all actually travis-ci/travis-ci#4641 (comment) I could also skip the |
thanks! having a look |
all the context is lost cause now those builds are passing. can you show me what the errors were? |
It said to install httr "Please install httr". To reproduce, take that repo without httr in Suggests |
thanks |
change to enable and disable to just throw message when pkg not installed onload remains the same as it wasnt loading pkgs, just remove commented out code wrap adapter httr egs in requireNamespace so not run when httr not avail adjust enable/disable tests so skip if httr not installed bump version install httr on travis
@maelle can you reinstall and try again? I think it's fixed, tried locally by removing httr from suggests in the pkg and it seemed to work |
Thanks, trying now 🤞 |
Yay it works! https://travis-ci.org/lockedata/HIBPwned/builds/449196009 Thank you! |
👋 @sckott!
We're seeing a test failure over at
HIBPwned
, one of which is due towebmockr
https://travis-ci.org/lockedata/HIBPwned/jobs/447952505#L1729 I'd like to enablewebmockr
forcrul
only. Would it be possible to pass an option? Or maybe couldwebmockr
find which adapter to use automatically by e.g. looking at dependencies when called from a test context?By the way it seems that the docs/comments of
webmockr::enable()
& ofonLoad()
aren't up-to-datewebmockr/R/flipswitch.R
Line 12 in 897aff6
webmockr/R/onload.R
Line 8 in a25329b
The text was updated successfully, but these errors were encountered: