file_distribution 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
---- 
-SHA512: 
-  data.tar.gz: e7259171f747c761e79f4e04877438ceb32bcefc34bd7959439f881217b7f6a15e1c5b172926f7c0365b06d622880614b53fd00bd7b17298356df6734bcd0b84
-  metadata.gz: d77dcd75cfb199ac1e1e6bf9b4f46389681fc853ab27076b5d0beeeb4cb443cdb666d0d61ebf72f177df39ff338c2ce7985a89419d7947f55b1c60534295a1d7
-SHA1: 
-  data.tar.gz: 800b6d7ecf62418a426111682e9a304284fc0e05
-  metadata.gz: d5b5cb72d357425952bf40930e5b76f633593f02
+---
+SHA1:
+  metadata.gz: eaa94a19b025255aaf8b025cbafb52eea7ead248
+  data.tar.gz: 031ebb61ba60285c500dbc8f98d25ee1e44bbc63
+SHA512:
+  metadata.gz: 9ad739b95a1fef51e32ff247c9b48c5b3fcf43ca29a697b61a0054e810184cf2aa03826e86e70e5174071a81d5e285dfa9f54e30941835833655bce91695ff00
+  data.tar.gz: 598187de802d1d9c14b0d60b783066a7e27203528aecaf12eb2ab07c3e493f888e94bc07350700e2841b532df0e654c7d70274691efe084606353a588fa972d5

data/Rakefile

@@ -4,5 +4,5 @@ Rake::TestTask.new do |t|
   t.libs << 'test'
 end
 
-desc "Run tests"
-task :default => :test
+desc 'Run tests'
+task default: :test

data/lib/file_distribution.rb

@@ -20,11 +20,9 @@ class FileDistribution
   # Params:
   # - ext: file extension.
   def set_extension(ext)
-    if ext.length > 0 && ext.chars.first != '.'
-      @ext = '.' + ext
-    else
-      @ext = ext
-    end
+    @ext = ext
+
+    @ext = sprintf('.%s', ext) if !ext.empty? && ext.chars.first != '.'
   end
 
   # Returns Destination path.
@@ -35,20 +33,20 @@ class FileDistribution
   # Params:
   # - id: database file ID etc.
   def hex_path(id)
-    hex = "%x" % id
-    hex = '0%s' % hex if hex.length % 2 != 0
-    @path = File.join(@prefix,hex.scan(/../))
+    hex = sprintf('%x', id)
+    hex = sprintf('0%s', hex) if hex.length.odd?
+    @path = File.join(@prefix, hex.scan(/../))
     @path += @ext
   end
 
   # Params:
   # - path: source file path.
   #
-  # Raises a SystemCallError if the file cannot be renamed.
+  # Raise a SystemCallError if the file cannot be renamed.
   def rename_from(path)
     dst_dir = File.dirname(@path)
-    FileUtils.mkpath(dst_dir) unless File.exists?(dst_dir)
+    FileUtils.mkpath(dst_dir) unless File.exist?(dst_dir)
 
-    File.rename(path,@path)
+    File.rename(path, @path)
   end
 end

data/test/test_file_distribution.rb

@@ -7,81 +7,77 @@ require 'fileutils'
 require 'test/unit'
 
 class TestFileDistribution < Test::Unit::TestCase
- 
+
   def setup
-    if not File.exists?("/tmp/storage")
-      FileUtils.mkdir("/tmp/storage")
-    end
+    FileUtils.mkdir('/tmp/storage') unless File.exist?('/tmp/storage')
 
-    @fd = FileDistribution.new("/tmp/storage//")
+    @fd = FileDistribution.new('/tmp/storage//')
   end
 
   def test_path
-    assert_equal("/tmp/storage", @fd.get_path())
-  end 
+    assert_equal('/tmp/storage', @fd.get_path)
+  end
 
   def test_case1
-    @fd.set_extension("tmp")
-    @fd.set_extension(".dat")
+    @fd.set_extension('tmp')
+    @fd.set_extension('.dat')
     @fd.hex_path(102423)
-    assert_equal("/tmp/storage/01/90/17.dat", @fd.get_path())
-  end 
+    assert_equal('/tmp/storage/01/90/17.dat', @fd.get_path)
+  end
 
   def test_case2
-    @fd.set_extension("dat")
+    @fd.set_extension('dat')
     @fd.hex_path(256)
-    assert_equal("/tmp/storage/01/00.dat", @fd.get_path())
+    assert_equal('/tmp/storage/01/00.dat', @fd.get_path)
   end
 
   # in most cases this is wrong way
   def test_case3
-    @fd.set_extension("")
+    @fd.set_extension('')
     @fd.hex_path(256)
-    assert_equal("/tmp/storage/01/00", @fd.get_path())
-  end 
+    assert_equal('/tmp/storage/01/00', @fd.get_path)
+  end
 
   def test_case4
     @fd.hex_path(1)
-    assert_equal("/tmp/storage/01.dat", @fd.get_path())
-  end 
+    assert_equal('/tmp/storage/01.dat', @fd.get_path)
+  end
 
   def test_case5
     f = File.open('/tmp/storage/test.txt', 'w')
-    f.close()
+    f.close
 
-    assert_equal(true, File.exists?("/tmp/storage/test.txt"))
+    assert_equal(true, File.exist?('/tmp/storage/test.txt'))
 
-    @fd.set_extension(".dat")
+    @fd.set_extension('.dat')
     @fd.hex_path(256)
 
-    @fd.rename_from("/tmp/storage/test.txt")
-    assert_equal(true, File.exists?("/tmp/storage/01/00.dat"))
-  end 	
+    @fd.rename_from('/tmp/storage/test.txt')
+    assert_equal(true, File.exist?('/tmp/storage/01/00.dat'))
+  end
 
   def test_case6
     f = File.open('/tmp/storage/test1.txt', 'w')
-    f.close()
+    f.close
 
     f = File.open('/tmp/storage/test2.txt', 'w')
-    f.close()
+    f.close
 
-    assert_equal(true, File.exists?("/tmp/storage/test1.txt"))
-    assert_equal(true, File.exists?("/tmp/storage/test2.txt"))
+    assert_equal(true, File.exist?('/tmp/storage/test1.txt'))
+    assert_equal(true, File.exist?('/tmp/storage/test2.txt'))
 
-    @fd.set_extension(".dat")
+    @fd.set_extension('.dat')
 
     @fd.hex_path(1)
-    @fd.rename_from("/tmp/storage/test1.txt")
-    assert_equal(true, File.exists?("/tmp/storage/01.dat"))
-    
+    @fd.rename_from('/tmp/storage/test1.txt')
+    assert_equal(true, File.exist?('/tmp/storage/01.dat'))
+
     @fd.hex_path(256)
-    @fd.rename_from("/tmp/storage/test2.txt")
-    assert_equal(true, File.exists?("/tmp/storage/01/00.dat"))
-  end 
+    @fd.rename_from('/tmp/storage/test2.txt')
+    assert_equal(true, File.exist?('/tmp/storage/01/00.dat'))
+  end
 
   def teardown
-    if File.exists?("/tmp/storage")
-      FileUtils.rm_r("/tmp/storage")
-    end
+    FileUtils.rm_r('/tmp/storage') unless File.exist?('/tmp/storage')
   end
-end
+end

metadata

@@ -1,65 +1,70 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: file_distribution
-version: !ruby/object:Gem::Version 
-  version: 0.1.1
+version: !ruby/object:Gem::Version
+  version: 0.2.0
 platform: ruby
-authors: 
+authors:
 - Adam Kubica
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2015-04-13 00:00:00 Z
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2016-06-16 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: rake
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    requirements: 
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
     - - ">="
-      - !ruby/object:Gem::Version 
+      - !ruby/object:Gem::Version
         version: 10.0.0
   type: :development
-  version_requirements: *id001
-description: Simple library that allows organize distribution of files within hex based tree.
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 10.0.0
+description: Simple library that allows organize distribution of files within hex
+  based tree.
 email: caffecoder@kaizen-step.com
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
-- Rakefile
+files:
 - LICENSE
+- Rakefile
 - lib/file_distribution.rb
 - test/test_file_distribution.rb
 homepage: http://github.org/caffecoder/fdist-ruby
-licenses: 
+licenses:
 - MIT
 metadata: {}
-
 post_install_message: 
-rdoc_options: []
-
-require_paths: 
+rdoc_options:
+- "--charset=UTF-8"
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  requirements: 
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      version: 1.8.7
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  requirements: 
+    - !ruby/object:Gem::Version
+      version: 2.2.1
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 2.0.15
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Simple file distribution library.
-test_files: 
+test_files:
 - test/test_file_distribution.rb