test coverage is far too low

Bug #1237509 reported by Charles Kerr on 2013-10-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Indicator Date and Time
Fix Released
Medium
Charles Kerr
indicator-datetime (Ubuntu)
Undecided
Unassigned

Bug Description

Improving test coverage is an ongoing goal across lots of our codebases, but the ratio of features to tests in datetime is just insane.

I'm adding the alarm features right now on a tight schedule and it would take me days to add tests for this code because there's absolutely no framework in tests/ for unit testing features in datetime, much less features whose states change over time.

So, adding this ticket as a line in the sand to remind myself to improve this in the 14.04 cycle.

Related branches

PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:indicator-datetime at revision 278, scheduled for release in indicator-datetime, milestone Unknown

Changed in indicator-datetime:
status: Triaged → Fix Committed
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:indicator-datetime at revision None, scheduled for release in indicator-datetime, milestone Unknown

Charles Kerr (charlesk) on 2013-11-03
Changed in indicator-datetime:
status: Fix Committed → In Progress
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-datetime - 13.10.0+14.04.20131125-0ubuntu1

---------------
indicator-datetime (13.10.0+14.04.20131125-0ubuntu1) trusty; urgency=low

  [ Charles Kerr ]
  * Switch the build system to cmake. (LP: #1237509)
  * As per the spec, let users show the year in indicator-datetime. (LP:
    #465689)
  * Don't generate g_critical() messages in the log when the timezone
    object is clearing its value. . (LP: #1246532)
  * When in the desktop-greeter mode, don't allow clicking on calendar
    dates to launch evolution. (LP: #1246812)
  * Use g_icon_serialize() output when setting menuitems' "icon"
    property.
  * In src/clock-live.c, pass the right pointer to
    on_detect_location_changed(). (LP: #1251460)

  [ Ted Gould ]
  * Adding a phone_greeter profile.
  * Upstart Job. (LP: #1185565)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 289
 -- Ubuntu daily release <email address hidden> Mon, 25 Nov 2013 03:55:52 +0000

Changed in indicator-datetime (Ubuntu):
status: New → Fix Released
Charles Kerr (charlesk) on 2014-02-06
Changed in indicator-datetime:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers