vedeu 0.8.12 → 0.8.13

data/lib/vedeu/input/mapper.rb

@@ -32,12 +32,12 @@ module Vedeu
         # @macro raise_missing_required
         # @return [Boolean]
         def registered?(key = nil, name = nil)
-          fail Vedeu::Error::MissingRequired,
-               'Cannot check whether a key is registered to a keymap without ' \
-               'the key.' if absent?(key)
-          fail Vedeu::Error::MissingRequired,
-               'Cannot check whether a key is registered to a keymap without ' \
-               'the keymap name.' if absent?(name)
+          raise Vedeu::Error::MissingRequired,
+                'Cannot check whether a key is registered to a keymap ' \
+                'without the key.' if absent?(key)
+          raise Vedeu::Error::MissingRequired,
+                'Cannot check whether a key is registered to a keymap ' \
+                'without the keymap name.' if absent?(name)
 
           new(key, name).registered?
         end

data/lib/vedeu/input/read.rb

@@ -102,8 +102,8 @@ module Vedeu
       # @return [Symbol]
       def mode
         unless valid_mode?
-          fail Vedeu::Error::InvalidSyntax,
-               ':mode must be `:raw`, `:fake` or `:cooked`.'
+          raise Vedeu::Error::InvalidSyntax,
+                ':mode must be `:raw`, `:fake` or `:cooked`.'
         end
 
         options[:mode]

data/lib/vedeu/interfaces/dsl.rb

@@ -39,8 +39,8 @@ module Vedeu
         # @return [Vedeu::Interfaces::Interface]
         # @todo More documentation required.
         def interface(name, &block)
-          fail Vedeu::Error::MissingRequired unless name
-          fail Vedeu::Error::RequiresBlock unless block_given?
+          raise Vedeu::Error::MissingRequired unless name
+          raise Vedeu::Error::RequiresBlock unless block_given?
 
           attributes = { client: client(&block), name: name }
 

data/lib/vedeu/logging/debug.rb

@@ -41,11 +41,11 @@ module Vedeu
       # @macro raise_requires_block
       # @return [void]
       # @yieldreturn [void] The section of the application to profile.
-      def profile(filename = '/tmp/profile.html', &block)
-        fail Vedeu::Error::RequiresBlock unless block_given?
-
+      def profile(filename = '/tmp/vedeu_profile', &block)
         Vedeu.requires_gem!('ruby-prof')
 
+        raise Vedeu::Error::RequiresBlock unless block_given?
+
         # ::RubyProf.measure_mode = ::RubyProf::WALL_TIME
         # ::RubyProf.measure_mode = ::RubyProf::PROCESS_TIME
         ::RubyProf.measure_mode = ::RubyProf::CPU_TIME
@@ -69,28 +69,28 @@ module Vedeu
           # ::RubyProf::CallTreePrinter.new(result).print(file)
 
           # Creates a flat report in text format
-          # ::RubyProf::FlatPrinter
+          # ::RubyProf::FlatPrinter.new(result).print(file)
 
           # - same as above but more verbose
-          # ::RubyProf::FlatPrinterWithLineNumbers
+          # ::RubyProf::FlatPrinterWithLineNumbers.new(result).print(file)
 
           # - Creates a call graph report in text format
-          # ::RubyProf::GraphPrinter
+          # ::RubyProf::GraphPrinter.new(result).print(file)
 
           # - Creates a call graph report in HTML (separate files per
           #   thread)
-          # ::RubyProf::GraphHtmlPrinter
+          # ::RubyProf::GraphHtmlPrinter.new(result).print(file)
 
           # - Creates a call graph report in GraphViz's DOT format
           #   which can be converted to an image
-          # ::RubyProf::DotPrinter
+          # ::RubyProf::DotPrinter.new(result).print(file)
 
           # - Creates a call tree report compatible with KCachegrind.
-          # ::RubyProf::CallTreePrinter
+          # ::RubyProf::CallTreePrinter.new(result).print(file)
 
           # - Uses the other printers to create several reports in one
           #   profiling run
-          # ::RubyProf::MultiPrinter
+          # ::RubyProf::MultiPrinter.new(result).print(file)
         end
 
         work

data/lib/vedeu/logging/timer.rb

@@ -41,22 +41,18 @@ module Vedeu
       # @return [void|NilClass] The return value of the executed
       #   block if given, or nil if debugging is disabled.
       def measure(&block)
-        if block_given?
-          if Vedeu.config.debug?
-            work = yield
+        raise Vedeu::Error::RequiresBlock unless block_given?
 
-            Vedeu.log(type:    :timer,
-                      message: "#{message} took #{elapsed}ms.")
+        if Vedeu.config.debug?
+          work = yield
 
-            work
+          Vedeu.log(type:    :timer,
+                    message: "#{message} took #{elapsed}ms.")
 
-          else
-            yield
-
-          end
+          work
 
         else
-          fail Vedeu::Error::RequiresBlock if Vedeu.config.debug?
+          yield
 
         end
       end

data/lib/vedeu/menus/dsl.rb

@@ -38,8 +38,8 @@ module Vedeu
         # @macro raise_missing_required
         # @return [API::Menu]
         def menu(name, &block)
-          fail Vedeu::Error::MissingRequired unless name
-          fail Vedeu::Error::RequiresBlock unless block_given?
+          raise Vedeu::Error::MissingRequired unless name
+          raise Vedeu::Error::RequiresBlock unless block_given?
 
           attributes = { client: client(&block), name: name }
 

data/lib/vedeu/models/focus.rb

@@ -209,9 +209,9 @@ module Vedeu
 
       # @raise [Vedeu::Error::ModelNotFound]
       def not_registered!(name)
-        fail Vedeu::Error::ModelNotFound,
-             "Cannot focus '#{name}' as this interface has not been " \
-             'registered.'
+        raise Vedeu::Error::ModelNotFound,
+              "Cannot focus '#{name}' as this interface has not been " \
+              'registered.'
       end
 
       # Return the name of the interface in focus after triggering the

data/lib/vedeu/models/page.rb

@@ -64,7 +64,7 @@ module Vedeu
       # @param other [Vedeu::Models::Page]
       # @return [Boolean]
       def eql?(other)
-        self.class == other.class && rows == other.rows
+        self.class.equal?(other.class) && rows == other.rows
       end
       alias == eql?
 

data/lib/vedeu/models/row.rb

@@ -63,7 +63,7 @@ module Vedeu
       # @param other [Vedeu::Models::Row]
       # @return [Boolean]
       def eql?(other)
-        self.class == other.class && cells == other.cells
+        self.class.equal?(other.class) && cells == other.cells
       end
       alias == eql?
 

data/lib/vedeu/presentation/style.rb

@@ -55,7 +55,7 @@ module Vedeu
       # @param other [void]
       # @return [Boolean]
       def eql?(other)
-        self.class == other.class && value == other.value
+        self.class.equal?(other.class) && value == other.value
       end
       alias == eql?
 

data/lib/vedeu/renderers/escape.rb

@@ -50,12 +50,6 @@ module Vedeu
         empty
       end
 
-      # @return [Boolean]
-      def positionable?(char)
-        char.respond_to?(:position) &&
-        char.position.is_a?(Vedeu::Geometries::Position)
-      end
-
       # @return [Array<Class>]
       def renderables
         [

data/lib/vedeu/renderers/options.rb

@@ -53,7 +53,7 @@ module Vedeu
 
       # @macro raise_not_implemented
       def write
-        fail Vedeu::Error::NotImplemented, 'Including classes implement this.'
+        raise Vedeu::Error::NotImplemented, 'Including classes implement this.'
       end
 
       private
@@ -81,7 +81,7 @@ module Vedeu
 
       # @macro raise_not_implemented
       def content
-        fail Vedeu::Error::NotImplemented, 'Including classes implement this.'
+        raise Vedeu::Error::NotImplemented, 'Including classes implement this.'
       end
 
       # @macro defaults_method
@@ -90,7 +90,7 @@ module Vedeu
           compression:   Vedeu.config.compression?,
           end_tag:       '</td>',
           end_row_tag:   '</tr>',
-          filename:      'out',
+          filename:      '/tmp/vedeu_out',
           output:        '',
           start_tag:     '<td',
           start_row_tag: '<tr>',

data/lib/vedeu/renderers/text.rb

@@ -45,12 +45,6 @@ module Vedeu
         empty
       end
 
-      # @return [Boolean]
-      def positionable?(char)
-        char.respond_to?(:position) &&
-        char.position.is_a?(Vedeu::Geometries::Position)
-      end
-
       # @return [Array<Class>]
       def renderables
         [

data/lib/vedeu/repositories/assemblage.rb

@@ -50,7 +50,7 @@ module Vedeu
       # @param other [void]
       # @return [Boolean]
       def eql?(other)
-        self.class == other.class && collection == other.collection
+        self.class.equal?(other.class) && collection == other.collection
       end
       alias == eql?
 

data/lib/vedeu/repositories/defaults.rb

@@ -41,8 +41,8 @@ module Vedeu
       # @macro raise_invalid_syntax
       # @return [Hash]
       def validate(attributes)
-        fail Vedeu::Error::InvalidSyntax,
-             'Argument :attributes is not a Hash.' unless hash?(attributes)
+        raise Vedeu::Error::InvalidSyntax,
+              'Argument :attributes is not a Hash.' unless hash?(attributes)
 
         attributes.keep_if { |key, _| defaults.key?(key) }
       end

data/lib/vedeu/repositories/repositories.rb

@@ -38,11 +38,22 @@ module Vedeu
       end
     end
 
+    # Access all the repositories stored.
+    #
+    # @return [Array]
+    def repositories
+      self
+    end
+
     # Remove all stored models from the repository.
     #
     # @return [Boolean]
     def reset!
-      all.map { |repository| repository.send(:reset) }
+      storage.map do |repository|
+        Vedeu.log(type:    :red,
+                  message: "Resetting: #{repository.name}")
+        repository.reset!
+      end
 
       true
     end
@@ -59,4 +70,9 @@ module Vedeu
 
   end # Repositories
 
+  # @!method repositories
+  #   @see Vedeu::Repositories
+  def_delegators Vedeu::Repositories,
+                 :repositories
+
 end # Vedeu

data/lib/vedeu/repositories/repository.rb

@@ -95,8 +95,8 @@ module Vedeu
       #   found with this name.
       # @return [Hash<String => Object>]
       def find!(name)
-        find(name) || fail(Vedeu::Error::ModelNotFound,
-                           "Cannot find model by name: '#{name}'")
+        find(name) || raise(Vedeu::Error::ModelNotFound,
+                            "Cannot find model by name: '#{name}'")
       end
 
       # Find a model by name, registers the model by name when not
@@ -182,9 +182,9 @@ module Vedeu
       # @macro raise_missing_required
       # @return [Boolean]
       def valid_model?(model)
-        fail Vedeu::Error::MissingRequired,
-             "Cannot store model '#{model.class}' without a name " \
-             'attribute.' unless present?(model.name)
+        raise Vedeu::Error::MissingRequired,
+              "Cannot store model '#{model.class}' without a name " \
+              'attribute.' unless present?(model.name)
       end
 
     end # Repository

data/lib/vedeu/runtime/application.rb

@@ -127,9 +127,9 @@ module Vedeu
       # @param block [Proc]
       # @return [void]
       def run_once(&block)
-        yield.tap do
-          # Vedeu.debug(binding)
-        end
+        # Vedeu.debug(binding)
+
+        yield
       end
 
       # :nocov:

data/lib/vedeu/runtime/main_loop.rb

@@ -25,7 +25,7 @@ module Vedeu
 
           Vedeu.trigger(:_drb_restart_)
 
-          fail Vedeu::Error::ModeSwitch
+          raise Vedeu::Error::ModeSwitch
         end
 
         # Start the main loop.
@@ -64,7 +64,7 @@ module Vedeu
         #   the running application.
         # @return [void]
         def safe_exit_point!
-          fail Vedeu::Error::Interrupt if @started && !@loop
+          raise Vedeu::Error::Interrupt if @started && !@loop
         end
 
         # :nocov:

data/lib/vedeu/runtime/router.rb

@@ -25,8 +25,8 @@ module Vedeu
       # @return [void]
       def add_controller(controller, klass)
         unless present?(controller)
-          fail Vedeu::Error::MissingRequired,
-               'Cannot store controller without a name attribute.'
+          raise Vedeu::Error::MissingRequired,
+                'Cannot store controller without a name attribute.'
         end
 
         Vedeu.log(type:    :create,
@@ -68,9 +68,9 @@ module Vedeu
           storage
 
         else
-          fail Vedeu::Error::MissingRequired,
-               'Cannot store action without a controller or name ' \
-               'attribute.'
+          raise Vedeu::Error::MissingRequired,
+                'Cannot store action without a controller or name ' \
+                'attribute.'
 
         end
       end
@@ -111,12 +111,12 @@ module Vedeu
         if registered?(controller)
           return true if storage[controller][:actions].include?(action)
 
-          fail Vedeu::Error::ActionNotFound,
-               "#{action} is not registered for #{controller}."
+          raise Vedeu::Error::ActionNotFound,
+                "#{action} is not registered for #{controller}."
 
         else
-          fail Vedeu::Error::ControllerNotFound,
-               "#{controller} is not registered."
+          raise Vedeu::Error::ControllerNotFound,
+                "#{controller} is not registered."
 
         end
       end
@@ -143,8 +143,8 @@ module Vedeu
           storage[controller][:klass]
 
         else
-          fail Vedeu::Error::MissingRequired,
-               "Cannot route to #{controller} as no class defined."
+          raise Vedeu::Error::MissingRequired,
+                "Cannot route to #{controller} as no class defined."
 
         end
       end

data/lib/vedeu/support/point.rb

@@ -11,6 +11,12 @@ module Vedeu
   #
   class Point
 
+    include Vedeu::Common
+
+    # @!attribute [r] value
+    # @return [Fixnum]
+    attr_reader :value
+
     # @param (see #initialize)
     # @macro raise_invalid_syntax
     # @return (see #initialize)
@@ -31,14 +37,16 @@ module Vedeu
     def initialize(value: nil, min: 1, max: Float::INFINITY)
       @min   = min
       @max   = max
-      @value = value
+      @value = value || @min
+
+      freeze
     end
 
     # @macro raise_invalid_syntax
     # @return [Vedeu::Point]
     def coerce
-      fail Vedeu::Error::InvalidSyntax,
-           "Expecting 'min' to be less than 'max'." if min > max
+      raise Vedeu::Error::InvalidSyntax,
+            "Expecting 'min' to be less than 'max'." if min > max
 
       if value < min
         Vedeu::Point.coerce(value: min, min: min, max: max)
@@ -54,12 +62,7 @@ module Vedeu
 
     # @return [Boolean]
     def valid?
-      @value.is_a?(Fixnum) && @value >= min && @value <= max
-    end
-
-    # @return [Fixnum]
-    def value
-      @value ||= min
+      numeric?(value) && value >= min && value <= max
     end
 
     private
@@ -67,19 +70,19 @@ module Vedeu
     # @macro raise_invalid_syntax
     # @return [Fixnum]
     def min
-      return @min if @min.is_a?(Fixnum)
+      return @min if numeric?(@min)
 
-      fail Vedeu::Error::InvalidSyntax,
-           "Expecting 'min' to be a Fixnum."
+      raise Vedeu::Error::InvalidSyntax,
+            "Expecting 'min' to be a Fixnum."
     end
 
     # @macro raise_invalid_syntax
     # @return [Fixnum]
     def max
-      return @max if @max.is_a?(Fixnum) || @max == Float::INFINITY
+      return @max if numeric?(@max) || @max == Float::INFINITY
 
-      fail Vedeu::Error::InvalidSyntax,
-           "Expecting 'max' to be a Fixnum or Float::INFINITY."
+      raise Vedeu::Error::InvalidSyntax,
+            "Expecting 'max' to be a Fixnum or Float::INFINITY."
     end
 
   end # Point

data/lib/vedeu/templating/helpers.rb

@@ -51,7 +51,7 @@ module Vedeu
 
       # @see Vedeu::Templating::Helpers#colour
       def define_stream(attributes = {}, &block)
-        fail Vedeu::Error::RequiresBlock unless block_given?
+        raise Vedeu::Error::RequiresBlock unless block_given?
 
         value = yield
 

data/lib/vedeu/templating/template.rb

@@ -52,12 +52,12 @@ module Vedeu
       # @macro raise_missing_required
       # @return [String]
       def path
-        fail Vedeu::Error::MissingRequired,
-             'No path to template specified.' if @path.empty?
+        raise Vedeu::Error::MissingRequired,
+              'No path to template specified.' if @path.empty?
 
         unless File.exist?(@path)
-          fail Vedeu::Error::MissingRequired,
-               "Template file cannot be found. (#{@path})"
+          raise Vedeu::Error::MissingRequired,
+                "Template file cannot be found. (#{@path})"
         end
 
         @path

data/lib/vedeu/terminal/terminal.rb

@@ -28,7 +28,7 @@ module Vedeu
     # @macro raise_requires_block
     # @return [Array]
     def open(&block)
-      fail Vedeu::Error::RequiresBlock unless block_given?
+      raise Vedeu::Error::RequiresBlock unless block_given?
 
       if raw_mode? || fake_mode?
         console.raw    { initialize_screen(mode) { yield } }

data/lib/vedeu/version.rb

@@ -3,6 +3,6 @@
 module Vedeu
 
   # The current version of Vedeu.
-  VERSION = '0.8.12'.freeze
+  VERSION = '0.8.13'
 
 end

data/lib/vedeu/views/chars.rb

@@ -45,9 +45,9 @@ module Vedeu
           end
 
         else
-          fail Vedeu::Error::InvalidSyntax,
-               'Cannot coerce for Vedeu::View::Chars, as collection is ' \
-               'unrecognised.'
+          raise Vedeu::Error::InvalidSyntax,
+                'Cannot coerce for Vedeu::View::Chars, as collection is ' \
+                'unrecognised.'
 
         end
       end

data/lib/vedeu/views/line.rb

@@ -81,7 +81,7 @@ module Vedeu
       # @param other [Vedeu::Repositories::Collection]
       # @return [Boolean]
       def eql?(other)
-        self.class == other.class && value == other.value
+        self.class.equal?(other.class) && value == other.value
       end
       alias == eql?
 

data/lib/vedeu/views/lines.rb

@@ -36,9 +36,9 @@ module Vedeu
           new([collection], parent, name)
 
         else
-          fail Vedeu::Error::InvalidSyntax,
-               'Cannot coerce for Vedeu::View::Lines, as collection ' \
-               'unrecognised.'
+          raise Vedeu::Error::InvalidSyntax,
+                'Cannot coerce for Vedeu::View::Lines, as collection ' \
+                'unrecognised.'
 
         end
       end

data/lib/vedeu/views/stream.rb

@@ -85,7 +85,7 @@ module Vedeu
       # @param other [void]
       # @return [Boolean]
       def eql?(other)
-        self.class == other.class && value == other.value
+        self.class.equal?(other.class) && value == other.value
       end
       alias == eql?
 

data/lib/vedeu/views/streams.rb

@@ -47,9 +47,9 @@ module Vedeu
           new([Vedeu::Views::Stream.new(value: collection)], parent, name)
 
         else
-          fail Vedeu::Error::InvalidSyntax,
-               'Cannot coerce for Vedeu::View::Streams, as collection ' \
-               "unrecognised. (#{collection.class.name})"
+          raise Vedeu::Error::InvalidSyntax,
+                'Cannot coerce for Vedeu::View::Streams, as collection ' \
+                "unrecognised. (#{collection.class.name})"
 
         end
       end

data/lib/vedeu/views/view.rb

@@ -129,8 +129,8 @@ module Vedeu
           buffer.add(self, refresh)
 
         else
-          fail Vedeu::Error::MissingRequired,
-               'Cannot store a view without a name.'
+          raise Vedeu::Error::MissingRequired,
+                'Cannot store a view without a name.'
 
         end
 

data/test/lib/vedeu/common_test.rb

@@ -208,6 +208,8 @@ module Vedeu
 
       subject { instance.numeric?(_value) }
 
+      it { instance.must_respond_to(:fixnum?) }
+
       context 'when the value is numeric' do
         let(:_value) { 16 }
 
@@ -219,6 +221,31 @@ module Vedeu
       end
     end
 
+    describe '#positionable?' do
+      subject { instance.positionable?(_value) }
+
+      context 'when it does not respond to position' do
+        let(:_value) { '' }
+
+        it { subject.must_equal(false) }
+      end
+
+      context 'when it responds to position' do
+        let(:_value)   { Vedeu::Cells::Empty.new(position: position) }
+        let(:position) { false }
+
+        context 'when the position is a Vedeu::Geometries::Position' do
+          let(:position) { Vedeu::Geometries::Position.new }
+
+          it { subject.must_equal(true) }
+        end
+
+        context 'when the position is not a Vedeu::Geometries::Position' do
+          it { subject.must_equal(false) }
+        end
+      end
+    end
+
     describe '#present?' do
       subject { instance.defined_value_test(_value) }
 
@@ -328,6 +355,22 @@ module Vedeu
       end
     end
 
+    describe '#symbol?' do
+      let(:_value) {}
+
+      subject { instance.symbol?(_value) }
+
+      context 'when the value is a Symbol' do
+        let(:_value) { :test }
+
+        it { subject.must_equal(true) }
+      end
+
+      context 'when the value is not a Symbol' do
+        it { subject.must_equal(false) }
+      end
+    end
+
     describe '#truthy?' do
       subject { instance.truthy?(_value) }