Closed Bug 940295 Opened 6 years ago Closed 6 years ago

Modify and re-enable test_that_new_event_appears_on_all_calendar_views


(Firefox OS Graveyard :: Gaia::UI Tests, defect, P4)



(b2g-v1.2 affected)

Tracking Status
b2g-v1.2 --- affected


(Reporter: zcampbell, Assigned: zcampbell)




(2 files)

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

A workaround to test it despite bug 877611:
1. Comment lines, 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!
I'll be working on this.
Assignee: nobody → karan.basi
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
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
Attached file github pr
Attachment #8340370 - Flags: review?(viorela.ioia)
Attachment #8340370 - Flags: review?(florin.strugariu)
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 on attachment 8341734 [details]
Pointer to Github pull request:

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:

Attachment #8341734 - Flags: review?(trifandreialin) → review+
Comment on attachment 8341734 [details]
Pointer to Github pull request:

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.