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

 
 

bug #54832: __run_test_suite__ fails with nested directories

Submitted by:  John Donoghue <lostbard>
Submitted on:  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

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project Administrator
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>
Project Member
Fri 12 Oct 2018 01:47:24 PM UTC, original submission:

With the following directory structure:

a1.m and a2.m can be empty

Then running

Running in 4.2.2:

Running 5.0.0:

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>
Project 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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project 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.4