storazzo 0.5.7 → 0.6.1

data/lib/storazzo/ric_disk_config.rb

@@ -1,261 +1,279 @@
+# frozen_string_literal: true
+
+require 'English'
+
 require 'singleton'
 require 'yaml'
 
-# require 'storazzo/media/abstract_ric_disk'
-Dir[File.dirname(__FILE__) + '/../lib/*.rb'].each do |file|
+Dir["#{File.dirname(__FILE__)}/../lib/*.rb"].each do |file|
   require File.basename(file, File.extname(file))
 end
-# require_all 'media/directory'
-
-=begin
-    This is a singleton class. You call me this way..
-    You call me with:
-
-    Storazzo::RicDiskConfig.instance()
 
-    or better
-
-    Storazzo::RicDiskConfig.safe_instance()
-
-    Note that being a Singleton, in Unit Tests it's hard to use the /etc/storazzo_config.sample.yaml instead
-    of the real one - yiikes. How do I fix it? Do I unsingleton it? :) Or do I create TWO singletons? :)
-=end
+#     This is a singleton class. You call me this way..
+#     You call me with:
+#
+#     Storazzo::RicDiskConfig.instance()
+#
+#     or better
+#
+#     Storazzo::RicDiskConfig.safe_instance()
+#
+#     Note that being a Singleton, in Unit Tests it's hard to use the /etc/storazzo_config.sample.yaml instead
+#     of the real one - yiikes. How do I fix it? Do I unsingleton it? :) Or do I create TWO singletons? :)
 
 module Storazzo
   # class Storazzo::Blah
   # end
 
-  class Storazzo::RicDiskConfig
-    include Singleton
-    include Storazzo::Common
-    include Storazzo::Colors
-
-    DefaultConfigLocation = File.expand_path "~/.storazzo.yaml"
+  module Storazzo
+    class RicDiskConfig
+      include Singleton
+      include Storazzo::Common
+      include Storazzo::Colors
 
-    DefaultConfigLocations = [
-      File.expand_path("~/.storazzo.yaml"), # HOME
-      File.expand_path("./.storazzo.yaml"), # LOCAL DIR
-    ]
+      DefaultConfigLocation = File.expand_path '~/.storazzo.yaml'
 
-    DefaultGemLocationForTests =
-      File.expand_path('../../../', __FILE__) +
-      "/etc/storazzo_config.sample.yaml"
+      DEFAULT_CONFIG_LOCATIONS = [
+        File.expand_path('~/.storazzo.yaml'), # HOME
+        File.expand_path('./.storazzo.yaml') # LOCAL DIR
+      ].freeze
 
-    attr_accessor :config, :config_file, :load_called, :project_id
+      DefaultGemLocationForTests =
+        "#{File.expand_path('../..', __dir__)}/etc/storazzo_config.sample.yaml"
 
-    public
+      attr_accessor :config, :config_file, :load_called, :project_id
 
-    # Load from the first valid config.
-    def load(config_path = nil, opts = {})
-      verbose = opts.fetch :verbose, false
+      # Load from the first valid config.
+      def load(config_path = nil, opts = {})
+        verbose = opts.fetch :verbose, false
 
-      if already_loaded? # and not self.config.nil?
-        pverbose verbose, "[#{self.class}] Config.Load: already loaded"
-        return self.config
-      end
-      pverbose verbose, "Storazzo::RicDiskConfig.load(): BEGIN"
-      # trying default location
-      raise "DefaultConfigLocation is not a string" unless DefaultConfigLocation.is_a?(String)
-
-      possible_locations = DefaultConfigLocations #  [ default_config_locations .. , "./.storazzo.yaml"]
-      deb "[Config.load] Possible Locations: #{possible_locations}"
-      if config_path.is_a?(String)
-        # possible_locations = [config_path] + possible_locations # .append()
-        possible_locations = possible_locations.unshift(config_path) # append to front
-        # OR: possible_locations.instert(0, config_path)
-        pverbose verbose, "[LOAD] possible_locations: #{possible_locations}"
-      end
-      pverbose verbose, "Searching these paths in order: #{possible_locations}"
-      # bug "This is not always an array of sTRINGS."
-      raise "possible_locations is not an array" unless possible_locations.is_a?(Array)
-
-      possible_locations.each do |possible_path|
-        # ASSERT is a string
-        raise "possible_path is not a string" unless possible_path.is_a?(String)
+        if already_loaded? # and not self.config.nil?
+          pverbose verbose, "[#{self.class}] Config.Load: already loaded"
+          return config
+        end
+        pverbose verbose, 'Storazzo::RicDiskConfig.load(): BEGIN'
+        # trying default location
+        raise 'DefaultConfigLocation is not a string' unless DefaultConfigLocation.is_a?(String)
+
+        possible_locations = DEFAULT_CONFIG_LOCATIONS #  [ default_config_locations .. , "./.storazzo.yaml"]
+        deb "[Config.load] Possible Locations: #{possible_locations}"
+        if config_path.is_a?(String)
+          # possible_locations = [config_path] + possible_locations # .append()
+          possible_locations.unshift(config_path) # append to front
+          # OR: possible_locations.instert(0, config_path)
+          pverbose verbose, "[LOAD] possible_locations: #{possible_locations}"
+        end
+        pverbose verbose, "Searching these paths in order: #{possible_locations}"
+        # bug "This is not always an array of sTRINGS."
+        raise 'possible_locations is not an array' unless possible_locations.is_a?(Array)
+
+        possible_locations.each do |possible_path|
+          # ASSERT is a string
+          raise 'possible_path is not a string' unless possible_path.is_a?(String)
+
+          deb "before buggy expand_path paz: '#{possible_path}''"
+          paz = begin
+            File.expand_path(possible_path)
+          rescue StandardError
+            possible_path
+          end
+          raise "Not a string: #{paz}" unless paz.is_a?(String)
 
-        deb "before buggy expand_path paz: '#{possible_path}''"
-        paz = File.expand_path(possible_path) rescue possible_path
-        raise "Not a string: #{paz}" unless paz.is_a?(String)
+          next unless File.exist?(paz)
 
-        if File.exists?(paz)
           @config_file = paz
-          @config = YAML.load(File.read paz) # YAML.load(File.read("file_path"))
+          @config = YAML.safe_load(File.read(paz)) # YAML.load(File.read("file_path"))
 
-          unless (@config["kind"] == 'StorazzoConfig' rescue false)
-            puts white "RicDiskConfig.load(): Sorry this is wrong Config File. Kind=#{@config["kind"] rescue $!}"
+          unless begin
+            @config['kind'] == 'StorazzoConfig'
+          rescue StandardError
+            false
+          end
+            puts white "RicDiskConfig.load(): Sorry this is wrong Config File. Kind=#{begin
+              @config['kind']
+            rescue StandardError
+              $ERROR_INFO
+            end}"
             next
           end
           #
           # pp @config if verbose
-          config_ver = @config["apiVersion"]
+          config_ver = @config['apiVersion']
           # puts @config[:ConfigVersion]
           deb("OK. Storazzo::RicDiskConfig v'#{config_ver}' parsed correctly")
           @load_called = true
-          return self.config
+          return config
         end
+        @load_called = true
+        # only get here if nothing is found
+        raise "No config found across these locations: #{possible_locations}. Consider copying and editing: #{RicDiskConfig.gem_default_config_path}"
       end
-      @load_called = true
-      # only get here if nothing is found
-      raise "No config found across these locations: #{possible_locations}. Consider copying and editing: #{RicDiskConfig.gem_default_config_path}"
-    end
 
-    # Obsolete, call another class instead.
-    def load_sample_version
-      raise "DEPRECATED! USE SampleRicDiskConfig.load() instead!"
-    end
+      # Obsolete, call another class instead.
+      def load_sample_version
+        raise 'DEPRECATED! USE SampleRicDiskConfig.load() instead!'
+      end
 
-    def config_ver
-      raise "I cant compute Version since I cant compute @config. Are you sure you didnt instance this Singleton without calling load?" if @config.nil?
+      def config_ver
+        raise 'I cant compute Version since I cant compute @config. Are you sure you didnt instance this Singleton without calling load?' if @config.nil?
 
-      @config['apiVersion'] # rescue :StillUnknown
-      # config['ConfigVersion']
-    end
+        @config['apiVersion'] # rescue :StillUnknown
+        # config['ConfigVersion']
+      end
 
-    def config_default_folder
-      # self.
-      @config['Config']['DefaultFolder'] # rescue "Unknown config_default_folder: #{$!}"
-    end
+      def config_default_folder
+        # self.
+        @config['Config']['DefaultFolder'] # rescue "Unknown config_default_folder: #{$!}"
+      end
 
-    def config_project_id
-      @config['Config']['Backends']['GoogleCloudStorage']['ProjectId']
-    end
+      def config_project_id
+        @config['Config']['Backends']['GoogleCloudStorage']['ProjectId']
+      end
 
-    #doesnt work :/ alias :project_id, :config_project_id
-    def project_id
-      config_project_id
-    end
+      # doesnt work :/ alias :project_id, :config_project_id
+      def project_id
+        config_project_id
+      end
 
-    # This should return true if and only if the user has configured the YAML
-    # to use GCS. For now we can just say TRUE since gsutil ls returns without project_id.
-    # However, we might have a flag enabled in storazzo config in next versions.
-    def gcs_enabled?
-      #true
-      defined?(get_config['Config']['Backends']['GoogleCloudStorage'])
-      #get_config['Config']['Backends']['GoogleCloudStorage DOESNT EXIST']
-      # defined? Backends:     GoogleCloudStorage
-    end
+      # This should return true if and only if the user has configured the YAML
+      # to use GCS. For now we can just say TRUE since gsutil ls returns without project_id.
+      # However, we might have a flag enabled in storazzo config in next versions.
+      def gcs_enabled?
+        # true
+        defined?(get_config['Config']['Backends']['GoogleCloudStorage'])
+        # get_config['Config']['Backends']['GoogleCloudStorage DOESNT EXIST']
+        # defined? Backends:     GoogleCloudStorage
+      end
 
-    def already_loaded?
-      # return
-      load_called == true
-    end
+      def already_loaded?
+        # return
+        load_called == true
+      end
 
-    def to_s
-      size = File.size(@config_file) rescue -1
-      # puts yellow "DEB: #{@config["apiVersion"]}"
-      # "RicDiskConfig(ver=#{config_ver}, file=#{config_file}), #{white(size)} bytes" # - config_default_folder=#{self.config_default_folder}"
-      "POLY_#{self.class}_(ver=#{config_ver}, file=#{config_file}), #{white(size)} bytes" # - config_default_folder=#{self.config_default_folder}"
-    end
+      def to_s
+        size = begin
+          File.size(@config_file)
+        rescue StandardError
+          -1
+        end
+        # puts yellow "DEB: #{@config["apiVersion"]}"
+        # "RicDiskConfig(ver=#{config_ver}, file=#{config_file}), #{white(size)} bytes" # - config_default_folder=#{self.config_default_folder}"
+        "POLY_#{self.class}_(ver=#{config_ver}, file=#{config_file}), #{white(size)} bytes" # - config_default_folder=#{self.config_default_folder}"
+      end
 
-    def to_verbose_s
-      h = {}
-      h[:description] =
-        "This is a Verbose Hash describing a RicDiskConfig or its child RicDiskSampleConfig to understand why it keeps failing.."
-      h[:to_s] = self.to_s
-      h[:class] = self.class
-      h[:file] = __FILE__
-      h[:id] = self.object_id
-      h[:get_bucket_paths] = self.get_bucket_paths()
-      h[:get_local_folders] = self.get_local_folders()
-      h[:config_project_id] = self.config_project_id()
-      return h
-    end
+      def to_verbose_s
+        h = {}
+        h[:description] =
+          'This is a Verbose Hash describing a RicDiskConfig or its child RicDiskSampleConfig to understand why it keeps failing..'
+        h[:to_s] = to_s
+        h[:class] = self.class
+        h[:file] = __FILE__
+        h[:id] = object_id
+        h[:get_bucket_paths] = get_bucket_paths
+        h[:get_local_folders] = get_local_folders
+        h[:config_project_id] = config_project_id
+        h
+      end
 
-    def get_config(opts = {})
-      return load(opts) if @config.nil?
+      def get_config(opts = {})
+        return load(opts) if @config.nil?
 
-      @config
-    end
+        @config
+      end
 
-    def self.gem_default_config_path
-      Storazzo.root + "/etc/storazzo_config.sample.yaml"
-    end
+      def self.gem_default_config_path
+        "#{Storazzo.root}/etc/storazzo_config.sample.yaml"
+      end
 
-    # Storazzo::RicDiskConfig
-
-    # returns all folders from file which are Directories
-    # This method is FLAKY! Sometimes gives error.
-    #         LocalFolderTest#test_show_all_shouldnt_fail_and_should_return_a_non_empty_array:
-    # TypeError: no implicit conversion of Hash into String
-    #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:38:in `expand_path'
-    #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:38:in `block in load'
-    #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:37:in `each'
-    #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:37:in `load'
-    #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:83:in `get_config'
-    #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:95:in `get_local_folders'
-    def get_local_folders
-      config = get_config
-      # puts config['Config']['AdditionalMountDirs']
-      config['Config']['AdditionalMountDirs'].map { |folder|
-        File.expand_path(folder) rescue folder # TypeError: no implicit conversion of Hash into String
-      }.filter { |f| File.directory?(f) }
-    end
+      # Storazzo::RicDiskConfig
+
+      # returns all folders from file which are Directories
+      # This method is FLAKY! Sometimes gives error.
+      #         LocalFolderTest#test_show_all_shouldnt_fail_and_should_return_a_non_empty_array:
+      # TypeError: no implicit conversion of Hash into String
+      #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:38:in `expand_path'
+      #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:38:in `block in load'
+      #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:37:in `each'
+      #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:37:in `load'
+      #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:83:in `get_config'
+      #     /Users/ricc/git/storazzo/lib/storazzo/ric_disk_config.rb:95:in `get_local_folders'
+      def get_local_folders
+        config = get_config
+        # puts config['Config']['AdditionalMountDirs']
+        config['Config']['AdditionalMountDirs'].map do |folder|
+          File.expand_path(folder)
+        rescue StandardError
+          folder
+        end.filter { |f| File.directory?(f) }
+      end
 
-    def get_bucket_paths
-      get_config['Config']['Backends']['GoogleCloudStorage']['BucketPaths'].map { |complex_gcs_struct|
-        complex_gcs_struct['path']
-      }
-    end
+      def get_bucket_paths
+        get_config['Config']['Backends']['GoogleCloudStorage']['BucketPaths'].map do |complex_gcs_struct|
+          complex_gcs_struct['path']
+        end
+      end
 
-    # UGLY CODE, copipasted from binary for ARGV, ex autosbrodola
-    def iterate_through_file_list_for_disks(files_list = [], opts = {})
-      verbose = opts.fetch :verbose, false
-      raise "[iterate_through_file_list_for_disks] Wrong input, I need an array here: #{files_list} " unless files_list.is_a?(Array)
-
-      # I decided i wont accept an emopty list, this is not how you use the gem, you lazy XXX!
-      # if files_list == [] # or files_list.nil?  # empty -> ALL
-      #     deb "iterate_through_file_list_for_disks(): no args provided"
-      #     dirs = RicDisk.find_active_dirs()
-      #     puts "DEB find_active_dirs: #{green dirs}"
-      #     dirs.each {|dir|
-      #         RicDisk.write_config_yaml_to_disk(dir)
-      #         RicDisk.calculate_stats_files(dir) # dir is inutile
-      #     } # TODO refactor in option sbrodola_afterwards=true. :)
-      # else
-      pverbose verbose, "iterate_through_file_list_for_disks(): I consider files_list as a list of directories to parse :)"
-
-      # dirs = RicDisk.find_active_dirs()
-      files_list.each do |dir|
-        dir = File.expand_path(dir)
-        if File.directory?(dir)
-          # if dirs.include?(dir)
-          pverbose verbose, "iterate_through_file_list_for_disks() Legit dir: #{green dir}"
-          rd = RicDisk.new(Storazzo::Media::AbstractRicDisk.DirFactory(dir))
-          pverbose verbose, "RicDisk from Factory (woohoo): #{rd}"
-          rd.write_config_yaml_to_disk(dir)
-          # RicDisk.write_config_yaml_to_disk(dir)
-          # RicDisk.calculate_stats_files (CLASS) => will become OBJECT compute_stats_files
-          rd.compute_stats_files() # dir is inutile # TODO
-        else
-          raise("Doesnt seem a dir to me, quitting: #{dir}")
+      # UGLY CODE, copipasted from binary for ARGV, ex autosbrodola
+      def iterate_through_file_list_for_disks(files_list = [], opts = {})
+        verbose = opts.fetch :verbose, false
+        raise "[iterate_through_file_list_for_disks] Wrong input, I need an array here: #{files_list} " unless files_list.is_a?(Array)
+
+        # I decided i wont accept an emopty list, this is not how you use the gem, you lazy XXX!
+        # if files_list == [] # or files_list.nil?  # empty -> ALL
+        #     deb "iterate_through_file_list_for_disks(): no args provided"
+        #     dirs = RicDisk.find_active_dirs()
+        #     puts "DEB find_active_dirs: #{green dirs}"
+        #     dirs.each {|dir|
+        #         RicDisk.write_config_yaml_to_disk(dir)
+        #         RicDisk.calculate_stats_files(dir) # dir is inutile
+        #     } # TODO refactor in option sbrodola_afterwards=true. :)
+        # else
+        pverbose verbose,
+                 'iterate_through_file_list_for_disks(): I consider files_list as a list of directories to parse :)'
+
+        # dirs = RicDisk.find_active_dirs()
+        files_list.each do |dir|
+          dir = File.expand_path(dir)
+          if File.directory?(dir)
+            # if dirs.include?(dir)
+            pverbose verbose, "iterate_through_file_list_for_disks() Legit dir: #{green dir}"
+            rd = RicDisk.new(Storazzo::Media::AbstractRicDisk.DirFactory(dir))
+            pverbose verbose, "RicDisk from Factory (woohoo): #{rd}"
+            rd.write_config_yaml_to_disk(dir)
+            # RicDisk.write_config_yaml_to_disk(dir)
+            # RicDisk.calculate_stats_files (CLASS) => will become OBJECT compute_stats_files
+            rd.compute_stats_files # dir is inutile # TODO
+          else
+            raise("Doesnt seem a dir to me, quitting: #{dir}")
+          end
         end
+        # end
       end
-      # end
-    end # /iterate_through_file_list_for_disks
 
-    def config_hash
-      config['Config']
-    end
+      def config_hash
+        config['Config']
+      end
 
-    def self.safe_instance()
-      puts "This is a safe instance :)"
-      my_config = self.instance()
-      my_config.load
-      my_config
-    end
+      def self.safe_instance
+        puts 'This is a safe instance :)'
+        my_config = instance
+        my_config.load
+        my_config
+      end
 
-    def self.get_config
-      self.instance.load unless self.instance.load_called
-      self.instance.get_config
-    end
+      def self.get_config
+        instance.load unless instance.load_called
+        instance.get_config
+      end
 
-    # # gem 'after_do'
-    # require 'after_do'
-    # Storazzo::RicDiskConfig.extend AfterDo
-    # Storazzo::RicDiskConfig.after :instance do # |activity|  #:pause, :finish, :resurrect, :do_today, :do_another_day
-    #     puts yellow("after INSTANCE() has been called I call LOAD!!!")
-    #     self.load
-    #     #persistor.save activity
-    # end
-  end #     class Storazzo::RicDiskConfig
-end # module Storazzo
+      # # gem 'after_do'
+      # require 'after_do'
+      # Storazzo::RicDiskConfig.extend AfterDo
+      # Storazzo::RicDiskConfig.after :instance do # |activity|  #:pause, :finish, :resurrect, :do_today, :do_another_day
+      #     puts yellow("after INSTANCE() has been called I call LOAD!!!")
+      #     self.load
+      #     #persistor.save activity
+      # end
+    end
+  end
+end

data/lib/storazzo/ric_disk_sample_config.rb

@@ -1,31 +1,33 @@
+# frozen_string_literal: true
+
 require 'singleton'
 require 'yaml'
 
-require_relative "./common"
-require_relative "./ric_disk_config"
-
-=begin
-    This is a singleton class. You call me this way..
-    You call me with:
-
-    Storazzo::RicDiskSampleConfig.instance()
-
-=end
+require_relative './common'
+require_relative './ric_disk_config'
 
 module Storazzo
-  class Storazzo::RicDiskSampleConfig < Storazzo::RicDiskConfig
-    # include Storazzo::Common
-
-    public
-
-    def load # _sample_version
-      deb("[RicDiskSampleConfig] Wheew 1! We're NOT destroying the world here. We're actually instancing a second Singleton which is a child of the mother, and this time doing things nicely and Rubily.")
-      super(DefaultGemLocationForTests, :verbose => false)
-    end
-
-    def load_sample_version
-      deb white("[RicDiskSampleConfig] Wheew 2! We're NOT destroying the world here. We're actually instancing a second Singleton which is a child of the mother, and this time doing things nicely and Rubily.")
-      super(DefaultGemLocationForTests, :verbose => false)
+  module Storazzo
+    #     This is a singleton class. You call me this way..
+    #     You call me with:
+    #
+    #     Storazzo::RicDiskSampleConfig.instance()
+    #
+    # It loads a sample config in local gem etc/. Usuful for testing or if no other config is found..
+
+    class RicDiskSampleConfig < Storazzo::RicDiskConfig
+      # include Storazzo::Common
+
+      # _sample_version
+      def load
+        deb("[RicDiskSampleConfig] Wheew 1! We're NOT destroying the world here. We're actually instancing a second Singleton which is a child of the mother, and this time doing things nicely and Rubily.")
+        super(DefaultGemLocationForTests, verbose: false)
+      end
+
+      def load_sample_version
+        deb white("[RicDiskSampleConfig] Wheew 2! We're NOT destroying the world here. We're actually instancing a second Singleton which is a child of the mother, and this time doing things nicely and Rubily.")
+        super(DefaultGemLocationForTests, verbose: false)
+      end
     end
   end
-end # module Storazzo
+end

data/lib/storazzo/ric_disk_statsfile.rb

@@ -1,24 +1,26 @@
-# This class wraps the RDS file: we're going to write this RDS file
-# directly in the disk: /mount/
-module Storazzo
-  class Storazzo::RicDiskStatsFile
-    # Please keep these two in sync, until you fix them and DRY the behaviour.
-    DefaultName = "ricdisk_stats_v11.rds" # => RicDiskStatsFile
-    Version = "1.1" # @@version
+# frozen_string_literal: true
 
-    # AttrAccessor for class - thanks StackOverflow from Android since Im in roaming :)
-    class << self
-      attr_accessor :default_name, :version
-    end
+module Storazzo
+  module Storazzo
+    # This class wraps the RDS file: we're going to write this RDS file
+    # directly in the disk: /mount/
+    class RicDiskStatsFile
+      # Please keep these two in sync, until you fix them and DRY the behaviour.
+      DefaultName = 'ricdisk_stats_v11.rds' # => RicDiskStatsFile
+      Version = '1.1' # @@version
 
-    public
+      # AttrAccessor for class - thanks StackOverflow from Android since Im in roaming :)
+      class << self
+        attr_accessor :my_default_name, :my_version # without MY it redefines whats below :/
+      end
 
-    def self.default_name
-      DefaultName
-    end
+      def self.default_name
+        DefaultName
+      end
 
-    def self.version
-      Version
+      def self.version
+        Version
+      end
     end
   end
 end

data/lib/storazzo/ric_disk_ugly.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # # TODO remove this! Ugly code to just borrow and cleanup a bit,
 
 # module Storazzo

data/lib/storazzo/version.rb

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # copied from https://github.com/rails/strong_parameters/edit/master/lib/strong_parameters/version.rb
 # I;'m copying the DHH philosophy here.
 module Storazzo
   # DHH_VERSION = "0.2.3._TODOFileRead.1" # TODO file.read ../../VERSION . chomp
-  RICC_VERSION = File.read("../VERSION").chomp
-
-  public
+  RICC_VERSION = File.read('../VERSION').chomp
 
   def self.version
     RICC_VERSION