apjanke / octave-testify

New BIST (Built-In Self Test) functions for GNU Octave
GNU General Public License v3.0
4 stars 2 forks source link

forge tester: Doesn't work on Windows #28

Closed apjanke closed 5 years ago

apjanke commented 5 years ago

The forge pkg tester doesn't work on Windows. pkg uninstall <package> won't work for a package that has been loaded in the Octave session.

Testing package arduino 0.3.0
warning: isdir is obsolete; use isfolder or dir_in_loadpath instead
Processing files in C:\Octave\OCTAVE~1.0\mingw64\share\octave\packages\arduino-0.3.0:

  addon.m ..................................................... PASS      0/1
                                                                  FAIL    1
  isarduino.m ................................................. PASS      4/5
                                                                  FAIL    1
  listArduinoLibraries.m ...................................... PASS      1/1
  scanForArduinos.m ........................................... PASS      1/2
                                                                  FAIL    1
  scanI2Cbus.m ................................................ PASS      0/1
                                                                  FAIL    1

The following files in C:\Octave\OCTAVE~1.0\mingw64\share\octave\packages\arduino-0.3.0 have no t
ests:

arduinosetup.m

TESTS FAILED!
4 failures in tests for packages: arduino

testify.ForgePkgTool: pkg unload arduino instrument-control
testify.ForgePkgTool: pkg uninstall arduino instrument-control
error: couldn't delete directory C:\Octave\OCTAVE~1.0\mingw64\lib\octave\packages\instrument-cont
rol-0.3.1\x86_64-w64-mingw32-api-v53: Permission denied

Looks like it's because of loaded libs, probably oct-files. Windows holds on to those files so you can't delete them until they're unloaded.

apjanke commented 5 years ago

It works on Windows now that the install-ordering is working and we don't have to do redundant package installations in a single session.