Skip to content
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

t/error.t fails #476

Closed
eserte opened this issue Oct 16, 2021 · 2 comments
Closed

t/error.t fails #476

eserte opened this issue Oct 16, 2021 · 2 comments
Assignees
Labels

Comments

@eserte
Copy link

eserte commented Oct 16, 2021

Currently the test suite fails for me:

#   Failed test 'Saucelabs errors are passed to user'
#   at t/error.t line 66.
#                   'Could not create new session: Server returned error message {"value": {"stacktrace": "", "message": "failed serving request POST /wd/hub/session: Unauthorized", "error": "unknown error"}} instead of data at t/error.t line 59.
# '
#     doesn't match '(?^:Sauce Labs)'
# Looks like you failed 1 test of 4.
t/error.t ............................... 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/4 subtests 

In rare cases it passes, but most of the times not.

@teodesian
Copy link
Owner

When I took over maint of this module, I was disturbed that it was reaching out to remote resources with this test.

Clearly I should have done something about it at the time. I'll look into this monday.

@teodesian teodesian self-assigned this Oct 16, 2021
@teodesian teodesian added the bug label Oct 16, 2021
@eserte
Copy link
Author

eserte commented Oct 17, 2021

When depending on external resources (and this may even be other CPAN modules which may break compatibility) my approach is to setup a "cronjob" in travis-ci or github actions, so to be informed as early as possible, especially if there is not much development work happening in the repository. That said, I don't know how long the other site is broken --- the last cpantesters reports were created on 2021-09-11 (then it were passes), so maybe it's broken for a longer time, or maybe just this weekend.

KohaAloha pushed a commit to KohaAloha/Selenium-Remote-Driver that referenced this issue Sep 29, 2022
KohaAloha pushed a commit to KohaAloha/Selenium-Remote-Driver that referenced this issue Sep 29, 2022
KohaAloha pushed a commit to KohaAloha/Selenium-Remote-Driver that referenced this issue Sep 29, 2022
KohaAloha pushed a commit to KohaAloha/Selenium-Remote-Driver that referenced this issue Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants