-
Notifications
You must be signed in to change notification settings - Fork 0
database integration. using id from database of OGW-Microservice's da… #92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: next
Are you sure you want to change the base?
Conversation
…tabase rm filename from register json schema new functions for data_file_path get_data_info, get_data_file_path, load_data
…ta_helpers. Is test_data_helpers needed ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
mypy
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
OpenGeodeWeb-Viewer/src/tests/conftest.py
Line 127 in 4ce8770
def _init_ws(self): |
🚫 [mypy] reported by reviewdog 🐶
Call to untyped function "call" in typed context [no-untyped-call]
OpenGeodeWeb-Viewer/src/tests/conftest.py
Line 137 in 4ce8770
self.call("viewport.image.push.observer.add", [-1]) |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
OpenGeodeWeb-Viewer/src/tests/conftest.py
Line 139 in 4ce8770
def _drain_initial_messages(self, max_messages: int = 5, timeout: float = 4.0): |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a type annotation [no-untyped-def]
OpenGeodeWeb-Viewer/src/tests/conftest.py
Line 200 in 4ce8770
def _seed_database(session): |
🚫 [mypy] reported by reviewdog 🐶
Call to untyped function "_seed_database" in typed context [no-untyped-call]
OpenGeodeWeb-Viewer/src/tests/conftest.py
Line 295 in 4ce8770
_seed_database(session) |
🚫 [mypy] reported by reviewdog 🐶
Untyped decorator makes function "dataset_factory" untyped [misc]
OpenGeodeWeb-Viewer/src/tests/conftest.py
Line 307 in 4ce8770
@pytest.fixture |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
OpenGeodeWeb-Viewer/src/tests/conftest.py
Line 327 in 4ce8770
def _copy_asset(fname: str): |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
def __init__(self): |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
def initialize(self, database_path: str): |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
def get_session(self): |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a return type annotation [no-untyped-def]
def close(self): |
🚫 [mypy] reported by reviewdog 🐶
Call to untyped function "DatabaseManager" in typed context [no-untyped-call]
db_manager = DatabaseManager() |
🚫 [mypy] reported by reviewdog 🐶
Module "src.opengeodeweb_viewer.database.connection" does not explicitly export attribute "Data" [attr-defined]
from .database.connection import db_manager, Data |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a type annotation [no-untyped-def]
def get_data(self, data_id): |
🚫 [mypy] reported by reviewdog 🐶
Call to untyped function "get_session" in typed context [no-untyped-call]
session = db_manager.get_session() |
🚫 [mypy] reported by reviewdog 🐶
Function is missing a type annotation [no-untyped-def]
def get_data_file_path(self, data_id, filename=None): |
🚫 [mypy] reported by reviewdog 🐶
Call to untyped function "get_data" in typed context [no-untyped-call]
data = self.get_data(data_id) |
🚫 [mypy] reported by reviewdog 🐶
Argument 1 to "join" has incompatible type "str | None"; expected "str" [arg-type]
return os.path.join(self.DATA_FOLDER_PATH, data_id, filename) |
🚫 [mypy] reported by reviewdog 🐶
Call to untyped function "get_data_base" in typed context [no-untyped-call]
if id in self.get_data_base(): |
🚫 [mypy] reported by reviewdog 🐶
Call to untyped function "get_data_base" in typed context [no-untyped-call]
del self.get_data_base()[id] |
missing viewer_object = self.get_viewer_object_type(params["id"])
if Data is None: | ||
raise Exception("Data model not available") | ||
|
||
with get_session() as session: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [mypy] reported by reviewdog 🐶
"scoped_session[Session]" has no attribute "enter"; maybe "iter"? [attr-defined]
if Data is None: | ||
raise Exception("Data model not available") | ||
|
||
with get_session() as session: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [mypy] reported by reviewdog 🐶
"scoped_session[Session]" has no attribute "exit" [attr-defined]
…tabase
rm filename from register json schema
new functions for data_file_path get_data_info, get_data_file_path, load_data