Get rid of fake tasks where they are not required

Bug #1440671 reported by Aleksey Kasatkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
In Progress
Medium
Fuel Sustaining
Mitaka
Won't Fix
Medium
Unassigned
Newton
In Progress
Medium
Fuel Sustaining

Bug Description

A lot of our nailgun tests use fake tasks even to check simple things where just mock of rpc.cast is enough (or even not needed).
Let's keep simple things simple and get rid of fake tasks where they are not required.

Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
Changed in fuel:
status: New → Confirmed
tags: added: qa-agree-7.0
Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Does not break deployment. It touches Nailgun unit tests only. Tech debt. Moved to 8.0.

Changed in fuel:
status: Confirmed → Won't Fix
Dmitry Pyzhov (dpyzhov)
tags: added: tech-debt
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
status: Won't Fix → Confirmed
no longer affects: fuel/8.0.x
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

Moving medium bug to 9.0 because we have SCF in action for 8.0

Changed in fuel:
milestone: 8.0 → 9.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/314667
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=73032343bb8c4c5770fb43952d6561b283b45f12
Submitter: Jenkins
Branch: master

commit 73032343bb8c4c5770fb43952d6561b283b45f12
Author: Nikita Zubkov <email address hidden>
Date: Tue May 10 19:12:35 2016 +0300

    Get rid of fake tasks where they are not required

    Replace fake_tasks with rpc mock for some tests. This patch covers not
    all files with tests, the fix will be done in series of few patches.

    Change-Id: I6d316a45974ea5518c32a78f069e3e88b167d1a4
    Partial-Bug: #1440671

Ilya Kharin (akscram)
Changed in fuel:
assignee: Nikita Zubkov (zubchick) → Fuel Sustaining (fuel-sustaining-team)
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.