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

Avoid MySQL integrity constraint violation #62

Conversation

vernard
Copy link
Contributor

@vernard vernard commented Jul 6, 2014

I followed the sample in README and on my second run of the test, it showed this error.

Unable to create the Magento fixture SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '1-1' for key 'UNQ_CATALOGINVENTORY_STOCK_ITEM_PRODUCT_ID_STOCK_ID'

I followed the solution in this page (http://www.magentocommerce.com/boards/viewthread/300959/)

@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@vernard
Copy link
Contributor Author

vernard commented Jul 6, 2014

This is my first PR and I'm quite sure I did something wrong. Please tell me what I did wrong and how to improve it.

@jamescowie
Copy link
Member

Hello @vernard the reason the build failed ( red cross ) next to your pull request is because you modified the behaviour of the code without updating the specs. We use PHPSpec against most of the code. You should be able to click through into Travis to get the error. Alternatively if you run composer install and run PHPSpec you will see the issue locally. If you have any issues please let us know.

@vernard
Copy link
Contributor Author

vernard commented Jul 7, 2014

I have an issue. The error report. Is there any other way to fix this error(Except for my PR)?

@jamescowie
Copy link
Member

Hello @vernard your PR fixes the issue I tested last night. However we need to update the PHPSpecs in order to keep the build green. If I have some more free time tonight I will have a look and add them to your PR so we can merge it in.

@vernard
Copy link
Contributor Author

vernard commented Jul 8, 2014

Awesome. Thanks.

@jamescowie jamescowie merged commit e7876c0 into MageTest:develop Nov 17, 2014
jamescowie pushed a commit that referenced this pull request Nov 17, 2014
…aint

Avoid MySQL integrity constraint violation (updated tests for #62)
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

3 participants