no need for a ternary

amtrivedi91 · Aug 30, 2016 · 5022585 · 5022585
1 parent a4a1267
commit 5022585
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/powerline_shell_base.py b/powerline_shell_base.py
@@ -118,7 +118,7 @@ def dirty(self):
             self.staged,
             self.conflicted,
         ]
-        return (True if sum(qualifiers) > 0 else False)
+        return sum(qualifiers) > 0
 
     def __getitem__(self, _key):
         return getattr(self, _key)

diff --git a/test/repo_stats_test.py b/test/repo_stats_test.py
@@ -9,6 +9,9 @@ def setUp(self):
         self.repo_stats.not_staged = 1
         self.repo_stats.conflicted = 4
 
+    def test_dirty(self):
+        self.assertTrue(self.repo_stats.dirty)
+
     def test_simple(self):
         self.assertEqual(self.repo_stats.untracked, 0)