.get.tmp.url is broken, causing regression tests to fail

[charles-plessy]

Hello,

At the moment, the regression tests at https://bioconductor.org/checkResults/3.17/bioc-LATEST/KEGGREST/nebbiolo1-checksrc.html fail with:

KEGGREST RUnit Tests - 11 test functions, 1 error, 0 failures
ERROR in test_mark_and_color_pathways_by_objects: Error in strsplit(urlLine, "\"", fixed = TRUE)[[1]] : 
  subscript out of bounds

Test files with failing tests

   test_KEGGREST.R 
     test_mark_and_color_pathways_by_objects 


Error in BiocGenerics:::testPackage("KEGGREST") : 
  unit tests failed for package KEGGREST
Execution halted

I could reduce this to:

> KEGGREST:::.get.tmp.url("https://www.kegg.jp/pathway/eco00260+eco:b0002+eco:c00263")
No encoding supplied: defaulting to UTF-8.
Error in strsplit(urlLine, "\"", fixed = TRUE)[[1]] : 
  subscript out of bounds

The function fails when running strsplit(…)[[1]] on urlLine, which is the output of grep("<img src=\"/tmp", lines…, which is character(0)

Probably the structure of the KEGG webpages changed? But as I am not familiar enough with the package, I can not propose a patch.

I hope it helps,

Charles