-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix some fetcher tests #6587
Fix some fetcher tests #6587
Conversation
I think the INSPIRE test can also be fixed, seems like the bib item changed |
The INSPIRE test leaves me clueless. The text is exactly the same. Maybe it's about the line seperators? I was unable to debug the test myself. Can you maybe take a look at it, @Siedlerchr ? |
For inspire change the line in the second entry to this: |
@Siedlerchr and @tobiasdiez , could you do me a favor and take a look on the buildinfo stuff, i'm not sure if I did that right and i'm unable to debug that. According to the test log, instead of the api key the IEEE fetcher only recieves three asterisks from the build script. Is this normal? Did I do something wrong? |
The test log does not output the API keys, it just outputs starts. This is perfectly OK. |
This PR is done for now and should fix all the fetcher tests, except IEEE fetcher, which depends most likely on a new api key. |
So we now only have MrDlib and IEEE not working |
Strange, MrDlib tests ran through an hour ago without complaining... |
refs #6369