yahoo-eng-team team mailing list archive

[OpenStack Infra <1989268@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/glance/+/856922
Committed: https://opendev.org/openstack/glance/commit/5d9cd0e711dae15bccd4012ceed3a8e16cb403e9
Submitter: "Zuul (22348)"
Branch:    master

commit 5d9cd0e711dae15bccd4012ceed3a8e16cb403e9
Author: Takashi Natsume <takanattie@xxxxxxxxx>
Date:   Sun Sep 11 13:44:04 2022 +0900

    Fix a wrong assertion method
    
    Replace 'has_calls' with 'assert_has_calls'.
    
    Change-Id: I0af1e8cd33eb7785aeffa1ac774063303ffbb090
    Closes-Bug: 1989268
    Signed-off-by: Takashi Natsume <takanattie@xxxxxxxxx>


** Changed in: glance
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Glance.
https://bugs.launchpad.net/bugs/1989268

Title:
  Wrong assertion method

Status in Glance:
  Fix Released

Bug description:
  There is misuse of an assert statement in the master.

  https://opendev.org/openstack/glance/src/commit/d7db4e56231059a15d4451c0eaa417c9375162a7/glance/tests/unit/v2/test_image_data_resource.py#L256

         mock_enforce.has_calls(expected_call)

  'has_calls' should be 'assert_has_calls'.

To manage notifications about this bug go to:
https://bugs.launchpad.net/glance/+bug/1989268/+subscriptions