Vauxoo / maintainer-quality-tools

QA tools for Odoo maintainers
GNU Affero General Public License v3.0
3 stars 9 forks source link

[IMP] test_server: env var to add modules to preinstall #342

Closed luisg123v closed 2 years ago

luisg123v commented 2 years ago

This allows to specify modules to be preinstalled in the DB template using the environment variable PREINSTALL_MODULES. Modules specified there will be installed along with dependencies of the main app.

Dummy MR:

luisg123v commented 2 years ago

Hi @moylop260,

What do you think?

moylop260 commented 2 years ago

If I understood well it is to install in demo instance the l10n_cr_edi,l10n_mx modules

FYI in the future, we will migrate from DeployV-CD to DeployV-CI docker images and it will not work since that DeployV doesn't use MQT

Even the developer's local instances will not work using the normal installation Odoo way

Maybe you could try another workaround

I mean,

<odoo>
    <function model="MODEL" name="_install_modules(['l10n_cr_edi', 'l10n_mx'])" />
</odoo>

Or using a hook method to identify if the current module has demo=True

I mean,

if self.env.ref("base.module_l10n_edi_deactivate_sh").demo: 
    # code to install modules
luisg123v commented 2 years ago

@moylop260,

I think none of those would work, I remember trying to trigger a module installation from another module and it was not possible due to registry.

I took another approach, using variable

export INSTALL_OPTIONS="--init=l10n_cr_edi"

And it worked.

So, closing this PR.

Regards,