unit test coverage is low
Bug #1040137 reported by
Charles Kerr
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| The Sync Menu |
Fix Released
|
High
|
Charles Kerr | ||
Bug Description
I omitted the tests to expedite the 12.10.0 release for the Feature Freeze deadline, but this is only a nod to deadlines and isn't the right way to do releases.
The 12.10 tests need to land ASAP.
Related branches
lp://qastaging/~charlesk/indicator-sync/lp-1040137
- Lars Karlitski (community): Approve
- jenkins (community): Approve (continuous-integration)
-
Diff: 392 lines (+211/-46)3 files modifiedsrc/service/app-menu-item.c (+4/-4)
src/service/sync-service.c (+25/-24)
test/test-client.cpp (+182/-18)
| summary: |
- Need to land the tests for 12.10 + unit tests not included in 12.10.0 |
Revision history for this message
| Charles Kerr (charlesk) wrote Re: unit tests not included in 12.10.0 | #1 |
| Changed in indicator-sync: | |
| milestone: | 12.10.1 → none |
| summary: |
- unit tests not included in 12.10.0 + unit test coverage is low |
| Changed in indicator-sync: | |
| status: | In Progress → Fix Committed |
| Changed in indicator-sync: | |
| milestone: | none → 12.10.2 |
Revision history for this message
| Charles Kerr (charlesk) wrote | #2 |
| Changed in indicator-sync: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I've merged some of my tests into trunk for 12.10.1.
Leaving this ticket open for 12.10.2 because coverage is still fairly low.