Skip to content

Commit ac8cbed

Browse files
committed
missing imports
1 parent d8b786c commit ac8cbed

File tree

5 files changed

+19
-25
lines changed

5 files changed

+19
-25
lines changed

src/opengeodeweb_back/utils_functions.py

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,9 @@ def save_all_viewables_and_return_info(
193193
data_entry.viewable_file_name = os.path.basename(saved_viewable_file_path)
194194
data_entry.light_viewable = os.path.basename(saved_light_viewable_file_path)
195195

196-
database.session.commit()
196+
session = get_session()
197+
if session:
198+
session.commit()
197199

198200
return {
199201
"native_file_name": data_entry.native_file_name,
@@ -247,18 +249,18 @@ def generate_native_viewable_and_light_viewable_from_file(
247249

248250
data = geode_functions.load(geode_object, copied_full_path)
249251

250-
database.session.delete(temp_data_entry)
251-
database.session.flush()
252+
# Remplacer :
253+
# database.session.delete(temp_data_entry)
254+
# database.session.flush()
255+
# Par :
256+
session = get_session()
257+
if session:
258+
session.delete(temp_data_entry)
259+
session.flush()
252260

253261
return save_all_viewables_and_return_info(
254262
geode_object,
255263
data,
256264
input_file=input_filename,
257265
additional_files=additional_files_copied,
258266
)
259-
260-
# Modifier toutes les utilisations de database.session par :
261-
# database.session.add() -> get_session().session.add()
262-
# database.session.commit() -> get_session().session.commit()
263-
# database.session.delete() -> get_session().session.delete()
264-
# database.session.flush() -> get_session().session.flush()

tests/conftest.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
# Local application imports
1010
from app import app
11-
from src.opengeodeweb_back.database import initialize_database
11+
from opengeodeweb_microservice.database.connection import init_database
1212

1313
TEST_ID = "1"
1414

@@ -28,8 +28,7 @@ def copy_data():
2828
print("Current working directory:", os.getcwd())
2929
print("Directory contents:", os.listdir("."))
3030

31-
initialize_database(app)
32-
31+
init_database(app)
3332

3433
@pytest.fixture
3534
def client():

tests/test_models_routes.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,8 @@
33
import flask
44

55
from src.opengeodeweb_back import geode_functions
6-
from src.opengeodeweb_back.data import Data
7-
from src.opengeodeweb_back.database import database
8-
6+
from opengeodeweb_microservice.microservice.data import Data
7+
from opengeodeweb_microservice.database.connection import get_session
98

109
def test_model_mesh_components(client, test_id):
1110
route = f"/models/vtm_component_indices"
@@ -35,7 +34,6 @@ def test_extract_brep_uuids(client, test_id):
3534
with client.application.app_context():
3635
data_entry = Data.create(geode_object="BRep", input_file=brep_filename)
3736
data_entry.native_file_name = brep_filename
38-
database.session.commit()
3937

4038
data_path = geode_functions.data_file_path(data_entry.id, brep_filename)
4139
os.makedirs(os.path.dirname(data_path), exist_ok=True)

tests/test_routes.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@
77

88
# Local application imports
99
from src.opengeodeweb_back import geode_functions, test_utils
10-
from src.opengeodeweb_back.data import Data
11-
from src.opengeodeweb_back.database import database
1210

1311

1412
def test_allowed_files(client):
@@ -173,7 +171,6 @@ def test_texture_coordinates(client, test_id):
173171
with client.application.app_context():
174172
data = Data.create(geode_object="PolygonalSurface3D", input_file="hat.vtp")
175173
data.native_file_name = "hat.vtp"
176-
database.session.commit()
177174

178175
data_path = geode_functions.data_file_path(data.id, "hat.vtp")
179176
print(data_path)
@@ -197,7 +194,6 @@ def test_vertex_attribute_names(client, test_id):
197194
with client.application.app_context():
198195
data = Data.create(geode_object="PolygonalSurface3D", input_file="test.vtp")
199196
data.native_file_name = "test.vtp"
200-
database.session.commit()
201197

202198
data_path = geode_functions.data_file_path(data.id, "test.vtp")
203199
os.makedirs(os.path.dirname(data_path), exist_ok=True)
@@ -218,7 +214,6 @@ def test_polygon_attribute_names(client, test_id):
218214
with client.application.app_context():
219215
data = Data.create(geode_object="PolygonalSurface3D", input_file="test.vtp")
220216
data.native_file_name = "test.vtp"
221-
database.session.commit()
222217

223218
data_path = geode_functions.data_file_path(data.id, "test.vtp")
224219
os.makedirs(os.path.dirname(data_path), exist_ok=True)
@@ -238,7 +233,6 @@ def test_polyhedron_attribute_names(client, test_id):
238233
with client.application.app_context():
239234
data = Data.create(geode_object="PolyhedralSolid3D", input_file="test.vtu")
240235
data.native_file_name = "test.vtu"
241-
database.session.commit()
242236

243237
data_path = geode_functions.data_file_path(data.id, "test.vtu")
244238
os.makedirs(os.path.dirname(data_path), exist_ok=True)

tests/test_utils_functions.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@
88
import uuid
99

1010
# Local application imports
11-
from src.opengeodeweb_back.database import database
12-
from src.opengeodeweb_back.data import Data
11+
from opengeodeweb_microservice.microservice.data import Data
12+
from opengeodeweb_microservice.database.connection import get_session
1313
from src.opengeodeweb_back import geode_functions, utils_functions
1414

1515

@@ -143,7 +143,8 @@ def test_save_all_viewables_commits_to_db(client):
143143
db_entry_before = Data.get(data_id)
144144
assert db_entry_before is not None
145145
assert db_entry_before.native_file_name == result["native_file_name"]
146-
database.session.rollback()
146+
session = get_session()
147+
session.rollback()
147148
db_entry_after = Data.get(data_id)
148149
assert (
149150
db_entry_after is not None

0 commit comments

Comments
 (0)