365212 - Use xpcshell based unit test framework

Status: VERIFIED FIXED

(Calendar :: Internal Components, defect)

Description

[Michiel van Leeuwen (email: mvl+moz@)]

We have a calendar/test directory, with some js based tests in there. But I don't think they really work.
If we switch to the xpcshell based unit test framework, the one in mozilla/tools/test-hardness, we can make tests more easily. It does include an eventloop, so we can test async providers. It also integrates with the build system, so we can make tinderboxes go orange on failed tests.

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

We can't make tinderbox run the tests just yet, because we use the tinderboxes also for distributing builds: bug 351968.

Comment 2

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: makefile changes

This patch changes the makefiles such that unit tests can be added to calendar/test/unit, as described in http://developer.mozilla.org/en/docs/Writing_xpcshell-based_unit_tests

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: firsts tests

Those first tests are partly conversions from the existing tests in test/homegrown/misc. Some came from libical/test-data/recur.txt. And I just made up some others.

Comment 4

[Joey Minta]

Comment on attachment 249887 [details] [diff] [review]
firsts tests

+++ test/unit/test_attendee.js	2006-12-28 11:41:18.000000000 +0100
+    // Create event
+    var event = eventClass.createInstance(eventIID);
Can you move this down closer to where it's actually going to be used?

+function findById(event, id, a) {
+    var foundAttendee = event.getAttendeeById(id);
+    do_check_eq(foundAttendee, a);
Do we need to be concerned about the wrapping problems we encounter elsewhere in the code?

+++ test/unit/test_datetime.js	2006-12-27 23:44:15.000000000 +0100

+  cd.timezone = "/mozilla.org/20050126_1/America/Bogota";
+  cd.normalize();
+ 
+  do_check_eq(cd.hour, 10);
+  do_check_eq(cd.icalString, "20051113T100000");
Why does icalString not include TZID= ?

+++ test/unit/test_ics_roundtrip.js	2006-12-28 16:09:01.000000000 +0100

+function checkRoundtrip(expectedProps, obj) {
+    for (var key in expectedProps) {
+        // Need translation
+        icskey = key;
Strict warning, and the comment should probably be a bit more verbose.

r=jminta with that.

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

(In reply to comment #4)
> Why does icalString not include TZID= ?

Because the tzid is a parameter, and not part of the property value (in rfc2445 speak).
Yes, it is confusing, and i'm not sure what to do about it.

Comment 6

[Matthew (lilmatt) Willis]

mvl:
I think this can go in now.  I suspect dmose won't be maintaining his old tests anyway :)

Comment 7

[Matthew (lilmatt) Willis]

Per our calendar-qa phone conference meeting today, we've landed these patches.
The old tests were left until we can confirm with dmose that they're okay to delete.

-> FIXED