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.

Sign up for GitHub

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

Jump to bottom

Multi revert #23

Open

comexpressao wants to merge 3 commits into master from multi_revert

Contributor

comexpressao commented Oct 18, 2016

1. prerendering_spec was failing in Chromium

Updated tests so that they differentiate from each other

2. When changing multiple attributes on a record and then reverting; it only reverts every other attribute

Added a test to change multiple attributes
Changes revert in base.rb to use a dup of @changed_attributes (when attributes are synced they are taken out of @changed_attributes which was causing the wrong attribute to be selected and therefore missing the revert of every other one).

comexpressao added 3 commits

October 18, 2016 15:10


          Updates prerendering_spec to distinguish between tests

62e03e4


          Adds to revert_record_spec to revert multiple attributes

dcda3d3


          Updates prerendering_spec to distinguish between tests

ed8d6bb

comexpressao assigned catmando

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet