import * is used in a lot of test files

Bug #1264724 reported by Sushil Kumar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
Undecided
Sushil Kumar

Bug Description

import * is used in a lot of test files.
This is suppressed by a flake8 ignore F403.

When we enable this flake8 rule, we start getting errors for import *

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to trove (master)

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

Changed in trove:
assignee: nobody → Sushil Kumar (sushil-kumar2)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove (master)

Reviewed: https://review.openstack.org/64342
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=167c0ceb7a49ba3a687883eff86074d15b917493
Submitter: Jenkins
Branch: master

commit 167c0ceb7a49ba3a687883eff86074d15b917493
Author: Sushil Kumar <email address hidden>
Date: Sat Dec 28 18:29:14 2013 +0000

    Enabled F403 flake8 rule

    Reasons:
    - F403 is disabled.
    - code contains "import *" statements.

    Changes:
    - Updates tox.ini to enable F403 rules.
    - Updates code for F403 violation.

    Change-Id: If932a910c073ae7bffb74908495f35dbc4f4d1f9
    Closes-Bug: #1264724

Changed in trove:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in trove:
milestone: none → icehouse-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in trove:
milestone: icehouse-2 → 2014.1
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.