mihok / horizon-jpl

NASA's JPL Horizons On-Line Ephemeris System Python API
MIT License
44 stars 8 forks source link

test_minor_bodies fails #1

Closed astrojuanlu closed 11 years ago

astrojuanlu commented 11 years ago

A long output is shown and then the test fails:

test_connection (horizon.test.test_horizon.TestHorizonInterface) ... ok
test_get (horizon.test.test_horizon.TestHorizonInterface) ... ok
test_major_bodies (horizon.test.test_horizon.TestHorizonInterface) ... ok
test_minor_bodies (horizon.test.test_horizon.TestHorizonInterface) ...  RAD > 0

Comet AND asteroid parameter search:
 RAD > 0.;
Continue [ <cr>=yes, n=no, ? ] : 

*******************************************************************************
JPL/DASTCOM3               Small-body Search Results       2013-Apr-28 08:04:35

 Comet AND asteroid parameter search:                                    
    RAD > 0.;

 Matching small-bodies: 

    Record #  Epoch-yr  Primary Desig  Name                      RAD      
    --------  --------  -------------  ------------------------- -------  
          1     2010    (undefined)     Ceres                    476.200
          2     2010    (undefined)     Pallas                   272.500
[...]
     901842     1997    C/1995 O1       Hale-Bopp                 30.000
     901848     1996    C/1996 B2       Hyakutake                  2.100
     902398     2001    C/2001 OG108    LONEOS                     6.800

 (3214 matches. To SELECT, enter record # (integer), followed by semi-colon.)
*******************************************************************************
 Select ... [F]tp, [M]ail, [R]edisplay, ?, <cr>:
FAIL

======================================================================
FAIL: test_minor_bodies (horizon.test.test_horizon.TestHorizonInterface)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "horizon/test/test_horizon.py", line 49, in test_minor_bodies
    self.assertTrue(len(data))
AssertionError: 0 is not true
mihok commented 11 years ago

Tests should pass now, the issue was in the regex statement.