This is because the string genome is used directly in constructing the BiocFileCache reference. One could instead standardize the string using bsgenomeName(bsgenome) or similar to ensure the shorthand and long versions of the above genome string both fetch the same files.
However to add the unit test to reproduce this fault would require splitting off the caching functionality into a separate function that returns the BiocFileCache string reference we construct. Such a reference string function would be useful later on for providing functions to export or import cached mappable files, as well as reducing the code in the current mappable cache unit test unit case.
Currently, different files would be written for these 2 cases, which should be equivalent:
This is because the string
genomeis used directly in constructing theBiocFileCachereference. One could instead standardize the string usingbsgenomeName(bsgenome)or similar to ensure the shorthand and long versions of the abovegenomestring both fetch the same files.However to add the unit test to reproduce this fault would require splitting off the caching functionality into a separate function that returns the
BiocFileCachestring reference we construct. Such a reference string function would be useful later on for providing functions to export or import cached mappable files, as well as reducing the code in the currentmappablecache unit test unit case.