Merge lp:~acerisara/ubuntu-calendar-app/failing-tests into lp:ubuntu-calendar-app

[Andrea Cerisara]

Fixes failing autopilot tests.

Fixes failing autopilot tests:

1. test_weekview: simply old buggy code I forgot to change before sending the mp.
2. test_yearview: I need further investigation (at runtime we have just ten MonthComponent instances while we should have twelve), so I disabled the test for the time being.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

FAILED: Continuous integration, rev:165
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/185/
Executed test runs:
    FAILURE: http://91.189.93.70:8080/job/ubuntu-calendar-app-raring-amd64-ci/185/console
    FAILURE: http://91.189.93.70:8080/job/ubuntu-calendar-app-saucy-amd64-ci/185/console
    FAILURE: http://91.189.93.70:8080/job/ubuntu-calendar-app-trusty-amd64-ci/19/console

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/185/rebuild

review: Needs Fixing (continuous-integration)

[lp:~acerisara/ubuntu-calendar-app/failing-tests]

166. By Andrea Cerisara on 2013-11-03

Removed unused import.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:166
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/186/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-raring-amd64-ci/186
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-saucy-amd64-ci/186
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-trusty-amd64-ci/20

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/186/rebuild

review: Approve (continuous-integration)

[Nicholas Skaggs (nskaggs)]

Can you provide better reasoning for the @unit.skip? A bug report would be most useful, but otherwise something more than "too be fixed" :-)

review: Needs Fixing

[Victor Thompson (vthompson)]

It seems like all the MonthComponents aren't loaded into the GridView. It seems like a quick fix might be to get the test to scroll/flick to the bottom and then loop through the months array when the returned number of months is less than the datetime.now().month value.

[lp:~acerisara/ubuntu-calendar-app/failing-tests]

167. By Andrea Cerisara on 2013-11-04

Fixes autopilot tests for the yearly view.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:167
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/187/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-raring-amd64-ci/187
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-saucy-amd64-ci/187
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-trusty-amd64-ci/21

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/187/rebuild

review: Approve (continuous-integration)

[Nicholas Skaggs (nskaggs)]

Andrea, if the month is before June everything works? Seems odd we only need to swipe if now.month > 6. I mean, if the month is before June, we we have to swipe down to get the months we need?

[Andrea Cerisara (acerisara)]

By default (when the view is open) we get the months till October, then we
have to swipe the page up to get the last two months. So to me, it could be
even more than June. If you have time please check this with the vis tool
so we could come up with a better implementation.

2013/11/5 Nicholas Skaggs <email address hidden>

> Andrea, if the month is before June everything works? Seems odd we only
> need to swipe if now.month > 6. I mean, if the month is before June, we we
> have to swipe down to get the months we need?
> --
>
> https://code.launchpad.net/~acerisara/ubuntu-calendar-app/failing-tests/+merge/193706
> You are the owner of lp:~acerisara/ubuntu-calendar-app/failing-tests.
>

--
** Think about the environment before printing

[Andrea Cerisara (acerisara)]

By default (when the view is open) we get months till October, then we have to swipe the page up to get the last two months. So to me, it could be even more than June. If you have time please check this with the vis tool so we could come up with a better implementation.

[Nicholas Skaggs (nskaggs)]

Time to take a look at this again now that ap 1.4 has landed :-)

[Nicholas Skaggs (nskaggs)]

Ok, I think the logic for yearview is fine. However, this is currently failing for me on the desktop after I rebased with trunk. I'll try the phone in a second.

[Nicholas Skaggs (nskaggs)]

Mmm, indeed there are some issues with this, it's not getting the proper values.. For instance, get_days_of_week doesn't give the proper week dates for the view, and fails accordingly. Investigating why.

[Nicholas Skaggs (nskaggs)]

review: Needs Fixing

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

FAILED: Continuous integration, rev:167
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/192/
Executed test runs:
    UNSTABLE: http://91.189.93.70:8080/job/generic-mediumtests-trusty/114
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-raring-amd64-ci/192
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-saucy-amd64-ci/192
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-trusty-amd64-ci/26

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/192/rebuild

review: Needs Fixing (continuous-integration)

[Nicholas Skaggs (nskaggs)]

FYI, I made the fix for the new event test in a different MP, so we can focus on the issues encountered by this one in this MP

https://code.launchpad.net/~nskaggs/ubuntu-calendar-app/fix-new-event-test/+merge/195421

=== modified file 'tests/autopilot/calendar_app/tests/test_weekview.py'
--- tests/autopilot/calendar_app/tests/test_weekview.py	2013-10-28 21:08:48 +0000
+++ tests/autopilot/calendar_app/tests/test_weekview.py	2013-11-04 22:14:24 +0000
@@ -9,7 +9,6 @@
 Calendar app autopilot tests for the week view.
 """
 
-import calendar
 import datetime
 
 from autopilot.matchers import Eventually
@@ -49,15 +48,11 @@
 
         now = datetime.datetime.now()
         days = self.get_days_of_week()
-        monday = (now - datetime.timedelta(days=now.weekday())).day
-        current_month_days = calendar.monthrange(now.year, now.month)[1]
+        monday = (now - datetime.timedelta(days=now.weekday()))
 
         for i in xrange(7):
             current_day = int(days[i].text)
-            expected_day = (monday + i) % current_month_days
-
-            if (monday + i) == current_month_days:
-                expected_day = current_month_days
+            expected_day = (monday + datetime.timedelta(days=i)).day
 
             self.assertThat(current_day, Equals(expected_day))
 

=== modified file 'tests/autopilot/calendar_app/tests/test_yearview.py'
--- tests/autopilot/calendar_app/tests/test_yearview.py	2013-10-28 21:08:48 +0000
+++ tests/autopilot/calendar_app/tests/test_yearview.py	2013-11-04 22:14:24 +0000
@@ -31,7 +31,10 @@
     def test_selecting_a_month_switch_to_month_view(self):
         """It must be possible to select a month and open the month view."""
 
-        months = self.months_from_selected_year()
+        # TODO: the component indexed at 1 is the one currently displayed,
+        # investigate a way to validate this assumption visually.
+        year_grid = self.year_view.select_many("QQuickGridView")[1]
+        months = year_grid.select_many("MonthComponent")
         self.assert_current_year_is_default_one(months[0])
 
         february = months[1]
@@ -56,19 +59,22 @@
     def test_current_day_is_selected(self):
         """The current day must be selected."""
 
-        months = self.months_from_selected_year()
-        current_month = months[datetime.now().month - 1]
+        current_month = self.current_month()
         month_grid = current_month.select_single(objectName="monthGrid")
 
-        current_day_label = month_grid.select_single(
+        # there could actually be two labels with
+        # the current day: one is the current day of the current month,
+        # the other one is the current day of the previous or next month. Both
+        # shouldn't have the standard white color.
+        current_day_labels = month_grid.select_many(
             "Label", text=str(datetime.now().day))
 
         # probably better to check the surrounding UbuntuShape object,
         # upgrade when python-autopilot 1.4 will be available (get_parent).
-        color = current_day_label.color
-        label_color = (color[0], color[1], color[2], color[3])
-
-        self.assertThat(label_color, NotEquals((255, 255, 255, 255)))
+        for current_day_label in current_day_labels:
+            color = current_day_label.color
+            label_color = (color[0], color[1], color[2], color[3])
+            self.assertThat(label_color, NotEquals((255, 255, 255, 255)))
 
     def test_show_next_years(self):
         """It must be possible to show next years by swiping the view."""
@@ -93,9 +99,33 @@
         self.assertThat(self.main_view.get_year(month_component),
                         Equals(datetime.now().year))
 
-    def months_from_selected_year(self):
-        # TODO: the component indexed at 1 is the one currently displayed,
-        # investigate a way to validate this assumption visually.
+    def current_month(self):
+        now = datetime.now()
+        current_month_name = now.strftime("%B")
+
+        # for months after June, we must scroll down the page to have
+        # the month components loaded in the view.
+        if now.month > 6:
+            self.drag_page_up()
 
         year_grid = self.year_view.select_many("QQuickGridView")[1]
-        return year_grid.select_many("MonthComponent")
+        months = year_grid.select_many("MonthComponent")
+
+        for month in months:
+            current_month_label = month.select_single(
+                "Label", objectName="monthLabel")
+
+            if current_month_name == current_month_label.text:
+                return month
+
+        return None
+
+    def drag_page_up(self):
+        x_line = (self.year_view.globalRect[0] +
+                  self.year_view.globalRect[2] / 2)
+
+        y_stop = self.year_view.globalRect[1]
+        y_start = self.year_view.globalRect[3]
+
+        self.pointing_device.drag(x_line, y_start, x_line, y_stop)
+        self.pointing_device.drag(x_line, y_start, x_line, y_stop)