Merge lp:~ralsina/ubuntuone-windows-installer/fix_800376 into lp:ubuntuone-windows-installer

[Roberto Alsina]

First branch of the implementation of "Syncing your computer to the cloud" pages.

Initial implementation of a page to select which local folders will be synced to Ubuntu One.

Currently it doesn't do UDF creation or validation, because that functionality should be imported from control panel.

To test IRL (windows only):

You need ubuntuone-client and ubuntu-one-control-panel in your PYTHONPATH, for example:

PYTHONPATH=.;..\ubuntuone-client;..\ubuntuone-control-panel

On another terminal, start the SSO windows client.

Then you can start the wizard:

python bin\ubuntuone-installer-qt

You should be able to click "Agree & Install", then login with a valid user/password, then click "Next" and see this page.

In the page, you should get your "My Documents" folder added by default, and after some seconds,
the space used should be visible on that item and in the header of the list.

I intentionally faked the quota to be very low. Because of that, you should see a widget asking you to buy more space.

If you click "remove" in that row, the item should disappear, and the space used adjust accordingly to 0.

This branch doesn't close the bug, it's just a step towards that.

[Natalia Bidart (nataliabidart)]

A couple of needs fixing:

* always call super() instead of explicitly calling the parent class

* the diff seems to add several (at first) not needed blank lines, can you please remove those, if they are not needed? (we usually do not add empty lines after the docstrings)

* by convention, setUp and tearDown, if defined, should be the first 2 methods after the class declaration

* this docstring:

506 + """After removing all folders, offer_frame should be hidden.
507 +
508 + Push the user over quota, it should be visible"""

should be:

506 + """After removing all folders, offer_frame should be hidden.
507 +
508 + Push the user over quota, it should be visible.

          """

review: Needs Fixing

[Roberto Alsina (ralsina)]

> A couple of needs fixing:
>
> * always call super() instead of explicitly calling the parent class

Ok.

>
> * the diff seems to add several (at first) not needed blank lines, can you
> please remove those, if they are not needed? (we usually do not add empty
> lines after the docstrings)

Ok.

> * by convention, setUp and tearDown, if defined, should be the first 2 methods
> after the class declaration

Ok.

> * this docstring:
>
> 506 + """After removing all folders, offer_frame should be hidden.
> 507 +
> 508 + Push the user over quota, it should be visible"""
>
> should be:
>
> 506 + """After removing all folders, offer_frame should be hidden.
> 507 +
> 508 + Push the user over quota, it should be visible.
>
> """

Ok.

All of those hsould be fixed now.

[lp:~ralsina/ubuntuone-windows-installer/fix_800376]

9. By Roberto Alsina on 2011-06-29

Implement the page that lets the user choose between "sync now" / "sync later" / "sync selectively"

10. By Roberto Alsina on 2011-06-29

Resolve conflicts with trunk

[Natalia Bidart (nataliabidart)]

review: Approve

[lp:~ralsina/ubuntuone-windows-installer/fix_800376]

11. By Roberto Alsina on 2011-06-29

PEP 252

[Manuel de la Peña (mandel)]

review: Approve

=== added file 'data/qt/local_folders.ui'
--- data/qt/local_folders.ui	1970-01-01 00:00:00 +0000
+++ data/qt/local_folders.ui	2011-06-29 15:13:36 +0000
@@ -0,0 +1,167 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ui version="4.0">
+ <class>Form</class>
+ <widget class="QWidget" name="Form">
+  <property name="geometry">
+   <rect>
+    <x>0</x>
+    <y>0</y>
+    <width>460</width>
+    <height>536</height>
+   </rect>
+  </property>
+  <property name="windowTitle">
+   <string>Form</string>
+  </property>
+  <layout class="QVBoxLayout" name="verticalLayout_2">
+   <item>
+    <widget class="QLabel" name="label">
+     <property name="text">
+      <string>Ok! Now it's time to choose wich folder sync to the cloud from this computer.
+We started by suggesting a few</string>
+     </property>
+     <property name="textFormat">
+      <enum>Qt::RichText</enum>
+     </property>
+     <property name="wordWrap">
+      <bool>true</bool>
+     </property>
+    </widget>
+   </item>
+   <item>
+    <layout class="QHBoxLayout" name="horizontalLayout_2">
+     <item>
+      <widget class="QLabel" name="label_2">
+       <property name="text">
+        <string>Folders on my computer</string>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <spacer name="horizontalSpacer_3">
+       <property name="orientation">
+        <enum>Qt::Horizontal</enum>
+       </property>
+       <property name="sizeHint" stdset="0">
+        <size>
+         <width>40</width>
+         <height>20</height>
+        </size>
+       </property>
+      </spacer>
+     </item>
+     <item>
+      <widget class="QPushButton" name="pushButton_2">
+       <property name="text">
+        <string>Add a folder</string>
+       </property>
+      </widget>
+     </item>
+    </layout>
+   </item>
+   <item>
+    <widget class="QTreeWidget" name="folder_list">
+     <attribute name="headerStretchLastSection">
+      <bool>false</bool>
+     </attribute>
+     <column>
+      <property name="text">
+       <string>My Folders</string>
+      </property>
+     </column>
+     <column>
+      <property name="text">
+       <string>Space (Total)</string>
+      </property>
+     </column>
+     <column>
+      <property name="text">
+       <string>Remove</string>
+      </property>
+     </column>
+    </widget>
+   </item>
+   <item>
+    <widget class="QCheckBox" name="checkBox">
+     <property name="text">
+      <string>Connect automatically when computer starts</string>
+     </property>
+    </widget>
+   </item>
+   <item>
+    <widget class="QCheckBox" name="checkBox_2">
+     <property name="text">
+      <string>Automatically sync all selected folders from this computer to the cloud</string>
+     </property>
+    </widget>
+   </item>
+   <item>
+    <widget class="QFrame" name="offer_frame">
+     <property name="frameShape">
+      <enum>QFrame::StyledPanel</enum>
+     </property>
+     <property name="frameShadow">
+      <enum>QFrame::Raised</enum>
+     </property>
+     <layout class="QVBoxLayout" name="verticalLayout">
+      <property name="leftMargin">
+       <number>0</number>
+      </property>
+      <property name="rightMargin">
+       <number>0</number>
+      </property>
+      <item>
+       <widget class="QLabel" name="offer_label">
+        <property name="text">
+         <string>The folders you have selected to sync take over your 2GB space. You can remove some folders or add some extra space</string>
+        </property>
+        <property name="wordWrap">
+         <bool>true</bool>
+        </property>
+       </widget>
+      </item>
+      <item>
+       <layout class="QHBoxLayout" name="horizontalLayout">
+        <item>
+         <spacer name="horizontalSpacer">
+          <property name="orientation">
+           <enum>Qt::Horizontal</enum>
+          </property>
+          <property name="sizeHint" stdset="0">
+           <size>
+            <width>40</width>
+            <height>20</height>
+           </size>
+          </property>
+         </spacer>
+        </item>
+        <item>
+         <widget class="QPushButton" name="pushButton">
+          <property name="text">
+           <string>add more space</string>
+          </property>
+         </widget>
+        </item>
+        <item>
+         <spacer name="horizontalSpacer_2">
+          <property name="orientation">
+           <enum>Qt::Horizontal</enum>
+          </property>
+          <property name="sizeHint" stdset="0">
+           <size>
+            <width>40</width>
+            <height>20</height>
+           </size>
+          </property>
+         </spacer>
+        </item>
+       </layout>
+      </item>
+     </layout>
+    </widget>
+   </item>
+  </layout>
+ </widget>
+ <resources/>
+ <connections/>
+</ui>

=== modified file 'ubuntuone_installer/gui/qt/gui.py'
--- ubuntuone_installer/gui/qt/gui.py	2011-06-28 19:45:45 +0000
+++ ubuntuone_installer/gui/qt/gui.py	2011-06-29 15:13:36 +0000
@@ -72,6 +72,7 @@
     license_ui,
     congratulations_ui,
 )
+from ubuntuone_installer.gui.qt.local_folders import LocalFoldersPage
 from ubuntuone_installer.gui.qt.sync_now_or_later import SyncNowOrLaterPage
 
 _ = gettext.gettext
@@ -87,7 +88,7 @@
     """Wizard Page that displays the license info and links to the GPL."""
 
     def __init__(self, parent=None):
-        QtGui.QWizardPage.__init__(self, parent)
+        super(LicensePage, self).__init__(parent)
         self.ui = license_ui.Ui_Form()
         self.ui.setupUi(self)
         self.setCommitPage(False)
@@ -97,7 +98,6 @@
 
     def initializePage(self):
         """Setup UI details."""
-
         # Set the right texts and connections for buttons
         self.setButtonText(QtGui.QWizard.NextButton, _("Agree && Install"))
         self.setButtonText(QtGui.QWizard.CancelButton,
@@ -124,7 +124,6 @@
 
     def print_document(self, button_id):
         """Print the document displayed in textBrowser."""
-
         if button_id == QtGui.QWizard.CustomButton1:
             document = self.ui.textBrowser.document()
             printer = QtGui.QPrinter(QtGui.QPrinter.HighResolution)
@@ -143,7 +142,7 @@
     """Wizard Page that lets the user Sign into Ubuntu One."""
 
     def __init__(self, ui=None, controller=None, parent=None):
-        QtGui.QWizardPage.__init__(self, parent)
+        super(SignInPage, self).__init__(parent)
         self.ui = ui
         self.ui.setupUi(self)
         self.controller = controller
@@ -155,7 +154,6 @@
 
     def initializePage(self):
         """Setup UI details."""
-
         self.setButtonText(QtGui.QWizard.CancelButton,
             _("Close window and setup later"))
         # Layout without custom button 1,
@@ -175,7 +173,7 @@
     """Wizard Page that lets a current user Sign into Ubuntu One."""
 
     def __init__(self, ui=None, controller=None, parent=None):
-        QtGui.QWizardPage.__init__(self, parent)
+        super(CurrentUserSignInPage, self).__init__(parent)
         self.ui = ui
         self.ui.setupUi(self)
         self.controller = controller
@@ -185,9 +183,6 @@
     # Invalid names of Qt-inherited methods
     # pylint: disable=C0103
 
-    def initializePage(self):
-        """Setup UI details."""
-
     def nextId(self):
         """Provide the next id."""
         return self.next
@@ -197,7 +192,7 @@
     """Shown after SSO login, before setup."""
 
     def __init__(self, ui=None, controller=None, parent=None):
-        QtGui.QWizardPage.__init__(self, parent)
+        super(SuccessPage, self).__init__(parent)
         self.ui = ui
         self.ui.setupUi(self)
         self.controller = controller
@@ -225,7 +220,7 @@
     """Final page of the wizard."""
 
     def __init__(self, parent=None):
-        QtGui.QWizardPage.__init__(self, parent)
+        super(CongratulationsPage, self).__init__(parent)
         self.ui = congratulations_ui.Ui_Form()
         self.ui.setupUi(self)
         self.setFinalPage(True)
@@ -235,7 +230,6 @@
 
     def initializePage(self):
         """Setup UI details."""
-
         # We need custom buttons
         self.wizard().setButtonText(QtGui.QWizard.FinishButton,
             _("Go to my Ubuntu One dashboard"))
@@ -269,7 +263,6 @@
 
     def __init__(self, close_callback=None):
         """Initialize this instance."""
-
         # Used to decide the next page dynamically
         self._next_id = None
 
@@ -279,7 +272,7 @@
         self.tc_url = TC_URL
         self.ping_url = PING_URL
 
-        QtGui.QWizard.__init__(self)
+        super(MainWindow, self).__init__()
         self.close_callback = close_callback
 
         self.creds = Credentials(
@@ -346,6 +339,8 @@
 
         # End of SSO pages
 
+        self.local_folders_page = LocalFoldersPage()
+        self.local_folders_page_id = self.addPage(self.local_folders_page)
         self.SYNC_NOW_OR_LATER_PAGE = self.addPage(SyncNowOrLaterPage())
         self.CONGRATULATIONS_PAGE = self.addPage(CongratulationsPage())
 
@@ -363,13 +358,13 @@
         """Called on successful login."""
         self._next_id = self.SUCCESS_PAGE
         self.next()
-        self._next_id = self.SYNC_NOW_OR_LATER_PAGE
+        self._next_id = self.local_folders_page_id
 
     def registration_success_slot(self):
         """Called on successful registration."""
         self._next_id = self.SUCCESS_PAGE
         self.next()
-        self._next_id = self.SYNC_NOW_OR_LATER_PAGE
+        self._next_id = self.local_folders_page_id
 
     def done(self, result):
         """The main window is being closed, call any custom callback."""

=== added file 'ubuntuone_installer/gui/qt/local_folders.py'
--- ubuntuone_installer/gui/qt/local_folders.py	1970-01-01 00:00:00 +0000
+++ ubuntuone_installer/gui/qt/local_folders.py	2011-06-29 15:13:36 +0000
@@ -0,0 +1,159 @@
+# -*- coding: utf-8 -*-
+
+# Authors: Roberto Alsina <[e-mail]>
+#
+# Copyright 2011 Canonical Ltd.
+#
+# This program is free software: you can redistribute it and/or modify it
+# under the terms of the GNU General Public License version 3, as published
+# by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranties of
+# MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
+# PURPOSE.  See the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+"""Widget to create UDFs in the Windows Install Wizard."""
+
+import ctypes
+import gettext
+import os
+import threading
+import Queue
+
+from PyQt4 import QtCore, QtGui
+
+from ubuntuone.controlpanel.gui import humanize
+
+from ubuntuone_installer.gui.qt.ui import local_folders_ui
+
+_ = gettext.gettext
+
+
+class FolderItem(QtGui.QTreeWidgetItem):
+    """Class representing a folder in the folder list UI."""
+    def __init__(self, strings, path=None, queue=None):
+        super(FolderItem, self).__init__(strings)
+        self.thread = CalculateSize(path, queue)
+        self.thread.start()
+        self.size = None
+        self.path = path
+
+
+class CalculateSize(threading.Thread):
+    """Class to calculate the size of a folder in the background."""    
+    def __init__(self, path_name, queue):
+        self.path_name = path_name
+        self.queue = queue
+        self._stop = False
+        super(CalculateSize, self).__init__()
+        self.daemon = True
+
+    def run(self):
+        total_size = 0
+        for dirpath, dirnames, filenames in os.walk(self.path_name):
+            for f in filenames:
+                fp = os.path.join(dirpath, f)
+                total_size += os.path.getsize(fp)
+        self.queue.put([self.path_name, total_size])
+
+
+class LocalFoldersPage(QtGui.QWizardPage):
+    """Wizard page to create UDFs in the Windows Installer."""
+    def __init__(self, parent=None):
+        super(LocalFoldersPage, self).__init__(parent)
+        self.setTitle(_("Syncing your computer with the cloud"))
+        self.ui = local_folders_ui.Ui_Form()
+        self.ui.setupUi(self)
+
+        header_view = self.ui.folder_list.header()
+        header_view.setResizeMode(0, header_view.Stretch)
+
+        self.queue = Queue.Queue()
+        self.timer = QtCore.QTimer()
+        self.items = {}
+        for folder_name in self.default_folders():
+            self.add_folder(folder_name)
+        self.update_sizes()
+        self.timer.start(2000)
+        self.timer.timeout.connect(self.update_sizes)
+
+    def initializePage(self):
+        self.wizard()._next_id = None
+
+    def quota(self):
+        """The quota available to the user."""
+        # FIXME: get this from real life
+        return 2 * 1024 * 1024
+
+    def default_folders(self):
+        """Return a list of the folders to add by default."""
+        # Special Folder "My Documents"
+        dll = ctypes.windll.shell32
+        buf = ctypes.create_string_buffer(300)
+        dll.SHGetSpecialFolderPathA(None, buf, 0x0005, False)
+        return [buf.value, ]
+
+    def add_folder(self, path):
+        """Add a folder to the list."""
+        if path in self.items:
+            return None
+        # FIXME: the path should actually be sent to u1cp to verify as valid
+        item = FolderItem([path, "", _("Remove")], path=path, queue=self.queue)
+        self.ui.folder_list.addTopLevelItem(item)
+        self.items[path] = item
+        return item
+
+    def update_sizes(self):
+        """Poll the queue were the threads put the size info."""
+        try:
+            path, size = self.queue.get(False)
+            item = self.items.get(path)
+            if item:
+                item.size = size
+                item.setText(1, humanize(size))
+        except Queue.Empty:
+            pass
+        total = 0
+        for path, item in self.items.items():
+            if item.size is None:
+                total = _("Calculating")
+                break
+            total += item.size
+
+        if isinstance(total, long):
+            self.show_hide_offer(total)
+            total = humanize(total)
+        else:
+            self.show_hide_offer(0)
+        self.ui.folder_list.headerItem().setText(1, _("Space (%s)" % total))
+
+    def show_hide_offer(self, cur_size):
+        """Show or hide the offer to buy space according to the total size."""
+        quota = self.quota()
+
+        if cur_size > quota:
+            self.ui.offer_frame.setVisible(True)
+        else:
+            self.ui.offer_frame.setVisible(False)
+
+        self.ui.offer_label.setText(_("The folders you have selected to sync "
+            "take over your %s space. You can remove some folders or add "
+            "some extra space" % humanize(quota)))
+
+    def stop_threads(self):
+        """Stop all pending threads."""
+        for path, item in self.items:
+            item.thread._stop = True
+
+    def on_folder_list_itemClicked(self, item, column):
+        """Delete folder from the list."""
+        if column == 2:
+            del(self.items[item.path])
+            item.thread._stop = True
+            self.ui.folder_list.takeTopLevelItem(
+                self.ui.folder_list.indexOfTopLevelItem(item))
+        self.update_sizes()

=== modified file 'ubuntuone_installer/gui/qt/tests/test_gui.py'
--- ubuntuone_installer/gui/qt/tests/test_gui.py	2011-06-28 19:18:35 +0000
+++ ubuntuone_installer/gui/qt/tests/test_gui.py	2011-06-29 15:13:36 +0000
@@ -19,6 +19,11 @@
 
 """Tests for the Qt UI."""
 
+import os
+import Queue
+import shutil
+import tempfile
+
 from PyQt4 import QtCore
 
 from ubuntuone.credentials import (
@@ -32,6 +37,7 @@
 from ubuntuone_installer.gui.qt import gui
 from ubuntuone_installer.gui.qt.tests import BaseTestCase
 from ubuntuone_installer.gui.qt.embedded_sso import UbuntuSSOClientGUI
+from ubuntuone_installer.gui.qt import local_folders
 
 TOKEN = {u'consumer_key': u'xQ7xDAz',
          u'consumer_secret': u'KzCJWCTNbbntwfyCKKjomJDzlgqxLy',
@@ -96,7 +102,8 @@
 
     def test_start_control_panel_on_finishing(self):
         """If done is called with result=1, the control panel
-        should be called."""
+        should be called.
+        """
         self.patch(gui.subprocess, "Popen", self._set_called)
         self.ui.done(result=1)
         self.assertEqual(self._called,
@@ -104,7 +111,8 @@
 
     def test_not_start_control_panel_on_cancel(self):
         """If done is called with result=0, the control panel
-        should NOT be called."""
+        should NOT be called.
+        """
         self.patch(gui.subprocess, "Popen", self._set_called)
         self.ui.done(result=0)
         self.assertEqual(self._called, False)
@@ -143,8 +151,7 @@
             congrats_page.ui.progressContainer.isVisible(), False)
 
     def test_credential_parameters(self):
-        """Test if the credentials are created the same way
-        as in control panel."""
+        """Compare credential parameters with control panel's."""
         self.assertEqual(self.ui.creds.kwargs, {
             'app_name': APP_NAME,
             'ui_module': "ubuntuone_installer.gui.qt.embedded_sso",
@@ -160,7 +167,6 @@
 
     def setUp(self):
         """Initialize this test instance."""
-
         self.kwargs = {
             'app_name': APP_NAME,
             'tc_url': TC_URL,
@@ -171,8 +177,109 @@
 
     def test_sso_client_gui(self):
         """Ensure the class stores the right parameters."""
-
         self.assertEqual(isinstance(
             self.ui.controller,
             ubuntu_sso.qt.controllers.UbuntuSSOWizardController), True)
         self.assertEqual(self.ui.view, True)
+
+
+class LocalFoldersTestCase(BaseTestCase):
+    """Test the LocalFoldersPage code."""
+
+    class_ui = local_folders.LocalFoldersPage
+
+    def setUp(self):
+        """Initialize this test instance."""
+        # Create a test folder with a known size
+        self.tmpdir = tempfile.mkdtemp()
+        f = open(os.path.join(self.tmpdir, 'test_file'), 'wb')
+        f.write(" " * 600)
+        f.close()
+        os.mkdir(os.path.join(self.tmpdir, 'test_dir'))
+        f = open(os.path.join(self.tmpdir, 'test_dir', 'test_file_2'), 'wb')
+        f.write(" " * 737)
+        f.close()
+        super(LocalFoldersTestCase, self).setUp()
+
+    def tearDown(self):
+        """Remove the temporary tree."""
+        shutil.rmtree(self.tmpdir)
+        BaseTestCase.tearDown(self)
+
+    def test_size_calculation(self):
+        """Test the recursive folder size calculation."""
+        self.queue = Queue.Queue()
+        self.csize = local_folders.CalculateSize(self.tmpdir, self.queue)
+        self.csize.run()
+        path, size = self.queue.get()
+        self.assertEqual(path, self.tmpdir)
+        self.assertEqual(size, 1337)
+
+    def test_item_addition_removal(self):
+        """Add an item (plus the default one), then remove them."""
+        self.ui.add_folder(self.tmpdir)
+        self.assertEqual(2, self.ui.ui.folder_list.topLevelItemCount())
+        self.ui.on_folder_list_itemClicked(
+            self.ui.ui.folder_list.topLevelItem(0), 2)
+        self.ui.on_folder_list_itemClicked(
+            self.ui.ui.folder_list.topLevelItem(0), 2)
+        self.assertEqual(0, self.ui.ui.folder_list.topLevelItemCount())
+        self.assertEqual({}, self.ui.items)
+
+    def test_total_size(self):
+        """Test that the header reflects the change in item sizes."""
+        while self.ui.ui.folder_list.topLevelItemCount():
+            self.ui.on_folder_list_itemClicked(
+                self.ui.ui.folder_list.topLevelItem(0), 2)
+        self.assertEqual(0, self.ui.ui.folder_list.topLevelItemCount())
+        item = self.ui.add_folder(self.tmpdir)
+        item.size = 1337
+        self.ui.update_sizes()
+        self.assertEqual(unicode(self.ui.ui.folder_list.headerItem().text(1)),
+            u"Space (1337)")
+
+    def test_add_twice(self):
+        """Behaviour for adding the same folder twice:
+
+        * It's added only once.
+        """
+        while self.ui.ui.folder_list.topLevelItemCount():
+            self.ui.on_folder_list_itemClicked(
+                self.ui.ui.folder_list.topLevelItem(0), 2)
+        self.assertEqual(0, self.ui.ui.folder_list.topLevelItemCount())
+        self.ui.add_folder(self.tmpdir)
+        self.ui.add_folder(self.tmpdir)
+        self.assertEqual(1, self.ui.ui.folder_list.topLevelItemCount())
+
+    def test_add_missing_folder(self):
+        """Behaviour for adding a folder that doesn't exist:
+
+        * It's added.
+        * Has size 0.
+        """
+
+        while self.ui.ui.folder_list.topLevelItemCount():
+            self.ui.on_folder_list_itemClicked(
+                self.ui.ui.folder_list.topLevelItem(0), 2)
+        self.assertEqual(0, self.ui.ui.folder_list.topLevelItemCount())
+        item = self.ui.add_folder(os.path.join("xyzzy", "xyzzy", "xyzzy"))
+        self.assertEqual(1, self.ui.ui.folder_list.topLevelItemCount())
+        item.thread.run()
+        self.ui.update_sizes()
+        self.assertEqual(0, item.size)
+        # world did not explode
+
+    def test_over_quota(self):
+        """After removing all folders, offer_frame should be hidden.
+
+        Push the user over quota, it should be visible.
+        """
+        self.patch(self.ui.ui.offer_frame, "setVisible", self._set_called)
+        while self.ui.ui.folder_list.topLevelItemCount():
+            self.ui.on_folder_list_itemClicked(
+                self.ui.ui.folder_list.topLevelItem(0), 2)
+        self.assertEqual(0, self.ui.ui.folder_list.topLevelItemCount())
+        self.ui.update_sizes()
+        self.assertEqual(self._called, ((False,), {}))
+        self.ui.show_hide_offer(self.ui.quota() + 1)
+        self.assertEqual(self._called, ((True,), {}))