Menu

#432 Channel transfer, extended testsuite

open
6
2005-08-13
2005-08-04
No

The attached patch is something I have sitting in my
sandbox for TIP #219 for some time now. It extends the
tcltest code with a very minimal attach/detach
facility, as Threads has it, and then uses it for basic
testing of transfering of channel transfer in the core
testsuite.

The package Thread has some testing of this as well,
however and IMHO everything the core provides should be
tested by the core testsuite as well, otherwise it will
be forgotten.

The tests in the patch do _not_ test the scenarios
which have come up on tcl-core in the last few days,
but might be able to. An initial step to improve this
area. Should maybe try to write an 'after-the-fact-TIP'
for the facility to give it some specification we can
follow.

Please review.

Discussion

  • Andreas Kupries

    Andreas Kupries - 2005-08-04

    Channel transfer testing

     
  • Don Porter

    Don Porter - 2005-08-13

    Logged In: YES
    user_id=80530

    ./unix/../generic/tclThreadTest.c
    ../generic/tclThreadTest.c: In function `NewTestThread':
    ../generic/tclThreadTest.c:487: warning: implicit
    declaration of function `Tcltest_Init'

     
  • Don Porter

    Don Porter - 2005-08-13
    • assigned_to: dgp --> andreas_kupries
     
  • Don Porter

    Don Porter - 2005-08-13

    Logged In: YES
    user_id=80530

    other than that compiler warning, the
    patch applies, builds, and tests fine,
    both with and without threads.

    gave the code a very brief look and saw
    nothing to object to. Our quality bar isn't
    very high in the *Test*.c files. They tend
    to be a mishmash of whatever was needed
    to get a test done, and that's fine.

     
Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.