Fix calendar->item() returning an old item in some cases.

On dataChange() from the ETM, we notify observers and update internal
maps. We should only notify the observers *after* we update the internal
calendar data, otherwise the observer might ask the calendar for the item
and it still has old data.

Improved unit-test.