- May 07, 2016
- May 06, 2016
- May 05, 2016
-
-
dwcoder authored
This commit fixes #23 Add the `string.strip()` function to the `get_config_dict()`, to remove inverted commas around string values in the diceware.ini file. The strip removes both `"` and `'`. Add unit test to ensure the line: delimiter=" " in the diceware.ini file is being parsed correctly as a space.
-
- May 03, 2016
-
-
ulif authored
This helps to recognize options as such.
-
- May 01, 2016
- Apr 30, 2016
-
-
ulif authored
-
ulif authored
-
ulif authored
-
ulif authored
The modifications undone here, in `test_handle_options_considers_configfile()` could have gone into an own test. Each single test should only test a clear minimum of things. This specific test checked, whether configiles are considered at all. It was not its purpose to examine correct interpretation of keywords in the .ini-file; only to see whether these keyword values can be changed at all. If we wanted a complete and strict checking also of values, we had at least to check all possible keywords, including 'randomsource' and 'wordlist'. Instead we only check a boolean, int, and string type keyword. Therefore the change to `test_handle_options_considers_configfile()` should have gone to another test. But then, the added testing is not neccessary. We had a test already that should have checked the 'caps' value properly. This test was done wrong before and is now done correctly, thanks to @dwcoder . I therefore removed the added test functionality.
-
ulif authored
Running `pep8` or `flake8` over `tests/` complained about this one blank line.
-
- Apr 29, 2016
-
-
dwcoder authored
This commit fixes issue #21. Change all occurences of the option `capitalize` to `caps` in __init.py__. Make the same change in the test_diceware.py. Add a unit test that checks the output of main() after setting the `caps = off` in the .diceware.ini.
-
- Apr 28, 2016
- Apr 27, 2016
- Apr 25, 2016
-
-
ulif authored
Add a common place where components can write messages to. Any userinterface can decide, what messages should be visible (depending on their severity).
-
- Apr 24, 2016
-
-
ulif authored
-
ulif authored
-
ulif authored
With one item in sequence we do not have to roll dice. This holds for all sorts of dice.
-
ulif authored
The moved comment was at the wrong place.
-
ulif authored
I guess it is obvious, how the first if-clause works.
-
ulif authored
We can simply do the (cheap) modulo operation after each die role.
-
- Apr 21, 2016
- Apr 20, 2016