Merge pull request #3 from eventbrite/fix-issue-2

Fix for Issue #2: Should not be passing "commit=True" to django model in...

src/rebar/group.py

@@ -213,8 +213,8 @@ def save(self):
             if isinstance(form, BaseForm):
                 form.save(commit=False)
 
-        # call save on the instance and commit
-        self.instance.save(commit=True)
+        # call save on the instance
+        self.instance.save()
 
         # call any post-commit hooks that have been stashed on Forms
         for form in self.forms:

src/rebar/tests/helpers.py

@@ -8,10 +8,8 @@ def __init__(self, **kwargs):
         self.id = None
         self.__dict__.update(kwargs)
 
-    def save(self, commit=False):
-
-        if commit:
-            self.id = 42
+    def save(self):
+        self.id = 42
 
 
 class CallSentinel(object):

src/rebar/tests/test_groups.py

@@ -548,7 +548,7 @@ def test_save_calls_instance_save_once(self):
 
         with patch.object(FakeModel, 'save', autospec=True) as save_mock:
 
-            save_mock.side_effect = lambda s, commit: setattr(s, 'id', 42)
+            save_mock.side_effect = lambda s: setattr(s, 'id', 42)
             form_data = {
                 'group-name-first_name': 'John',
                 'group-name-last_name': 'Doe',
@@ -561,7 +561,7 @@ def test_save_calls_instance_save_once(self):
 
             form_group.save()
 
-            save_mock.assert_called_once_with(ANY, commit=True)
+            save_mock.assert_called_once_with(ANY)
 
     def test_inline_formsets_saved_after_parent_saved(self):