SourceForge has been redesigned. Learn more.
Close

#172 String~space testGroup

v4.2.0
closed
None
1
2012-08-22
2012-08-22
No

I wanted to re-use the String~space testGroup for MutableBuffer~space testGroup
but couldn't find it. So I made the ~space testGroup first for Strings.

The TestCases tracker doesn't let me create a ticket, that's why I put it here.

The testGroup runs ok with trunk rev. 8164, but in trunk rev. 8237 the String~space method doesn't work.

1 Attachments

Discussion

  • Mark Miesfeld

    Mark Miesfeld - 2012-08-22

    Uli, space doesn't work because your patch mistakenly removed it from Setup.cpp.

    C:\work.ooRexx\wc\ooTest\4.x>qt
    4 - say 'str:' str 'str~space:' str~space
    Error 97 running C:\work.ooRexx\wc\ooTest\4.x\qt.rex line 4: Object method not found
    Error 97.1: Object "help" does not understand message "SPACE"

    I added it back in and it seems okay.

    C:\work.ooRexx\wc\ooTest\4.x>qt
    str: help str~space: help

    I updated the test cases tracker to allow people to open new tickets.

    Your test case runs okay:

    C:\work.ooRexx\wc\ooTest\4.x>testOORexx.rex -f space
    Searching for test containers..
    Executing automated test suite..

    ooTest Framework - Automated Test of the ooRexx Interpreter

    Interpreter: REXX-ooRexx_4.2.0(MT) 6.04 21 Aug 2012
    Addressing Mode: 64
    ooRexxUnit: 2.0.0_3.2.0 ooTest: 1.0.0_4.0.0

    Tests ran: 363
    Assertions: 370
    Failures: 0
    Errors: 0
    Skipped files: 0

    File search: 00:00:00.046000
    Suite construction: 00:00:00.026000
    Test execution: 00:00:00.076000
    Total time: 00:00:01.044000

    Committed revision 8240.
    Committed [r8240]

    Thanks a lot Uli for this work.

    • status: open --> accepted
    • assigned_to: Mark Miesfeld

    • status: accepted --> closed

     

    Related

    Commit: [r8240]

  • Mark Miesfeld

    Mark Miesfeld - 2012-08-22

    Pending doesn't seem appropriate to me for a patch applied to the test suite, so I'm going to mark this closed.

     

Anonymous
Anonymous

Cancel  Add attachments