Let's move all QLabel font sizes and color styles used in <span>s into one file with all styling constants.
This is not blocking for this branch, so a new bug is being opened.
---
Empty lines before the docstrings in class ActionsButtons and class EnhancedLineEdit.
----
It's never a good idea to touch class variables in tests, because the state of the class variable is not resetted between tests. Please use regular instances for the fake objects instead every time you can, something like:
class FakeDesktopService(object):
"""Fake QDesktopService."""
Let's move all QLabel font sizes and color styles used in <span>s into one file with all styling constants.
This is not blocking for this branch, so a new bug is being opened.
---
Empty lines before the docstrings in class ActionsButtons and class EnhancedLineEdit.
----
It's never a good idea to touch class variables in tests, because the state of the class variable is not resetted between tests. Please use regular instances for the fake objects instead every time you can, something like:
class FakeDesktopServ ice(object) :
"""Fake QDesktopService."""
def __init__(self):
self.opened_ url = None
def openUrl(self, url):
self.opened_ url = url
"""Fake openUrl."""
[...]
def test_open_ in_browser( self):
fake_desktop_ service = FakeDesktopServ ice()
self.patch( QtGui, "QDesktopServices", fake_desktop_ service) ubuntuone. com/asd123'
self.ui. ui.line_ copy_link. setText( url)
self.ui. _open_in_ browser( )
self.assertEqu al(expected, fake_desktop_ service. opened_ url)
"""Test the execution of open_in_browser."""
url = 'http://
expected = QtCore.QUrl(url)
And similarly in the two tests in ActionsButtonsT estCase.
----
Please, fix the docstrings in: to_main_ list public_ files
* test_move_
* test_get_
* test_copy