fec 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2032e219e5071d35f5eae7eece797031994c814d
-  data.tar.gz: 1ca6a9bd9b00455a7811e80d6fcd42c313242dbe
+  metadata.gz: 0202ab36bcf03852f076930b33f1adab85f10227
+  data.tar.gz: 2d2013c30e2b0f272b6a36ed62df1ed5e05889aa
 SHA512:
-  metadata.gz: dbc8cedac359048814607e8c42910e5c32fe404d1cc086b2e4252bfc45f623f3be41df2b3b3c5ef626195f581d928560df2a1f065fcfc691fc8599ad2e3a9170
-  data.tar.gz: 39be9f13523d1b095b93a66e37c0e71db98177114d8116272a0c3b6b31becff2c07c7cf8ebe0fc669f2a7b198bcb2beaa80256f7f9fc325c2171a24aabfd0b06
+  metadata.gz: fc9c1ea1067c7cb28e76a78e5074f9706298fc39f56cb1c26e1d1610a8259b693f4d38f0ab0c22b9e8e745b1c2bdf7116a471c9d2eb5320e2e4ae68a2c3b2a75
+  data.tar.gz: 60c0e879320a4d810b3dc047f12c182d364d0a353b7f15f3990deedffab77421ca62d0be1d44e9a1fc1a3d55796cd7ffadc806cee4b63902139a7e8236f79876

data/.rspec

@@ -0,0 +1,2 @@
+--format documentation
+--color

data/fec.gemspec

@@ -25,6 +25,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'bundler', '~> 1.11.2'
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rubocop'
 
   # comment linter directly on pull request

data/lib/fec/cli.rb

@@ -3,6 +3,7 @@ require 'thor'
 require 'rainbow'
 require_relative 'file_helper'
 require_relative 'messenger'
+require 'pry'
 
 module Fec
   class FileExtensionChanger < Thor
@@ -23,8 +24,8 @@ module Fec
       new_extension = options.fetch('new_extension')
 
       Dir.entries("#{folder_path}/").each do |name|
-        if directory?(folder_path, name)
-          path = "#{folder_path}/#{name}"
+        path = "#{folder_path}/#{name}"
+        if directory?(path)
           notice_message("Processing #{path}")
           rename(path)
         elsif name.downcase.include? old_extension

data/lib/fec/file_helper.rb

@@ -1,7 +1,10 @@
+require 'pry'
 module Fec
   module FileHelper
-    def directory?(parrent, path)
-      File.directory?(File.join(parrent, path) && !(path == '.' || path == '..'))
+    def directory?(path)
+      parrent = File.expand_path('..', path)
+      path = File.basename(path)
+      File.directory?(File.join(parrent, path)) && !(path == '.' || path == '..')
     end
   end
 end

data/lib/fec/version.rb

@@ -1,3 +1,3 @@
 module Fec
-  VERSION = '0.1.2'.freeze
+  VERSION = '0.1.3'.freeze
 end

data/spec/fec_spec.rb

@@ -1,11 +1,28 @@
 require 'spec_helper'
+require 'file_helper'
 
 describe Fec do
+  include FileHelper
+
+  let(:fec) { Fec::FileExtensionChanger.new }
+
   it 'has a version number' do
     expect(Fec::VERSION).not_to be nil
   end
 
-  it 'does something useful' do
-    expect(false).to eq(true)
+  it 'Rename success' do
+    old_file_extension = 'txt'
+    new_file_extension = 'new'
+    root = create_file(old_file_extension)
+    fec.options = {
+      'old_extension' => old_file_extension,
+      'new_extension' => new_file_extension
+    }
+    fec.rename root
+    files = files_in_folder(root)
+    true_expectation = all_files_contain_same_extension?(files, new_file_extension)
+    expect(true_expectation).to be true
+    false_expectation = all_files_contain_same_extension?(files, old_file_extension)
+    expect(false_expectation).to be false
   end
 end

data/spec/file_helper.rb

@@ -0,0 +1,44 @@
+require 'fec/file_helper'
+
+module FileHelper
+  include Fec::FileHelper
+  def all_files_contain_same_extension?(files, extension)
+    files.each do |item|
+      return false unless File.extname(item).include? extension
+    end
+    true
+  end
+
+  def create_file(file_extension)
+    root = "#{Dir.tmpdir}/foo"
+    bar_path = "#{root}/bar"
+    lorem_path = "#{root}/lorem"
+    mkdir_and_create_file(root, file_extension)
+    mkdir_and_create_file(bar_path, file_extension)
+    mkdir_and_create_file(lorem_path, file_extension)
+    root
+  end
+
+  def files_in_folder(path)
+    files = []
+    Dir.entries(path).each do |name|
+      current_path = "#{path}/#{name}"
+      if directory?(current_path)
+        files_in_folder(current_path).each do |item|
+          files.push(item)
+        end
+      elsif name.length > 2
+        files.push(name)
+      end
+    end
+    files
+  end
+
+  private
+
+  def mkdir_and_create_file(folder_path, file_extension)
+    FileUtils.mkdir_p folder_path
+    file_name = "#{Time.now.to_f}.#{file_extension}"
+    File.open("#{folder_path}/#{file_name}", 'w') { |f| f.write('') }
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fec
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - CAO Quang Binh
@@ -66,6 +66,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -148,6 +162,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - ".rubocop.yml"
 - Gemfile
 - LICENSE.txt
@@ -165,6 +180,7 @@ files:
 - lib/fec/messenger.rb
 - lib/fec/version.rb
 - spec/fec_spec.rb
+- spec/file_helper.rb
 - spec/spec_helper.rb
 homepage: https://github.com/CQBinh/fec
 licenses:
@@ -192,4 +208,5 @@ specification_version: 4
 summary: File Extension Changer.
 test_files:
 - spec/fec_spec.rb
+- spec/file_helper.rb
 - spec/spec_helper.rb