Skip to content

[14.0][FIX] auth_signup_verify_email: Use real email address for tests #388

Merged
OCA-git-bot merged 1 commit intoOCA:14.0from
Tecnativa:14.0-ci
Jun 12, 2022
Merged

[14.0][FIX] auth_signup_verify_email: Use real email address for tests #388
OCA-git-bot merged 1 commit intoOCA:14.0from
Tecnativa:14.0-ci

Conversation

@pedrobaeza
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza commented Jun 12, 2022

In one part of the CI pipeline, the IANA example domains are not accepted, getting this message from email_validator library:

The domain name example.com does not accept email

So we switch to a real domain for avoiding the problem, using our famous main OCA list address, which at the end, it's not secret and won't be disabled in a short future.

@Tecnativa

@pedrobaeza pedrobaeza added this to the 14.0 milestone Jun 12, 2022
@pedrobaeza pedrobaeza changed the title [14.0][WIP] auth_signup_verify_email: See CI problem [14.0][FIX] auth_signup_verify_email: Pin library for avoiding CI error Jun 12, 2022
@rvalyi
Copy link
Copy Markdown
Member

rvalyi commented Jun 12, 2022

@pedrobaeza the requirements.txt is now generated based on the modules __manifest__.py and I have seen no option to specify a package version in external_dependencies so I wonder what would be the best way to work around such issues. cc @sbidoul

@pedrobaeza pedrobaeza changed the title [14.0][FIX] auth_signup_verify_email: Pin library for avoiding CI error [14.0][WIP] auth_signup_verify_email: See CI error Jun 12, 2022
@pedrobaeza
Copy link
Copy Markdown
Member Author

I'm just trying some things. Pinning doesn't serve for this case, but if it would serve, the option to "consolidate" it is to put that version in the manifest in the external_dependencies part.

@pedrobaeza pedrobaeza force-pushed the 14.0-ci branch 4 times, most recently from bd09f0e to 167dafb Compare June 12, 2022 16:27
@sbidoul
Copy link
Copy Markdown
Member

sbidoul commented Jun 12, 2022

I have seen no option to specify a package version in external_dependencies so I wonder what would be the best way to work around such issues

I'm not sure what the problem is here, but to answer this specific question, I'm pretty sure you can put version specifiers in external_dependencies in Odoo >= 13.

In one part of the CI pipeline, the IANA example domains are not
accepted, getting this message from `email_validator` library:

The domain name example.com does not accept email

So we switch to a real domain for avoiding the problem, using our
famous main OCA list address, which at the end, it's not secret and
won't be disabled in a short future.
@pedrobaeza pedrobaeza changed the title [14.0][WIP] auth_signup_verify_email: See CI error [14.0][FIX] auth_signup_verify_email: Use real email address for tests Jun 12, 2022
@pedrobaeza
Copy link
Copy Markdown
Member Author

At the end, I have needed to switch to a real email address for passing the tests, as example domain is no longer valid in the CI environment (it was passing in local environment though), but it's not a big deal.

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-388-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e1c9995 into OCA:14.0 Jun 12, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at d715179. Thanks a lot for contributing to OCA. ❤️

@pedrobaeza pedrobaeza deleted the 14.0-ci branch June 12, 2022 17:04
SiesslPhillip pushed a commit to grueneerde/OCA-server-auth that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-auth (16.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants