doc: test: update TestShell example instructions/options
What the developer wrote
Authored by Sebastian Falbesoner
doc: test: update TestShell example instructions/options
The `createwallet` RPC doesn't return the empty passphrase
warning anymore if no passphrase was passed explicitly.
The `noshutdown` parameter key was removed in commit
fa0dc09b9002f0bcae63af6af8d37fb3e0040ef4, so remove it from
the table.
What changed, and why it matters
This commit only updates documentation for Bitcoin Core's test shell. It changes an example output to remove a wallet passphrase warning that is no longer returned, and removes a 'noshutdown' option from a table because that option was removed in an earlier commit. There is no code change and no security impact.
No security action needed; this is a documentation-only update.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies test/functional/test-shell.md only. It updates the documented example output of createwallet to reflect that the empty passphrase warning is no longer emitted, and removes the noshutdown parameter from the TestShell options table because that parameter was removed in commit fa0dc09b9002f0bcae63af6af8d37fb3e0040ef4. No functional code is changed.
Changed components
test/functional/test-shell.mdInspect captured patch +1 / −2
diff --git a/test/functional/test-shell.md b/test/functional/test-shell.md
index 0a05658e..526c67f6 100644
--- a/test/functional/test-shell.md
+++ b/test/functional/test-shell.md
@@ -97,7 +97,7 @@ rewards to a wallet address owned by the mining node.
```
>>> test.nodes[0].createwallet('default')
-{'name': 'default', 'warning': 'Empty string given as passphrase, wallet will not be encrypted.'}
+{'name': 'default'}
>>> address = test.nodes[0].getnewaddress()
>>> test.generatetoaddress(test.nodes[0], 101, address)
['2b98dd0044aae6f1cca7f88a0acf366a4bfe053c7f7b00da3c0d115f03d67efb', ...
@@ -179,7 +179,6 @@ can be called after the TestShell is shut down.
| `coveragedir` | `None` | Records bitcoind RPC test coverage into this directory if set. |
| `loglevel` | `INFO` | Logs events at this level and higher. Can be set to `DEBUG`, `INFO`, `WARNING`, `ERROR` or `CRITICAL`. |
| `nocleanup` | `False` | Cleans up temporary test directory if set to `True` during `shutdown`. |
-| `noshutdown` | `False` | Does not stop bitcoind instances after `shutdown` if set to `True`. |
| `num_nodes` | `1` | Sets the number of initialized bitcoind processes. |
| `perf` | False | Profiles running nodes with `perf` for the duration of the test if set to `True`. |
| `rpc_timeout` | `60` | Sets the RPC server timeout for the underlying bitcoind processes. |
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.