refactor library_loader to library_loader and sketchbook module

Author: monkstone

lib/jruby_art/library_loader.rb

@@ -2,15 +2,15 @@
 # frozen_string_literal: false
 
 require_relative '../jruby_art'
-
+require_relative 'sketchbook'
 # The processing wrapper module
 module Processing
   # Encapsulate library loader functionality as a class
   class LibraryLoader
     attr_reader :sketchbook_library_path
 
     def initialize
-      @sketchbook_library_path = File.join(find_sketchbook_path, 'libraries')
+      @sketchbook_library_path = File.join(Sketchbook.find_path, 'libraries')
       @loaded_libraries = Hash.new(false)
     end
 
@@ -117,27 +117,6 @@ def get_library_directory_path(library_name, extension = nil)
         end
       end
       nil
-    end
-
-    def find_sketchbook_path
-      preferences_paths = []
-      sketchbook_paths = []
-      sketchbook_path = Processing::RP_CONFIG.fetch('sketchbook_path', false)
-      return sketchbook_path if sketchbook_path
-      [
-        "'Application Data/Processing'", 'AppData/Roaming/Processing',
-        'Library/Processing', 'Documents/Processing',
-        '.processing', 'sketchbook'
-      ].each do |prefix|
-        spath = format('%s/%s', ENV['HOME'], prefix)
-        pref_path = format('%s/preferences.txt', spath)
-        preferences_paths << pref_path if FileTest.file?(pref_path)
-        sketchbook_paths << spath if FileTest.directory?(spath)
-      end
-      return sketchbook_paths.first if preferences_paths.empty?
-      lines = IO.readlines(preferences_paths.first)
-      matchedline = lines.grep(/^sketchbook/).first
-      matchedline[/=(.+)/].delete('=')
-    end
+    end    
   end
 end

lib/jruby_art/sketchbook.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: false
+# Utility to find sketchbook and hence java libraries
+module Sketchbook
+  def self.find_path
+    preferences_paths = []
+    sketchbook_paths = []
+    sketchbook_path = Processing::RP_CONFIG.fetch('sketchbook_path', false)
+    return sketchbook_path if sketchbook_path
+    [
+      "'Application Data/Processing'", 'AppData/Roaming/Processing',
+      'Library/Processing', 'Documents/Processing',
+      '.processing', 'sketchbook'
+    ].each do |prefix|
+      spath = format('%s/%s', ENV['HOME'], prefix)
+      pref_path = format('%s/preferences.txt', spath)
+      preferences_paths << pref_path if FileTest.file?(pref_path)
+      sketchbook_paths << spath if FileTest.directory?(spath)
+    end
+    return sketchbook_paths.first if preferences_paths.empty?
+    lines = IO.readlines(preferences_paths.first)
+    matchedline = lines.grep(/^sketchbook/).first
+    matchedline[/=(.+)/].delete('=')
+  end
+end