https://git.schokokeks.org/derivepassphrase.git/tree/7e9e0e24feaedfe8465a080e2bfee991bbbb6a40 Recent commits feed provided by GitList. Previously, we would define a "worst-case" hypothesis profile for use under instrumentation (coverage measurement), and decorate every hypothesis test which we experienced to have deadline problems with this decorator. This step was manual, and very error-prone when generating new hypothesis tests. Conceptually, it also indicated that adjusted deadlines applied for some tests, but not for others, even though the deadline adjustment is actually a side-effect of global instrumentation. We therefore now adopt a global solution, where the pytest configuration file detects which coverage instrumentation (if any) is running, and adjusts the default hypothesis settings appropriately. The decorator is removed. In the course of this, we re-evaluated the profiles and the slowdown factors, instead of assuming the worst-case instrumentation. hypothesis also defines a "ci" profile for use in continuous integration, together with some commentary on how this profile is expected to be used. We cede this profile name to hypothesis (and remove it from our testing matrix in hatch), and define our own "intense" profile for high-volume hypothesis testing based on the "ci" profile, to be invoked manually as a "pre-release" (or perhaps "nightly") run. Finally, because the test suite now is rather slow in sequential use, we add `--dist=worksteal` to the test options to better distribute load during parallel testing. We choose this over other more "balanced" strategies because our single-test runtimes wildly fluctuate thanks to hypothesis, so work stealing appears to be the most reliable distribution heuristic for us. https://git.schokokeks.org/derivepassphrase.git/commit/7e9e0e24feaedfe8465a080e2bfee991bbbb6a40 software@the13thletter.info (Marco Ricci) Fri, 31 Jan 2025 14:43:03 +0100 7e9e0e24feaedfe8465a080e2bfee991bbbb6a40 * t/split-cli-module: Document the trimming of the `cli` module in the changelog Split off cli_helpers and cli_machinery internal modules Move CLI messages module into a new `_internals` subpackage https://git.schokokeks.org/derivepassphrase.git/commit/af0cd7e9e4b0c823bf1385d6942ef7ba44f53649 software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 22:30:02 +0100 af0cd7e9e4b0c823bf1385d6942ef7ba44f53649 https://git.schokokeks.org/derivepassphrase.git/commit/aa0968564bd32aff89877e91cea4da237998aeae software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 22:29:01 +0100 aa0968564bd32aff89877e91cea4da237998aeae Split off the `logging` and `click` support code (the "machinery") and the command-specific helper functions (the "helpers") into separate modules under `derivepassphrase._internals`. Change all functions, classes and attributes within those new modules to be public with respect to the module, since the module itself is already non-public. Also document all previously undocumented classes and functions: `cli_helpers.ORIGIN`, `cli_helpers.check_for_misleading_passphrase`, `cli_helpers.key_to_phrase` and `cli_helpers.print_config_as_sh_script`. Add a better default value (`cli_helpers.default_error_callback`) for the `error_callback` parameter of `cli_helpers.key_to_phrase`, and document that too. Finally, since the shell completion parts are now split across two modules, add an explicit TODO to the `cli_machinery.ZshComplete` class to add context. https://git.schokokeks.org/derivepassphrase.git/commit/623f749ed8245822f7b00520bd4096d008673685 software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 22:23:26 +0100 623f749ed8245822f7b00520bd4096d008673685 Also adjust the headings in the development build of the documentation to clearly mark internal documentation as internal. (The absolute imports within the CLI messages module make the move itself relatively painless: only the callers need to be adjusted to the new path.) https://git.schokokeks.org/derivepassphrase.git/commit/733aeec69f03e5b8cfa025ee381dce10902f821c software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 20:34:40 +0100 733aeec69f03e5b8cfa025ee381dce10902f821c * t/vault-module-api: Document API changes in the vault module Adjust coverage on vault tests for error-only code paths Adjust docstrings for vault "symbol" character class Make the vault UUID and CHARSETS attributes public Let `Vault` accept Buffer-type values wherever it accepts bytes https://git.schokokeks.org/derivepassphrase.git/commit/258929497c5a8582f02e6b3fe3e42fad45e1bb86 software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 16:35:02 +0100 258929497c5a8582f02e6b3fe3e42fad45e1bb86 https://git.schokokeks.org/derivepassphrase.git/commit/9cca65fc7dbc704b832332abdf441745e77b85b5 software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 16:34:22 +0100 9cca65fc7dbc704b832332abdf441745e77b85b5 Code paths in the tests that only do error handling due to factors beyond their control (here: unsatisfied hypothesis assumptions, or re-raising the error) should not be counted towards coverage. https://git.schokokeks.org/derivepassphrase.git/commit/a9025de171c37b4a8ba78945906e800616c1d9a3 software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 15:36:12 +0100 a9025de171c37b4a8ba78945906e800616c1d9a3 The "dash" class is a subset of "symbol", so we can't talk about "all hitherto unlisted ASCII printable characters", but should list the exceptions explicitly. https://git.schokokeks.org/derivepassphrase.git/commit/45152120fe7dbcfdac8ba2786507a3bad882be53 software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 15:33:36 +0100 45152120fe7dbcfdac8ba2786507a3bad882be53 They are constants, they are not priviledged information, and having to keep indirectly referring to them instead of directly is rather irritating. https://git.schokokeks.org/derivepassphrase.git/commit/689cf84c5396b0d92909c554d5c2a75453c55c24 software@the13thletter.info (Marco Ricci) Wed, 29 Jan 2025 15:28:07 +0100 689cf84c5396b0d92909c554d5c2a75453c55c24