View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006479 | Taler | exchange | public | 2020-08-12 13:18 | 2021-08-24 16:23 |
| Reporter | Christian Grothoff | Assigned To | Christian Grothoff | ||
| Priority | normal | Severity | major | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
| Product Version | git (master) | ||||
| Target Version | 0.8 | Fixed in Version | 0.8 | ||
| Summary | 0006479: re-enable auditor integration test | ||||
| Description | Currently the auditor test is disabled as we need to make sure the webex integration test generates a good DB for the new code. We should re-enable the test and fix any breakage detected. | ||||
| Tags | No tags attached. | ||||
|
|
The wallet now has re-enabled the integration test functionalty. I've updated the invocation syntax in the auditor test scripts. Additionally, the "integrationtest" functionality has now a test that covers it in the wallet's integration test suite: https://git.taler.net/wallet-core.git/tree/packages/taler-integrationtests/src/test-wallettesting.ts |
|
|
The integration test now fails in some part that seems to be a problem with the test harness, not the wallet: """ Revoking denomination null (to affect coin null) Argument `revoke' malformed. Expected base32 (Crockford) encoded value. Generation failed, running only on existing DB SKIP test-revocation.sh (exit status: 77) """ |
|
|
For the later parts (that I can't test yet because of the previous issue): The integration test now modifies the passed in wallet DB, so this might affect the test (i.e. a different temp DB file might have to be passed). |
|
|
Looks like the SELECT which is expected to find out the key to identify the denomination to revoke fails to select any record. Probably because something changed (coin selection) in the integration test. |
|
|
Looking at the old code, it seems like the "spendAmount" has changed from 4 to 5. I'll adjust this and see what'll happen. These amounts are not configured in the JSON passed to the runIntegrationTest command and not fixed anymore, |
|
|
Once the auditor test works again, we should add some additional checks in the wallet's integration test for the functionality to make sure that we notice in directly in the wallet when the coin selection changes unexpectedly. |
|
|
fa27c116..d453c47a re-enables the test. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2020-08-12 13:18 | Christian Grothoff | New Issue | |
| 2020-08-12 13:18 | Christian Grothoff | Status | new => assigned |
| 2020-08-12 13:18 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2020-08-12 13:18 | Christian Grothoff | Assigned To | Christian Grothoff => |
| 2020-08-12 13:18 | Christian Grothoff | Status | assigned => confirmed |
| 2020-08-14 13:03 | Florian Dold | Note Added: 0016626 | |
| 2020-08-14 13:30 | Florian Dold | Note Added: 0016627 | |
| 2020-08-14 13:32 | Florian Dold | Note Added: 0016628 | |
| 2020-08-14 13:37 | Christian Grothoff | Note Added: 0016629 | |
| 2020-08-14 13:47 | Florian Dold | Note Added: 0016630 | |
| 2020-08-14 13:47 | Florian Dold | Note Added: 0016631 | |
| 2020-08-15 04:36 | Thehornet17 | Tag Attached: bug | |
| 2020-08-15 04:36 | Thehornet17 | Tag Attached: android | |
| 2020-08-15 10:00 | Christian Grothoff | Tag Detached: android | |
| 2020-08-15 10:00 | Christian Grothoff | Tag Detached: bug | |
| 2020-08-28 23:51 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2020-08-28 23:51 | Christian Grothoff | Status | confirmed => assigned |
| 2020-08-29 13:48 | Christian Grothoff | Status | assigned => resolved |
| 2020-08-29 13:48 | Christian Grothoff | Resolution | open => fixed |
| 2020-08-29 13:48 | Christian Grothoff | Fixed in Version | => 0.8 |
| 2020-08-29 13:48 | Christian Grothoff | Note Added: 0016797 | |
| 2021-08-24 16:23 | Christian Grothoff | Status | resolved => closed |
