Glance v1 unit test code can do with some refactoring

Bug #1264639 reported by David Koo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Won't Fix
Wishlist
Shweta Patil

Bug Description

The Glance v1 unit test code (tests/unit/v1/test_api.py) can do with some refactoring - there are lots of repeated patterns throughout the code. Refactoring it can help with understanding and maintaining the code and also make adding future tests simpler.

This bug is an action item to track the review comment for https://review.openstack.org/#/c/64079/4 raised by Steve Kowalik.

David Koo (kpublicmail)
Changed in glance:
assignee: nobody → David Koo (kpublicmail)
status: New → In Progress
Feilong Wang (flwang)
Changed in glance:
importance: Undecided → Wishlist
David Koo (kpublicmail)
Changed in glance:
assignee: David Koo (kpublicmail) → nobody
Shweta Patil (89shweta)
Changed in glance:
assignee: nobody → Shweta Patil (89shweta)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

Fix proposed to branch: master
Review: https://review.openstack.org/87020

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/92193

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/92218

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on glance (master)

Change abandoned by Flavio Percoco (<email address hidden>) on branch: master
Review: https://review.openstack.org/87020
Reason: after all this time, I'd assume we don't need this patch anymore. The bug/blueprint was probably fixed already or invalidated. Please, feel free to re-open it if you think otherwise.

Revision history for this message
Abhishek Kekane (abhishek-kekane) wrote :

V1 tests has been removed in rocky cycle, no need to act on this anymore.

Changed in glance:
status: In Progress → Invalid
Changed in glance:
status: Invalid → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.