Fixed HollowAccountCompletionFuzzing

[iwsimon]

Description:

Related issue(s):

Fixes #9505

Notes for reviewer:

Checklist

• Documented (Code comments, README, etc.)
• Tested (unit, integration, etc.)

[github-actions]

Node: Unit Test Results

    2 281 files  ±0      2 281 suites  ±0   1h 24m 32s ⏱️ - 1m 52s
118 353 tests ±0  118 319 ✔️ ±0  34 💤 ±0  0 ❌ ±0 
126 671 runs  ±0  126 637 ✔️ ±0  34 💤 ±0  0 ❌ ±0 

Results for commit 5613408. ± Comparison against base commit 87ab17a.

♻️ This comment has been updated with latest results.

[github-actions]

Node: E2E Test Results

    1 files      1 suites   22m 18s ⏱️
310 tests 310 ✔️ 0 💤 0 ❌
332 runs  332 ✔️ 0 💤 0 ❌

Results for commit 5613408.

♻️ This comment has been updated with latest results.

[github-actions]

Node: Integration Test Results

279 tests   279 ✔️  28m 14s ⏱️
    5 suites      0 💤
    5 files        0 ❌

Results for commit 5613408.

♻️ This comment has been updated with latest results.

[github-actions]

Node: HAPI Test Results

1 245 tests   910 ✔️  1h 33m 49s ⏱️
   166 suites  335 💤
   166 files        0 ❌

Results for commit 5613408.

♻️ This comment has been updated with latest results.

[tinker-michaelj]

I confirmed that if we just add PAYER_ACCOUNT_DELETED here a 120s run of this @HapiTest was stable (with no other changes needed anywhere).

I would prefer this instead of introducing extra cases to the node software here. 😞

[iwsimon]

I confirmed that if we just add PAYER_ACCOUNT_DELETED here a 120s run of this @HapiTest was stable (with no other changes needed anywhere).

I would prefer this instead of introducing extra cases to the node software here. 😞

It makes sense to modify the test-clients code to include more precheck scenarios than node code to check where the code is executed throw different PreCheckException. With this change though, we still had FAIL_INVALID errors. It's fixed by the change in HandleWorkflow.

[tinker-michaelj]

LGTM, tyvm @iwsimon !