watch_tower 0.0.1.beta10 → 0.0.1.beta11

data/README.md

@@ -111,12 +111,24 @@ following guidelines:
   colon and a space, for example 'README: fix a typo' or 'Server/Project: Fix
   a typo'.
 - Include tests.
-- Do not change the version, We will take care of that.
+- __Do not change the version__, We will take care of that.
 
 You can also take a look at the [TODO
 list](https://github.com/TechnoGate/watch_tower/blob/master/TODO) for what's
 in mind for the project
 
+# Contact
+
+For bugs and feature request, please use __Github issues__, for other
+requests, you can contact us:
+
+- [Github private
+  message](https://github.com/inbox/new/eMxyzptlk)
+- Email: [contact@technogate.fr](mailto:contact@technogate.fr)
+
+Don't forget to follow me on [Github](https://github.com/eMxyzptlk) and
+[Twitter](https://twitter.com/eMxyzptlk) for news and updates.
+
 # Credits
 
 Please see the

data/lib/watch_tower.rb

@@ -41,7 +41,7 @@ module WatchTower
   LOG = Logger.new(File.join(LOG_PATH, "#{ENV['WATCH_TOWER_ENV']}.log"))
 
   # Make it by default warn level
-  LOG.level = Logger::WARN
+  LOG.level = Logger::INFO
 
   # Threads
   # Hash

data/lib/watch_tower/cli/start.rb

@@ -49,12 +49,24 @@ module WatchTower
             def start
               # Set the logger to debug mode if necessary
               LOG.level = Logger::DEBUG if options[:debug]
-              if Config[:enabled] &&
-                (!options[:bootloader] || (options[:bootloader] && Config[:launch_on_boot]))
-                LOG.info "Starting WatchTower."
-                start!
-              else
-                abort "You need to edit the config file located at #{Config::CONFIG_FILE}."
+
+              begin
+                if Config[:enabled] &&
+                  (!options[:bootloader] || (options[:bootloader] && Config[:launch_on_boot]))
+                  LOG.info "#{__FILE__}:#{__LINE__}: Starting WatchTower."
+                  start!
+                  LOG.info "#{__FILE__}:#{__LINE__}: WatchTower has finished."
+                else
+                  abort "You need to edit the config file located at \#{Config.config_file}."
+                end
+              rescue ConfigNotReadableError => e
+                LOG.fatal "#{__FILE__}:#{__LINE__}: The config file is not readable."
+                STDERR.puts "The config file is not readable, please make sure \#{Config.config_file} exists and you have the necessary permissions to read it."
+                exit(1)
+              rescue ConfigNotValidError => e
+                LOG.fatal "#{__FILE__}:#{__LINE__}: The config file is not valid: \#{e.message}."
+                STDERR.puts "The config file \#{Config.config_file} is not valid: \#{e.message}."
+                exit(1)
               end
             end
 
@@ -65,8 +77,6 @@ module WatchTower
 
                   # Start WatchTower
                   start_watch_tower
-
-                  LOG.debug "#{__FILE__}:#{__LINE__}: WatchTower has finished."
                 else
                   LOG.debug "#{__FILE__}:#{__LINE__}: Running WatchTower in the background."
                   pid = fork do
@@ -79,8 +89,6 @@ module WatchTower
 
                       # Start WatchTower
                       start_watch_tower
-
-                      LOG.debug "#{__FILE__}:#{__LINE__}: WatchTower has finished."
                     rescue => e
                       LOG.fatal "#{__FILE__}:#{__LINE__ - 2}: The process raised an exception \#{e.message}"
                       LOG.fatal "#{__FILE__}:#{__LINE__ - 3}: ==== Backtrace ===="

data/lib/watch_tower/config.rb

@@ -6,9 +6,6 @@ module WatchTower
   module Config
     extend self
 
-    # Define the config file's path
-    CONFIG_FILE = File.join(USER_PATH, 'config.yml')
-
     # Define the config class variable
     @@config = nil
 
@@ -18,23 +15,56 @@ module WatchTower
     # @return mixed
     # @raise [Void]
     def [](config)
-      ensure_config_file_exists
-      @@config ||= HashWithIndifferentAccess.new(YAML.parse_file(CONFIG_FILE).to_ruby)
+      if @@config.nil?
+        check_config_file
+        @@config ||= parse_config_file
+      end
+
       @@config[:watch_tower].send(:[], config)
     end
 
+    # Get the config file
+    #
+    # @return [String] Absolute path to the config file
+    def config_file
+      File.join(USER_PATH, 'config.yml')
+    end
+
     protected
-      # Ensures config file exists in the user config folder
+      # Initialize the configuration file
+      def initialize_config_file
+        File.open(config_file, 'w') do |f|
+          f.write(File.read(File.join(TEMPLATE_PATH, 'config.yml')))
+        end
+      end
+
+      # Check the config file
+      def check_config_file
+        # Check that config_file is defined
+        raise ConfigNotDefinedError unless config_file
+        # Check that the config file exists
+        initialize_config_file unless ::File.exists?(config_file)
+        # Check that the config file is readable?
+        raise ConfigNotReadableError unless ::File.readable?(config_file)
+      end
+
+      # Parse the config file
       #
-      # @param [Void]
-      # @return [Void]
-      # @raise [Void]
-      def ensure_config_file_exists
-        unless File.exists?(CONFIG_FILE)
-          File.open(CONFIG_FILE, 'w') do |f|
-            f.write(File.read(File.join(TEMPLATE_PATH, 'config.yml')))
-          end
+      # @return [HashWithIndifferentAccess] The config
+      def parse_config_file
+        begin
+          parsed_yaml = YAML.parse_file config_file
+        rescue Psych::SyntaxError => e
+          raise ConfigNotValidError,
+            "Not valid YAML file: #{e.message}."
         end
+        raise ConfigNotValidError,
+          "Not valid YAML file: The YAML does not respond_to to_ruby." unless parsed_yaml.respond_to?(:to_ruby)
+        config = HashWithIndifferentAccess.new(parsed_yaml.to_ruby)
+        raise ConfigNotValidError,
+          "Not valid YAML file: It doesn't contain watch_tower root key." unless config.has_key?(:watch_tower)
+
+        config
       end
   end
-end
+end

data/lib/watch_tower/errors.rb

@@ -29,4 +29,11 @@ module WatchTower
 
   # Exceptions raised by the Eye module
   EyeError = Class.new WatchTowerError
+
+  # Exceptions raised by the Config module
+  ConfigError = Class.new WatchTowerError
+  ConfigNotReadableError = Class.new ConfigError
+  ConfigNotFound = Class.new ConfigError
+  ConfigNotDefinedError = Class.new ConfigError
+  ConfigNotValidError = Class.new ConfigError
 end

data/lib/watch_tower/server/database.rb

@@ -21,11 +21,11 @@ module WatchTower
         # Migrate the database
         migrate!
       rescue DatabaseConfigNotFoundError
-        STDERR.puts "Database configurations are missing, please edit #{Config::CONFIG_FILE} and try again."
-        exit(1)
+        raise ConfigNotFound,
+          "Database configurations are missing, please edit #{Config.config_file} and try again."
       rescue ::ActiveRecord::ConnectionNotEstablished => e
-        STDERR.puts "There was an error connecting to the database: #{e}"
-        exit(1)
+        raise DatabaseError,
+          "There was an error connecting to the database: #{error}"
       end
 
       # Stop the database server
@@ -36,11 +36,11 @@ module WatchTower
         # Disconnect from the database
         disconnect!
       rescue DatabaseConfigNotFoundError
-        STDERR.puts "Database configurations are missing, please edit #{Config::CONFIG_FILE} and try again."
-        exit(1)
+        raise ConfigNotFound,
+          "Database configurations are missing, please edit #{Config.config_file} and try again."
       rescue ::ActiveRecord::ConnectionNotEstablished => e
-        STDERR.puts "There was an error connecting to the database: #{e}"
-        exit(1)
+        raise DatabaseError,
+          "There was an error connecting to the database: #{error}"
       end
 
       def is_connected?
@@ -103,4 +103,4 @@ module WatchTower
         end
     end
   end
-end
+end

data/lib/watch_tower/version.rb

@@ -4,7 +4,7 @@ module WatchTower
   MAJOR = 0
   MINOR = 0
   TINY = 1
-  PRE = 'beta10'
+  PRE = 'beta11'
 
   def self.version
     # Init the version

data/spec/watch_tower/config_spec.rb

@@ -1,6 +1,21 @@
 require 'spec_helper'
 
 describe WatchTower::Config do
+  before(:each) do
+    @config = {watch_tower: {enabled: true}}
+    @config_path = '/valid/path'
+    @invalid_config_path = '/invalid/path'
+    @yaml = mock
+    @yaml.stubs(:to_ruby).returns(@config)
+    Psych.stubs(:parse_file).with(@config_path).returns(@yaml)
+    WatchTower::Config.stubs(:config_file).returns(@config_path)
+
+    ::File.stubs(:exists?).with(@config_path).returns(true)
+    ::File.stubs(:readable?).with(@config_path).returns(true)
+
+    ::File.stubs(:exists?).with(@invalid_config_path).returns(false)
+    ::File.stubs(:readable?).with(@invalid_config_path).returns(false)
+  end
 
   describe "@@config" do
     it "should have and class_variable @@config" do
@@ -8,18 +23,135 @@ describe WatchTower::Config do
     end
   end
 
-  describe "#ensure_config_file_exists" do
-    it "should respond_to ensure_config_file_exists" do
-      -> { subject.send :ensure_config_file_exists }.should_not raise_error NoMethodError
+  describe "#check_config_file" do
+    before(:each) do
+      WatchTower::Config.stubs(:initialize_config_file)
+    end
+
+    it { should respond_to :check_config_file }
+
+    it "should call File.exists?" do
+      ::File.expects(:exists?).with(@config_path).returns(true).once
+
+      subject.send(:check_config_file)
     end
 
-    it "should be able to create the config file from the template if it doesn't exist" do
+    it "should call File.readable?" do
+      ::File.expects(:readable?).with(@config_path).returns(true).once
+
+      subject.send(:check_config_file)
+    end
+  end
+
+  describe "#initialize_config_file" do
+    it { should respond_to :initialize_config_file }
+
+    it "should be able to create the config file from the template" do
       config_file = mock
       config_file.expects(:write).once
-      File.expects(:exists?).with(WatchTower::Config::CONFIG_FILE).returns(false).once
-      File.expects(:open).with(WatchTower::Config::CONFIG_FILE, 'w').yields(config_file).once
+      File.expects(:open).with(WatchTower::Config.config_file, 'w').yields(config_file).once
+
+      subject.send :initialize_config_file
+    end
+  end
+
+  describe "#parse_config_file" do
+    before(:each) do
+      WatchTower::Config.send(:class_variable_set, :@@config, nil)
+      WatchTower::Config.stubs(:initialize_config_file)
+    end
+
+    it { should respond_to :parse_config_file }
+
+    it "should parse the config file and return an instance of HashWithIndifferentAccess" do
+      subject.send(:parse_config_file).should be_instance_of HashWithIndifferentAccess
+    end
+
+    it "should handle the case where config is not a valid YAML file." do
+      Psych.stubs(:parse_file).raises(Psych::SyntaxError)
+
+      -> { subject.send :parse_config_file }.should raise_error ConfigNotValidError
+    end
+
+    it "should handle the case where Psych returns nil." do
+      Psych.stubs(:parse_file).with(@config_path).returns(nil)
+
+      -> { subject.send :parse_config_file }.should raise_error ConfigNotValidError
+    end
+
+    it "should handle the case where :watch_tower key does not exist" do
+      config = {}
+      yaml = mock
+      yaml.stubs(:to_ruby).returns(config)
+      Psych.stubs(:parse_file).with(@config_path).returns(yaml)
+
+      -> { subject.send :parse_config_file }.should raise_error ConfigNotValidError
+    end
+  end
+
+  describe "#[]" do
+    before(:each) do
+      WatchTower::Config.send(:class_variable_set, :@@config, nil)
+      WatchTower::Config.stubs(:initialize_config_file)
+    end
+
+    it "should call check_config_file" do
+      WatchTower::Config.expects(:check_config_file).once
+
+      subject[:enabled]
+    end
+
+    it "should call config_file" do
+      WatchTower::Config.expects(:config_file).returns(@config_path).once
+
+      subject[:enabled]
+    end
+
+    it "should call YAML.parse_file" do
+      Psych.expects(:parse_file).with(@config_path).returns(@yaml).once
+
+      subject[:enabled]
+    end
+
+    it "should call to_ruby on the YAML result" do
+      @yaml.expects(:to_ruby).returns(@config).once
+
+      subject[:enabled]
+    end
+
+    it "should create a new HashWithIndifferentAccess" do
+      HashWithIndifferentAccess.expects(:new).returns(@config).once
+
+      subject[:enabled]
+    end
+
+    it "should raise ConfigNotDefinedError if config not found" do
+      WatchTower::Config.stubs(:config_file).returns(nil)
+      ::File.stubs(:exists?).with(@invalid_config_path).returns(false)
+
+      -> { subject[:enabled] }.should raise_error ConfigNotDefinedError
+    end
+
+    it "should raise ConfigNotReadableError if config not found" do
+      WatchTower::Config.stubs(:config_file).returns(@invalid_config_path)
+      ::File.stubs(:readable?).with(@invalid_config_path).returns(false)
+
+      -> { subject[:enabled] }.should raise_error ConfigNotReadableError
+    end
+
+    it "should handle the case where config is not a valid YAML file." do
+      Psych.stubs(:parse_file).with(@config_path).returns(nil)
+
+      -> { subject[:enabled] }.should raise_error ConfigNotValidError
+    end
+
+    it "should handle the case where :watch_tower key does not exist" do
+      config = {}
+      yaml = mock
+      yaml.stubs(:to_ruby).returns(config)
+      Psych.stubs(:parse_file).with(@config_path).returns(yaml)
 
-      subject.send :ensure_config_file_exists
+      -> { subject[:enabled] }.should raise_error ConfigNotValidError
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: watch_tower
 version: !ruby/object:Gem::Version
-  version: 0.0.1.beta10
+  version: 0.0.1.beta11
   prerelease: 6
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-10-27 00:00:00.000000000 Z
+date: 2011-10-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &4456300 !ruby/object:Gem::Requirement
+  requirement: &5402520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 3.1.1
   type: :runtime
   prerelease: false
-  version_requirements: *4456300
+  version_requirements: *5402520
 - !ruby/object:Gem::Dependency
   name: i18n
-  requirement: &4469840 !ruby/object:Gem::Requirement
+  requirement: &4966260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 0.6.0
   type: :runtime
   prerelease: false
-  version_requirements: *4469840
+  version_requirements: *4966260
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &4466500 !ruby/object:Gem::Requirement
+  requirement: &5093740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 3.1.1
   type: :runtime
   prerelease: false
-  version_requirements: *4466500
+  version_requirements: *5093740
 - !ruby/object:Gem::Dependency
   name: sinatra
-  requirement: &4465200 !ruby/object:Gem::Requirement
+  requirement: &5092860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: 1.3.0
   type: :runtime
   prerelease: false
-  version_requirements: *4465200
+  version_requirements: *5092860
 - !ruby/object:Gem::Dependency
   name: sinatra-snap
-  requirement: &4464520 !ruby/object:Gem::Requirement
+  requirement: &5092160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,10 +65,10 @@ dependencies:
         version: 0.3.2
   type: :runtime
   prerelease: false
-  version_requirements: *4464520
+  version_requirements: *5092160
 - !ruby/object:Gem::Dependency
   name: haml
-  requirement: &4463680 !ruby/object:Gem::Requirement
+  requirement: &5089700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -76,10 +76,10 @@ dependencies:
         version: 3.1.3
   type: :runtime
   prerelease: false
-  version_requirements: *4463680
+  version_requirements: *5089700
 - !ruby/object:Gem::Dependency
   name: grit
-  requirement: &4475980 !ruby/object:Gem::Requirement
+  requirement: &5048100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -87,10 +87,10 @@ dependencies:
         version: 2.4.1
   type: :runtime
   prerelease: false
-  version_requirements: *4475980
+  version_requirements: *5048100
 - !ruby/object:Gem::Dependency
   name: coffee-script
-  requirement: &4473680 !ruby/object:Gem::Requirement
+  requirement: &5439660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -98,10 +98,10 @@ dependencies:
         version: 2.2.0
   type: :runtime
   prerelease: false
-  version_requirements: *4473680
+  version_requirements: *5439660
 - !ruby/object:Gem::Dependency
   name: uglifier
-  requirement: &4485180 !ruby/object:Gem::Requirement
+  requirement: &5523540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -109,10 +109,10 @@ dependencies:
         version: 1.0.3
   type: :runtime
   prerelease: false
-  version_requirements: *4485180
+  version_requirements: *5523540
 - !ruby/object:Gem::Dependency
   name: sass
-  requirement: &5352920 !ruby/object:Gem::Requirement
+  requirement: &5608600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -120,10 +120,10 @@ dependencies:
         version: 3.1.10
   type: :runtime
   prerelease: false
-  version_requirements: *5352920
+  version_requirements: *5608600
 - !ruby/object:Gem::Dependency
   name: sprockets
-  requirement: &5349780 !ruby/object:Gem::Requirement
+  requirement: &5608120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -131,10 +131,10 @@ dependencies:
         version: 2.0.2
   type: :runtime
   prerelease: false
-  version_requirements: *5349780
+  version_requirements: *5608120
 - !ruby/object:Gem::Dependency
   name: cronedit
-  requirement: &5347480 !ruby/object:Gem::Requirement
+  requirement: &5607480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -142,10 +142,10 @@ dependencies:
         version: 0.3.0
   type: :runtime
   prerelease: false
-  version_requirements: *5347480
+  version_requirements: *5607480
 - !ruby/object:Gem::Dependency
   name: guard
-  requirement: &5362620 !ruby/object:Gem::Requirement
+  requirement: &5606940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -153,10 +153,10 @@ dependencies:
         version: 0.8.4
   type: :development
   prerelease: false
-  version_requirements: *5362620
+  version_requirements: *5606940
 - !ruby/object:Gem::Dependency
   name: guard-bundler
-  requirement: &5361640 !ruby/object:Gem::Requirement
+  requirement: &5606120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -164,10 +164,10 @@ dependencies:
         version: 0.1.3
   type: :development
   prerelease: false
-  version_requirements: *5361640
+  version_requirements: *5606120
 - !ruby/object:Gem::Dependency
   name: guard-rspec
-  requirement: &5360740 !ruby/object:Gem::Requirement
+  requirement: &5604680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -175,10 +175,10 @@ dependencies:
         version: 0.4.5
   type: :development
   prerelease: false
-  version_requirements: *5360740
+  version_requirements: *5604680
 - !ruby/object:Gem::Dependency
   name: guard-sprockets2
-  requirement: &5359540 !ruby/object:Gem::Requirement
+  requirement: &5603880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -186,10 +186,10 @@ dependencies:
         version: 0.0.5
   type: :development
   prerelease: false
-  version_requirements: *5359540
+  version_requirements: *5603880
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &5358300 !ruby/object:Gem::Requirement
+  requirement: &5603200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -197,10 +197,10 @@ dependencies:
         version: 0.7.2
   type: :development
   prerelease: false
-  version_requirements: *5358300
+  version_requirements: *5603200
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &5356340 !ruby/object:Gem::Requirement
+  requirement: &5602120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -208,10 +208,10 @@ dependencies:
         version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *5356340
+  version_requirements: *5602120
 - !ruby/object:Gem::Dependency
   name: rspec-rails
-  requirement: &5368240 !ruby/object:Gem::Requirement
+  requirement: &5601240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -219,10 +219,10 @@ dependencies:
         version: 2.6.1
   type: :development
   prerelease: false
-  version_requirements: *5368240
+  version_requirements: *5601240
 - !ruby/object:Gem::Dependency
   name: capybara
-  requirement: &5365440 !ruby/object:Gem::Requirement
+  requirement: &5678180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -230,10 +230,10 @@ dependencies:
         version: 1.1.1
   type: :development
   prerelease: false
-  version_requirements: *5365440
+  version_requirements: *5678180
 - !ruby/object:Gem::Dependency
   name: launchy
-  requirement: &5394640 !ruby/object:Gem::Requirement
+  requirement: &5677000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -241,10 +241,10 @@ dependencies:
         version: 2.0.5
   type: :development
   prerelease: false
-  version_requirements: *5394640
+  version_requirements: *5677000
 - !ruby/object:Gem::Dependency
   name: mocha
-  requirement: &5402720 !ruby/object:Gem::Requirement
+  requirement: &5674720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -252,10 +252,10 @@ dependencies:
         version: 0.10.0
   type: :development
   prerelease: false
-  version_requirements: *5402720
+  version_requirements: *5674720
 - !ruby/object:Gem::Dependency
   name: factory_girl
-  requirement: &4967600 !ruby/object:Gem::Requirement
+  requirement: &5672560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -263,10 +263,10 @@ dependencies:
         version: 2.1.2
   type: :development
   prerelease: false
-  version_requirements: *4967600
+  version_requirements: *5672560
 - !ruby/object:Gem::Dependency
   name: timecop
-  requirement: &5094520 !ruby/object:Gem::Requirement
+  requirement: &5671000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -274,10 +274,10 @@ dependencies:
         version: 0.3.5
   type: :development
   prerelease: false
-  version_requirements: *5094520
+  version_requirements: *5671000
 - !ruby/object:Gem::Dependency
   name: pry
-  requirement: &5090800 !ruby/object:Gem::Requirement
+  requirement: &5724100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -285,7 +285,7 @@ dependencies:
         version: 0.9.6.2
   type: :development
   prerelease: false
-  version_requirements: *5090800
+  version_requirements: *5724100
 description: ! 'Did you ever want to keep track of how much time you _really_ spend
   on all of