r-hyperspec / hyperSpec

hyperSpec: Tools for Spectroscopy (R package)
https://r-hyperspec.github.io/hyperSpec/
GNU General Public License v3.0
12 stars 3 forks source link

deprecation process #9

Closed cbeleites closed 3 years ago

cbeleites commented 3 years ago

remove unit tests for deprecated file import functions, they are anyways skipped with skip_if_not_fileio_available()

This will remove lots of messages about skipped unit tests.

The unit tests themselves can still be looked up easily in the old hyperSpec repo at https://github.com/cbeleites/hyperSpec/

GegznaV commented 3 years ago

While it is exported (why?), it is a function that exclusively relates to the old directory structure of the hyperspec project.

Unit tests in other packages use this function. These unit tests must be updated/fixed.

cbeleites commented 3 years ago

Yes. Having this function in the other packages means that the unit tests were always skipped, so that needs to be fixed in the new packages.

GegznaV commented 3 years ago

The only package that uses skip_if_not_fileio_available() is hySpc.read.ENVI