Modify and re-enable test_that_new_event_appears_on_all_calendar_views

RESOLVED FIXED

Status

Firefox OS
Gaia::UI Tests
P4
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: zac, Assigned: zac)

Tracking

unspecified
Other
Linux

Firefox Tracking Flags

(b2g-v1.2 affected)

Details

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
In Gaia pull 13607, Johan suggested a workaround to bug 877611:

-----------------
A workaround to test it despite bug 877611:
1. Comment lines test_calendar_new_event_appears_on_all_calendar_views.py:50-51, in order to create a new event with the default calendar UI values.
2. In the same file, replace hours=1 in line 40 by the default values in calendar UI. Example: If your current time is 5:11 AM, the default start value is 18 (for 6:00 PM).
3. In the same file, replace hours=2 in line 41 by start_value += 1. Following the previous example: the new value should be 19 (for 7:00 PM)
-------------

Basically this just anticipates the default values for a calendar event and then we have values to assert with.

I think we should modify test_that_new_event_appears_on_all_calendar_views in this manner so that we can re-enable the test coverage.

We can also follow this model of using the default values for extra test cases like deleting events.

Johan might like to take this task as it was his idea!

Comment 1

5 years ago
I'll be working on this.
(Assignee)

Updated

5 years ago
Assignee: nobody → karan.basi
Status: NEW → ASSIGNED
(Assignee)

Comment 2

5 years ago
The task here is to set the "expected" values that we assert with values from the phone's current time.


* remove the lines noted in point 1 above.
* change the lines in point 2 above to fill with values based on the current hour/time on the phone
* same for point 3 using the current hour/time
(Assignee)

Comment 3

5 years ago
Karan I'm going to take this one as I'd like to re-enable the coverage.

Ping me, I can find another task for you in the future :)
Assignee: karan.basi → zcampbell
(Assignee)

Comment 4

5 years ago
Created attachment 8340370 [details] [review]
github pr
Attachment #8340370 - Flags: review?(viorela.ioia)
Attachment #8340370 - Flags: review?(florin.strugariu)
(Assignee)

Updated

5 years ago
status-b2g-v1.2: --- → affected

Comment 5

5 years ago
Ah no worries. I was working on it but my macbooks hard drive failed and I won't be getting it back for a couple of weeks. I forgot to post an update. My bad.
Attachment #8340370 - Flags: review?(viorela.ioia) → review+
Comment on attachment 8340370 [details] [review]
github pr

I have a concern in the test but don't know if this is the right pull to solve it
Attachment #8340370 - Flags: review?(florin.strugariu) → review-
Attachment #8340370 - Flags: review- → review+

Comment 8

5 years ago
Created attachment 8341734 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/14315

Pointer to Github pull-request

Comment 9

5 years ago
Comment on attachment 8341734 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/14315

Uplifted to v1.2
Attachment #8341734 - Flags: review?(trifandreialin)
Attachment #8341734 - Flags: review?(florin.strugariu)
Attachment #8341734 - Flags: review?(bob.silverberg)
Comment on attachment 8341734 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/14315

r+
Attachment #8341734 - Flags: review?(trifandreialin) → review+
Comment on attachment 8341734 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/14315

Good to merge when the tree reopens.
Attachment #8341734 - Flags: review?(bob.silverberg) → review+
Attachment #8341734 - Flags: review?(florin.strugariu) → review+
You need to log in before you can comment on or make changes to this bug.