vedeu 0.6.22 → 0.6.23

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 962d118b37be3f266143bd483c590ea61fcb045b
-  data.tar.gz: a10a95a4e722f747a4faa9b07653d1a74ba98ccb
+  metadata.gz: 4242ab9d58186c12c3f4884987b54da9740b138d
+  data.tar.gz: 9129f04bc359fa77f802414074136695417003f7
 SHA512:
-  metadata.gz: 658eae376a443bde02be432fd0c91c46f972c59da102ea7f198198831b0dc2371776566cb74545829ba4b2a7c3a3f0e8196993be1faadaa3ec195f87db24cbf8
-  data.tar.gz: 3950ca1569a8e8ebf80fe8b14692fcf0796fa61d5eb9d9267ed7606a654ecde8519ac79c4f6d591166f988aad10e8fbee74bbb22ee29c2a3eff23353e28cd2db
+  metadata.gz: 449cf59c90d7017af8b6558e59dcf077ec89cb935f750365e36d0ba8b36d6a09e75f07f329ea6ab7acfa5eeaab55b9f14a35613aaef72c39b45a3be71bf21be0
+  data.tar.gz: 723cd0a76e4d71187bd28390f61fef953fe1ed73de24b976a21eaff46928d595f0d5ab8682016dd29cebf9a2aac28aa062aa364f63798c9eac1f65a920327b36

data/.codeclimate.yml

@@ -5,6 +5,6 @@ ratings:
   paths:
     - lib/**
 exclude_paths:
-  - examples/**/*
-  - test/**/*
-  - vendor/**/*
+- "examples/**/*"
+- "test/**/*"
+- "vendor/**/*"

data/docs/events/refresh.md

@@ -14,6 +14,11 @@ will be still refreshed in memory but not shown.
 
     Vedeu.trigger(:_refresh_)
 
+### :_refresh_border_
+Will cause the named border to refresh.
+
+    Vedeu.trigger(:_refresh_border_, name)
+
 ### :_refresh_cursor_
 Will cause the named cursor to refresh, or the cursor of the interface
 which is currently in focus.

data/lib/vedeu/bindings/refresh.rb

@@ -14,6 +14,7 @@ module Vedeu
       # @return [TrueClass]
       def setup!
         refresh!
+        refresh_border!
         refresh_cursor!
         refresh_group!
         refresh_view!
@@ -29,6 +30,13 @@ module Vedeu
         Vedeu.bind(:_refresh_) { Vedeu::Output::Refresh.all }
       end
 
+      # See {file:docs/events/refresh.md#\_refresh_border_}
+      def refresh_border!
+        Vedeu.bind(:_refresh_border_) do |name|
+          Vedeu::Borders::Refresh.by_name(name)
+        end
+      end
+
       # See {file:docs/events/refresh.md#\_refresh_cursor_}
       def refresh_cursor!
         Vedeu.bind(:_refresh_cursor_) do |name|

data/lib/vedeu/borders/all.rb

@@ -36,5 +36,5 @@ end # Vedeu
 require 'vedeu/borders/border'
 require 'vedeu/borders/dsl'
 require 'vedeu/borders/null'
-require 'vedeu/borders/render'
+require 'vedeu/borders/refresh'
 require 'vedeu/borders/repository'

data/lib/vedeu/borders/border.rb

@@ -172,7 +172,7 @@ module Vedeu
 
       # @return [Array<Array<Vedeu::Views::Char>>]
       def render
-        Vedeu::Borders::Render.render(self)
+        Vedeu.trigger(:_refresh_border_, name)
       end
 
       private

data/lib/vedeu/borders/null.rb

@@ -45,6 +45,11 @@ module Vedeu
       end
       alias_method :yn, :byn
 
+      # @return [Boolean]
+      def enabled?
+        false
+      end
+
       # @return [Fixnum]
       def height
         (by..byn).size

data/lib/vedeu/borders/{render.rb → refresh.rb}

@@ -2,9 +2,9 @@ module Vedeu
 
   module Borders
 
-    # Renders the provided border.
+    # Renders and refreshes the named border.
     #
-    class Render
+    class Refresh
 
       extend Forwardable
       include Vedeu::Common
@@ -23,7 +23,6 @@ module Vedeu
                      :height,
                      :horizontal,
                      :left?,
-                     :name,
                      :right?,
                      :style,
                      :title,
@@ -39,33 +38,41 @@ module Vedeu
                      :y,
                      :yn
 
+      # @example
+      #   Vedeu.trigger(:_refresh_border_, name)
+      #
+      # @param name [String|Symbol] The name of the border to render.
       # @return [Array<Array<Vedeu::Views::Char>>]
-      # @param (see #initialize)
-      def self.render(border)
-        new(border).render
+      def self.by_name(name)
+        new(name).render
       end
 
-      # Returns a new instance of Vedeu::Borders::Render.
+      # Returns a new instance of Vedeu::Borders::Refresh.
       #
-      # @param border [Vedeu::Borders::Border]
-      # @return [Vedeu::Borders::Render]
-      def initialize(border)
-        @border = border
+      # @param name [String|Symbol]
+      # @return [Vedeu::Borders::Refresh]
+      def initialize(name)
+        @name = name
       end
 
       # @return [Array<Array<Vedeu::Views::Char>>]
       def render
-        Vedeu::Output::Output.render(output) if enabled?
+        Vedeu::Output::Output.render(output) if enabled? && Vedeu.ready?
       end
 
       protected
 
-      # @!attribute [r] border
-      # @return [Vedeu::Borders::Border]
-      attr_reader :border
+      # @!attribute [r] name
+      # @return [String|Symbol]
+      attr_reader :name
 
       private
 
+      # @return [Vedeu::Borders::Border]
+      def border
+        @border ||= Vedeu.borders.by_name(name)
+      end
+
       # @return [Array<Array<Vedeu::Views::Char>>]
       def output
         Vedeu.timer("Bordering: '#{name}'") do
@@ -348,7 +355,7 @@ module Vedeu
         caption.chomp.slice(0..(width - 5))
       end
 
-    end # Render
+    end # Refresh
 
   end # Borders
 

data/lib/vedeu/buffers/refresh.rb

@@ -40,10 +40,13 @@ module Vedeu
 
       # @return [Array|Vedeu::Error::ModelNotFound]
       def by_name
+        Vedeu.trigger(:_clear_view_content_, name)
+
         Vedeu.timer("Refresh Buffer: '#{name}'") do
           Vedeu.buffers.by_name(name).render
-          Vedeu.borders.by_name(name).render unless content_only?
-        end if Vedeu.ready?
+        end
+
+        Vedeu.trigger(:_refresh_border_, name) unless content_only?
       end
 
       protected

data/lib/vedeu/groups/clear.rb

@@ -11,6 +11,7 @@ module Vedeu
         # Clears the group of interfaces belonging to the given name.
         #
         # @example
+        #   Vedeu.trigger(:_clear_group_, name)
         #   Vedeu.clear_by_group(name)
         #
         # @param name [String|Symbol]

data/lib/vedeu/models/views/view.rb

@@ -78,8 +78,6 @@ module Vedeu
 
       # @return [Array<Array<Vedeu::Views::Char>>]
       def render
-        Vedeu.trigger(:_clear_view_content_, name)
-
         Vedeu::Output::Viewport.render(self)
       end
 

data/lib/vedeu/output/clear/interface.rb

@@ -51,7 +51,7 @@ module Vedeu
 
       # @return [Array<Array<Vedeu::Views::Char>>]
       def render
-        Vedeu::Output::Output.render(output)
+        Vedeu::Output::Output.render(output) if Vedeu.ready?
       end
 
       protected

data/lib/vedeu/output/compressor.rb

@@ -48,7 +48,7 @@ module Vedeu
       def content
         return [] if absent?(output)
 
-        @content ||= output.content.delete_if(&:cell?)
+        @content ||= output.content.reject(&:cell?)
       end
 
       # @return [String]

data/lib/vedeu/output/renderers/all.rb

@@ -49,9 +49,6 @@ module Vedeu
       threads = storage.map do |renderer|
         Thread.new(renderer) do
           mutex.synchronize do
-            Vedeu.log(type:    :output,
-                      message: "Renderer: '#{renderer.class.name}'")
-
             renderer.render(output)
           end
         end

data/lib/vedeu/output/viewport.rb

@@ -44,7 +44,7 @@ module Vedeu
       #
       # @return [Array<Array<String>>|NilClass]
       def render
-        Vedeu::Output::Output.render(output) if visible?
+        Vedeu::Output::Output.render(output) if visible? && Vedeu.ready?
       end
 
       # Returns a string representation of the viewport.
@@ -63,6 +63,7 @@ module Vedeu
 
       private
 
+      # @return [Array<Array<Vedeu::Views::Char>>]
       def output
         Vedeu.timer("Rendering: '#{name}'") do
           out = []

data/lib/vedeu/runtime/launcher.rb

@@ -68,6 +68,7 @@ module Vedeu
 
     rescue StandardError => uncaught_exception
       output = if configuration.debug?
+                 uncaught_exception.message + "\n" +
                  uncaught_exception.backtrace.join("\n")
                else
                  uncaught_exception.message

data/lib/vedeu/runtime/main_loop.rb

@@ -22,15 +22,9 @@ module Vedeu
           @loop    = true
 
           while @loop
-            yield
-
-            Vedeu.trigger(:_hide_cursor_)
+            Vedeu.trigger(:_refresh_cursor_, Vedeu.focus)
 
-            Vedeu.log(type:    :info,
-                      message: 'MainLoop rendering terminal buffer.')
-            Vedeu::Terminal::Buffer.render
-
-            Vedeu.trigger(:_show_cursor_)
+            yield
 
             safe_exit_point!
           end

data/lib/vedeu/terminal/buffer.rb

@@ -38,7 +38,7 @@ module Vedeu
 
       # @return [Array<Array<Vedeu::Models::Cell>>]
       def empty_buffer
-        Array.new(Vedeu.height) do |y|
+        @empty_buffer ||= Array.new(Vedeu.height) do |y|
           Array.new(Vedeu.width) do |x|
             Vedeu::Models::Cell.new(position: [y + 1, x + 1])
           end

data/lib/vedeu/version.rb

@@ -1,6 +1,6 @@
 module Vedeu
 
   # The current version of Vedeu.
-  VERSION = '0.6.22'
+  VERSION = '0.6.23'
 
 end

data/test/lib/vedeu/bindings/refresh_test.rb

@@ -8,6 +8,7 @@ module Vedeu
 
       context 'the system events needed by Vedeu to run are defined' do
         it { Vedeu.bound?(:_refresh_).must_equal(true) }
+        it { Vedeu.bound?(:_refresh_border_).must_equal(true) }
         it { Vedeu.bound?(:_refresh_cursor_).must_equal(true) }
         it { Vedeu.bound?(:_refresh_group_).must_equal(true) }
         it { Vedeu.bound?(:_refresh_view_).must_equal(true) }

data/test/lib/vedeu/borders/border_test.rb

@@ -17,7 +17,7 @@ module Vedeu
           colour:       nil,
           enabled:      false,
           horizontal:   'q',
-          name:         'borders',
+          name:         _name,
           repository:   Vedeu.borders,
           show_bottom:  true,
           show_left:    true,
@@ -31,6 +31,7 @@ module Vedeu
         }
       }
       let(:geometry) {}
+      let(:_name)    { 'borders' }
 
       describe '.build' do
         subject {
@@ -96,7 +97,7 @@ module Vedeu
             bottom_right: 'D',
             enabled:      enabled,
             horizontal:   'H',
-            name:         'Border#bxbxnbybyn',
+            name:         _name,
             show_top:     top,
             show_bottom:  bottom,
             show_left:    left,
@@ -106,13 +107,14 @@ module Vedeu
             vertical:     'V'
           }
         }
+        let(:_name)   { 'Border#bxbxnbybyn' }
         let(:enabled) { false }
         let(:top)     { false }
         let(:bottom)  { false }
         let(:left)    { false }
         let(:right)   { false }
         let(:geometry) {
-          Vedeu::Geometry::Geometry.new(name: 'Border#bxbxnbybyn',
+          Vedeu::Geometry::Geometry.new(name: _name,
                                         x:    2,
                                         xn:   6,
                                         y:    2,
@@ -212,7 +214,7 @@ module Vedeu
 
       describe '#width' do
         let(:geometry) {
-          Vedeu::Geometry::Geometry.new(name: 'borders', width: 8)
+          Vedeu::Geometry::Geometry.new(name: _name, width: 8)
         }
 
         before { Vedeu.geometries.stubs(:by_name).returns(geometry) }
@@ -230,7 +232,7 @@ module Vedeu
             let(:attributes) {
               {
                 enabled: true,
-                name:    'borders',
+                name:    _name,
               }
             }
 
@@ -241,7 +243,7 @@ module Vedeu
             let(:attributes) {
               {
                 enabled:   true,
-                name:      'borders',
+                name:      _name,
                 show_left: false
               }
             }
@@ -253,7 +255,7 @@ module Vedeu
             let(:attributes) {
               {
                 enabled:    true,
-                name:       'borders',
+                name:       _name,
                 show_left:  false,
                 show_right: false
               }
@@ -266,7 +268,7 @@ module Vedeu
 
       describe '#height' do
         let(:geometry) {
-          Vedeu::Geometry::Geometry.new(name: 'borders', height: 5)
+          Vedeu::Geometry::Geometry.new(name: _name, height: 5)
         }
 
         before { Vedeu.geometries.stubs(:by_name).returns(geometry) }
@@ -284,7 +286,7 @@ module Vedeu
             let(:attributes) {
               {
                 enabled: true,
-                name:    'borders',
+                name:    _name,
               }
             }
 
@@ -295,7 +297,7 @@ module Vedeu
             let(:attributes) {
               {
                 enabled:  true,
-                name:     'borders',
+                name:     _name,
                 show_top: false
               }
             }
@@ -307,7 +309,7 @@ module Vedeu
             let(:attributes) {
               {
                 enabled:     true,
-                name:        'borders',
+                name:        _name,
                 show_top:    false,
                 show_bottom: false
               }
@@ -327,7 +329,7 @@ module Vedeu
           let(:attributes) {
             {
               enabled: true,
-              name:    'borders',
+              name:    _name,
             }
           }
 
@@ -344,7 +346,7 @@ module Vedeu
           let(:attributes) {
             {
               enabled:     true,
-              name:        'borders',
+              name:        _name,
               show_bottom: false,
             }
           }
@@ -362,7 +364,7 @@ module Vedeu
           let(:attributes) {
             {
               enabled:   true,
-              name:      'borders',
+              name:      _name,
               show_left: false,
             }
           }
@@ -380,7 +382,7 @@ module Vedeu
           let(:attributes) {
             {
               enabled:    true,
-              name:       'borders',
+              name:       _name,
               show_right: false,
             }
           }
@@ -398,7 +400,7 @@ module Vedeu
           let(:attributes) {
             {
               enabled:  true,
-              name:     'borders',
+              name:     _name,
               show_top: false,
             }
           }
@@ -408,12 +410,12 @@ module Vedeu
       end
 
       describe '#render' do
-        before { Vedeu::Borders::Render.stubs(:render) }
+        before { Vedeu.stubs(:trigger).with(:_refresh_border_, _name) }
 
         subject { instance.render }
 
         it {
-          Vedeu::Borders::Render.expects(:render).with(instance)
+          Vedeu.expects(:trigger).with(:_refresh_border_, _name)
           subject
         }
       end

data/test/lib/vedeu/borders/{render_test.rb → refresh_test.rb}

@@ -4,10 +4,10 @@ module Vedeu
 
   module Borders
 
-    describe Render do
+    describe Refresh do
 
-      let(:described)   { Vedeu::Borders::Render }
-      let(:instance)    { described.new(border) }
+      let(:described)   { Vedeu::Borders::Refresh }
+      let(:instance)    { described.new(_name) }
       let(:border)      {
         Vedeu::Borders::Border.new(enabled:     enabled,
                                    name:        _name,
@@ -20,7 +20,7 @@ module Vedeu
       }
       let(:visible)     { false }
       let(:enabled)     { false }
-      let(:_name)       { 'Vedeu::RenderBorder' }
+      let(:_name)       { 'Vedeu::Borders::Refresh' }
       let(:title)       {}
       let(:caption)     {}
       let(:show_top)    { true }
@@ -30,14 +30,14 @@ module Vedeu
 
       describe '#initialize' do
         it { instance.must_be_instance_of(described) }
-        it { instance.instance_variable_get('@border').must_equal(border) }
+        it { instance.instance_variable_get('@name').must_equal(_name) }
       end
 
       describe '#render' do
         it { instance.must_respond_to(:render) }
       end
 
-      describe '.render' do
+      describe '.by_name' do
         let(:geometry) {
           Vedeu::Geometry::Geometry.new(name: _name, x: 1, xn: 7, y: 1, yn: 4)
         }
@@ -45,11 +45,12 @@ module Vedeu
           Vedeu::Models::Interface.new(name: _name, visible: true)
         }
         before do
+          Vedeu.borders.stubs(:by_name).returns(border)
           Vedeu.geometries.stubs(:by_name).returns(geometry)
           Vedeu.interfaces.stubs(:by_name).returns(interface)
         end
 
-        subject { described.render(border) }
+        subject { described.by_name(_name) }
 
         context 'when the border is enabled' do
           let(:enabled) { true }

data/test/lib/vedeu/buffers/refresh_test.rb

@@ -26,7 +26,7 @@ module Vedeu
         context 'when Vedeu is not yet ready' do
           let(:ready) { false }
 
-          it { subject.must_equal(nil) }
+          it { subject.must_equal([nil]) }
         end
 
         context 'when Vedeu is ready' do

data/test/lib/vedeu/models/views/view_test.rb

@@ -61,17 +61,11 @@ module Vedeu
 
       describe '#render' do
         before do
-          Vedeu.stubs(:trigger).with(:_clear_view_content_, _name)
           Vedeu::Output::Viewport.stubs(:render).with(instance)
         end
 
         subject { instance.render }
 
-        it {
-          Vedeu.expects(:trigger).with(:_clear_view_content_, _name)
-          subject
-        }
-
         it {
           Vedeu::Output::Viewport.expects(:render).with(instance)
           subject

data/test/lib/vedeu/output/viewport_test.rb

@@ -25,6 +25,7 @@ module Vedeu
         Vedeu::Models::Interface.new(style: nil, visible: visible)
       }
       let(:geometry)  { Vedeu::Geometry::Geometry.new(height: 3, width: 3) }
+      let(:ready)     { true }
 
       before do
         Vedeu.interfaces.stubs(:by_name).returns(interface)
@@ -40,15 +41,24 @@ module Vedeu
       describe '.render' do
         before do
           Vedeu::Output::Output.stubs(:render)
+          Vedeu.stubs(:ready?).returns(ready)
         end
 
         subject { described.render(view) }
 
         context 'when the interface is visible' do
-          it {
-            Vedeu::Output::Output.expects(:render)
-            subject
-          }
+          context 'and Vedeu is ready' do
+            it {
+              Vedeu::Output::Output.expects(:render)
+              subject
+            }
+          end
+
+          context 'but Vedeu is not ready' do
+            let(:ready) { false }
+
+            it { subject.must_equal(nil) }
+          end
         end
 
         context 'when the interface is not visible' do

data/test/lib/vedeu/terminal/buffer_test.rb

@@ -18,19 +18,9 @@ module Vedeu
       end
 
       describe '#buffer' do
-        let(:expected) {
-          Array.new(Vedeu.height) do |y|
-            Array.new(Vedeu.width) do |x|
-              Vedeu::Models::Cell.new(position: [y + 1, x + 1])
-            end
-          end
-        }
-
         subject { described.buffer }
 
         it { subject.must_be_instance_of(Array) }
-        it { described.instance_variable_get('@output').must_equal(expected) }
-        it { subject.must_equal(expected) }
       end
 
       describe '#clear' do
@@ -57,18 +47,9 @@ module Vedeu
       end
 
       describe '#empty_buffer' do
-        let(:expected) {
-          Array.new(height) do |y|
-            Array.new(width) do |x|
-              Vedeu::Models::Cell.new(position: [y + 1, x + 1])
-            end
-          end
-        }
-
         subject { described.empty_buffer }
 
         it { subject.must_be_instance_of(Array) }
-        it { subject.must_equal(expected) }
       end
 
       # describe '#output' do
@@ -121,19 +102,9 @@ module Vedeu
       end
 
       describe '#reset' do
-        let(:output) {
-          Array.new(height) do |y|
-            Array.new(width) do |x|
-              Vedeu::Models::Cell.new(position: [y + 1, x + 1])
-            end
-          end
-        }
-
         subject { described.reset }
 
-        it { described.instance_variable_get('@output').must_equal(output) }
         it { subject.must_be_instance_of(Array) }
-        it { subject.must_equal(output) }
       end
 
       describe '#write' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vedeu
 version: !ruby/object:Gem::Version
-  version: 0.6.22
+  version: 0.6.23
 platform: ruby
 authors:
 - Gavin Laking
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-01 00:00:00.000000000 Z
+date: 2015-10-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: guard
@@ -290,7 +290,7 @@ files:
 - lib/vedeu/borders/border.rb
 - lib/vedeu/borders/dsl.rb
 - lib/vedeu/borders/null.rb
-- lib/vedeu/borders/render.rb
+- lib/vedeu/borders/refresh.rb
 - lib/vedeu/borders/repository.rb
 - lib/vedeu/buffers/all.rb
 - lib/vedeu/buffers/buffer.rb
@@ -491,7 +491,7 @@ files:
 - test/lib/vedeu/borders/border_test.rb
 - test/lib/vedeu/borders/dsl_test.rb
 - test/lib/vedeu/borders/null_test.rb
-- test/lib/vedeu/borders/render_test.rb
+- test/lib/vedeu/borders/refresh_test.rb
 - test/lib/vedeu/borders/repository_test.rb
 - test/lib/vedeu/buffers/buffer_test.rb
 - test/lib/vedeu/buffers/null_test.rb
@@ -704,7 +704,7 @@ test_files:
 - test/lib/vedeu/borders/border_test.rb
 - test/lib/vedeu/borders/dsl_test.rb
 - test/lib/vedeu/borders/null_test.rb
-- test/lib/vedeu/borders/render_test.rb
+- test/lib/vedeu/borders/refresh_test.rb
 - test/lib/vedeu/borders/repository_test.rb
 - test/lib/vedeu/buffers/buffer_test.rb
 - test/lib/vedeu/buffers/null_test.rb