Changes between Version 3 and Version 4 of Ticket #16244


Ignore:
Timestamp:
04/26/14 21:44:05 (6 years ago)
Author:
nthiery
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #16244 – Description

    v3 v4  
    1313This ticket reduces the role of `InstanceTester.some_elements` to just do some argument parsing and ensure that at most "max_run" elements are returned. It only requires the input to be iterable.
    1414
    15 If we want to have fancy random samples, we should define a specific protocol (typically P.sample()) for it, or just let parents decide on the appropriate strategy by defining some_elements appropriately.
     15If we want to have fancy random samples, we should define a specific protocol (typically P.sample()) for it, or just let parents decide on the strategy by defining some_elements appropriately.
    1616
    1717This was originaly analyzed in #15919.
     18
     19TODO: decide whether InstanceTester.some_elements should return a list or an iterator. A list would be more user friendly (though that's not critical since this is pretty low level) and maybe less error-prone (if a _test_method attempt to reuse the result twice). On the other hand, in case of failure of a _test_method, using an iterator would waste a bit less time before the failure occurs (no need to build all the elements).