Re: Changes to Testcase generator

Hello Josie, thanks for all the work done! Hopefully we will see more from you at some point. And if you could add to the existing documentation in any form, than that would be awsome. thx marcus Thursday, April 10, 2008, 3:04:45 PM, you wrote: > Hi, > As I mentioned earlier these are the changes that I usually end up making > to the tests created by the test case generator, if you have time would be > great if they could get changed! > All Tests: > In the prototype: > remove the word "proto" from the start > add dollar signs to the argument name, e.g. array arr1 becomes array $arr1 > If there is no alias to the function then delete the "Alias to:" line > Remove echo "Done"; and the new line before it and put ===DONE=== after > the closing tag (before the EXPECTF/CLEAN section) > Basic: > Don't have an "add comment here to say what the test is supposed to do" > line. > Variation: > Not numbered (i.e. array_diff_variation.phpt) : > Title should be Test <<<function name>>> function : usage variations > First echo statement should also say usage variations instead of > variation. > I'm using the following for generic (numbered variations, i.e. > array_diff_variation1.phpt) variation tests: > http://www.pastebin.ca/979401 > Optional: (I'm not that fussed about these but would be neat if you could > do something!) > If changing the scripts to use ===DONE===, maybe the EXPECTF should > reflect that? > In the prototype when an undefined number of arguments can be used "..." > is used as the argument name, this then causes a variable called $... to > be generated. Was wondering if there was anyway of renaming this? Also > this should trigger the error test case to not try to pass too many > parameters to the function > So far that's all I can think of, I'm sure I'll remember something later > which I will send through to you as well! > As I'm leaving next week I was also thinking of writing a lessons learned > type document to pass onto Sanjay and Shathosh, which I could forward onto > the QA list if that sounds like a good idea? > Thanks, > Josie Messa > Unless stated otherwise above: > IBM United Kingdom Limited - Registered in England and Wales with number > 741598. > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU Best regards, Marcus