Rename method to 'delta.update()' #82 #83

Signed-off-by: John M. Horan <johnmhoran@gmail.com>

Author: johnmhoran

src/deltacode/__init__.py

@@ -199,18 +199,18 @@ def license_diff(self):
                 old_licenses = delta.old_file.licenses or []
 
                 if len(delta.new_file.licenses) > 0 and delta.old_file.licenses == []:
-                    delta.add_score(20, 'license info added')
+                    delta.update(20, 'license info added')
                     return
 
                 if delta.new_file.licenses == [] and len(delta.old_file.licenses) > 0:
-                    delta.add_score(15, 'license info removed')
+                    delta.update(15, 'license info removed')
                     return
 
                 new_keys = set(license.key for license in new_licenses)
                 old_keys = set(license.key for license in old_licenses)
 
                 if new_keys != old_keys:
-                    delta.add_score(10, 'license change')
+                    delta.update(10, 'license change')
 
     def copyright_diff(self):
         """
@@ -228,10 +228,10 @@ def copyright_diff(self):
                 old_copyrights = delta.old_file.copyrights or []
 
                 if len(delta.new_file.copyrights) > 0 and delta.old_file.copyrights == []:
-                    delta.add_score(10, 'copyright info added')
+                    delta.update(10, 'copyright info added')
                     return
                 elif delta.new_file.copyrights == [] and len(delta.old_file.copyrights) > 0:
-                    delta.add_score(10, 'copyright info removed')
+                    delta.update(10, 'copyright info removed')
                     return
 
                 new_statements = set(statement for copyright in new_copyrights for statement in copyright.statements)
@@ -242,7 +242,7 @@ def copyright_diff(self):
 
                 if ((new_statements != old_statements) or
                         (new_holders != old_holders)):
-                    delta.add_score(5, 'copyright change')
+                    delta.update(5, 'copyright change')
 
     def index_deltas(self, index_key='path', delta_list=[]):
         """
@@ -280,12 +280,11 @@ def __init__(self, score=0, new_file=None, old_file=None):
         self.factors = []
         self.score = score
 
-    def add_score(self, score=0, factor=''):
+    def update(self, score=0, factor=''):
         """
-        For each Delta object identified in DeltaCode.license_diff() or
-        DeltaCode.copyright_diff(), add the score to the object's 'score'
-        attribute and add a string, summarizing the factor associated with the
-        score, to the object's 'factors' attribute (a list).
+        Add the score to the Delta object's 'score' attribute and add a string,
+        summarizing the factor associated with the score, to the object's
+        'factors' attribute (a list).
         """
         self.factors.append(factor)
         self.score += score

tests/test_deltacode.py

@@ -1661,7 +1661,7 @@ def test_Delta_add_score_added(self):
 
         delta = deltacode.Delta(100, new, None)
 
-        delta.add_score(25, 'This is a test of an added file')
+        delta.update(25, 'This is a test of an added file')
 
         assert delta.score == 125
         assert delta.factors == ['This is a test of an added file']
@@ -1686,7 +1686,7 @@ def test_Delta_add_score_modified(self):
 
         delta = deltacode.Delta(20, new, old)
 
-        delta.add_score(25, 'This is a test of a modified file')
+        delta.update(25, 'This is a test of a modified file')
 
         assert delta.score == 45
         assert delta.factors == ['This is a test of a modified file']
@@ -1708,7 +1708,7 @@ def test_Delta_add_score_license_change_no_copyright_change(self):
 
         for d in deltas_object:
             if d.new_file.path == 'path.txt':
-                d.add_score(25, 'This is a test of a license change')
+                d.update(25, 'This is a test of a license change')
 
         assert [d.score for d in deltas_object if d.new_file.path == 'path.txt'] == [55]
         assert [d.factors for d in deltas_object if d.new_file.path == 'path.txt'].pop() == ['modified', 'license change', 'This is a test of a license change']