Closed Bug 1258835 Opened 7 years ago Closed 7 years ago

Give tests easier means to spin up calendar services


(Calendar :: General, defect)

Not set


(Not tracked)



(Reporter: Fallen, Assigned: Fallen)




(1 file, 1 obsolete file)

Right now we have a lot of tests that run either one of the calendar services, or manually spin up various services. It would be nice to simplify this a bit.

I noticed some of our tests are swallowing errors that timezones are not defined because they are not spinning up the timezone service.
Attached patch Fix - v1 (obsolete) β€” β€” Splinter Review
Geoff, can you take a look at this patch?
Attachment #8733544 - Flags: review?(geoff)
Comment on attachment 8733544 [details] [diff] [review]
Fix - v1

Review of attachment 8733544 [details] [diff] [review]:

::: calendar/test/unit/test_storage.js
@@ +2,5 @@
>   * License, v. 2.0. If a copy of the MPL was not distributed with this
>   * file, You can obtain one at */
>  function run_test() {
> +    do_calendar_startup(run_next_test);

This doesn't apply properly, I think you must have another patch applied.

@@ +7,3 @@
>  }
> +add_task(function testAttachRoundtrip() {

Should be add_test, no?
Attachment #8733544 - Flags: review?(geoff) → review+
(In reply to Geoff Lankow (:darktrojan) from comment #2)
> This doesn't apply properly, I think you must have another patch applied.

Yes, I have bug 1115667 applied, sorry about that!

> > +add_task(function testAttachRoundtrip() {
> Should be add_test, no?
Good catch!
Attached patch Fix - v2 β€” β€” Splinter Review
Attachment #8733544 - Attachment is obsolete: true
Attachment #8741097 - Flags: review+
Please push this together with bug 1115667 (applies on top of the patches in that bug)
Keywords: checkin-needed
Depends on: 1115667
There was a slight conflict in test_items.js. Hopefully I fixed it correctly:
Closed: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 5.0
This regressed Bug 1264782.
Depends on: 1264782
You need to log in before you can comment on or make changes to this bug.