Add Google Test to projects lacking test suites
Bug #985060 reported by
Allan LeSage
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Indicator Date and Time |
Fix Released
|
Medium
|
Charles Kerr | ||
| Indicator Display Objects |
Fix Released
|
Medium
|
Ted Gould | ||
| Print Indicator |
Confirmed
|
Medium
|
Lars Karlitski | ||
| Session Menu |
Fix Released
|
Medium
|
Charles Kerr | ||
| indicator-power |
Fix Released
|
Medium
|
Charles Kerr | ||
| indicator-datetime (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
Some indicators need test suites or test-suite improvements; add Google Test to these, as described here: https:/
I'll add tasks for each affected project.
Related branches
lp://staging/~ted/ido/gtest
- jenkins (community): Approve (continuous-integration)
- Charles Kerr (community): Needs Fixing
-
Diff: 303 lines (+232/-1)7 files modified.bzrignore (+4/-0)
Makefile.am (+1/-1)
configure.ac (+8/-0)
m4/gtest.m4 (+63/-0)
src/idorange.h (+2/-0)
tests/Makefile.am (+52/-0)
tests/gtest-menuitems.cpp (+102/-0)
lp://staging/~charlesk/indicator-datetime/lp-985060
- Lars Karlitski (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 333 lines (+275/-5)6 files modifiedMakefile.am (+7/-5)
configure.ac (+22/-0)
m4/gtest.m4 (+63/-0)
tests/Makefile.am (+52/-0)
tests/Makefile.am.strings (+38/-0)
tests/test-indicator.cc (+93/-0)
lp://staging/~cyphermox/indicator-datetime/bootstrap
- Charles Kerr (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 15 lines (+6/-0)1 file modifieddebian/changelog (+6/-0)
| Changed in indicator-datetime: | |
| assignee: | nobody → Charles Kerr (charlesk) |
| Changed in ido: | |
| assignee: | nobody → Ted Gould (ted) |
| Changed in indicator-session: | |
| assignee: | nobody → Ted Gould (ted) |
| Changed in indicator-power: | |
| assignee: | nobody → Charles Kerr (charlesk) |
| Changed in indicator-datetime: | |
| importance: | Undecided → Medium |
| Changed in ido: | |
| importance: | Undecided → Medium |
| Changed in indicator-session: | |
| importance: | Undecided → Medium |
| Changed in indicator-power: | |
| importance: | Undecided → Medium |
| Changed in indicator-printers: | |
| assignee: | nobody → Lars Uebernickel (larsu) |
| importance: | Undecided → Medium |
| description: | updated |
| Changed in indicator-datetime: | |
| status: | New → Confirmed |
| Changed in ido: | |
| status: | New → Confirmed |
| Changed in indicator-printers: | |
| status: | New → Confirmed |
| Changed in indicator-session: | |
| status: | New → Confirmed |
| Changed in indicator-power: | |
| status: | New → Confirmed |
| Changed in ido: | |
| status: | Confirmed → In Progress |
| Changed in indicator-datetime: | |
| status: | Confirmed → In Progress |
| Changed in indicator-session: | |
| assignee: | Ted Gould (ted) → Charles Kerr (charlesk) |
| status: | Confirmed → In Progress |
| Changed in indicator-power: | |
| status: | Confirmed → In Progress |
| Changed in ido: | |
| status: | In Progress → Fix Released |
| Changed in indicator-power: | |
| status: | In Progress → Fix Released |
| Changed in indicator-session: | |
| status: | In Progress → Fix Committed |
| Changed in indicator-session: | |
| milestone: | none → 12.10.0 |
| status: | Fix Committed → Fix Released |
| Changed in indicator-power: | |
| milestone: | none → 12.10.0 |
| Changed in ido: | |
| milestone: | none → 12.10.0 |
| Changed in indicator-datetime: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #1 |
| Changed in indicator-datetime (Ubuntu): | |
| status: | New → Fix Released |
| Changed in indicator-datetime: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This bug was fixed in the package indicator-datetime - 12.10.3daily12.
---------------
indicator-datetime (12.10.
[ Mathieu Trudel-Lapierre ]
* Automatic snapshot from revision 197 (bootstrap):
- Add scaffolding for Google Tests. (LP: #985060)
- Add "check-news" to our AM_INIT_AUTOMAKE() call. (LP: #1075753)
- Don't use geoclue until the user clicks the "Show time in auto-detected
location" checkbox. (LP: #1074999)
- Remove stray semicolon found by Coverity. (LP: #1064546)
* debian/control:
- Update to follow style like other indicator packages: use trailing commas
at the end of lists.
- Reorganize Build-Depends for clarity.
- Drop Build-Depends on cdbs, bump debhelper to (>= 9).
- Update Vcs-Bzr and Vcs-Browser, add a notice for uploaders.
- Add gnome-common to Build-Depends.
- Add libxorg-gtest-dev, libgtest-dev to Build-Depends.
* debian/compat: bump compat level to 9.
* debian/rules:
- Switch to using the dh sequencer instead of cdbs rules, adjust targets
accordingly.
- Override dh_autoreconf to make it use autogen.sh from upstream.
[ Didier Roche ]
* debian/copyright:
- Update the copyright
[ Mathieu Trudel-Lapierre ]
* Don't use geoclue if "show-auto-
(LP: #1074999)
* Add check-news to indicators' AM_INIT_AUTOMAKE rules (LP: #1075753)
* Add Google Test to projects lacking test suites (LP: #985060)
* Stray semicolon in datetime-prefs (LP: #1064546)
[ Automatic PS uploader ]
* Automatic snapshot from revision 199
-- Automatic PS uploader <email address hidden> Fri, 23 Nov 2012 07:39:58 +0000