* ubuntu_sso/qt/tests/test_reset_password.py:
82: [W0212, SetupAccountTestCase.test_focus_changed_1] Access to a protected member _called of a client class
85: [W0212, SetupAccountTestCase.test_focus_changed_1] Access to a protected member _called of a client class
93: [W0212, SetupAccountTestCase.test_focus_changed_2] Access to a protected member _called of a client class
96: [W0212, SetupAccountTestCase.test_focus_changed_2] Access to a protected member _called of a client class
* Question: should showEvent and hideEvent call super()?
* Silly fix: ubuntu_sso/qt/tests/test_common.py should have you as author, not mandel :-). Same for ubuntu_sso/qt/tests/test_reset_password.py.
* ubuntu_ sso/qt/ tests/test_ reset_password. py: tCase.test_ focus_changed_ 1] Access to a protected member _called of a client class tCase.test_ focus_changed_ 1] Access to a protected member _called of a client class tCase.test_ focus_changed_ 2] Access to a protected member _called of a client class tCase.test_ focus_changed_ 2] Access to a protected member _called of a client class
82: [W0212, SetupAccountTes
85: [W0212, SetupAccountTes
93: [W0212, SetupAccountTes
96: [W0212, SetupAccountTes
* Question: should showEvent and hideEvent call super()?
* Silly fix: ubuntu_ sso/qt/ tests/test_ common. py should have you as author, not mandel :-). Same for ubuntu_ sso/qt/ tests/test_ reset_password. py.
* This is not valid style for our project!
+from ubuntu_ sso.qt. common import (check_as_invalid, assistance, check_match,
+ check_as_valid,
+ password_
+ password_
+ BAD,
+ GOOD,
+ NORMAL,
+ PASSWORD_DIGIT,
+ PASSWORD_LENGTH,
+ PASSWORD_MATCH,
+ PASSWORD_UPPER)
+
it should be:
+from ubuntu_ sso.qt. common import (check_as_invalid, assistance, check_match,
+ check_as_valid,
+ password_
+ password_
+ BAD,
+ GOOD,
+ NORMAL,
+ PASSWORD_DIGIT,
+ PASSWORD_LENGTH,
+ PASSWORD_MATCH,
+ PASSWORD_UPPER,
+)
* I advice to use twisted TestCase instead of unittest's.
* I think there is no need to cut off some sentences like:
+ password_ assistance( line_edit,
+ label_assistance)
can you please check the rest?
* Can we have this moved to a setUp? seems to be used in every test:
+ line_edit = QtGui.QLineEdit()
+ label_assistance = QtGui.QLabel()
* This two should be together by alphabetical order:
+from PyQt4 import QtCore, QtGui trial.unittest import TestCase
+from twisted.
It looks really good! Big applause for all the tests!