Changeset 63

Timestamp:

Feb 23, 2010 10:31:16 PM (15 years ago)

Author:

retracile

Message:

Mergebot: add testcases to cover property related delete/modify conflicts

File:

• mergebot/trunk/utils/test.py (modified) (5 diffs)

mergebot/trunk/utils/test.py

@@ -256 +256 @@
         self.assertEqual(retval, 0, "svn prposet failed with error %s" % (retval))
         
+    def propdel(self, propname, filename):
+        retval = call(['svn', 'propdel', propname, filename],
+            cwd=self.get_workdir(),
+            stdout=logfile, stderr=logfile)
+        self.assertEqual(retval, 0, "svn prposet failed with error %s" % (retval))
 
 
@@ -397 +402 @@
 
 
+class MergeBotTestMergeWithPropertyBranchDeleteConflict(FunctionalSvnTestCaseSetup):
+    def runTest(self):
+        """Verify that the 'merge' button detects property deleted on branch and modified on trunk"""
+        ticket_id = self._tester.create_ticket(summary=self.__class__.__name__,
+            info={'component':'stuff', 'version':'trunk'})
+        basename = self.__class__.__name__
+
+        self.checkout()
+        self.propset('svn:ignore', basename, '.')
+        self.commit('set property on trunk')
+
+        # create the branch
+        self._tester.branch(ticket_id, 'stuff')
+
+        # modify the property on trunk
+        self.propset('svn:ignore', basename + '\ntrunk', '.')
+        self.commit('Modify the property on trunk')
+
+        # delete the property on the branch
+        self.switch(ticket_id)
+        self.propdel('svn:ignore', '.')
+        self.commit('Delete the property on branch')
+
+        # merge, make sure it shows a conflict
+        self._tester.merge_conflict(ticket_id, 'stuff')
+
+
+class MergeBotTestMergeWithPropertyTrunkDeleteConflict(FunctionalSvnTestCaseSetup):
+    def runTest(self):
+        """Verify that the 'merge' button detects property deleted on trunk and modified on branch"""
+        ticket_id = self._tester.create_ticket(summary=self.__class__.__name__,
+            info={'component':'stuff', 'version':'trunk'})
+        basename = self.__class__.__name__
+
+        self.checkout()
+        self.propset('svn:ignore', basename, '.')
+        self.commit('set property on trunk')
+
+        # create the branch
+        self._tester.branch(ticket_id, 'stuff')
+
+        # delete the property on trunk
+        self.propdel('svn:ignore', '.')
+        self.commit('Delete the property on trunk')
+
+        # delete the property on the branch
+        self.switch(ticket_id)
+        self.propset('svn:ignore', basename + '\nbranch', '.')
+        self.commit('Modify the property on branch')
+
+        # merge, make sure it shows a conflict
+        self._tester.merge_conflict(ticket_id, 'stuff')
+
+
 class MergeBotTestMergeWithBranchRenameConflict(FunctionalSvnTestCaseSetup):
     def runTest(self):
@@ -544 +603 @@
 
         # modify the property on the branch
+        self.switch(ticket_id)
+        self.propset('svn:ignore', basename + '\nbranch', '.')
+        self.commit('Modify the property on branch')
+
+        # rebranch, make sure it shows a conflict
+        self._tester.rebranch_conflict(ticket_id, 'stuff')
+
+
+class MergeBotTestRebranchWithPropertyBranchDeleteConflict(FunctionalSvnTestCaseSetup):
+    def runTest(self):
+        """Verify that the 'rebranch' button detects property deleted on branch and modified on trunk"""
+        ticket_id = self._tester.create_ticket(summary=self.__class__.__name__,
+            info={'component':'stuff', 'version':'trunk'})
+        basename = self.__class__.__name__
+
+        self.checkout()
+        self.propset('svn:ignore', basename, '.')
+        self.commit('set property on trunk')
+
+        # create the branch
+        self._tester.branch(ticket_id, 'stuff')
+
+        # modify the property on trunk
+        self.propset('svn:ignore', basename + '\ntrunk', '.')
+        self.commit('Modify the property on trunk')
+
+        # delete the property on the branch
+        self.switch(ticket_id)
+        self.propdel('svn:ignore', '.')
+        self.commit('Delete the property on branch')
+
+        # rebranch, make sure it shows a conflict
+        self._tester.rebranch_conflict(ticket_id, 'stuff')
+
+
+class MergeBotTestRebranchWithPropertyTrunkDeleteConflict(FunctionalSvnTestCaseSetup):
+    def runTest(self):
+        """Verify that the 'rebranch' button detects property deleted on trunk and modified on branch"""
+        ticket_id = self._tester.create_ticket(summary=self.__class__.__name__,
+            info={'component':'stuff', 'version':'trunk'})
+        basename = self.__class__.__name__
+
+        self.checkout()
+        self.propset('svn:ignore', basename, '.')
+        self.commit('set property on trunk')
+
+        # create the branch
+        self._tester.branch(ticket_id, 'stuff')
+
+        # delete the property on trunk
+        self.propdel('svn:ignore', '.')
+        self.commit('Delete the property on trunk')
+
+        # delete the property on the branch
         self.switch(ticket_id)
         self.propset('svn:ignore', basename + '\nbranch', '.')
@@ -651 +764 @@
     suite.addTest(MergeBotTestRebranchWithBranchRenameConflict())
     suite.addTest(MergeBotTestRebranchWithTrunkRenameConflict())
+    suite.addTest(MergeBotTestRebranchWithPropertyBranchDeleteConflict())
+    suite.addTest(MergeBotTestRebranchWithPropertyTrunkDeleteConflict())
     suite.addTest(MergeBotTestMergeWithChange())
     suite.addTest(MergeBotTestMergeWithChangeAndTrunkChange())
@@ -657 +772 @@
     suite.addTest(MergeBotTestMergeWithBranchRenameConflict())
     suite.addTest(MergeBotTestMergeWithTrunkRenameConflict())
+    suite.addTest(MergeBotTestMergeWithPropertyBranchDeleteConflict())
+    suite.addTest(MergeBotTestMergeWithPropertyTrunkDeleteConflict())
     suite.addTest(MergeBotTestSingleUseCase())
     return suite