again in initializePage? The terms legend was set on __init__, is there any specific issue that requires setting the text again?
* In the test_overlay_connection_setup_account, can you call self.ui.set_up_button.clicked.emit(False) instead of self.ui.button(QtGui.QWizard.CustomButton3).clicked.emit(False)?
* If you're patching self.patch(self.ui, 'wizard', FakeWizard) in every test method in SetupAccountTestCase, please add that to the setUp method to avoid duplication.
* Please call super() instead of QtGui.QCheckBox .__init_ _(self) .
* Why re you removing the overlay shown when the setup button is clicked?
* Since we're building strings with the hope they get translated, we always try to use formatting with keywords. So, instead of this
"By signing up to Ubuntu One you agree to our {0} and {1}"
we should use:
"By signing up to Ubuntu One you agree to our {terms_ and_conditions} and {privacy_policy}"
and then call format like this:
terms = TERMS.format( terms_and_ conditions= TERMS_LINK,
privacy_ policy= PRIVACY_ POLICY_ LINK) check_box. EnhancedCheckBo x(terms)
self.terms_checkbox = enhanced_
* Why are you calling
again in initializePage? The terms legend was set on __init__, is there any specific issue that requires setting the text again?
* In the test_overlay_ connection_ setup_account, can you call self.ui. set_up_ button. clicked. emit(False) instead of self.ui. button( QtGui.QWizard. CustomButton3) .clicked. emit(False) ?
* If you're patching self.patch(self.ui, 'wizard', FakeWizard) in every test method in SetupAccountTes tCase, please add that to the setUp method to avoid duplication.
Great work on the tests for the new widget!