storazzo 0.4.2 → 0.4.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6a489f7c8e545a123a224850369894e939e8bc541ca2b00ce4f2306493144ec2
-  data.tar.gz: 35f76e4a92849cb0547910ac8f240b38dda4cecaf70ceec0b6d665d6d5d3ce5f
+  metadata.gz: 474eb4be081a7edac52377a5269a24e7c826ed0afbf200ac6b315a220bece1f4
+  data.tar.gz: 9d6bbb34885de5bf25ef075dd4bcce0e2a0aaba6714d5a34bee1b220096be140
 SHA512:
-  metadata.gz: 3d69dbf50b340e06833ba28cf4219ee6aca33b5466e442377597c825ae0c2adaef2d8be3b4330adee8dde52d693e84f3f69c5208df767e988268f45359e36f27
-  data.tar.gz: ef9ce00d9af13c84aa2db03996ea383af7494007a2bdd94f799d4720d2cc2aae63106e7bbf74db555c6574180f3948305712d115f757f07a196d96958da74ef5
+  metadata.gz: 37ce7a58b0ab2f36dcc681579bb55dfa147c195d37016d1fff9ac5ea390fa469d363e2a0df70a614d0c7acf97fd5db9c6d9eb9275f2926155910d9d8e15d9d6f
+  data.tar.gz: 0d6f2863ee75c3a1e653100c7904371028f0838d627c83a286d4bdd0ccfdc699190f4a331c942b6bb4834f4218e50a37853c73827c69d69868294f71dd1172f6

data/Gemfile

@@ -1,7 +1,8 @@
-# useless now
-
+source "https://rubygems.org"
 
 # to troubleshoot/debug things
 #gem 'rubocop'
 gem 'rake'
-gem 'pry'
+#gem 'pry'
+gem 'pry', :group => :development
+#gem 'require_all' # yup I'm lazy: https://stackoverflow.com/questions/735073/best-way-to-require-all-files-from-a-directory-in-ruby

data/Makefile

@@ -51,4 +51,17 @@ watch-test:
 	watch -c make test
 
 test-gcs-bucket:
-	ruby -I test test/test_gcs_bucket.rb
+	#echo "Warning this uses the INCLUDED gem not the latest one in development'
+	#ruby -I test test/media/test_gcs_bucket.rb
+	rake test TEST=test/media/test_gcs_bucket.rb
+
+test-local-folder:
+	echo "Warning this uses the INCLUDED gem not the latest one in development'
+	ruby -I test test/media/test_local_folder.rb
+# https://medium.com/@tegon/quick-guide-to-minitest-arguments-745bf9fe4b3
+test-media-subfolder:
+	rake test TEST="test/media/*.rb"
+test-verbose:
+	rake test:verbose --verbose 
+test-single-file-continuously:
+	\watch -n 5 --color rake test TEST="test/media/test_local_folder.rb"

data/README.md

@@ -9,6 +9,24 @@
 
 (Latest version is hosted in https://rubygems.org/gems/storazzo)
 
+# Tests
+
+I still struggle to enforce the include of LOCAL unchecked code rather than latest required system gem (cmon Ruby!)
+but I found loads of interesting ways to test my code by googling and StoackOverflowing:
+
+* `rake test TEST="test/sum_test.rb"`
+* test-gcs-bucket: `ruby -I test test/test_gcs_bucket.rb` (meh - see below)
+* test-media-subfolder: `rake test TEST="test/media/*.rb"`
+
+Single test in single file:
+
+* `rake test TEST="test/sum_test.rb" TESTOPTS="--name=test_returns_two"` (sample)
+* `rake test TEST="test/media/test_local_folder.rb" TESTOPTS="--name=test_1_first_directory_parsing_actually_works"`
+* `ruby -I test test/test_local_folder.rb -n test_first_directory_parsing_actually_works` (note this includes `storazzo` latest gem 
+    and doesnt benefit from LATEST code so its NOT good for testing: use RAKE for that).
+
+
+Now to toggle verbosity I believe I need to go into Rakefile (bummer)
 # Thanks
 
 Inspiration from:

data/Rakefile

@@ -3,15 +3,42 @@
 # from hola: https://guides.rubygems.org/make-your-own-gem/#adding-an-executable
 require "rake/testtask"
 
-Rake::TestTask.new do |t|
+desc "Run Unit tests"
+Rake::TestTask.new(:test) do |t|
   t.libs << "test"
+  t.libs << "test/media"
+  # note this is only useful for this: https://chriskottom.com/articles/command-line-flags-for-minitest-in-the-raw/
   t.verbose = false
   t.warning = false 
+  #puts "[RiccardoOnly]: t.pattern: #{t.pattern}" 
+  t.pattern = 'test/**/test_*.rb'
 end
 
-desc "Run tests"
+desc "By default, Run Unit tests"
 task default: :test
 
+# Adding test/media directory to rake test.
+desc "Test tests/media/* code sobenem"
+namespace :test do
+  desc "Test Verbosely by default since I'm too stupid to toggle via ENV var dammit"
+  Rake::TestTask.new(:verbose) do |t|
+    t.libs << "test"
+    t.libs << "test/media"
+    t.verbose = true
+    t.warning = true 
+    t.pattern = 'test/**/test_*.rb'
+    $DEBUG = true
+  end
+end
+# namespace :verbose_test do
+#   desc "Test tests/media/* code"
+#   Rake::TestTask.new do |t|
+#     t.libs << "test/media"
+#     # Rails::TestTask.new(media: 'test:prepare') do |t|
+#     t.pattern = 'test/**/test_*.rb'
+#   end
+# end
+#Rake::Task['test:run'].enhance ["test:media"]
 
 # begin
 #     require 'bundler/setup'
@@ -29,4 +56,7 @@ task default: :test
 #   RuboCop::RakeTask.new do |task|
 #     task.requires << 'rubocop-performance'
 #     task.requires << 'rubocop-rspec'
-#   end
+#   end
+
+
+

data/VERSION

@@ -1 +1 @@
-0.4.2
+0.4.5

data/bin/ricdisk-magic

@@ -168,7 +168,6 @@ end
 def main(filename)
   deb "I'm called by #{white filename}"
   deb "HISTORY: #{gray HISTORY}"
-  #deb "To remove this shit, just set $DEBUG=false :)"
   init        # Enable this to have command line parsing capabilities!
   puts white("$DEBUG is #{$DEBUG }. Tu turn on, call me with -d") unless $DEBUG
   #warn "[warn] template v#{$TEMPLATE_VER }: proviamo il warn che magari depreca il DEB"

data/bin/stats-with-md5

@@ -111,42 +111,9 @@ def compute_stats_and_md5(file)
 		stats = File.stat(file)
 		ret[:stats_object] = stats # TODO deprecate
 		deb("Stats methods: #{stats.methods.sort.join(', ')}")
-		#deb(stats.ftype)
-		#puts(stats.methods)
 		deb(stats.ctime)
 		#puts(stats.birthtime rescue (stats.ctime))
-
-		# On Mac:
-		#<File::Stat                     
-		# dev=0x100000f,                  
-		# ino=1247768,                    
-		# mode=0100644 (file rw-r--r--),  
-		# nlink=1,
-		# uid=164825 (ricc),
-		# gid=89939 (primarygroup),
-		# rdev=0x0 (0, 0),
-		# size=564,
-		# blksize=4096,
-		# blocks=8,
-		# atime=2022-03-05 22:36:48.373362127 +0100 (1646516208),
-		# mtime=2022-03-05 22:36:48.176789949 +0100 (1646516208),
-		# ctime=2022-03-05 22:36:48.176789949 +0100 (1646516208)>
-
-		# On LInux:
-		#<File::Stat                                
-		#  dev=0xfe01,                                
-		#  ino=6293055,                               
-		#  mode=0100644 (file rw-r--r--),             
-		#  nlink=1,                                   
-		#  uid=164825 (ricc),                         
-		#  gid=89939 (primarygroup),                  
-		#  rdev=0x0 (0, 0),                           
-		#  size=7,                                    
-		#  blksize=4096,                              
-		#  blocks=8,                                  
-		#  atime=2022-06-27 08:49:38.586706861 +0200 (1656312578),
-		#  mtime=2022-03-23 14:28:45 +0100 (1648042125),
-		#  ctime=2022-05-30 10:12:10.381629305 +0200 (1653898330)>
+		# On Mac/Linux: see test/dumps/***.yaml
 		ret[:size] = stats.size
 		ret[:mode] = stats.mode
 		# datetimes
@@ -235,7 +202,6 @@ def print_stats_and_md5_for_gcs(mybucket_with_subdir, opts={})
 		puts "print_stats_and_md5_for_gcs(): Generic Error: #{$!}"
 		exit 1
 	end
-
 	autowrite_to_dir_or_gcs(:gcs, mybucket_with_subdir) if opts_autowrite
 end 
 

data/lib/storazzo/common.rb

@@ -1,10 +1,12 @@
 # Ric common stuff! :)
 #
-#
 # Usage:
-#      include Storazzo::Common 
+#      include Storazzo::Common  (def to def)
+# or
+#      extend Storazzo::Common   (def to def self.XX)
 #
 require_relative 'colors'
+require 'pry'
 
 module Storazzo::Common 
 
@@ -12,16 +14,37 @@ module Storazzo::Common
 
 
     def deb(s)
-        puts "[DEB] #{yellow(s)}" if $DEBUG
+        puts "[DEB🐞] #{yellow(s)}" if _debug_true # $DEBUG
+    end
+    # this has a yield
+    def if_deb?()
+        if _debug_true # $DEBUG
+            deb "== yield START =="
+            yield 
+            deb "== yield END =="
+        end
     end
     def warn(s)
-        puts "[WRN] #{azure(s)}"
+        puts "[W⚠️RN] #{azure(s)}"
     end
     def err(str)
-        puts "[ERR] #{red(s)}"
+        puts "[ERR⛔] #{red(s)}"
     end
     def bug(s)
         puts "[🐛] #{gray s}"
     end
+    def pverbose(is_verbose, str)
+        puts "[V📚RB💀S📚] #{gray str}"
+    end
+    def ppp(complex_object_to_colorize)
+        # TODO i need to learn to return without printing..
+        Pry::ColorPrinter.pp(complex_object_to_colorize)
+    end
+
+private
+    def _debug_true
+        $DEBUG or ENV["DEBUG"] == 'true'
+    end
 
+#    puts "[DEBUG ENABLED!]" if _debug_true
 end

data/lib/storazzo/media/abstract_ric_disk.rb

@@ -1,18 +1,21 @@
 module Storazzo::Media
     class Storazzo::Media::AbstractRicDisk
+        #include Storazzo::Common
+        extend Storazzo::Common
 
-        #@@default_stats_filename = Storazzo::RicDiskStatsFile.default_name
+        #DefaultStatsFilename = Storazzo::RicDiskStatsFile.default_name
 
             # looks like there's not Abstract Class in Ruby, but also SO says he best
             # way to do this is this: 
             
             #    attr_accessor :name, :description, :ricdisk_file, :local_mountpoint, :wr, :path, :ricdisk_file_empty, :size, :active_dirs
 
+
             ########################
             # Abstract methods START
             ########################
             def initialize(local_mount)
-                puts "[DEB] [AbstractRicDisk.init()] Some child of AbstractRicDisk (#{self}) called me! Yummie." # disable when you dont need me anymore..
+                pverbose true, "[AbstractRicDisk.init()] Some child of AbstractRicDisk (#{self}) called me! Yummie." # disable when you dont need me anymore..
                 validate
             end
             def self.list_all
@@ -50,7 +53,7 @@ module Storazzo::Media
                     # if not writeable, I will:
                     # 1. create a dir based on its unique format.
                     # 2. create a file of same look and feel (alternative - used a DASH)
-                    return "{get_local_folder}/#{unique_id}::#{self.default_stats_filename}"
+                    return "TODO FIXME {get_local_folder}/#{unique_id}::#{self.default_stats_filename}"
                     #"{get_local_folder}"/#{unique_id}/#{Storazzo::RicDiskStatsFile.default_name}"
                 end                     
             end
@@ -63,6 +66,18 @@ module Storazzo::Media
                 "MD5::v1::#{hash}"
             end
 
+            def to_verbose_s
+                h = {}
+                h[:class] = self.class 
+                h[:unique_id] = self.unique_id
+                h[:inspect] = self.inspect
+                h[:to_s] = self.to_s
+                h[:local_mountpoint] = local_mountpoint
+                h[:writeable] = self.writeable?
+                h[:stats_file_smart_fullpath] = stats_file_smart_fullpath
+                return h
+            end
+
             # # Todo check on instances these 3 methods exist
             def self.abstract_class_mandatory_methods
                 %W{
@@ -92,15 +107,27 @@ module Storazzo::Media
                 "#{local_mountpoint}/#{my_stats_filename}"
             end
 
-            def validate
-                puts "DEB We're trying to see if your object is valid, across 3 possible sub-classes."
+            def validate(opts={})
+                verbose = opts.fetch(:verbose, true)
+                puts "[VERBOSE] validate(): We're trying to see if your object is valid, across 3 possible sub-classes." if verbose
                 #1. check for 
                 raise "Unknown local mount " unless local_mount.is_a?(String)
                 #2. check thaty writeable? is true or false
                 my_writeable = wr
                 raise "Writeable is not boolean" unless (my_writeable.is_a? TrueClass or my_writeable.is_a? FalseClass )
             end
+
+            # TODO use a proper Factory pattern.
+            def self.DirFactory(path)
+                raise "I need a path/directory string: #{path}" unless path.is_a?(String)
+
+                deb "TODO: if coincides with MountPoint, instance THAT"
+                if path =~ /^gs:\/\//
+                    deb "Smells like GCS"
+                    return GcsBucket.new(path)
+                end
+                deb "Smells like LocalFolder :)"
+                return LocalFolder.new(path)
+            end
     end
-    #puts "DEB lib/storazzo/media/abstract_ric_disk Media::ARD inside module"
 end
-#puts "DEB lib/storazzo/media/abstract_ric_disk Media::ARD outside module"

data/lib/storazzo/media/local_folder.rb

@@ -3,28 +3,26 @@
 
 module Storazzo::Media
     class Storazzo::Media::LocalFolder < Storazzo::Media::AbstractRicDisk
+        #extend Storazzo::Common
+        include Storazzo::Common
 
         attr_accessor :local_mountpoint, :wr
 
         def initialize(local_mount)
-            puts "[Storazzo::Media::LocalFolder] initialize"
+            deb "[Storazzo::Media::LocalFolder] initialize"
             
             @local_mountpoint = File.expand_path(local_mount)
             raise "Sorry local mount doesnt exist!" unless File.exist?(@local_mountpoint)
             @wr = File.writable?(stats_filename_default_fullpath) # .writeable? stats_file_smart_fullpath
 
             #super.initialize(local_mount) rescue "SUPER_ERROR: #{$!}"
-            super(local_mount) rescue "SUPER_ERROR: #{$!}"
+            super(local_mount) rescue "SUPER_ERROR(#{local_mount}): #{$!}"
         end
         
-        def self.list_all
-            # get lisrts from Config singletone
-            #puts " self.list_all: loading config "
-            config = Storazzo::RicDiskConfig.instance # # ).get_config
-            #puts config['Config']['AdditionalMountDirs']
-            #puts "TODO see config: #{config}"
-            #[42, 43]
-            #deb config.get_local_folders
+        def self.list_all(config=nil)
+            # get lists from Config singleton
+            config ||= Storazzo::RicDiskConfig.instance # # ).get_config
+            config.load
             config.get_local_folders
         end
 

data/lib/storazzo/media/mount_point.rb

@@ -1,4 +1,9 @@
 # TODO
+module Storazzo::Media
+    class Storazzo::Media::MountPoint < Storazzo::Media::AbstractRicDisk
+        puts "[REMOVEME] Storazzo::Media::MountPoint being read. REMOVEME when you see this :)"
+    end
+end
 
 #         def writeable?
 #File.writable?(@local_mountpoint)

data/lib/storazzo/ric_disk.rb

@@ -40,10 +40,21 @@ module Storazzo
   ## INSTANCE methods
   ################################
 
+    def initialize_old_way(path, opts={})
+      raise "Now I dont want a string in input, I want an OBJECT <  Storazzo::Media::AbstractRicDisk"
+    end
+
 
-    def initialize(path, opts={})
+    def initialize(ric_disk_object, opts={})
+      verbose = opts.fetch :verbose, true 
+      pverbose verbose, "This needs an object of type Storazzo::Media::AbstractRicDisk now (this case: #{ric_disk_object.class})"
+      raise "Woopsie, not a Storazzo::Media::AbstractRicDisk! Intead its a #{ric_disk_object.class}" unless ric_disk_object.class.superclass == Storazzo::Media::AbstractRicDisk
+      # ok back to business, now path is a String :)
+      path =  ric_disk_object.path
+      deb "RicDisk initialize.. path=#{path}"
       deb "RicDisk initialize.. path=#{path}"
       @local_mountpoint = File.expand_path(path)
+      @ard = ric_disk_object # AbstractRicDiskObject
       @description = "This is an automated RicDisk description from v.#{RicdiskVersion}. Created on #{Time.now}'"
       @ricdisk_version = RicdiskVersion
       @ricdisk_file = compute_ricdisk_file() # Storazzo::RicDisk.get_ricdisk_file(path)
@@ -53,7 +64,7 @@ module Storazzo
       #@wr = File.writable?("#{path}/#{ricdisk_file}" ) # .writeable?
       #@wr = writeable?
       @tags = ['ricdisk', 'storazzo']
-      @size = `du -s '#{path}'`.split(/\s/)[0] # self.size
+      @size = _compute_size_could_take_long(path) 
       @unique_hash = "MD5::" + Digest::MD5.hexdigest(File.expand_path(path)) #   hash = Digest::MD5.hexdigest(File.expand_path(get_local_mountpoint))
       @computation_hostname = Socket.gethostname
       @created_at = Time.now
@@ -91,18 +102,31 @@ module Storazzo
     end
 
     def to_s 
-      "RicDisk(paz=#{path}, r/w=#{writeable?}, size=#{size}B, f=#{ricdisk_file}, v#{ricdisk_version})"
+      "RicDisk(paz=#{path}, r/w=#{writeable?}, size=#{size}B, f=#{ricdisk_file}, v#{ricdisk_version}, ard=#{@ard})"
     end
 
     # could take long..
     # def size 
     #   `du -s '#{path}'`.split(/\s/)[0]
     # end
+    def self._compute_size_could_take_long(my_path) 
+      deb "Could take long. TODO(ricc): add some sort of cutoff/timeout to 5 seconds."
+      puts azure('could take long')
+      `du -s '#{my_path}' 2>/dev/null`.chomp.split(/\s/)[0] # self.size
+    end
 
     def writeable?() 
+      #memoize
       return @wr unless @wr.nil? 
-      # Otherwise I can do an EXPENSIVE calculation
-      puts yellow("TODO(ricc): Do expensive calculation if this FS is writeable: #{path}")
+      # NOW: CALCULATE it
+      # Now I can do ONCE an EXPENSIVE calculation
+      puts yellow("[RicDisk.writeable] TODO(ricc): Do expensive calculation if this FS is writeable: #{path} and write/memoize it on @wr once and for all")
+      puts yellow("[RicDisk.writeable]             I have a feeling this should be delegated to praecipuus Storazzo::Media::Object we refer to (WR is different on GCS vs Local):") # infinite loop dammit #{self.to_verbose_s}")
+      puts("Dir: #{ azure path}")
+      puts("absolute_path: #{azure absolute_path}")
+      puts("File.writable?(absolute_path): #{azure File.writable?(absolute_path)}")
+      bash_output = `if [ -w "#{absolute_path}" ]; then echo "WRITABLE"; else echo "NOT WRITABLE"; fi`
+      puts("bash_output: #{azure bash_output}")
       #@wr = File.writable?(File.expand_path(@ricdisk_file)) # rescue false
       raise "for some reason an important info (ricdisk_file='#{absolute_path}') is missing!" if ricdisk_file.nil?
       @wr = File.writable?(absolute_path) # rescue false
@@ -111,6 +135,16 @@ module Storazzo
       #false
     end
 
+    def to_verbose_s 
+      h = {}
+      h[:to_s] = self.to_s
+      h[:wr] = self.wr
+      h[:inspect] = self.inspect
+      h[:writeable] = self.writeable?
+      h[:ard] = @ard
+      return h
+    end
+
   ################################
   ## CLASS methods
   ################################
@@ -154,8 +188,8 @@ module Storazzo
           x = Dir.glob('*').select {|f| File.directory? f}
           subdirs = x.map{|subdir|   "#{dir}#{subdir}"}
           subdirs.each{|subdir| 
-            puts "DEB Subdir: #{subdir}"
-            puts `ls -al "#{subdir}"`
+            deb "Subdir: #{subdir}"
+            puts `ls -al "#{subdir}"` # TODO refactor in exec
             active_dirs << subdir if ok_dir? # self.ok_dir?(subdir)
           }
           #puts(white x)
@@ -239,7 +273,7 @@ module Storazzo
       # given a path, if .ricdisk exists i do stuff with it..
       disk_info = nil
       unless ok_dir? # self.ok_dir?(subdir)
-        puts("[write_config_yaml_to_disk] Nothing for me here: '#{subdir}'. Existing")
+        warn("[write_config_yaml_to_disk] Nothing for me here: '#{subdir}'. Existing")
         return 
       end
       ConfigFiles.each do |papable_configfile_name|
@@ -258,7 +292,7 @@ module Storazzo
       #   puts(yellow disk_info.to_yaml)
       # end
       if disk_info.is_a?(RicDisk)
-        puts yellow("DEB disk_info.class: #{disk_info.class}")
+        deb yellow("disk_info.class: #{disk_info.class}")
         if File.empty?(disk_info.absolute_path) # and (disk_info.wr)
           puts(green("yay, we can now write the file '#{disk_info.absolute_path}' (which is R/W, I just checked!) with proper YAML content.."))
           if disk_info.wr

data/lib/storazzo/ric_disk_config.rb

@@ -1,6 +1,13 @@
 require 'singleton'
 require 'yaml'
 
+#require 'storazzo/media/abstract_ric_disk'
+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:
@@ -20,18 +27,16 @@ module Storazzo
         include Storazzo::Common
         include Storazzo::Colors
         
-        #@@default_config_location = "~/.storazzo.yaml" 
         DefaultConfigLocation = File.expand_path "~/.storazzo.yaml" 
-        # @@default_config_locations = [
-        #     "~/.storazzo.yaml" , # HOME
-        #     "./.storazzo.yaml" , # LOCAL DIR
-        # ]        
+        
         DefaultConfigLocations = [
             File.expand_path("~/.storazzo.yaml") , # HOME
             File.expand_path("./.storazzo.yaml") , # LOCAL DIR
         ]
-        #@@default_gem_location_for_tests
-        DefaultGemLocationForTests = File.expand_path('../../../', __FILE__) + "/etc/storazzo_config.sample.yaml" 
+
+        DefaultGemLocationForTests = 
+            File.expand_path('../../../', __FILE__) + 
+            "/etc/storazzo_config.sample.yaml" 
         
         attr_accessor :config, :config_file, :load_called
 
@@ -41,23 +46,22 @@ public
             verbose = opts.fetch :verbose, false
 
             if already_loaded? # and not self.config.nil?
-                puts "[#{self.class}] VERBOSE load: already loaded" if verbose
+                pverbose verbose, "[#{self.class}] VERBOSE load: already loaded"
                 return self.config
             end
-
-            puts "[VERBOSE] Storazzo::RicDiskConfig.load(): BEGIN " if verbose
+            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_location , "./.storazzo.yaml"]
-            puts "DEB possible_locations: #{possible_locations}"
-            if config_path.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)
-                puts "[LOAD] possible_locations: #{possible_locations}" if verbose
+                pverbose verbose, "[LOAD] possible_locations: #{possible_locations}"
             end
             puts "[VERBOSE] Searching these paths in order: #{possible_locations}" if verbose
-            bug "This is not always an array of sTRINGS."
+            #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
@@ -90,9 +94,7 @@ public
 
         # Obsolete, call another class instead.
         def load_sample_version
-        #     puts("Warning! We're destroying the world here. We're taking a Singletong and changing the way it behaves by moving the config file by under her feet. Don't be mad at me if this misbehaves. You saw it coming, my friends. This is why I would NEVER hire you as a Software Developer in my Company.")
              raise "DEPRECATED! USE SampleRicDiskConfig.load() instead!"
-        #     load(DefaultGemLocationForTests, :verbose => true )
         end
 
         def config_ver
@@ -162,7 +164,9 @@ public
 
         # UGLY CODE, copipasted from binary for ARGV, ex autosbrodola
         def iterate_through_file_list_for_disks(files_list=[], opts={})
-            verbose = opts.fetch :verbose, true
+            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"
@@ -173,7 +177,6 @@ public
             #         RicDisk.calculate_stats_files(dir) # dir is inutile
             #     } # TODO refactor in option sbrodola_afterwards=true. :)
         # else
-            raise "Wrong input, I need an array here: #{files_list} " unless files_list.is_a?(Array)
             puts "iterate_through_file_list_for_disks(): I consider files_list as a list of directories to parse :)" if verbose
 
             #dirs = RicDisk.find_active_dirs()
@@ -182,8 +185,8 @@ public
                 if File.directory?(dir)
                 #if dirs.include?(dir)
                     puts "iterate_through_file_list_for_disks() Legit dir: #{green dir}" if verbose
-                    rd = RicDisk.new(dir)
-                    puts "RicDisk: #{rd}"
+                    rd = RicDisk.new(Storazzo::Media::AbstractRicDisk.DirFactory(dir))
+                    pverbose true, "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

data/lib/storazzo/ric_disk_sample_config.rb

@@ -13,20 +13,16 @@ require_relative "./ric_disk_config"
 =end
 
 module Storazzo
-
     class Storazzo::RicDiskSampleConfig < Storazzo::RicDiskConfig
         #include Storazzo::Common
 
-        #include Singleton
-
-        public
+    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.load DefaultGemLocationForTests #super.load(DefaultGemLocationForTests, :verbose => true )
             super(DefaultGemLocationForTests, :verbose => false )
         end
         def load_sample_version
-            puts 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.")
+            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

data/lib/storazzo/ric_disk_statsfile.rb

@@ -3,7 +3,6 @@
 module Storazzo
     class Storazzo::RicDiskStatsFile
         # Please keep these two in sync, until you fix them and DRY the behaviour.
-        #@@default_name
         DefaultName = "ricdisk_stats_v11.rds" # => RicDiskStatsFile
         Version      = "1.1" # @@version
 

data/lib/storazzo.rb

@@ -30,22 +30,27 @@ end
 
 # nice to paste nice output
 require 'pp'
+require 'require_all'
+
+#require_all './' , 'media/'
+#require_all 'lib/**/*.rb'
+#require_rel '.', 'media'
+Dir[File.dirname(__FILE__) + '/../lib/*.rb'].each do |file| 
+    puts "+ Requiring... #{file}"
+    require File.basename(file, File.extname(file))
+end
 
 require 'storazzo'
 require 'storazzo/common'
 require 'storazzo/colors'
 require 'storazzo/hashify'
 require 'storazzo/ric_disk'          # NEW and will build from ground up using multiple files..
-#require 'storazzo/ric_disk/asbtract_ric_disk'
 require 'storazzo/media/abstract_ric_disk'
 require 'storazzo/media/gcs_bucket' 
 require 'storazzo/media/local_folder' 
+require 'storazzo/media/mount_point' 
 require 'storazzo/ric_disk_ugly'     # OLD and 90% working
 require 'storazzo/ric_disk_config'         # => RicDiskConfif
 require 'storazzo/ric_disk_sample_config'   # => NOTHING!!
-#require 'storazzo/ric_disk_config_example'  # => NEW!!!
 require 'storazzo/ric_disk_statsfile'
 require 'storazzo/main'
-#require 'storazzo/translator'
-
-#puts Storazzo::Main.say_hi 

data/storazzo.gemspec

@@ -13,9 +13,8 @@ Gem::Specification.new do |s|
         "ricdisk-magic",
         "stats-with-md5",
     ]
-
     s.homepage    = "https://rubygems.org/gems/storazzo" # maybe https://github.com/palladius/storazzo
     s.license       = "MIT"
-
     #s.add_dependency "activesupport", "~> 3.0"
+    s.add_dependency "pry" # , "~> 3.0"
   end

data/test/media/test_abstract_ric_disk.rb

@@ -0,0 +1,3 @@
+# TODO
+require "minitest/autorun"
+require "storazzo"

data/test/{test_gcs_bucket.rb → media/test_gcs_bucket.rb}

@@ -4,20 +4,9 @@ require "storazzo/ric_disk"
 require "storazzo/ric_disk_config"
 require 'storazzo/colors'
 require "storazzo/media/local_folder"
-require "storazzo/ric_disk_config_example"
 
 require 'pry' # must install the gem... but you ALWAYS want pry installed anyways
 
-#require "storazzo/ric_disk_config"
-#require "storazzo/ric_disk_sample_config"
-#require 'storazzo/ric_disk_sample_config'  # => NOTHING!!
-#require 'storazzo/ric_disk_config_sample'  # => NOTHING!!
-
-#require "storazzo/ric_disk"
-#require "lib/storazzo/ric_disk_sample_config"
-#require "storazzo/media/local_folder"
-#require "storazzo/ric_disk/gcs_bucket"
-#require "storazzo/media/local_folder"
 
 class GcsBucketTest < Minitest::Test
 
@@ -52,11 +41,13 @@ class GcsBucketTest < Minitest::Test
         deb " Storazzo.constants: #{ Storazzo.constants}"
         #puts Storazzo::RicDiskSampleConfig
         #config_obj = Storazzo::RicDiskSampleConfig.instance()
-        Pry::ColorPrinter.pp($sample_config_obj.to_verbose_s())
+        if_deb? do
+            Pry::ColorPrinter.pp($sample_config_obj.to_verbose_s())
+        end 
         #pp green(config_obj.to_verbose_s())
 
         l = $sample_config_obj.load
-        Pry::ColorPrinter.pp(l)
+        Pry::ColorPrinter.pp(l) if $DEBUG
 
         puts "$sample_config_obj: #{$sample_config_obj}"
         #config_obj.load # _sample_version

data/test/{test_local_folder.rb → media/test_local_folder.rb}

@@ -1,18 +1,16 @@
 require "minitest/autorun"
 require "storazzo"
+require 'storazzo/common'
 require "storazzo/ric_disk"
 require "storazzo/ric_disk_config"
 require 'storazzo/colors'
 require "storazzo/media/local_folder"
 
-#require "storazzo/ric_disk/gcs_bucket"
-#require "storazzo/media/local_folder"
-
-
-#puts yellow("DISABLING FOR NOW TODO restore")
-
 class LocalFolderTest < Minitest::Test
-    include Storazzo::Colors
+#    include Storazzo::Colors
+#    include Storazzo::Common
+    include Storazzo::Common
+
     # def test_fail_on_purpOSE # test_storazzo_hi_with_argument
     #     assert_match 42, 42 , "change me when it failes from makefile" 
     #     #"Hello from Storazzo", Storazzo::Main.hi("ruby this should fail")
@@ -65,7 +63,7 @@ class LocalFolderTest < Minitest::Test
         stats_file = disk.stats_filename_default_fullpath
         puts "stats_file: #{stats_file}"
         disk.parse()
-        puts "[DEB] config: ''#{config}''"
+        deb "config: ''#{config}''"
         # config.iterate_through_file_list_for_disks([test_dir])
         # assert(
         #     File.exists?(stats_file),
@@ -105,7 +103,7 @@ class LocalFolderTest < Minitest::Test
         )
     end
 
-    def test_readonly_folder
+    def TODO_test_readonly_directory_creates_configfile_outside_of_dir
         test_dir = "/etc/"
         disk = Storazzo::Media::LocalFolder.new(test_dir)
         stats_file = disk.stats_filename_default_fullpath
@@ -118,4 +116,40 @@ class LocalFolderTest < Minitest::Test
         )
         # ...
     end
+
+    def test_readonly_directory_is_indeed_readonly
+        test_dir = "/etc/"
+        readonly_rdisk = Storazzo::Media::LocalFolder.new(test_dir)
+        # stats_file = disk.stats_filename_default_fullpath
+        # config = Storazzo::RicDiskSampleConfig.instance()
+        # config.load
+        # config.iterate_through_file_list_for_disks([test_dir])
+        # assert(
+        #     not(File.exists?(stats_file)),
+        #     "parse on LocalFolder should NOT create file '#{stats_file}' but another in another TODO place"
+        # )
+        #readonly_rdisk = Storazzo::RicDisk.new(test_dir)
+        ppp(readonly_rdisk.to_verbose_s)
+        assert_equal(
+            readonly_rdisk.wr,
+            false,
+            "Folder #{test_dir} is NOT writeble to the author's knowledge."
+        )
+        # ...
+    end
+
+
+    def test_writeable_directory_is_indeed_writeable
+        test_dir = '/tmp/'
+        writeable_rdisk = Storazzo::Media::LocalFolder.new(test_dir)
+        #writeable_rdisk = Storazzo::RicDisk.new(test_dir)
+        assert_equal(
+            writeable_rdisk.wr,
+            true,
+            "Folder #{test_dir} is INDEED writeble to the author's knowledge, although triggers lot of noise."
+        )
+
+    end
+
+
 end

data/test/media/test_mount_point.rb

@@ -0,0 +1,26 @@
+require "minitest/autorun"
+require "storazzo"
+
+#require 'pry' # must install the gem... but you ALWAYS want pry installed anyways
+
+class MediaMountPointTest < Minitest::Test
+
+    def test_test_in_subfolder
+        #raise "Does this even work?!?"
+        puts "Looks like this only works if you run this: ruby -I test test/media/test_media_mount_point.rb"
+    end
+
+    def test_mount_point_creation 
+        #x = Storazzo::Media::MountPoint.new
+        #assert class inherits from ...
+        skip "TODO Code is still missing but TODO implement that this class inherits from **ther class"
+        # assert(
+        #     false,
+        #     "TODO Code is still missing but TODO implement that this class inherits from **ther class"
+        # )
+    end
+
+    def test_what_skip_means
+        skip 'Check for affiliate link, credit card details, pledge history. Foujnd example online. Please Riccardo fix'
+    end
+end

data/test/test_ric_disk.rb

@@ -0,0 +1,19 @@
+require "minitest/autorun"
+require "storazzo"
+
+class RicDiskTest < Minitest::Test
+
+    def test_factory_works_for_gcs 
+        rd1 = RicDisk.new('/tmp')
+        hash = rd1.to_verbose_s()
+        pp hash
+        assert_equal(
+            hash.class,
+            Hash,
+            "rd1.to_verbose_s should return a Hash" 
+        )
+    end
+
+
+
+end

data/test/test_ric_disk_config.rb

@@ -7,13 +7,6 @@
 
 require "minitest/autorun"
 require "storazzo"
-#require "storazzo/ric_disk_config"
-#require "storazzo/ric_disk_sample_config"
-#require "storazzo/ric_disk_config_example"
-
-# require "storazzo/media/local_folder"
-#require "storazzo/ric_disk/gcs_bucket"
-#require "storazzo/media/local_folder"
 
 
 class RicDiskConfigTest < Minitest::Test
@@ -24,7 +17,7 @@ class RicDiskConfigTest < Minitest::Test
         config_obj = Storazzo::RicDiskSampleConfig.instance()
         puts "config_obj.class: #{config_obj.class}"
         config = config_obj.load # _sample_version
-        puts "Config: #{config}"
+        pverbose true, "[test_load_sample_version] Config: #{pp config}"
         #puts '[RicDiskConfigTest] TODO lets make sure the gem being imported is actually in /etc/storazz-config.smaple blah blah'
         #puts "[RicDiskConfigTest] config_file: ", config_obj.config_file
         assert_equal(

data/var/dumps/file_stat.linux.yaml

@@ -0,0 +1,15 @@
+		# On Linux:
+		<File::Stat                                
+		 dev=0xfe01,                                
+		 ino=6293055,                               
+		 mode=0100644 (file rw-r--r--),             
+		 nlink=1,                                   
+		 uid=164825 (ricc),                         
+		 gid=89939 (primarygroup),                  
+		 rdev=0x0 (0, 0),                           
+		 size=7,                                    
+		 blksize=4096,                              
+		 blocks=8,                                  
+		 atime=2022-06-27 08:49:38.586706861 +0200 (1656312578),
+		 mtime=2022-03-23 14:28:45 +0100 (1648042125),
+		 ctime=2022-05-30 10:12:10.381629305 +0200 (1653898330)>

data/var/dumps/file_stat.macosx.yaml

@@ -0,0 +1,15 @@
+		# On Mac:
+		<File::Stat                     
+		dev=0x100000f,                  
+		ino=1247768,                    
+		mode=0100644 (file rw-r--r--),  
+		nlink=1,
+		uid=164825 (ricc),
+		gid=89939 (primarygroup),
+		rdev=0x0 (0, 0),
+		size=564,
+		blksize=4096,
+		blocks=8,
+		atime=2022-03-05 22:36:48.373362127 +0100 (1646516208),
+		mtime=2022-03-05 22:36:48.176789949 +0100 (1646516208),
+		ctime=2022-03-05 22:36:48.176789949 +0100 (1646516208)>

data/var/test/disks/disk02-full/Rakefile

@@ -0,0 +1,13 @@
+# I fell in love with this: https://stackoverflow.com/questions/9274205/rake-watch-for-changes
+
+file 'main.o' => ["main.c", "greet.h"] do
+    sh "cc -c -o main.o main.c"
+end
+
+file 'greet.o' => ['greet.c'] do
+    sh "cc -c -o greet.o greet.c"
+end
+
+file "hello" => ["main.o", "greet.o"] do
+    sh "cc -o hello main.o greet.o"
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: storazzo
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.4.5
 platform: ruby
 authors:
 - Riccardo Carlesso
@@ -9,7 +9,21 @@ autorequire:
 bindir: bin
 cert_chain: []
 date: 2022-07-29 00:00:00.000000000 Z
-dependencies: []
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A simple gem to manage your external hard drives and extract MD5 and
   common stuff from them.
 email: name dot surname at popular Google-owned Mail
@@ -46,12 +60,18 @@ files:
 - lib/storazzo/ric_disk_ugly.rb
 - lib/storazzo/version.rb
 - storazzo.gemspec
-- test/test_gcs_bucket.rb
-- test/test_local_folder.rb
+- test/media/test_abstract_ric_disk.rb
+- test/media/test_gcs_bucket.rb
+- test/media/test_local_folder.rb
+- test/media/test_mount_point.rb
+- test/test_ric_disk.rb
 - test/test_ric_disk_config.rb
 - test/test_ric_disk_stats_file.rb
 - test/test_storazzo.rb
+- var/dumps/file_stat.linux.yaml
+- var/dumps/file_stat.macosx.yaml
 - var/test/README.md
+- var/test/disks/disk02-full/Rakefile
 - var/test/disks/disk02-full/fake file.touch
 - var/test/disks/disk02-full/ls.txt
 homepage: https://rubygems.org/gems/storazzo
@@ -80,9 +100,13 @@ summary: storazzo is an amazing gem. Code is in https://github.com/palladius/sto
 test_files:
 - test/test_storazzo.rb
 - test/test_ric_disk_stats_file.rb
+- test/test_ric_disk.rb
 - test/test_ric_disk_config.rb
-- test/test_gcs_bucket.rb
-- test/test_local_folder.rb
+- test/media/test_gcs_bucket.rb
+- test/media/test_mount_point.rb
+- test/media/test_abstract_ric_disk.rb
+- test/media/test_local_folder.rb
 - var/test/README.md
 - var/test/disks/disk02-full/fake file.touch
 - var/test/disks/disk02-full/ls.txt
+- var/test/disks/disk02-full/Rakefile