Open katrinleinweber opened 6 years ago
Right below https://github.com/TIBHannover/BacDiveR/blob/master/R/retrieve_search_results.R#L14, the queryURL should be validated ("defensive programming") somehow. Maybe using stringi::stri_startswith("https://bacdive.dsmz.de/advsearch?site=advsearch") and/or stringi::stri_endswith("&advsearch=search")?
queryURL
stringi::stri_startswith("https://bacdive.dsmz.de/advsearch?site=advsearch")
stringi::stri_endswith("&advsearch=search")
In either case, a test should be added whether an error is triggered by using a non-advsearch URL.
advsearch
BNLS
Related to #71.
Right below https://github.com/TIBHannover/BacDiveR/blob/master/R/retrieve_search_results.R#L14, the
queryURL
should be validated ("defensive programming") somehow. Maybe usingstringi::stri_startswith("https://bacdive.dsmz.de/advsearch?site=advsearch")
and/orstringi::stri_endswith("&advsearch=search")
?In either case, a test should be added whether an error is triggered by using a non-
advsearch
URL.BNLS