bugGNU Octave - Bugs: bug #54832, __run_test_suite__ fails with...

 
 

bug #54832: __run_test_suite__ fails with nested directories

Submitter:  John Donoghue <lostbard>
Submitted:  Fri 12 Oct 2018 01:47:24 PM UTC
   
 
Category:  Test Suite Severity:  2 - Minor
Priority:  1 - Later Item Group:  Regression
Status:  None Assigned to:  None
Originator Name:  lostbard Open/Closed:  * Open
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 31 Dec 2018 12:16:25 AM UTC, comment #2: 

This bug still affects Octave 5.0.1.

In a future version of Octave, I would rather see a non-internal function that can process tests in a directory. Essentially '__run_test_suite__' is a private function for Octave's own internal use, I would like to see 'runtests' or a new function that users can rely on for similar functionality.

So I would like to close this as won't fix, but maybe a discussion can be had first on what behavior is needed in a replacement.

Leaving as "regression", but in a function that is not meant to be used outside of Octave.

Mike Miller <mtmiller>
Group Member
Wed 17 Oct 2018 04:21:46 PM UTC, comment #1: 

Does work ok if the full path is given rather than folername in the current directory.

_run_test_suite_ ({"/home/john/testfolder"},{})

John Donoghue <lostbard>
Group Member
Fri 12 Oct 2018 01:47:24 PM UTC, original submission:  

With the following directory structure:


testfolder (dir)
  a1.m
  test1 (dir)
    a2.m


a1.m and a2.m can be empty


Then running

 __run_test_suite__ ({"testfolder"},{})


Running in 4.2.2:

Integrated test scripts:

Fixed test scripts:


Summary:

  PASS         0
  FAIL         0




Running 5.0.0:

Integrated test scripts:

__run_test_suite__: fopen failed: testfolder/test1/a2.m


same thing accurs with package +folders.

Looking at the code for _run_test_suite_, it does a cd into the test folder and then tries to open the file using the full path name.


John Donoghue <lostbard>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Digest:
   bug dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by lostbard (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-12-31 mtmiller Severity3 - Normal 2 - Minor
        Priority5 - Normal 1 - Later
    2018-10-16 lostbard Dependencies- Depends on bugs #54561

    Back to the top

    Powered by Savane 3.13-cf05.
    Corresponding source code