4 Commits (fa79af298917d501cee26370fdf9d44d05133d15)

Author	SHA1	Message	Date
Jon Atack	7195fa792f	test: Tool wallet test coverage for unexpected writes to wallet ... This commit adds test coverage in `test/functional/tool_wallet.py` to reproduce unexpected writes to the wallet as described in https://github.com/bitcoin/bitcoin/issues/15608: - wallet tool `info` unexpectedly writes to the wallet file if the wallet file permissions are read/write. - wallet tool `info` raises with "Error loading . Is wallet being used by another process?" if the wallet file permissions are read-only. 1. Reproduce the reported issue, define the current unexpected behavior, and add test coverage to guide a future fix in the form of commented-out assertions to be uncommented when testing/fixing. 2. Provisionally extend the same coverage to the wallet tool create test and the getwalletinfo test as regression tests while fixing the issue. 3. Add some logging for sanity checking. ------ Changes after rebase: 5. Make wallet_path an instance method instead of a function in tool_wallet.py as per Marco Falke review suggestion. 6. Assert wallet permissions instead of logging them in tool_wallet.py. This ran into an issue with Appveyor keeping permissions at 666 so allowed for 666 as a workaround. 7. Change the added logging from info to debug level. 8. More helpful assertions order in tool_wallet.py#assert_tool_output. This change makes #assert_tool_output raise "Error loading wallet.dat. Is wallet being used by another process?" rather than a less-helpful message when debugging the read-only wallet permissions issue.	5 years ago
Jon Atack	3bf2b3a37b	test: Split tool_wallet.py test into subtests ... as per Marco Falke review suggestion.	5 years ago
Jon Atack	1eb13f09a9	test: Add log messages to test/functional/tool_wallet.py ... and update code comments as per Python PEP 8 style guide.	5 years ago
João Barbosa	3c3e31c3a4	[tests] Add wallet-tool test ... Original tests by João Barbosa <joao.paulo.barbosa@gmail.com> Additional contribution by John Newbery <john@johnnewbery.com>	6 years ago