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

[FIX] payment_stripe: disable test 30 #39740

Closed
wants to merge 1 commit into from

Conversation

Julien00859
Copy link
Member

The test suite is an HttpCase so it must be started post install, as it
is started post install and payment_stripe_sca is automatically
installed. The test 30 is incorrect.

The test suite is an HttpCase so it must be started post install, as it
is started post install and `payment_stripe_sca` is automatically
installed. The test 30 is incorrect.
@Julien00859
Copy link
Member Author

should be forward to 12.3 too

@C3POdoo C3POdoo added the RD research & development, internal work label Nov 4, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Nov 4, 2019
@bouvyd
Copy link
Contributor

bouvyd commented Nov 4, 2019

@fw-bot up to saas-12.3

@fw-bot
Copy link
Contributor

fw-bot commented Nov 4, 2019

Forward-porting to 'saas-12.3'.

@bouvyd
Copy link
Contributor

bouvyd commented Nov 4, 2019

@robodoo r+

@robodoo robodoo added the r+ 👌 label Nov 4, 2019
@bouvyd
Copy link
Contributor

bouvyd commented Nov 4, 2019

@Julien00859 Thanks for trying 👍
you can't always get what you want

@Julien00859
Copy link
Member Author

robodoo pushed a commit that referenced this pull request Nov 4, 2019
The test suite is an HttpCase so it must be started post install, as it
is started post install and `payment_stripe_sca` is automatically
installed. The test 30 is incorrect.

closes #39740

Signed-off-by: Damien Bouvy (dbo) <dbo@odoo.com>
@robodoo
Copy link
Contributor

robodoo commented Nov 4, 2019

Merged at 6dba945, thanks!

@robodoo robodoo closed this Nov 4, 2019
@Julien00859 Julien00859 deleted the 12.0-r675511-stripe_30-juc branch November 5, 2019 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants