Skip to content

Commit

Permalink
Avoid circular imports (#32)
Browse files Browse the repository at this point in the history
  • Loading branch information
hluk committed Dec 8, 2022
1 parent fbd29ef commit 7ee88b4
Show file tree
Hide file tree
Showing 10 changed files with 14 additions and 20 deletions.
15 changes: 5 additions & 10 deletions resultsdb/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,10 @@
import logging.config as logging_config
import os

from resultsdb import proxy
from resultsdb.proxy import ReverseProxied
from resultsdb.controllers.main import main
from resultsdb.controllers.api_v2 import api as api_v2
from resultsdb.controllers.api_v3 import api as api_v3, oidc
from resultsdb.models import db
from . import config

Expand All @@ -44,7 +47,7 @@ def create_app(config_obj=None):
app.secret_key = "replace-me-with-something-random"

# make sure app behaves when behind a proxy
app.wsgi_app = proxy.ReverseProxied(app.wsgi_app)
app.wsgi_app = ReverseProxied(app.wsgi_app)

# Expose the __version__ variable in templates
app.jinja_env.globals["app_version"] = __version__
Expand Down Expand Up @@ -161,16 +164,8 @@ def not_found(error):


def register_blueprints(app):
from resultsdb.controllers.main import main # noqa: E402

app.register_blueprint(main)

from resultsdb.controllers.api_v2 import api as api_v2 # noqa: E402

app.register_blueprint(api_v2, url_prefix="/api/v2.0")

from resultsdb.controllers.api_v3 import api as api_v3, oidc # noqa: E402

app.register_blueprint(api_v3, url_prefix="/api/v3")

if app.config["AUTH_MODULE"] == "oidc":
Expand Down
3 changes: 2 additions & 1 deletion resultsdb/__main__.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,8 @@
from alembic.migration import MigrationContext
from flask.cli import FlaskGroup

from resultsdb import create_app, db
from resultsdb import create_app
from resultsdb.models import db
from resultsdb.models.results import Group, Testcase, Result, ResultData

from sqlalchemy.engine import reflection
Expand Down
2 changes: 1 addition & 1 deletion resultsdb/alembic/env.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
from flask import current_app as app
from sqlalchemy import engine_from_config, pool

from resultsdb import db
from resultsdb.models import db

# add '.' to the pythonpath to support migration inside development env
import sys
Expand Down
2 changes: 1 addition & 1 deletion resultsdb/controllers/api_v2.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@

from sqlalchemy.orm import exc as orm_exc

from resultsdb import db
from resultsdb.models import db
from resultsdb.controllers.common import commit_result, SERIALIZE
from resultsdb.parsers.api_v2 import (
CreateGroupParams,
Expand Down
2 changes: 1 addition & 1 deletion resultsdb/controllers/api_v3.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from flask_oidc import OpenIDConnect
from flask_pydantic import validate

from resultsdb import db
from resultsdb.models import db
from resultsdb.authorization import match_testcase_permissions, verify_authorization
from resultsdb.controllers.common import commit_result
from resultsdb.models.results import (
Expand Down
2 changes: 1 addition & 1 deletion resultsdb/controllers/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
from flask import jsonify
from flask import current_app as app

from resultsdb import db
from resultsdb.models import db
from resultsdb.messaging import (
load_messaging_plugin,
create_message,
Expand Down
2 changes: 1 addition & 1 deletion resultsdb/messaging.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
import pkg_resources
import stomp

from resultsdb import db
from resultsdb.models import db
from resultsdb.models.results import Result, ResultData
from resultsdb.serializers.api_v2 import Serializer

Expand Down
4 changes: 2 additions & 2 deletions runapp.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@
# Ralph Bean <rbean@redhat.com>


import resultsdb
from resultsdb import create_app

if __name__ == '__main__':
app = resultsdb.create_app()
app = create_app()
app.run(
host=app.config['HOST'],
port=app.config['PORT'],
Expand Down
1 change: 0 additions & 1 deletion testing/functest_api_v20.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@

from flask import current_app as app

import resultsdb
import resultsdb.messaging
from resultsdb.models import db

Expand Down
1 change: 0 additions & 1 deletion testing/functest_create_fedmsg.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@

from flask import current_app as app

import resultsdb
import resultsdb.messaging
from resultsdb.models import db

Expand Down

0 comments on commit 7ee88b4

Please sign in to comment.