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

sale_validity: backport 8.0 tests to 7.0 #36

Merged
merged 1 commit into from Sep 19, 2014
Merged

Conversation

gurneyalex
Copy link
Member

No description provided.

@pedrobaeza
Copy link
Member

There is something different in v7, because test is failing:

2014-09-08 10:18:16,757 23281 ERROR openerp_test openerp.tools.yaml_import: "Field '_onchange_spec' does not exist in object 'browse_record(sale.order, 10)'"

@coveralls
Copy link

Coverage Status

Coverage increased (+0.86%) when pulling 0074bf2 on gurneyalex:7.0 into f35e51b on OCA:7.0.

@gurneyalex
Copy link
Member Author

Travis is red because of OCA/OCB#37

@pedrobaeza
Copy link
Member

OK, the previous error has gone (I don't know why - I suppose because of maintainer quality tools change..).

👍

@gurneyalex
Copy link
Member Author

@pedrobaeza it's gone because I used git commit --amend and git push -f so that my various failed attempt would not clutter the commit tree 😸

@pedrobaeza
Copy link
Member

Ah, that's the trick 😉
Now, let's wait if we can merge soon the patch for OCB tests.

@eLBati
Copy link
Member

eLBati commented Sep 19, 2014

👍

pedrobaeza added a commit that referenced this pull request Sep 19, 2014
sale_validity: backport 8.0 tests to 7.0
@pedrobaeza pedrobaeza merged commit e5ae834 into OCA:7.0 Sep 19, 2014
@gurneyalex gurneyalex deleted the 7.0 branch September 25, 2014 11:51
lmignon pushed a commit to acsone/sale-workflow that referenced this pull request Feb 11, 2021
Signed-off-by jgrandguillaume
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants