| Commit message (Collapse) | Author | Age |
... | |
| | |
|
| | |
|
|/
|
|
|
| |
These are pretty basic, but they do cover most of the conditions, even
if they rely heavily on mocking.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
This covers mosts of the functions in the crypto module, but doesn't
bother with the hash_key function, which is slated for deletion in the
port to python-gpg. It also doesn't cover re-raising errors.
|
|
|
|
|
|
|
|
|
|
| |
gpgme.Context.verify doesn't raise an exception, instead it attaches the
error as an attribute of the return value. This means that we've been
returning that a signature is valid even when it isn't.
This patch checks the attribute instead of try/excepting. Because there
is a second bug (fixed in the next patch) signature verification will
always fail with this patch.
|
|
|
|
|
|
|
|
| |
This adds a couple of basic tests for the signing and verification of
signatures code in the crypto module. This relies on the utilities
module introduced in the last patch.
One of the tests in here is expected to fail
|
|
|
|
|
| |
This module currently only contains a class that adds a class level
addCleanup function (called addClassCleanup).
|
|
|
|
| |
The test file is also renamed.
|
|
|
|
|
|
|
|
|
|
|
| |
This patch does a bit of refactoring to the Settings class for it's
setup, and adds a reload method. The refactoring encapsulates the
handling of NoneType for the read_*config methods, and reworks the
constructor just a bit so that it doesn't rely on calling methods to
create instance variables.
The end result is slightly cleaner, and will be useful for adding a
reload command to alot itself.
|
| |
|
|
|
|
| |
makes pylint happier to run tests, doesn't affect main pytest.
|
|
|
|
| |
Or it wont be picked up by some test discovery methods.
|
|
|
|
|
|
|
| |
Instead of manual parsing with regexp and manual string formatting the
functions from email.utils are used. This fixes some small
inconsistencies with addresses with empty realnames and with commas in
realnames.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
This fixes a bug in determine_sender where an account that did match but
did not have a realname set was "not found" in case force_realname was
set.
Helped-By: Pol Van Aubel <dev@polvanaubel.com>
|
|
|
|
| |
The second test case exemplifies a bug in determine_sender.
|
|
|
|
|
| |
Otherwise the next account in the for loop might have been returned if
the account was found to match with its last alias.
|
|
|
|
|
| |
This test case exemplifies a bug in determine_sender. The address is
matched correctly but the wrong account is associated with it.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| | |
add some first tests for settings.manager.SettingsManager
|
| | |
|
|/
|
|
|
| |
This doesn't cover the actually complex part of the of the module, but
it's a few basic tests.
|
|
|
|
|
|
| |
Instead us %p, which differs only in the case of the am/pm.
Fixes #1048
|
|
|
|
|
| |
The OSError that is being caught should only be raised by the Popen
constructor: https://docs.python.org/3/library/subprocess.html#exceptions
|
| |
|
|
|
|
|
|
|
|
|
| |
There are a number of things pylint warns on that absolutely make sense
to fix in production code, but for unittests they either don't matter
(like naming variables "foo"), can't be fixed (TestCase methods that
don't use self because they use a mock assert), or the descriptive names
violate PEP8. These are annoying and create noise, so tell pylint to
ignore them.
|
|
|
|
|
| |
This both fixes a test that failed (since stderr wasn't being set) and
simplifies the function to only have one path that uses a ternary.
|
|
|
|
| |
This even finds a bug!
|
| |
|
| |
|
|\
| |
| | |
Remove remaining uses of cmp
|
| |
| |
| |
| |
| |
| | |
This is both a performance issue (since cmp is slower than rich
comparisons), and a python3 issue since cmp (and __cmp__) are gone in
python 3.
|
| | |
|
|\ \
| | |
| | | |
Silence argparse validator test
|