Fix coverage and pylint warnings
a5kin opened this issue · comments
We are going a bit more extreme. Now, for successful test suite run, it should get 100% branch coverage as well as code rated 10/10 by pylint
. Fortunately, most of the hard job is already done, as we have 99% coverage and 9.94/10 code rating. Let fix the rest of issues and then keep marks at highest possible level.
Now we have tests all green with pylint
and branch coverage maxed out along with self esteem =) In the process, some flaws in code were detected and fixed, and we finally get rid of highly insecure eval
. The only issue left is using pickle
to save/load board, it is not detecting by pylint
though. Let fix it separately.