Fixes #22: Ensure --force option is handed on

2 files changed, 12 insertions, 1 deletions

diff --git a/dotfiles/cli.py b/dotfiles/cli.py
index c0984d6..a071240 100644
--- a/dotfiles/cli.py
+++ b/dotfiles/cli.py
@@ -177,8 +177,10 @@ def parse_config(config_file):
 def dispatch(dotfiles, action, force, args):
     if action in ['list', 'check']:
         getattr(dotfiles, action)()
-    elif action in ['add', 'remove', 'sync']:
+    elif action in ['add', 'remove']:
         getattr(dotfiles, action)(args)
+    elif action == 'sync':
+        getattr(dotfiles, action)(files=args, force=force)
     elif action == 'move':
         if len(args) > 1:
             print("Error: Move cannot handle multiple targets.")
diff --git a/test_dotfiles.py b/test_dotfiles.py
index b5521d2..761496e 100755
--- a/test_dotfiles.py
+++ b/test_dotfiles.py
@@ -9,6 +9,7 @@ import tempfile
 import unittest
 
 from dotfiles import core
+from dotfiles import cli
 from dotfiles.utils import is_link_to
 
 
@@ -62,6 +63,14 @@ class DotfilesTestCase(unittest.TestCase):
                 os.path.join(self.homedir, '.lastpass'),
                 '/tmp')
 
+    def test_dispatch(self):
+        """Test that the force option is handed on to the sync method."""
+        class MockDotfiles(object):
+            def sync(self, files=None, force=False):
+                assert bool(force)
+        dotfiles = MockDotfiles()
+        cli.dispatch(dotfiles, 'sync', True, [])
+
     def test_move_repository(self):
         """Test the move() method for a Dotfiles repository."""