[Octave-bug-tracker] [bug #51928] [octave forge] (statistics) Failure in unit test of grp2idx function

[Rafael Laboissiere]

URL:
  <http://savannah.gnu.org/bugs/?51928>

                 Summary: [octave forge] (statistics) Failure in unit test of
grp2idx function
                 Project: GNU Octave
            Submitted by: rlaboiss
            Submitted on: Sun 03 Sep 2017 08:33:12 PM UTC
                Category: Octave Forge Package
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: Incorrect Result
                  Status: None
             Assigned to: None
         Originator Name: 
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
                 Release: other
        Operating System: Any

    _______________________________________________________

Details:

A unit test of the function grp2idx of the statistics package (version 1.3.0)
is failing with the following error message:
 

octave:1> pkg load statistics
octave:2> test ("grp2idx")
***** assert (nthargout (1:3, @grp2idx, ["sci"; "oct"; "sci"; "oct"; "oct"]),
        {[1; 2; 1; 2; 2] {"sci"; "oct"} ["sci"; "oct"]});
!!!!! test failed
ASSERT errors for:  assert (cond {i},expected {i},tol)

  Location  |  Observed  |  Expected  |  Reason
    (1)           2            1         Abs err 1 exceeds tol 0
    (2)           1            2         Abs err 1 exceeds tol 0
    (3)           2            1         Abs err 1 exceeds tol 0
    (4)           1            2         Abs err 1 exceeds tol 0
    (5)           1            2         Abs err 1 exceeds tol 0

ASSERT errors for:  assert (cond {i},expected {i},tol)

  Location  |  Observed  |  Expected  |  Reason
     []          oct          sci        Strings don't match

ASSERT errors for:  assert (cond {i},expected {i},tol)

  Location  |  Observed  |  Expected  |  Reason
     []          sci          oct        Strings don't match

ASSERT errors for:  assert (cond {i},expected {i},tol)

  Location  |  Observed  |  Expected  |  Reason
     []       osccti       soccit        Strings don't match


However, it is stated in the comment of the test block:


## FIXME this fails because unique() in core does not yet have set_order
##        option implemented.  See code for code to uncomment once it is
##        implemented in core.


In this case, the failing tests should be run with 'xtest', instead of 'test'.
 The attached patch fixes this.




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Sun 03 Sep 2017 08:33:12 PM UTC  Name: xtests-in-grp2idx.patch  Size:
1KiB   By: rlaboiss

<http://savannah.gnu.org/bugs/download.php?file_id=41724>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?51928>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/