watcher

watcher/tests are omitted twice

Bug #1552617 reported by Gábor Antal on 2016-03-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
​ watcher
Fix Released
Low
Gábor Antal
watcher mitaka-3 "m3"
Also affects project (?) Also affects distribution/package Nominate for series

Bug Description

watcher/tests is omitted twice, once in tox.ini, once in .coveragerc

Gábor Antal (gabor.antal) on 2016-03-03
Changed in watcher:
assignee: nobody → Gábor Antal (gabor.antal)
OpenStack Infra (hudson-openstack) wrote : Fix proposed to watcher (master)

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

Changed in watcher:
status: New → In Progress
Jean-Emile DARTOIS (jed56) on 2016-03-03
tags: added: low-hanging-fruit
Changed in watcher:
importance: Undecided → Low
milestone: none → mitaka-3
OpenStack Infra (hudson-openstack) wrote : Fix merged to watcher (master)

Reviewed: https://review.openstack.org/287658
Committed: https://git.openstack.org/cgit/openstack/watcher/commit/?id=63fffeacd8c6faf99448ea0d5914af97977e07ce
Submitter: Jenkins
Branch: master

commit 63fffeacd8c6faf99448ea0d5914af97977e07ce
Author: Gábor Antal <email address hidden>
Date: Thu Mar 3 09:45:41 2016 +0100

    Remove tests omission from coverage target in tox.ini

    In coverage target in tox.ini, there is the following argument:
      --omit="watcher/tests/*"

    However, in .coveragerc, the tests are also omitted,
    according to line 4 in .coveragerc:
      omit = watcher/tests/*

    So the watcher/tests/* directory is omitted twice. As it can be
    seen in other modules (e.g.: swift, nova) omitting only in
    .coveragerc should be enough.

    Change-Id: I72951196a346fb73a90c998138fc91dd171432cd
    Closes-Bug: #1552617

Changed in watcher:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.
Subscribing...

Other bug subscribers