Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#6 poppy has inadaquate test coverage

open
None
1
2013-01-24
2011-06-01
Brian Larsen
No

this needs more unit testing coverage

Discussion

  • Brian Larsen
    Brian Larsen
    2011-08-23

    • priority: 5 --> 1
     
  • Brian Larsen
    Brian Larsen
    2011-08-23

    Poppy needs some tests without the c backend used.

     
  • Steve Morley
    Steve Morley
    2013-01-24

    As in previous comment, a number of the omitted lines are when (lib.have_libspacepy == False), i.e. the C-backend is not being used.
    The rest of the untested code is plotting, so the rather low (46%) coverage doesn't really represent how well the code is covered.
    Of course... are we really testing what is happening in the C backend? We really need to make sure we have functional tests and regression tests to ensure the overall operation of this is solid and then close this bug.

    Assigned to Jon, but we should really have a drive to overhaul our tests after the next release where everyone is involved.

     
  • Steve Morley
    Steve Morley
    2013-01-24

    • assigned_to: nobody --> jnlanl