From: Tomas Babej <tomasbabej@gmail.com>
Date: Sat, 26 Dec 2015 15:35:56 +0000 (+0100)
Subject: tests: Expand and fix tests with set-based tags
X-Git-Url: https://git.madduck.net/etc/taskwarrior.git/commitdiff_plain/3adcc3e2236cb92b6e2cb80a7cc28fb9c6803e31

tests: Expand and fix tests with set-based tags
---

diff --git a/tasklib/tests.py b/tasklib/tests.py
index 4f0b23f..874a047 100644
--- a/tasklib/tests.py
+++ b/tasklib/tests.py
@@ -696,16 +696,24 @@ class TaskTest(TasklibTest):
     def test_adding_tag_by_appending(self):
         t = Task(self.tw, description="test task", tags=['test1'])
         t.save()
-        t['tags'].append('test2')
+        t['tags'].add('test2')
         t.save()
-        self.assertEqual(t['tags'], ['test1', 'test2'])
+        self.assertEqual(t['tags'], set(['test1', 'test2']))
+
+    def test_adding_tag_twice(self):
+        t = Task(self.tw, description="test task", tags=['test1'])
+        t.save()
+        t['tags'].add('test2')
+        t['tags'].add('test2')
+        t.save()
+        self.assertEqual(t['tags'], set(['test1', 'test2']))
 
     def test_adding_tag_by_appending_empty(self):
         t = Task(self.tw, description="test task")
         t.save()
-        t['tags'].append('test')
+        t['tags'].add('test')
         t.save()
-        self.assertEqual(t['tags'], ['test'])
+        self.assertEqual(t['tags'], set(['test']))
 
     def test_serializers_returning_empty_string_for_none(self):
         # Test that any serializer returns '' when passed None