https://git.schokokeks.org/derivepassphrase.git/tree/4f4a484f760953687ebd96d3bfdc105b10bd2dcf Recent commits feed provided by GitList. https://git.schokokeks.org/derivepassphrase.git/commit/4f4a484f760953687ebd96d3bfdc105b10bd2dcf software@the13thletter.info (Marco Ricci) Tue, 21 Jan 2025 21:02:31 +0100 4f4a484f760953687ebd96d3bfdc105b10bd2dcf The set of "consistently testable vault configurations" is determined by a specific settings to the validator function. The validator function no longer accepts the `allow_derivepassphrase_extensions` keyword parameter, but the vault test config selection logic was not yet updated to reflect this. https://git.schokokeks.org/derivepassphrase.git/commit/6360edaffe920cdeae6a661bcf477f8496225212 software@the13thletter.info (Marco Ricci) Tue, 21 Jan 2025 21:02:31 +0100 6360edaffe920cdeae6a661bcf477f8496225212 The two pytest fixtures `running_ssh_agent` and `spawn_ssh_agent` both deal with providing access to SSH agents, spawning them if necessary. They differ in the data they provide to the test functions (just the environment setup, or an actual connected `SSHAgentClient`) and in their parametrization (`running_ssh_agent` spawns any one, `spawn_ssh_agent` spawns all). However, despite the spawning logic being basically identical, both methods contained incompatible implementations of the logic, separately for named agent spawning and for system agent spawning. So, consolidate these functions into a single spawning function, and move all necessary supporting code (context manager for subprocess termination, typing protocol for spawn function) to the top level. https://git.schokokeks.org/derivepassphrase.git/commit/d0d1b6c0c5508419c327dd9e5e0fe3f783757d38 software@the13thletter.info (Marco Ricci) Tue, 21 Jan 2025 21:02:31 +0100 d0d1b6c0c5508419c327dd9e5e0fe3f783757d38 Introduce a new workhorse class that provides the validity checking and the falsy value cleanup functionality. The existing top-level functions are now mere facades. The primary reason is to keep the complexity and the nesting level of these functions low-ish: they still trigger linting errors that need to be silenced, but the nesting is tamer, and some common functionality (e.g., tree traversal) can be usefully extracted. https://git.schokokeks.org/derivepassphrase.git/commit/2cc4e4fffab112760803e80084f665d87b468069 software@the13thletter.info (Marco Ricci) Tue, 21 Jan 2025 21:02:31 +0100 2cc4e4fffab112760803e80084f665d87b468069 The `warnings.deprecated` decorator sadly does not do anything at runtime if merely one of several overloads is deprecated. And it seems to fail even if the deprecated signature is actively used; see `_types.is_vault_config`. Implement a known working solution to check if the deprecated keyword argument was supplied. (No tests yet, so excluded from coverage.) https://git.schokokeks.org/derivepassphrase.git/commit/387941e06b2539a702c22139d1f7475f30217c62 software@the13thletter.info (Marco Ricci) Tue, 21 Jan 2025 21:02:31 +0100 387941e06b2539a702c22139d1f7475f30217c62 * t/pathlib: Use `pathlib` for path or filename operations https://git.schokokeks.org/derivepassphrase.git/commit/80810bd4eaa1dd9db8434d95682347ae36ce8b07 software@the13thletter.info (Marco Ricci) Mon, 20 Jan 2025 18:00:28 +0100 80810bd4eaa1dd9db8434d95682347ae36ce8b07 The interface was already supported by all the user-facing code (interestingly enough) and by almost all of the internal code. Using `pathlib.Path` objects directly instead of manually dealing with string or byte string collections does cut down on a lot of the otherwise explicit bookkeeping and context management. Some changes to the tests are necessary because other (or more) functions need to be mocked, or other ways of testing successful or ensuring unsuccessful operations are needed. https://git.schokokeks.org/derivepassphrase.git/commit/65725c82b8466edb0c90061e0750a3d9aaa9f53b software@the13thletter.info (Marco Ricci) Mon, 20 Jan 2025 18:00:12 +0100 65725c82b8466edb0c90061e0750a3d9aaa9f53b * t/documentation-fixes: Make `T_Buffer` type variable visible at runtime again Adapt "storeroom" tests to use new names for helper functions https://git.schokokeks.org/derivepassphrase.git/commit/c539203b53c4317342f311d3ef2f1f9f50009e26 software@the13thletter.info (Marco Ricci) Mon, 20 Jan 2025 15:27:30 +0100 c539203b53c4317342f311d3ef2f1f9f50009e26 Apparently this is necessary to keep the class properly importable: subclassing `typing.Generic` with a runtime-invisible type variable or with a string fails. https://git.schokokeks.org/derivepassphrase.git/commit/751f013f7398e98809cba085cb1b1fcd11ce5f66 software@the13thletter.info (Marco Ricci) Mon, 20 Jan 2025 15:21:59 +0100 751f013f7398e98809cba085cb1b1fcd11ce5f66 The names changed in d58a3abf705eba597a358c323416c325aa649a02, but the tests were still using the old names. Oops. https://git.schokokeks.org/derivepassphrase.git/commit/c66ee5857ee1cea525f7d94596154bd3c80b31ca software@the13thletter.info (Marco Ricci) Mon, 20 Jan 2025 15:10:25 +0100 c66ee5857ee1cea525f7d94596154bd3c80b31ca