Noisy GMOCK WARNING in test_download_manager.cpp

Bug #1278545 reported by Alejandro J. Cura
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity-scope-click (Ubuntu)
Fix Released
Low
Mike McCracken

Bug Description

There's a series of lines like the following in the "make test" output:

GMOCK WARNING:
Uninteresting mock function call - taking default action specified at:
/home/alecu/canonical/unity-scope-click/get-installed/scope/tests/test_download_manager.cpp:158:
    Function call: rawHeaderPairs()
          Returns: {}
Stack trace:

We'd like to get rid of those, by using NiceMock where appropriate.

Related branches

Changed in unity-scope-click (Ubuntu):
importance: Undecided → Low
status: New → Confirmed
assignee: nobody → Mike McCracken (mikemc)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity-scope-click - 0.1+14.04.20140317-0ubuntu1

---------------
unity-scope-click (0.1+14.04.20140317-0ubuntu1) trusty; urgency=low

  [ Rodney Dawes ]
  * Require full URL host and path to be passed to Service::call. Add
    support for doing HEAD and POST in Service::call.
  * Don't run the vala tests during package build any more.
  * Implement signing in Service::call to allow performing authorized
    requests.
  * Rename the ::Service class to ::Client. Update the tests and mocks
    for the rename.

  [ Alejandro J. Cura ]
  * Include supported frameworks in click search (LP: #1267190)
  * A class to handle related signals and their disconnection
  * Bubble up errors in the webclient, and allow aborting webcalls.
  * Use grid layout instead of journal for searches

  [ Colin Watson ]
  * Use get_boolean_member to fetch the manifest's _removable field, not
    get_int_member.

  [ Michael McCracken ]
  * Fix broken tests for download manager. (LP: #1278545)
  * Add make target to run disabled tests .
 -- Ubuntu daily release <email address hidden> Mon, 17 Mar 2014 17:09:48 +0000

Changed in unity-scope-click (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers