diff options
| author | kali <kali@leap.se> | 2012-08-27 05:25:07 +0900 |
|---|---|---|
| committer | kali <kali@leap.se> | 2012-08-27 05:25:07 +0900 |
| commit | 09a68fab7d39521ffb5017abaffdb4c946cd5034 (patch) | |
| tree | 361a499bf4bd060174f9575ab8cb246f6d98575e | |
| parent | 72c64c11e5b77901606a3f732aefcfa64f5d14d7 (diff) | |
add new TODO comment
| -rw-r--r-- | src/leap/base/tests/test_config.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/leap/base/tests/test_config.py b/src/leap/base/tests/test_config.py index 73b0f32..0a98dba 100644 --- a/src/leap/base/tests/test_config.py +++ b/src/leap/base/tests/test_config.py @@ -104,6 +104,13 @@ class ProviderDefinitionTestCase(ProviderTest): # doing an extra wipe of the tempdir... but be careful!!!! do not mess with # os.environ home more than needed... that could potentially bite! +# XXX actually, another thing to fix here is separating tests: +# - test that requests has been called. +# - check deeper for error types/msgs + +# we SHOULD inject requests dep in the constructor +# (so we can pass mock easily). + class ProviderFetchConError(ProviderTest): def test_connection_error(self): |