shoes-swt 4.0.0.pre6 → 4.0.0.pre7

data/spec/shoes/swt/arrow_painter_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+
+describe Shoes::Swt::ArrowPainter do
+  include_context "painter context"
+
+  let(:corners) { 0 }
+  let(:app) { double('app', real: container, add_paint_listener: true, dsl: dsl) }
+  let(:container) { double('container', disposed?: false) }
+
+  let(:dsl) do
+    double("dsl object", hidden: false, rotate: 0,
+                         translate_left: 0, translate_top: 0,
+                         left: left, top: top,
+                         width: width, height: height,
+                         style: {}, strokewidth: 1).as_null_object
+  end
+
+  let(:left) { 55 }
+  let(:top) { 77 }
+  let(:width) { 222 }
+  let(:height) { 222 }
+  let(:shape) { Shoes::Swt::Arrow.new dsl, app }
+  subject { Shoes::Swt::ArrowPainter.new shape }
+
+  it_behaves_like "fill painter"
+  it_behaves_like "stroke painter"
+
+  it "draws on path" do
+    expect(gc).to receive(:draw_path).with(subject.path)
+    subject.paint_control(event)
+  end
+
+  it "fills on path" do
+    expect(gc).to receive(:fill_path).with(subject.path)
+    subject.paint_control(event)
+  end
+end

data/spec/shoes/swt/arrow_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe Shoes::Swt::Arrow do
+  include_context "swt app"
+
+  let(:left) { 55 }
+  let(:top) { 77 }
+  let(:width) { 222 }
+  let(:height) { 222 }
+  let(:dsl) { ::Shoes::Arrow.new shoes_app, parent, left, top, width }
+
+  subject { Shoes::Swt::Arrow.new dsl, swt_app }
+
+  it_behaves_like "paintable"
+  it_behaves_like "updating visibility"
+  it_behaves_like 'clickable backend'
+
+  it "clears the path" do
+    expect(subject.painter).to receive(:clear_path)
+    subject.update_position
+  end
+
+  it "disposes of the painter" do
+    expect(subject.painter).to receive(:dispose)
+    subject.dispose
+  end
+end

data/spec/shoes/swt/background_spec.rb

@@ -9,15 +9,17 @@ describe Shoes::Swt::Background do
   let(:height) { 111 }
   let(:corners) { 0 }
   let(:fill) { Shoes::COLORS[:salmon] }
-  let(:dsl) { double("dsl object", app: shoes_app,
-                                   element_left: left, element_top: top,
-                                   element_width: width, element_height: height,
-                                   strokewidth: 1, curve: corners, fill: fill,
-                                   hidden: false).as_null_object}
+  let(:dsl) do
+    double("dsl object", app: shoes_app,
+                         element_left: left, element_top: top,
+                         element_width: width, element_height: height,
+                         translate_left: 0, translate_top: 0,
+                         style: { strokewidth: 1 }, strokewidth: 1,
+                         curve: corners, fill: fill,
+                         hidden: false).as_null_object
+  end
 
-  subject {
-    Shoes::Swt::Background.new dsl, swt_app
-  }
+  subject { Shoes::Swt::Background.new dsl, swt_app }
 
   context "#initialize" do
     it { is_expected.to be_an_instance_of(Shoes::Swt::Background) }
@@ -41,7 +43,7 @@ describe Shoes::Swt::Background do
     include_context "painter context"
 
     let(:corners) { 0 }
-    let(:shape) { Shoes::Swt::Background.new dsl, swt_app}
+    let(:shape) { Shoes::Swt::Background.new dsl, swt_app }
     subject { Shoes::Swt::Background::Painter.new shape }
 
     it_behaves_like "fill painter"
@@ -50,7 +52,7 @@ describe Shoes::Swt::Background do
       let(:corners) { 0 }
 
       it "fills rect" do
-        expect(gc).to receive(:fill_round_rectangle).with(left, top, width, height, corners*2, corners*2)
+        expect(gc).to receive(:fill_round_rectangle).with(left, top, width, height, corners * 2, corners * 2)
         subject.paint_control(event)
       end
     end
@@ -59,7 +61,7 @@ describe Shoes::Swt::Background do
       let(:corners) { 13 }
 
       it "fills rect" do
-        expect(gc).to receive(:fill_round_rectangle).with(left, top, width, height, corners*2, corners*2)
+        expect(gc).to receive(:fill_round_rectangle).with(left + 1, top + 1, width - 2, height - 2, corners * 2, corners * 2)
         subject.paint_control(event)
       end
     end

data/spec/shoes/swt/border_spec.rb

@@ -8,9 +8,12 @@ describe Shoes::Swt::Border do
   let(:width) { 222 }
   let(:height) { 111 }
   let(:corners) { 0 }
-  let(:dsl) { double("dsl object", element_width: width, element_height: height,
-                                   element_left: left, element_top: top, parent: parent,
-                                   strokewidth: 1, curve: corners,hidden: false).as_null_object }
+  let(:dsl) do
+    double("dsl object", element_width: width, element_height: height,
+                         element_left: left, element_top: top, parent: parent,
+                         translate_left: 0, translate_top: 0,
+                         strokewidth: 1, curve: corners, hidden: false).as_null_object
+  end
 
   subject { Shoes::Swt::Border.new dsl, swt_app }
 
@@ -35,7 +38,7 @@ describe Shoes::Swt::Border do
       let(:corners) { 0 }
 
       specify "draws rect" do
-        expect(gc).to receive(:draw_round_rectangle).with(left+sw/2, top+sw/2, width-sw, height-sw, corners*2, corners*2)
+        expect(gc).to receive(:draw_round_rectangle).with(left + sw / 2, top + sw / 2, width - sw, height - sw, corners * 2, corners * 2)
         subject.paint_control(event)
       end
     end
@@ -44,7 +47,7 @@ describe Shoes::Swt::Border do
       let(:corners) { 13 }
 
       specify "draws rect" do
-        expect(gc).to receive(:draw_round_rectangle).with(left+sw/2, top+sw/2, width-sw, height-sw, corners*2, corners*2)
+        expect(gc).to receive(:draw_round_rectangle).with(left + sw / 2, top + sw / 2, width - sw, height - sw, corners * 2, corners * 2)
         subject.paint_control(event)
       end
     end

data/spec/shoes/swt/button_spec.rb

@@ -28,10 +28,10 @@ describe Shoes::Swt::Button do
     end
 
     describe 'size' do
-      let(:width) {50}
-      let(:height) {20}
-      let(:size) {double 'real size', x: width, y: height}
-      let(:real) {double('button real', size: size, pack: true).as_null_object}
+      let(:width) { 50 }
+      let(:height) { 20 }
+      let(:size) { double 'real size', x: width, y: height }
+      let(:real) { double('button real', size: size, pack: true).as_null_object }
 
       before :each do
         allow(::Swt::Widgets::Button).to receive(:new) { real }

data/spec/shoes/swt/check_spec.rb

@@ -4,14 +4,25 @@ describe Shoes::Swt::Check do
   include_context "swt app"
 
   let(:text) { "TEXT" }
-  let(:dsl) { double('dsl',
-                     :app    => shoes_app, :visible? => true,
-                     :left   => 42, :top => 66,
-                     :element_left => 42, element_top: 66,
-                     :width  => 100, :width= => true,
-                     :element_width => 100, element_height: 200,
-                     :height => 200, :height= => true,
-                     blk:    block, contents: []) }
+
+  let(:dsl) do
+    double('dsl', app: shoes_app,
+                  gui: real,
+                  visible?: true,
+                  left: 42,
+                  top: 66,
+                  element_left: 42,
+                  element_top: 66,
+                  width: 100,
+                  :width= => true,
+                  element_width: 100,
+                  element_height: 200,
+                  height: 200,
+                  :height= => true,
+                  blk: block,
+                  contents: [])
+  end
+
   let(:block) { proc {} }
   let(:real) { double('real').as_null_object }
 

data/spec/shoes/swt/click_listener_spec.rb

@@ -8,7 +8,7 @@ describe Shoes::Swt::ClickListener do
   let(:click_block)   { double("click block", call: nil) }
   let(:release_block) { double("release block", call: nil) }
 
-  subject   { Shoes::Swt::ClickListener.new(swt_app) }
+  subject { Shoes::Swt::ClickListener.new(swt_app) }
 
   before do
     allow(swt_app).to receive(:add_listener)

data/spec/shoes/swt/common/painter_spec.rb

@@ -1,20 +1,35 @@
 require 'spec_helper'
 
 describe Shoes::Swt::Common::Painter do
-  let(:object) {double 'object', dsl: dsl, transform: transform,
-                                 apply_fill: nil, apply_stroke: nil}
-  let(:parent) {double 'parent', absolute_left: 0, absolute_top: 0,
-                       width: 200, height: 100, fixed_height?: true}
-  let(:dsl) {double 'dsl', parent: parent,
-                    visible?: true, positioned?: true, style: {}}
-  let(:event) {double 'paint event', gc: graphics_context}
-  let(:graphics_context) { double 'graphics_context',
-                                  dispose: nil,
-                                  clipping: nil, set_clipping: nil,
-                                  set_antialias: nil, set_line_cap: nil,
-                                  set_transform: nil, setTransform: nil }
+  let(:object) do
+    double 'object', dsl: dsl, transform: transform, apply_fill: nil,
+                     apply_stroke: nil
+  end
+
+  let(:parent) do
+    double 'parent', absolute_left: 0, absolute_top: 0, width: 200, height: 100,
+                     fixed_height?: true
+  end
+
+  let(:dsl) do
+    double 'dsl', parent: parent, visible?: true, positioned?: true, style: {}
+  end
+
+  let(:event) { double 'paint event', gc: graphics_context }
+
+  let(:graphics_context) do
+    double 'graphics_context', dispose: nil,
+                               clipping: nil,
+                               set_clipping: nil,
+                               set_antialias: nil,
+                               set_line_cap: nil,
+                               set_transform: nil,
+                               setTransform: nil
+  end
+
   let(:transform) { double 'transform', disposed?: false }
-  subject {Shoes::Swt::Common::Painter.new object}
+
+  subject { Shoes::Swt::Common::Painter.new object }
 
   before do
     allow(::Swt::Transform).to receive(:new) { transform }
@@ -71,7 +86,7 @@ describe Shoes::Swt::Common::Painter do
       expect_transform_for_rotate
 
       subject.set_rotate graphics_context, 0, 0, 0 do
-        #no-op
+        # no-op
       end
     end
   end

data/spec/shoes/swt/common/remove_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Shoes::Swt::Common::Remove do
-  let(:clazz)  {
+  let(:clazz) do
     Class.new do
       include Shoes::Swt::Common::Remove
 
@@ -13,7 +13,7 @@ describe Shoes::Swt::Common::Remove do
         @real = real
       end
     end
-  }
+  end
 
   let(:app)  { double("app", click_listener: click_listener) }
   let(:dsl)  { double("dsl") }

data/spec/shoes/swt/dialog_spec.rb

@@ -26,7 +26,7 @@ describe Shoes::Swt::Dialog do
     @dialog = Shoes::Swt::Dialog.new
   end
 
-  TEXT = 'some random text'
+  TEXT = 'some random text'.freeze
 
   describe 'alert' do
     it 'pops up a window containing a short message.' do

data/spec/shoes/swt/flow_spec.rb

@@ -3,8 +3,10 @@ require 'spec_helper'
 describe Shoes::Swt::Flow do
   include_context "swt app"
 
-  let(:dsl) { double('dsl', app: shoes_app,
-                            pass_coordinates?: true).as_null_object }
+  let(:dsl) do
+    double('dsl', app: shoes_app, pass_coordinates?: true).as_null_object
+  end
+
   let(:real) { double('real', disposed?: false) }
   let(:parent_real) { double('parent_real', get_layout: "ok") }
 
@@ -24,7 +26,7 @@ describe Shoes::Swt::Flow do
   describe "#initialize" do
     before do
       allow(parent).to receive(:real) { parent_real }
-      allow(parent_real).to receive(:get_layout) {double(top_slot: true)}
+      allow(parent_real).to receive(:get_layout) { double(top_slot: true) }
     end
 
     it "sets readers" do

data/spec/shoes/swt/font_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Shoes::Swt::Font do
-  subject {Shoes::Swt::Font}
+  subject { Shoes::Swt::Font }
 
   describe 'Shoes::FONTS' do
     it 'has the FONTS array initially populared' do

data/spec/shoes/swt/gradient_spec.rb

@@ -3,9 +3,12 @@ require 'spec_helper'
 describe Shoes::Swt::Gradient do
   let(:color1) { Shoes::Color.create(Shoes::COLORS[:honeydew]) }
   let(:color2) { Shoes::Color.create(Shoes::COLORS[:salmon]) }
-  let(:applied_to) { double("applied to",
-                            element_left: 0, element_top: 0, angle: 0,
-                            element_width: 10, element_height: 10) }
+
+  let(:applied_to) do
+    double("applied to", element_left: 0, element_top: 0, angle: 0,
+                         element_width: 10, element_height: 10)
+  end
+
   let(:dsl) { Shoes::Gradient.new(color1, color2) }
   let(:gc) { double("gc", set_background_pattern: nil) }
 

data/spec/shoes/swt/image_spec.rb

@@ -7,19 +7,19 @@ describe Shoes::Swt::Image do
   IMAGE_HEIGHT = 1
 
   let(:blk) { double("block") }
-  let(:dsl) { Shoes::Image.new shoes_app, parent_dsl, image, opts}
-  let(:opts) {{left: left, top: top, width: width, height: height}}
+  let(:dsl) { Shoes::Image.new shoes_app, parent_dsl, image, opts }
+  let(:opts) { {left: left, top: top, width: width, height: height} }
   let(:left) { 100 }
   let(:top) { 200 }
   let(:height) { nil }
-  let(:width) {nil}
+  let(:width) { nil }
   let(:image_path) { File.dirname(__FILE__) + '/minimal.png' }
   let(:image) { image_path }
 
-  subject {
+  subject do
     allow(dsl).to receive(:file_path) { image }
     Shoes::Swt::Image.new(dsl, swt_app)
-  }
+  end
 
   it_behaves_like "paintable"
   it_behaves_like "removable"
@@ -71,14 +71,14 @@ describe Shoes::Swt::Image do
     end
 
     describe 'with a given width' do
-      let(:width) {(IMAGE_WIDTH * 5.8).to_i}
+      let(:width) { (IMAGE_WIDTH * 5.8).to_i }
       it 'scales the height' do
         expect(subject.height).to eq((IMAGE_HEIGHT * 5.8).to_i)
       end
     end
 
     describe 'with a given height' do
-      let(:height) {IMAGE_HEIGHT * 4}
+      let(:height) { IMAGE_HEIGHT * 4 }
 
       it 'scales the width' do
         expect(subject.width).to eq(IMAGE_WIDTH * 4)
@@ -86,8 +86,8 @@ describe Shoes::Swt::Image do
     end
 
     describe 'with a given width and height' do
-      let(:width) {1}
-      let(:height) {2}
+      let(:width) { 1 }
+      let(:height) { 2 }
       it 'sets the given width' do
         expect(subject.width).to eq(1)
       end

data/spec/shoes/swt/input_box_spec.rb

@@ -3,15 +3,20 @@ require 'spec_helper'
 describe Shoes::Swt::InputBox do
   include_context "swt app"
 
-  let(:dsl) { double('dsl', app: shoes_app, visible?: true, element_width: 80,
-                            element_height: 22, initial_text: 'Jay',
-                            secret?: secret,
-                            call_change_listeners: true).as_null_object }
-  let(:real) { double('real', disposed?: false, text: text,
-                              add_modify_listener: true).as_null_object }
-  let(:styles) {::Swt::SWT::SINGLE | ::Swt::SWT::BORDER}
-  let(:secret) {false}
-  let(:text) {'Some text...'}
+  let(:dsl) do
+    double('dsl', app: shoes_app, visible?: true, element_width: 80,
+                  element_height: 22, initial_text: 'Jay',
+                  secret?: secret, call_change_listeners: true).as_null_object
+  end
+
+  let(:real) do
+    double('real', disposed?: false, text: text, add_modify_listener: true)
+      .as_null_object
+  end
+
+  let(:styles) { ::Swt::SWT::SINGLE | ::Swt::SWT::BORDER }
+  let(:secret) { false }
+  let(:text) { 'Some text...' }
 
   subject { Shoes::Swt::InputBox.new dsl, swt_app, styles }
 
@@ -25,8 +30,8 @@ describe Shoes::Swt::InputBox do
   it_behaves_like "updating visibility"
 
   describe "#initialize" do
-    let(:event) {double 'Event', source: source}
-    let(:source) {double 'Source'}
+    let(:event) { double 'Event', source: source }
+    let(:source) { double 'Source' }
     it "sets text on real element" do
       subject.text = "some text"
       expect(real).to have_received(:text=).with("some text")
@@ -57,9 +62,9 @@ describe Shoes::Swt::InputBox do
         end
 
         describe 'with the same text' do
-          let(:event) {double 'Bad Event', source: source}
-          let(:source) {double 'Our source', text: text}
-          let(:text) {'Double call'}
+          let(:event) { double 'Bad Event', source: source }
+          let(:source) { double 'Our source', text: text }
+          let(:text) { 'Double call' }
 
           it 'does not call the change listeners' do
             subject.text = text
@@ -90,7 +95,7 @@ describe Shoes::Swt::InputBox do
   end
 
   describe Shoes::Swt::EditLine do
-    subject {Shoes::Swt::EditLine.new dsl, swt_app}
+    subject { Shoes::Swt::EditLine.new dsl, swt_app }
     describe ":secret option" do
       context "when NOT set" do
         it "does NOT set PASSWORD style" do