[IMP] allow containers to write in external filestore

odoo_openupgrade_wizard/tools_odoo.py

@@ -50,7 +50,7 @@ def get_docker_image_tag(ctx, odoo_version: dict) -> str:
 def get_docker_container_name(ctx, migration_step: dict) -> str:
     """Return a docker container name, based on project name,
     odoo release and migration step"""
-    return "odoo-openupgrade-wizard-container---%s---%s---step---%s" % (
+    return "odoo-openupgrade-wizard-container__%s__%s__step-%s" % (
         ctx.obj["config"]["project_name"],
         str(migration_step["release"]).rjust(4, "0"),
         str(migration_step["name"]).rjust(2, "0"),
@@ -94,7 +94,7 @@ def generate_odoo_command(
         f" --db_password odoo"
         f" --workers 0"
         f" --config /odoo_env/odoo.cfg"
-        # f" --data-dir /env/filestore/"
+        f" --data-dir /env/filestore/"
         f" --logfile {log_file}"
         f" --addons-path {addons_path}"
         f" {database_cmd}"

tests/cli_B_03_run_test.py

@@ -20,12 +20,11 @@ def test_cli_run():
         ]
     )
 
-    # # Ensure that a subfolder filestore/DB_NAME has been created
-    # db_filestore_path = output_folder_path / Path(
-    #     "./filestore/filestore/%s" % db_name
-    # )
-
-    # assert db_filestore_path.exists()
+    # Ensure that a subfolder filestore/DB_NAME has been created
+    db_filestore_path = output_folder_path / Path(
+        "./filestore/filestore/%s" % db_name
+    )
+    assert db_filestore_path.exists()
 
     # Ensure that all the containers are removed
     docker_client = get_docker_client()