shoes-swt 4.0.0.pre4 → 4.0.0.pre5

data/spec/shoes/swt/app_spec.rb

@@ -1,22 +1,34 @@
-require "shoes/swt/spec_helper"
+require 'spec_helper'
 
 describe Shoes::Swt::App do
-  let(:opts) { {:background => Shoes::COLORS[:salmon], :resizable => true} }
+  let(:opts) { {background: Shoes::COLORS[:salmon], resizable: true} }
   let(:app)  { double('app') }
-  let(:dsl)  { double('dsl', :app => app,
-                             :opts => opts,
-                             :width => width,
-                             :height => 0,
-                             :app_title => 'double') }
-
-  let(:opts_unresizable) { {:background => Shoes::COLORS[:salmon],
-                            :resizable => false} }
-  let(:app_unresizable) { double('app', :opts => opts_unresizable,
-                                        :width => 0,
-                                        :height => 0,
-                                        :app_title => 'double') }
+  let(:dsl)  { double('dsl', app: app,
+                             opts: opts,
+                             width: width,
+                             height: 0,
+                             app_title: 'double') }
+
+  let(:opts_unresizable) { { background: Shoes::COLORS[:salmon],
+                            resizable: false } }
+  let(:app_unresizable) { double('app', opts: opts_unresizable,
+                                        width: 0,
+                                        height: 0,
+                                        app_title: 'double') }
+  let(:opts_always_on_top) { {background: Shoes::COLORS[:salmon],
+                              always_on_top: true} }
+  let(:app_always_on_top) { double('app', opts: opts_always_on_top,
+                                          width: 0,
+                                          height: 0,
+                                          app_title: 'double') }
+  let(:plain_app) { double('app', opts: {},
+                                  width: 2,
+                                  height: 2,
+                                  app_title: 'double') }
   let(:width) {0}
 
+  let(:swt_salmon) { Shoes::Swt::Color.new(Shoes::COLORS[:salmon]).real }
+
   subject { Shoes::Swt::App.new(dsl) }
 
   it { is_expected.to respond_to :clipboard }
@@ -76,6 +88,11 @@ describe Shoes::Swt::App do
       not_resizable = Shoes::Swt::App.new app_unresizable
       expect(not_resizable.send(:main_window_style)).to eq(BASE_BITMASK)
     end
+
+    it "should return a bitmask that represents always being on top" do
+      always_on_top = Shoes::Swt::App.new app_always_on_top
+      expect(always_on_top.send(:main_window_style)).to eq(BASE_BITMASK | Swt::SWT::ON_TOP)
+    end
   end
 
   context "when attempting to copy text" do
@@ -123,4 +140,29 @@ describe Shoes::Swt::App do
     end
   end
 
+  describe 'App Background color' do
+    it 'has the given background when specified' do
+      not_resizable = Shoes::Swt::App.new app_unresizable
+      background = not_resizable.shell.background
+      expect(background).to eq swt_salmon
+    end
+
+
+    it 'has the default system background when unspecified' do
+      default_background = ::Swt.display.getSystemColor(::Swt::SWT::COLOR_WIDGET_BACKGROUND)
+      app = Shoes::Swt::App.new(Shoes::InternalApp.new(Shoes::App.new, {}))
+      background = app.shell.background
+      expect(background).to eq default_background
+    end
+
+    it 'setup_system_colors' do
+      default_background = ::Swt.display.getSystemColor(::Swt::SWT::COLOR_WIDGET_BACKGROUND)
+      expect(::Shoes::DSL).to receive(:define_shoes_color)
+                              .with(:system_background,
+                                    default_background.red,
+                                    default_background.green,
+                                    default_background.blue)
+      subject.class.setup_system_colors
+    end
+  end
 end

data/spec/shoes/swt/arc_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Arc do
   include_context "swt app"
@@ -10,10 +10,10 @@ describe Shoes::Swt::Arc do
   let(:angle1) { Shoes::PI }
   let(:angle2) { Shoes::HALF_PI }
   let(:dsl) { double("dsl object", app: shoes_app, element_width: width,
-                     element_height: height, element_left: left,
-                     element_top: top, angle1: angle1, angle2: angle2,
-                     wedge?: false, pass_coordinates?: nil,
-                     hidden: false).as_null_object }
+                                   element_height: height, element_left: left,
+                                   element_top: top, angle1: angle1, angle2: angle2,
+                                   wedge?: false, pass_coordinates?: nil,
+                                   hidden: false).as_null_object }
   let(:fill_color) { Shoes::Color.new(40, 50, 60, 70) }
   let(:stroke_color) { Shoes::Color.new(80, 90, 100, 110) }
 

data/spec/shoes/swt/background_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Background do
   include_context "swt app"
@@ -10,10 +10,10 @@ describe Shoes::Swt::Background do
   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}
+                                   element_left: left, element_top: top,
+                                   element_width: width, element_height: height,
+                                   strokewidth: 1, curve: corners, fill: fill,
+                                   hidden: false).as_null_object}
 
   subject {
     Shoes::Swt::Background.new dsl, swt_app

data/spec/shoes/swt/border_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Border do
   include_context "swt app"
@@ -9,8 +9,8 @@ describe Shoes::Swt::Border do
   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 }
+                                   element_left: left, element_top: top, parent: parent,
+                                   strokewidth: 1, curve: corners,hidden: false).as_null_object }
 
   subject { Shoes::Swt::Border.new dsl, swt_app }
 

data/spec/shoes/swt/button_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Button do
   include_context "swt app"
@@ -8,11 +8,9 @@ describe Shoes::Swt::Button do
   let(:block) { proc {} }
   let(:real) { double('real', disposed?: false).as_null_object }
 
-  subject { Shoes::Swt::Button.new dsl, parent }
+  subject { Shoes::Swt::Button.new dsl, swt_app }
 
   before :each do
-    allow(parent).to receive(:real)
-    allow(parent).to receive(:dsl){double(contents: [])}
     allow(dsl).to receive(:width=)
     allow(dsl).to receive(:height=)
     allow(::Swt::Widgets::Button).to receive(:new) { real }
@@ -36,8 +34,6 @@ describe Shoes::Swt::Button do
       let(:real) {double('button real', size: size, pack: true).as_null_object}
 
       before :each do
-        allow(parent).to receive(:real)
-        allow(parent).to receive(:dsl){double(contents: [])}
         allow(::Swt::Widgets::Button).to receive(:new) { real }
       end
 
@@ -50,7 +46,7 @@ describe Shoes::Swt::Button do
         dsl = dsl_for_dimensions input_width, input_height
         expect(dsl).to receive(:element_width=).with(expected_width) unless input_width
         expect(dsl).to receive(:element_height=).with(expected_height) unless input_height
-        Shoes::Swt::Button.new dsl, parent
+        Shoes::Swt::Button.new dsl, swt_app
       end
 
       it 'uses only real when width and height are not specified' do

data/spec/shoes/swt/check_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Check do
   include_context "swt app"

data/spec/shoes/swt/click_listener_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::ClickListener do
   include_context 'swt app'
@@ -100,7 +100,7 @@ describe Shoes::Swt::ClickListener do
 
       it "takes the last element to respond" do
         other_dsl = double("other dsl", in_bounds?: true,
-                           hidden?: false, pass_coordinates?: false)
+                                        hidden?: false, pass_coordinates?: false)
         other_block = double("other block", call: nil)
 
         subject.send(add_method, other_dsl, other_block)

data/spec/shoes/swt/color_factory_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::ColorFactory do
   let(:blue)     { Shoes::COLORS[:blue] }

data/spec/shoes/swt/color_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Color do
   subject(:color)  { Shoes::Swt::Color.create(Shoes::COLORS[:salmon]) }

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

@@ -1,12 +1,16 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Common::Painter do
   let(:object) {double 'object', dsl: dsl, transform: transform,
-                       apply_fill: nil, apply_stroke: nil}
-  let(:dsl) {double 'dsl', visible?: true, positioned?: true, style: Hash.new}
+                                 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(:transform) { double 'transform', disposed?: false }
@@ -34,6 +38,12 @@ describe Shoes::Swt::Common::Painter do
       subject.paint_control event
     end
 
+    it 'clips to parent region' do
+      allow(dsl).to receive(:needs_rotate?) { false }
+      expect(graphics_context).to receive(:set_clipping).with(0, 0, 200, 100)
+      subject.paint_control event
+    end
+
     it 'rotates' do
       allow(dsl).to receive(:needs_rotate?) { true }
       allow(dsl).to receive(:rotate) { 10 }

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

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Common::Remove do
   let(:clazz)  {

data/spec/shoes/swt/configuration_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Configuration do
   context ":swt" do
@@ -7,6 +7,5 @@ describe Shoes::Configuration do
         expect(Shoes.configuration.backend).to eq(Shoes::Swt)
       end
     end
-
   end
 end

data/spec/shoes/swt/dialog_spec.rb

@@ -1,9 +1,8 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 main_object = self
 
 describe Shoes::Swt::Dialog do
-
   def double_message_box
     create_double_message_box double(:mb, open: true, :message= => true)
   end
@@ -77,7 +76,6 @@ describe Shoes::Swt::Dialog do
 
   describe 'on the main object' do
     describe '#alert' do
-
       it 'returns nil' do
         double_message_box
         expect(main_object.alert('Something')).to be_nil
@@ -96,5 +94,4 @@ describe Shoes::Swt::Dialog do
       end
     end
   end
-
 end

data/spec/shoes/swt/flow_spec.rb

@@ -1,12 +1,12 @@
-require 'shoes/swt/spec_helper'
+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 }
+                            pass_coordinates?: true).as_null_object }
   let(:real) { double('real', disposed?: false) }
-  let(:parent_real) { double('parent_real', :get_layout => "ok") }
+  let(:parent_real) { double('parent_real', get_layout: "ok") }
 
   subject { Shoes::Swt::Flow.new(dsl, parent) }
 
@@ -24,7 +24,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,6 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Font do
-
   subject {Shoes::Swt::Font}
 
   describe 'Shoes::FONTS' do
@@ -29,5 +28,4 @@ describe Shoes::Swt::Font do
       subject.add_font font_path
     end
   end
-
 end

data/spec/shoes/swt/gradient_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Gradient do
   let(:color1) { Shoes::Color.create(Shoes::COLORS[:honeydew]) }

data/spec/shoes/swt/image_pattern_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::ImagePattern do
   let(:dsl)         { Shoes::ImagePattern.new("some/path/to") }

data/spec/shoes/swt/image_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Image do
   include_context "swt app"
@@ -18,7 +18,7 @@ describe Shoes::Swt::Image do
 
   subject {
     allow(dsl).to receive(:file_path) { image }
-    Shoes::Swt::Image.new(dsl, parent)
+    Shoes::Swt::Image.new(dsl, swt_app)
   }
 
   it_behaves_like "paintable"
@@ -42,7 +42,7 @@ describe Shoes::Swt::Image do
   end
 
   describe "painting raw images" do
-    let(:image) { File.read(image_path, :mode => "rb") }
+    let(:image) { File.read(image_path, mode: "rb") }
 
     specify "loads image from raw data" do
       subject.real.image_data.width = 3
@@ -52,7 +52,6 @@ describe Shoes::Swt::Image do
 
   # note the image used is 3x1 pixel big
   describe 'dimensions' do
-
     it 'has the given width' do
       expect(subject.width).to eq(IMAGE_WIDTH)
     end

data/spec/shoes/swt/input_box_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::InputBox do
   include_context "swt app"
@@ -8,12 +8,12 @@ describe Shoes::Swt::InputBox do
                             secret?: secret,
                             call_change_listeners: true).as_null_object }
   let(:real) { double('real', disposed?: false, text: text,
-                      add_modify_listener: true).as_null_object }
+                              add_modify_listener: true).as_null_object }
   let(:styles) {::Swt::SWT::SINGLE | ::Swt::SWT::BORDER}
   let(:secret) {false}
   let(:text) {'Some text...'}
 
-  subject { Shoes::Swt::InputBox.new dsl, parent, styles }
+  subject { Shoes::Swt::InputBox.new dsl, swt_app, styles }
 
   before :each do
     allow(::Swt::Widgets::Text).to receive(:new) { real }
@@ -33,7 +33,6 @@ describe Shoes::Swt::InputBox do
     end
 
     describe 'change listeners' do
-
       it "should set up a listener that delegates change events" do
         expect(dsl).to receive(:call_change_listeners)
         expect(real).to receive(:add_modify_listener) do |&blk|
@@ -74,7 +73,6 @@ describe Shoes::Swt::InputBox do
             expect(dsl).to_not have_received :call_change_listeners
           end
         end
-
       end
     end
   end
@@ -92,13 +90,13 @@ describe Shoes::Swt::InputBox do
   end
 
   describe Shoes::Swt::EditLine do
-    subject {Shoes::Swt::EditLine.new dsl, parent}
+    subject {Shoes::Swt::EditLine.new dsl, swt_app}
     describe ":secret option" do
       context "when NOT set" do
         it "does NOT set PASSWORD style" do
           options = Shoes::Swt::EditLine::DEFAULT_STYLES
           allow(dsl).to receive(:secret?) { false }
-          expect(::Swt::Widgets::Text).to receive(:new).with(parent.real, options)
+          expect(::Swt::Widgets::Text).to receive(:new).with(swt_app.real, options)
           subject
         end
       end
@@ -107,7 +105,7 @@ describe Shoes::Swt::InputBox do
         it "sets PASSWORD style" do
           options = Shoes::Swt::EditLine::DEFAULT_STYLES | ::Swt::SWT::PASSWORD
           allow(dsl).to receive(:secret?) { true }
-          expect(::Swt::Widgets::Text).to receive(:new).with(parent.real, options)
+          expect(::Swt::Widgets::Text).to receive(:new).with(swt_app.real, options)
           subject
         end
       end

data/spec/shoes/swt/integration_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::App do
   before :each do

data/spec/shoes/swt/key_listener_spec.rb

@@ -1,10 +1,10 @@
 # encoding: UTF-8
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Keypress do
   let(:app) { double('app', add_key_listener: nil, remove_key_listener: nil) }
   let(:dsl) { double('dsl') }
-  let(:block) { proc{ |key| key} }
+  let(:block) { proc { |key| key} }
   let(:key_listener) {Shoes::Swt::Keypress.new(dsl, app, &block)}
 
   describe '.get_swt_constant' do
@@ -14,7 +14,6 @@ describe Shoes::Swt::Keypress do
   end
 
   describe "subclasses" do
-
     describe "Subclass Keypress" do
       it "adds key listener on creation" do
         expect(app).to receive(:add_key_listener)
@@ -38,14 +37,15 @@ describe Shoes::Swt::Keypress do
   CTRL = ::Swt::SWT::CTRL
   ALT = ::Swt::SWT::ALT
   SHIFT = ::Swt::SWT::SHIFT
+  COMMAND = ::Swt::SWT::COMMAND
 
   subject {key_listener}
 
   def test_character_press(character, state_modifier = 0, result_char = character)
     expect(block).to receive(:call).with(result_char)
     event = double  character: character.ord,
-                  stateMask: 0 | state_modifier,
-                  keyCode: character.downcase.ord
+                    stateMask: 0 | state_modifier,
+                    keyCode: character.downcase.ord
     subject.handle_key_event(event)
   end
 
@@ -55,6 +55,11 @@ describe Shoes::Swt::Keypress do
     test_character_press(character, state_modifier, result)
   end
 
+  def test_command_character_press(character, state_mask_modifier = 0)
+    state_modifier = COMMAND | state_mask_modifier
+    result = ('super_' + character).to_sym
+    test_character_press(character, state_modifier, result)
+  end
 
   describe 'works with simple keys such as' do
     it '"a"' do
@@ -154,6 +159,20 @@ describe Shoes::Swt::Keypress do
     end
   end
 
+  describe 'works with command key pressed such as' do
+    it ':super_a' do
+      test_command_character_press 'a'
+    end
+
+    it ':super_z' do
+      test_command_character_press 'z'
+    end
+
+    it ':super_/' do
+      test_command_character_press '/'
+    end
+  end
+
   describe 'only modifier keys yield nothing' do
     def test_receive_nothing_with_modifier(modifier, last_key_press = modifier)
       expect(block).not_to receive :call
@@ -173,10 +192,18 @@ describe Shoes::Swt::Keypress do
       test_receive_nothing_with_modifier CTRL
     end
 
+    it 'command' do
+      test_receive_nothing_with_modifier COMMAND
+    end
+
     it 'shift + ctrl' do
       test_receive_nothing_with_modifier SHIFT | CTRL, SHIFT
     end
 
+    it 'shift + command' do
+      test_receive_nothing_with_modifier SHIFT | COMMAND, SHIFT
+    end
+
     it 'ctrl + alt' do
       test_receive_nothing_with_modifier CTRL | ALT, CTRL
     end
@@ -184,17 +211,20 @@ describe Shoes::Swt::Keypress do
     it 'shift + ctrl + alt' do
       test_receive_nothing_with_modifier CTRL | SHIFT | ALT, ALT
     end
+
+    it 'shift + ctrl + command + alt' do
+      test_receive_nothing_with_modifier CTRL | SHIFT | ALT | COMMAND, ALT
+    end
   end
 
   describe 'special keys' do
-
     ARROW_LEFT = ::Swt::SWT::ARROW_LEFT
 
     def special_key_test(code, expected, modifier = 0)
       expect(block).to receive(:call).with(expected)
       event = double stateMask: modifier,
-                   keyCode: code,
-                   character: 0
+                     keyCode: code,
+                     character: 0
       subject.handle_key_event(event)
     end
 
@@ -253,19 +283,17 @@ describe Shoes::Swt::Keypress do
   end
 
   describe '#ignore_event?' do
-
     let(:character) {'a'}
     let(:event) {double 'key event',
                         widget: widget,
                         stateMask: 0,
                         keyCode:  keyCode,
                         character: character.ord }
-    let(:shell){Java::OrgEclipseSwtWidgets::Shell.new}
+    let(:shell) {Java::OrgEclipseSwtWidgets::Shell.new}
     let(:style) {0}
     let(:keyCode) {character.downcase.ord}
 
-
-    subject{key_listener.ignore_event? event}
+    subject {key_listener.ignore_event? event}
 
     shared_examples_for 'ignores space and enter' do
       describe 'with a space' do
@@ -282,12 +310,12 @@ describe Shoes::Swt::Keypress do
     shared_examples_for 'accepts normal characters' do
       describe 'with a normal character' do
         let(:character) {'a'}
-        it{is_expected.to be_falsey}
+        it {is_expected.to be_falsey}
       end
     end
 
     context 'on a Shell' do
-      let(:widget){shell}
+      let(:widget) {shell}
       it {is_expected.to be_falsey}
 
       describe 'even with enter' do
@@ -297,7 +325,7 @@ describe Shoes::Swt::Keypress do
     end
 
     context 'on a Text' do
-      let(:widget){Java::OrgEclipseSwtWidgets::Text.new(shell, style)}
+      let(:widget) {Java::OrgEclipseSwtWidgets::Text.new(shell, style)}
       it {is_expected.to be_truthy}
     end
 
@@ -315,11 +343,10 @@ describe Shoes::Swt::Keypress do
 
       describe 'with up' do
         let(:keyCode) {::Swt::SWT::ARROW_UP}
-        it{is_expected.to be_truthy}
+        it {is_expected.to be_truthy}
       end
 
       it_behaves_like 'accepts normal characters'
     end
   end
-
 end

data/spec/shoes/swt/line_spec.rb

@@ -1,9 +1,9 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::Line do
   include_context "swt app"
 
-  let(:container) { double('container', :disposed? => false).as_null_object }
+  let(:container) { double('container', disposed?: false).as_null_object }
   let(:dsl) { Shoes::Line.new shoes_app, parent, point_a, point_b }
   let(:point_a) { Shoes::Point.new(10, 100) }
   let(:point_b) { Shoes::Point.new(300, 10) }

data/spec/shoes/swt/link_segment_spec.rb

@@ -1,4 +1,4 @@
-require 'shoes/swt/spec_helper'
+require 'spec_helper'
 
 describe Shoes::Swt::LinkSegment do
   let(:range)         { (2..10) }
@@ -6,7 +6,7 @@ describe Shoes::Swt::LinkSegment do
   let(:line_height)   { 14 }
   let(:layout)        { double('swt layout', width: line_width, line_count: 10) }
   let(:text_segment)  { double('text segment', layout: layout,
-                               element_left: 0, element_top: 0) }
+                                               element_left: 0, element_top: 0) }
 
   subject { Shoes::Swt::LinkSegment.new(text_segment, range) }
 
@@ -14,7 +14,6 @@ describe Shoes::Swt::LinkSegment do
     10.times {|i| stub_line_bounds(i)}
   end
 
-
   # ....................
   context "empty link" do
     let(:range) { (0...0) }