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

Test failures for get_study_org and get_isa_investigation #29

Open
sneumann opened this issue Dec 14, 2022 · 0 comments
Open

Test failures for get_study_org and get_isa_investigation #29

sneumann opened this issue Dec 14, 2022 · 0 comments

Comments

@sneumann
Copy link
Contributor

Hi,
the tests expect_true(tibble::is_tibble(get_study_org('MTBLS375')))
and expect_true(is.character(get_isa_investigation('MTBLS375'))) fail
without the MTBLS_API_KEY variable set, although the tests say 'tokenless-get-methods.
This also causes failures when installing metabolighteR as dependency.
The get_study_org() is setting the key in the function:

httr::add_headers(user_token = getOption('MTBLS_API_KEY'))

while e.g. get_study_desc() does not
httr::GET(paste0(getOption('BASE_URL'), '/studies')) %>% httr::content('parsed')

Either the tests or the code seem to need fixing.
Yours, Steffen

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

No branches or pull requests

1 participant