Skip to content
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

[MOV] test_*: move to a specific addons-path #117315

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mart-e
Copy link
Contributor

@mart-e mart-e commented Mar 31, 2023

There is no reason to have these modules to be mixed with functionnal addons. The addons path can be modified when wanting to test them. Same logic for hw_* modules that are only needed for the IoTbox

@mart-e mart-e requested a review from Xavier-Do March 31, 2023 09:40
@robodoo
Copy link
Contributor

robodoo commented Mar 31, 2023

@C3POdoo C3POdoo added the RD research & development, internal work label Mar 31, 2023
@mart-e mart-e force-pushed the master-move-addons-path-mat branch from 8a694aa to b381ca6 Compare March 31, 2023 10:17
@mart-e mart-e requested a review from a team March 31, 2023 10:24
@Xavier-Do
Copy link
Contributor

@robodoo override=ci/style

Codeowner was also disabled on this bundle

@mart-e mart-e marked this pull request as ready for review March 31, 2023 13:10
@mart-e
Copy link
Contributor Author

mart-e commented Mar 31, 2023

@robodoo override=ci/security
Reason: too many moved files

@mart-e mart-e force-pushed the master-move-addons-path-mat branch from b381ca6 to edede57 Compare April 4, 2023 09:30
@KangOl
Copy link
Contributor

KangOl commented Apr 4, 2023

In my attempt1 to isolate the test addons, I had to make more changes to avoid dependencies between the business and test addons.

Footnotes

  1. https://github.com/odoo/odoo/compare/master...odoo-dev:odoo:master-test-addons-path-chs

@Xavier-Do
Copy link
Contributor

In my attempt1 to isolate the test addons, I had to make more changes to avoid dependencies between the business and test addons.

We actually tests with all addons so we won't detect such issues for the moment. It would be a good idea to create a config to test that on runbot, maybe as a nightly build.

@mart-e mart-e force-pushed the master-move-addons-path-mat branch from edede57 to 8e01240 Compare April 6, 2023 06:44
@mart-e mart-e force-pushed the master-move-addons-path-mat branch from 8e01240 to d4d1299 Compare April 6, 2023 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants