repla 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e6c25d2a775a0a58b1b85c28fb7e659c031733ba
-  data.tar.gz: 7bbd40e204cb30ccc62931c0669725dd6dd91cff
+  metadata.gz: b9d072ea3122cc808168f77e02aa527bee84b97f
+  data.tar.gz: 3425a55a09e166dd54b32d526d295ffff411a106
 SHA512:
-  metadata.gz: 61e74ff0e82cc4e04b45495fbdd58ec26b828d0af6990efd856fabe575de63c9941b9e95a3edfc35357345746eab6dd2d3c8515503ed4dcfd57170fdb9fbb8c8
-  data.tar.gz: 01ffaf898ce45b8964e7a0e63b8e4571a85f2aae7c51ba3c0d3d5c2ff779e9771d20c1fea7d19fb24c4f05172cb39dbe7d55703898b31fb05d7f0db3e5279735
+  metadata.gz: 2b410101c334239ea91222c48ec893a4ee9187266a57e46748a4c64b6418bcda4778c11a35b8b27a030bdf4df4e54ce720ed8e3d921b5f514a3c65488d104e01
+  data.tar.gz: df7a9f1397e818791db811fbba2efff84a38b5c36e2b6f68c82f66eba1ce6f2b25c5a3996ae4740f46ad5e922c83a5f651f4bd14026f954ab6feed8b1065f6e6

data/lib/repla/dependencies/lib/controller.rb

@@ -2,33 +2,31 @@ require_relative 'tester'
 require_relative 'model'
 require_relative 'view'
 
-module Repla::Dependencies
-
-  class Controller < Repla::Controller
-
-    def initialize
-      @view = View.new
-    end
-
-    def missing_dependency(dependency)
-      name = dependency.name
-      type = self.class.string_for_type(dependency.type)
-      options = dependency.options
-      if options.has_key?(:installation_instructions)
-        installation_instructions = options[:installation_instructions]
-      end      
-      @view.add_missing_dependency(name, type, installation_instructions)
-    end
+module Repla
+  module Dependencies
+    # Dependencies controller
+    class Controller < Repla::Controller
+      def initialize
+        @view = View.new
+      end
 
-    private
+      def missing_dependency(dependency)
+        name = dependency.name
+        type = self.class.string_for_type(dependency.type)
+        options = dependency.options
+        if options.key?(:installation_instructions)
+          installation_instructions = options[:installation_instructions]
+        end
+        @view.add_missing_dependency(name, type, installation_instructions)
+      end
 
-    def self.string_for_type(type)
-      case type
-      when :shell_command
-        return "shell command"
+      private_class_method def self.string_for_type(type)
+        case type
+        when :shell_command
+          return 'shell command'
+        end
+        nil
       end
-      nil
     end
-
   end
 end

data/lib/repla/dependencies/lib/model.rb

@@ -1,10 +1,13 @@
-module Repla::Dependencies
-  class Dependency
-    attr_reader :name, :type, :options
-    def initialize(name, type,  options = {})
-      @name = name
-      @type = type
-      @options = options
-    end    
+module Repla
+  module Dependencies
+    # Dependency
+    class Dependency
+      attr_reader :name, :type, :options
+      def initialize(name, type, options = {})
+        @name = name
+        @type = type
+        @options = options
+      end
+    end
   end
 end

data/lib/repla/dependencies/lib/tester.rb

@@ -1,19 +1,19 @@
-module Repla::Dependencies
-  module Tester
-    def self.check(name, type)
-      case type
-      when :shell_command
-        return check_shell_command(name)
+module Repla
+  module Dependencies
+    # Tester
+    module Tester
+      def self.check(name, type)
+        case type
+        when :shell_command
+          check_shell_command(name)
+        end
       end
-    end
-    
-    private
-    
-    require 'shellwords'
-    def self.check_shell_command(name)
-      command = "type -a #{Shellwords.escape(name)} > /dev/null 2>&1"
-      system(command)
-    end
 
+      require 'shellwords'
+      private_class_method def self.check_shell_command(name)
+        command = "type -a #{Shellwords.escape(name)} > /dev/null 2>&1"
+        system(command)
+      end
+    end
   end
 end

data/lib/repla/dependencies/lib/view.rb

@@ -1,19 +1,24 @@
-module Repla::Dependencies
-  
-  class View < Repla::View
-    ROOT_ACCESS_DIRECTORY = File.join(File.dirname(__FILE__), '../../')
-    HTML_DIRECTORY = File.join(File.dirname(__FILE__), '../html/')
-    VIEW_TEMPLATE = File.join(HTML_DIRECTORY, 'index.html')
+module Repla
+  module Dependencies
+    # Dependency view
+    class View < Repla::View
+      ROOT_ACCESS_DIRECTORY = File.join(File.dirname(__FILE__), '../../')
+      HTML_DIRECTORY = File.join(File.dirname(__FILE__), '../html/')
+      VIEW_TEMPLATE = File.join(HTML_DIRECTORY, 'index.html')
 
-    def initialize
-      super
-      self.root_access_directory_path = File.expand_path(ROOT_ACCESS_DIRECTORY)
-      load_file(VIEW_TEMPLATE)
-    end
+      def initialize
+        super
+        self.root_access_directory_path = File.expand_path(
+          ROOT_ACCESS_DIRECTORY
+        )
+        load_file(VIEW_TEMPLATE)
+      end
 
-    ADD_MISSING_DEPENDENCY_FUNCTION = "addMissingDependency"
-    def add_missing_dependency(name, type, installation_instructions = nil)
-      do_javascript_function(ADD_MISSING_DEPENDENCY_FUNCTION, [name, type, installation_instructions])
+      ADD_MISSING_DEPENDENCY_FUNCTION = 'addMissingDependency'.freeze
+      def add_missing_dependency(name, type, installation_instructions = nil)
+        do_javascript_function(ADD_MISSING_DEPENDENCY_FUNCTION,
+                               [name, type, installation_instructions])
+      end
     end
   end
 end

data/lib/repla/dependencies.rb

@@ -1,33 +1,34 @@
 require_relative '../repla'
 
-module Repla::Dependencies
-  class Checker
-    require_relative 'dependencies/lib/model'
-    require_relative 'dependencies/lib/controller'
+module Repla
+  module Dependencies
+    # Checks whether bundle dependencies are installed
+    class Checker
+      require_relative 'dependencies/lib/model'
+      require_relative 'dependencies/lib/controller'
 
-    def check_dependencies(dependencies)
-      passed = true
-      dependencies.each { |dependency|  
-        dependency_passed = check(dependency)
-        passed = false unless dependency_passed
-      }
-      passed
-    end
+      def check_dependencies(dependencies)
+        passed = true
+        dependencies.each do |dependency|
+          dependency_passed = check(dependency)
+          passed = false unless dependency_passed
+        end
+        passed
+      end
 
-    def check(dependency)
-      name = dependency.name
-      type = dependency.type
-      passed = Tester::check(name, type)
-      controller.missing_dependency(dependency) unless passed
-      passed
-    end
+      def check(dependency)
+        name = dependency.name
+        type = dependency.type
+        passed = Tester.check(name, type)
+        controller.missing_dependency(dependency) unless passed
+        passed
+      end
 
-    private
+      private
 
-    def controller
-      @controller ||= Controller.new
+      def controller
+        @controller ||= Controller.new
+      end
     end
-
   end
-
 end

data/lib/repla/lib/constants.rb

@@ -1,9 +1,9 @@
 module Repla
   # Keys
-  PLUGIN_NAME_KEY = 'WC_PLUGIN_NAME'
-  SPLIT_ID_KEY = 'WC_SPLIT_ID'
-  WINDOW_ID_KEY = 'WC_WINDOW_ID'
+  PLUGIN_NAME_KEY = 'WC_PLUGIN_NAME'.freeze
+  SPLIT_ID_KEY = 'WC_SPLIT_ID'.freeze
+  WINDOW_ID_KEY = 'WC_WINDOW_ID'.freeze
 
   # Directories
-  APPLESCRIPT_DIRECTORY = File.join(File.dirname(__FILE__), "..", "..", "applescript")
+  APPLESCRIPT_DIRECTORY = File.join(File.dirname(__FILE__), '../../applescript')
 end

data/lib/repla/lib/controller.rb

@@ -1,4 +1,5 @@
 module Repla
+  # Controller
   class Controller
     attr_reader :view
   end

data/lib/repla/lib/escape.rb

@@ -0,0 +1,70 @@
+require 'Shellwords'
+
+# Escaping parameters
+module Escape
+  # Escaping floats
+  module FloatEscape
+    def javascript_argument
+      to_s
+    end
+  end
+  # Escaping integers
+  module IntegerEscape
+    def javascript_argument
+      to_s
+    end
+  end
+
+  # Escaping strings
+  module StringEscape
+    def javascript_argument
+      "'#{StringEscape.javascript_escape(self)}'"
+    end
+
+    def javascript_escape
+      StringEscape.javascript_escape(self)
+    end
+
+    def javascript_escape!
+      replace(StringEscape.javascript_escape(self))
+    end
+
+    def float?
+      true if Float(self)
+    rescue StandardError
+      false
+    end
+
+    def integer?
+      to_i.to_s == self
+    end
+
+    def shell_escape
+      StringEscape.shell_escape(self)
+    end
+
+    def shell_escape!
+      replace(StringEscape.shell_escape(self))
+    end
+
+    private_class_method def self.javascript_escape(string)
+      string.gsub('\\', '\\\\\\\\').gsub("\n", '\\\\n').gsub("'", "\\\\'")
+    end
+
+    private_class_method def self.shell_escape(string)
+      Shellwords.escape(string)
+    end
+  end
+
+  refine String do
+    include StringEscape
+  end
+
+  refine Integer do
+    include IntegerEscape
+  end
+
+  refine Float do
+    include FloatEscape
+  end
+end

data/lib/repla/lib/module.rb

@@ -1,72 +1,83 @@
 require 'Shellwords'
+require_relative 'escape'
+
+# Repla
 module Repla
-  LOAD_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "load_plugin.scpt")
+  using Escape
+
+  LOAD_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, 'load_plugin.scpt')
   def self.load_plugin(path)
-    self.run_applescript(LOAD_PLUGIN_SCRIPT, [path])
+    run_applescript(LOAD_PLUGIN_SCRIPT, [path])
   end
 
-  EXISTS_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "exists.scpt")
+  EXISTS_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, 'exists.scpt')
   def self.application_exists
-    result = self.run_applescript(EXISTS_SCRIPT)
-    result == "true" ? true : false
+    result = run_applescript(EXISTS_SCRIPT)
+    result == 'true'
   end
 
-  RUN_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "run_plugin.scpt")
+  RUN_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, 'run_plugin.scpt')
   def self.run_plugin(name, directory = nil, arguments = nil)
     parameters = [name]
     parameters.push(directory) unless directory.nil?
     parameters += arguments unless arguments.nil?
-    self.run_applescript(RUN_PLUGIN_SCRIPT, parameters)
+    run_applescript(RUN_PLUGIN_SCRIPT, parameters)
   end
-  
-  RUN_PLUGIN_IN_SPLIT_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "run_plugin_in_split.scpt")
+
+  RUN_PLUGIN_IN_SPLIT_SCRIPT = File.join(APPLESCRIPT_DIRECTORY,
+                                         'run_plugin_in_split.scpt')
   def self.run_plugin_in_split(name, window_id, split_id)
     parameters = [name, window_id, split_id]
-    self.run_applescript(RUN_PLUGIN_IN_SPLIT_SCRIPT, parameters)
+    run_applescript(RUN_PLUGIN_IN_SPLIT_SCRIPT, parameters)
   end
 
-  WINDOW_ID_FOR_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "window_id_for_plugin.scpt")
+  WINDOW_ID_FOR_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY,
+                                          'window_id_for_plugin.scpt')
   def self.window_id_for_plugin(name)
-    self.run_applescript(WINDOW_ID_FOR_PLUGIN_SCRIPT, [name])
+    run_applescript(WINDOW_ID_FOR_PLUGIN_SCRIPT, [name])
   end
 
-  SPLIT_ID_IN_WINDOW_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "split_id_in_window.scpt")
-  def self.split_id_in_window(window_id, pluginName = nil)
+  SPLIT_ID_IN_WINDOW_SCRIPT = File.join(APPLESCRIPT_DIRECTORY,
+                                        'split_id_in_window.scpt')
+  def self.split_id_in_window(window_id, plugin_name = nil)
     arguments = [window_id]
-    arguments.push(pluginName) unless pluginName.nil?
-    self.run_applescript(SPLIT_ID_IN_WINDOW_SCRIPT, arguments)
+    arguments.push(plugin_name) unless plugin_name.nil?
+    run_applescript(SPLIT_ID_IN_WINDOW_SCRIPT, arguments)
   end
 
-  SPLIT_ID_IN_WINDOW_LAST_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "split_id_in_window_last.scpt")
+  SPLIT_ID_IN_WINDOW_LAST_SCRIPT = File.join(APPLESCRIPT_DIRECTORY,
+                                             'split_id_in_window_last.scpt')
   def self.split_id_in_window_last(window_id)
     arguments = [window_id]
-    self.run_applescript(SPLIT_ID_IN_WINDOW_LAST_SCRIPT, arguments)
+    run_applescript(SPLIT_ID_IN_WINDOW_LAST_SCRIPT, arguments)
   end
 
-  CREATE_WINDOW_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "create_window.scpt")
+  CREATE_WINDOW_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, 'create_window.scpt')
   def self.create_window
-    self.run_applescript(CREATE_WINDOW_SCRIPT)
+    run_applescript(CREATE_WINDOW_SCRIPT)
   end
 
   # Resources
 
-  RESOURCE_PATH_FOR_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "resource_path_for_plugin.scpt")
+  RESOURCE_PATH_FOR_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY,
+                                              'resource_path_for_plugin.scpt')
   def self.resource_path_for_plugin(name)
-    self.run_applescript(RESOURCE_PATH_FOR_PLUGIN_SCRIPT, [name])
+    run_applescript(RESOURCE_PATH_FOR_PLUGIN_SCRIPT, [name])
   end
 
-  RESOURCE_URL_FOR_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "resource_url_for_plugin.scpt")
+  RESOURCE_URL_FOR_PLUGIN_SCRIPT = File.join(APPLESCRIPT_DIRECTORY,
+                                             'resource_url_for_plugin.scpt')
   def self.resource_url_for_plugin(name)
-    self.run_applescript(RESOURCE_URL_FOR_PLUGIN_SCRIPT, [name])
+    run_applescript(RESOURCE_URL_FOR_PLUGIN_SCRIPT, [name])
   end
 
-  private
-
-  def self.run_applescript(script, arguments = nil)
+  private_class_method def self.run_applescript(script, arguments = nil)
     command = "osascript #{script.shell_escape}"
 
     if arguments
-      command += " " + arguments.compact.map(&:to_s).map(&:shell_escape).join(' ')
+      command += ' ' + arguments.compact.map(&:to_s).map do |x|
+        x.shell_escape
+      end.join(' ')
     end
 
     result = `#{command}`
@@ -74,37 +85,9 @@ module Repla
     result.chomp!
 
     return nil if result.empty?
-    return result.to_i if result.is_integer?
-    return result.to_f if result.is_float?
-    
-    result
-  end
-
-  class ::String
-    def is_float?
-      true if Float(self) rescue false
-    end
-
-    def is_integer?
-      self.to_i.to_s == self
-    end
+    return result.to_i if result.integer?
+    return result.to_f if result.float?
 
-    def shell_escape
-      Shellwords.escape(self)
-    end
-
-    def shell_escape!
-      replace(shell_escape)
-    end
-
-  end
-
-  class ::Float
-    alias_method :javascript_argument, :to_s
-  end
-
-  class ::Integer
-    alias_method :javascript_argument, :to_s
+    result
   end
-
 end

data/lib/repla/lib/view/javascript.rb

@@ -1,6 +1,10 @@
+require_relative '../escape'
+
+# Repla
 module Repla
+  using Escape
+  # View
   class View < Window
-
     def do_javascript_function(function, arguments = nil)
       javascript = self.class.javascript_function(function, arguments)
       do_javascript(javascript)
@@ -11,41 +15,18 @@ module Repla
       function << '('
 
       if arguments
-        arguments.each { |argument|
-          if argument
-            function << argument.javascript_argument
-          else
-            function << "null"
-          end
+        arguments.each do |argument|
+          function << if argument
+                        argument.javascript_argument
+                      else
+                        'null'
+                      end
           function << ', '
-        }
+        end
         function = function[0...-2]
       end
 
       function << ');'
     end
-
-    private
-
-    class ::Fixnum
-      def javascript_argument
-        "#{self}"
-      end
-    end
-
-    class ::String
-      def javascript_argument
-        "'#{self.javascript_escape}'"
-      end
-
-      def javascript_escape
-        self.gsub('\\', "\\\\\\\\").gsub("\n", "\\\\n").gsub("'", "\\\\'")
-      end
-
-      def javascript_escape!
-        replace(self.javascript_escape)
-      end
-    end
-
   end
 end

data/lib/repla/lib/view.rb

@@ -1,10 +1,10 @@
-require_relative "window"
-require_relative "view/javascript"
-require_relative "view/resources"
+require_relative 'window'
+require_relative 'view/javascript'
+require_relative 'view/resources'
 
 module Repla
+  # View
   class View < Window
-
     # Properties
     def initialize(window_id = nil, view_id = nil)
       super(window_id)
@@ -12,16 +12,15 @@ module Repla
     end
 
     def view_id
-      @view_id ||= ENV.has_key?(SPLIT_ID_KEY) ? ENV[SPLIT_ID_KEY] : split_id
+      @view_id ||= ENV.key?(SPLIT_ID_KEY) ? ENV[SPLIT_ID_KEY] : split_id
     end
 
     private
-    
+
     # Web
 
     def arguments_with_target(arguments)
       super(arguments).push(view_id)
     end
-
   end
 end

data/lib/repla/lib/window.rb

@@ -1,6 +1,7 @@
 module Repla
+  # Window
   class Window
-    require_relative "constants"
+    require_relative 'constants'
 
     attr_accessor :root_access_directory_path
     def initialize(window_id = nil)
@@ -10,13 +11,16 @@ module Repla
     # Properties
 
     def window_id
-      @window_id ||= ENV.has_key?(WINDOW_ID_KEY) ? ENV[WINDOW_ID_KEY] : Repla::create_window
+      key = WINDOW_ID_KEY
+      @window_id ||= ENV.key?(key) ? ENV[key] : Repla.create_window
     end
 
     # Web
 
-    LOAD_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "load.scpt")
-    LOADWITHROOTACCESSDIRECTORY_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "load_with_root_access_directory.scpt")
+    LOAD_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, 'load.scpt')
+    LOADWITHROOTACCESSDIRECTORY_SCRIPT = File.join(
+      APPLESCRIPT_DIRECTORY, 'load_with_root_access_directory.scpt'
+    )
     def load_file(file)
       arguments = [file]
 
@@ -30,41 +34,41 @@ module Repla
       run_script(script, arguments)
     end
 
-    DOJAVASCRIPT_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "do_javascript.scpt")
+    DOJAVASCRIPT_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, 'do_javascript.scpt')
     def do_javascript(javascript)
       run_script(DOJAVASCRIPT_SCRIPT, [javascript])
     end
 
-    READ_FROM_STANDARD_INPUT_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "read_from_standard_input.scpt")
+    READ_FROM_STANDARD_INPUT_SCRIPT = File.join(APPLESCRIPT_DIRECTORY,
+                                                'read_from_standard_input.scpt')
     def read_from_standard_input(text)
       run_script(READ_FROM_STANDARD_INPUT_SCRIPT, [text])
     end
 
     # Window
 
-    CLOSEWINDOW_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, "close_window.scpt")
+    CLOSEWINDOW_SCRIPT = File.join(APPLESCRIPT_DIRECTORY, 'close_window.scpt')
     def close
-      Repla::run_applescript(CLOSEWINDOW_SCRIPT, [window_id])
+      Repla.run_applescript(CLOSEWINDOW_SCRIPT, [window_id])
     end
 
     def split_id
-      Repla::split_id_in_window(window_id)
+      Repla.split_id_in_window(window_id)
     end
 
     def split_id_last
-      Repla::split_id_in_window_last(window_id)
+      Repla.split_id_in_window_last(window_id)
     end
 
     private
 
     def run_script(script, arguments = [])
       arguments = arguments_with_target(arguments)
-      Repla::run_applescript(script, arguments)
+      Repla.run_applescript(script, arguments)
     end
 
     def arguments_with_target(arguments)
       arguments.push(window_id)
     end
-
   end
 end

data/lib/repla/logger/test/lib/test_setup.rb

@@ -1,4 +1,4 @@
-require_relative "../../../test"
+require_relative '../../../test'
 require Repla::Test::HELPER_FILE
 require Repla::Test::REPLA_FILE
 
@@ -7,4 +7,5 @@ TEST_MESSAGE_JAVASCRIPT = 'document.body.lastChild.innerText'.freeze
 TEST_MESSAGE_COUNT_JAVASCRIPT = 'document.body.children.length'.freeze
 
 TEST_JAVASCRIPT_DIRECTORY = File.join(File.dirname(__FILE__), '..', 'js')
-TEST_JAVASCRIPT_FILE = File.join(TEST_JAVASCRIPT_DIRECTORY, 'test_view_helper.js')
+TEST_JAVASCRIPT_FILE = File.join(TEST_JAVASCRIPT_DIRECTORY,
+                                 'test_view_helper.js')