plugin config aborts and disables when not enabled

features/lolcommits.feature

@@ -120,7 +120,7 @@ Feature: Basic UI functionality
     When I run `lolcommits --config --test -p loltext` interactively
       And I wait for output to contain "enabled:"
       Then I type "false"
-    Then the output should contain "Successfully configured plugin: loltext"
+      Then the output should contain "Disabling plugin: loltext - answer with 'true' to enable & configure"
     And a file named "~/.lolcommits/test/config.yml" should exist
     When I successfully run `lolcommits --test --show-config`
     Then the output should match /loltext:\s+enabled: false/

lib/lolcommits/configuration.rb

@@ -94,16 +94,15 @@ def do_configure!(plugin_name)
       config = read_configuration || {}
       plugin_name   = plugin.class.name
       plugin_config = plugin.configure_options!
-      # having a plugin_config, means configuring went OK
-      if plugin_config
-        # save plugin and print config
-        config[plugin_name] = plugin_config
-        save(config)
-        puts self
-        puts "\nSuccessfully configured plugin: #{plugin_name} at path '#{configuration_file}'"
-      else
-        puts "\nAborted plugin configuration for: #{plugin_name}"
-      end
+
+      # save plugin and print config if enabled
+      config[plugin_name] = plugin_config
+      save(config)
+
+      return unless plugin_config['enabled']
+
+      puts self
+      puts "\nSuccessfully configured plugin: #{plugin_name} at path '#{configuration_file}'"
     end
 
     def save(config)

lib/lolcommits/plugin/base.rb

@@ -45,13 +45,15 @@ def configure_options!
         options.reduce({}) do |acc, option|
           print "#{option}: "
           val = parse_user_input(gets.strip)
-          # check enabled option isn't a String
-          if (option == 'enabled') && ![true, false].include?(val)
-            puts "Aborting - please respond with 'true' or 'false'"
-            exit 1
-          else
-            acc.merge(option => val)
+          # check enabled option value, disable and abort config if not true
+          if option == 'enabled'
+            unless val == true
+              puts "Disabling plugin: #{self.class.name} - answer with 'true' to enable & configure"
+              break { option => false }
+            end
           end
+
+          acc.merge(option => val)
         end
       end