Changes between Initial Version and Version 1 of Ticket #20483, comment 24
- Timestamp:
- Jun 9, 2013, 6:13:12 AM (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #20483, comment 24
initial v1 1 Question: in current patches, the app containing the test is implicitly included. But with the new test discovery tests can live outside apps. I understand that including the current app in `available_apps` would be slightly more verbose, but it's also more consistent and explicit.1 In current patches, the app containing the test is implicitly included. But with the new test discovery tests can live outside apps. I understand that including the current app in `available_apps` would be slightly more verbose, but it's also more consistent and explicit. 2 2 3 The code for implicitly including the current app looks sufficiently robust, I don't have anything against i nfrom a technical point of view. I'm really considering this from an API point of view -- ie. terse vs. explicit.3 The code for implicitly including the current app looks sufficiently robust, I don't have anything against it from a technical point of view. I'm really considering this from an API point of view -- ie. terse vs. explicit.