View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005306 | Taler | other | public | 2018-03-27 11:01 | 2019-12-20 18:12 |
| Reporter | Marcello Stanisci | Assigned To | Marcello Stanisci | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | tweak | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.6 | Fixed in Version | 0.6 | ||
| Summary | 0005306: "Modular" tests for C components. | ||||
| Description | Every C component's test-suite contains a "run()" method that defines a gigantic 'commands[]' array that then feeds to the test interpreter. This array accounts for various parts of the tested code, so it would be more readable to include commands into this array in a modular fashion, instead of just heaping test commands in it. Right now, we do: commands[] = { /* Testing feature 1 */ TALER_TESTING_cmd_x (..), TALER_TESTING_cmd_y (..), ... /* Testing feature 2 */ TALER_TESTING_cmd_z (..), TALER_TESTING_cmd_y (..), ... /* Testing feature 3 */ ... }; Would be more readable to have: commands[] = { /* Testing feature 1 */ feature_1_tests (..), /* Testing feature 2 */ feature_2_tests (..), /* Testing feature 3 */ ... }; | ||||
| Tags | No tags attached. | ||||
| You could achieve this using macros, or by having a meta-command that takes an array of commands ("subroutine"). | |
| 402a8b42 implements the meta-command technique. Note that the various test suites do NOT use this yet. | |
| 010847f splits CMDs from merchant test cases; 193d95bd splits CMDs from exchange test cases. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2018-03-27 11:01 | Marcello Stanisci | New Issue | |
| 2018-03-27 15:26 | Christian Grothoff | Assigned To | => Marcello Stanisci |
| 2018-03-27 15:26 | Christian Grothoff | Severity | minor => tweak |
| 2018-03-27 15:26 | Christian Grothoff | Status | new => acknowledged |
| 2018-03-27 15:26 | Christian Grothoff | Product Version | => git (master) |
| 2018-03-27 15:26 | Christian Grothoff | Target Version | => 0.6 |
| 2018-03-27 15:26 | Christian Grothoff | Note Added: 0012905 | |
| 2018-06-20 21:20 | Marcello Stanisci | Note Added: 0013047 | |
| 2018-06-26 11:22 | Marcello Stanisci | Note Added: 0013081 | |
| 2018-06-26 11:22 | Marcello Stanisci | Status | acknowledged => informing |
| 2018-06-26 11:22 | Marcello Stanisci | Resolution | open => fixed |
| 2018-09-09 09:57 | Christian Grothoff | Fixed in Version | => 0.6 |
| 2019-12-20 18:12 | Christian Grothoff | Status | informing => closed |