]> git.madduck.net Git - etc/taskwarrior.git/commitdiff

madduck's git repository

Every one of the projects in this repository is available at the canonical URL git://git.madduck.net/madduck/pub/<projectpath> — see each project's metadata for the exact URL.

All patches and comments are welcome. Please squash your changes to logical commits before using git-format-patch and git-send-email to patches@git.madduck.net. If you'd read over the Git project's submission guidelines and adhered to them, I'd be especially grateful.

SSH access, as well as push access can be individually arranged.

If you use my repositories frequently, consider adding the following snippet to ~/.gitconfig and using the third clone URL listed for each project:

[url "git://git.madduck.net/madduck/"]
  insteadOf = madduck:

Add (failing) tests for appending/adding to empty lists/sets
authorRob Golding <rob@robgolding.com>
Fri, 9 Jan 2015 11:22:19 +0000 (11:22 +0000)
committerRob Golding <rob@robgolding.com>
Fri, 9 Jan 2015 11:22:19 +0000 (11:22 +0000)
tasklib/task.py
tasklib/tests.py

index b56936e77d17d949639ca9994602194d84c5a0c3..d5913af9adda62e66f05bb7931410c94a0000f37 100644 (file)
@@ -95,7 +95,7 @@ class SerializingObject(object):
     def deserialize_tags(self, tags):
         if isinstance(tags, basestring):
             return tags.split(',') if tags else []
     def deserialize_tags(self, tags):
         if isinstance(tags, basestring):
             return tags.split(',') if tags else []
-        return tags
+        return tags or []
 
     def serialize_depends(self, cur_dependencies):
         # Return the list of uuids
 
     def serialize_depends(self, cur_dependencies):
         # Return the list of uuids
index f3edd61c56422fd1f53c2c39db36ec1cecdc7780..a608cf802a6969850cc277c12c48ddc5d3293d05 100644 (file)
@@ -245,6 +245,18 @@ class TaskTest(TasklibTest):
 
         self.assertEqual(t['depends'], set([dependency1, dependency2]))
 
 
         self.assertEqual(t['depends'], set([dependency1, dependency2]))
 
+    def test_add_to_empty_dependency_set(self):
+        # Adds dependency to task with one dependencies
+        t = Task(self.tw, description="test task")
+        dependency = Task(self.tw, description="needs to be done first")
+
+        dependency.save()
+
+        t['depends'].add(dependency)
+        t.save()
+
+        self.assertEqual(t['depends'], set([dependency]))
+
     def test_simple_dependency_set_save_repeatedly(self):
         # Adds only one dependency to task with no dependencies
         t = Task(self.tw, description="test task")
     def test_simple_dependency_set_save_repeatedly(self):
         # Adds only one dependency to task with no dependencies
         t = Task(self.tw, description="test task")
@@ -416,7 +428,6 @@ class TaskTest(TasklibTest):
     def test_saving_unmodified_task(self):
         t = Task(self.tw, description="test task")
         t.save()
     def test_saving_unmodified_task(self):
         t = Task(self.tw, description="test task")
         t.save()
-        t.refresh()
         t.save()
 
     def test_adding_tag_by_appending(self):
         t.save()
 
     def test_adding_tag_by_appending(self):
@@ -424,9 +435,15 @@ class TaskTest(TasklibTest):
         t.save()
         t['tags'].append('test2')
         t.save()
         t.save()
         t['tags'].append('test2')
         t.save()
-        t.refresh()
         self.assertEqual(t['tags'], ['test1', 'test2'])
 
         self.assertEqual(t['tags'], ['test1', 'test2'])
 
+    def test_adding_tag_by_appending_empty(self):
+        t = Task(self.tw, description="test task")
+        t.save()
+        t['tags'].append('test')
+        t.save()
+        self.assertEqual(t['tags'], ['test'])
+
 
 class AnnotationTest(TasklibTest):
 
 
 class AnnotationTest(TasklibTest):