rstub 0.1.3 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ecffc042bdc6192d0e134540bf3952861dc70bac
-  data.tar.gz: dc688dd4721bd4cbde6f381b058cb55d2e6972d4
+  metadata.gz: fca65c701dc58118fbb3791d49778b7119309bce
+  data.tar.gz: f34d7b9e44d1df57392939bbd6579068a677f14d
 SHA512:
-  metadata.gz: 29c7d8a57f05805d818a1fe56b987cb303de7d9d0cca65829f259694c8944c5268ffa86ff539fcac9fb53231e8631d5cc95fe3f07832ecc959f4ef14cdcd241a
-  data.tar.gz: 2f6d0adc62a34381d6d269380c735e4285d90b81c7f01b5bcfbbf419d4573fbf31b2646b98e8a52f97b480e6a9369c9aeb1fe1c6209d7d2ff6a79b26c41b2400
+  metadata.gz: 5fefb10fd8466b63c231e99f3a41954c2d7579481a312fc9bbf8955cf3fd3e09ca03fc860bce1778f2881b4979bf4f204ed689eb6c12fcbedfe6518411d6b678
+  data.tar.gz: f884090380dde77d40408c6fbf5946f204ad0c8db9f8ff511c0d7a75c93c700de722135f42280c3e9e8ae70f87caca381859454d8dcff3e4d43a5b146e1a0063

data/lib/rstub/path_parser.rb

@@ -37,14 +37,22 @@ class PathParser
   end
 
   def get_directories(files)
-    files.select { |file| Dir.exist?(file) }
-      .concat(parse_out_directories(files))
+    dirs = []
+    files.each do |file|
+      if Dir.exist? file
+        dirs << file
+        dirs.concat(get_directories(Dir.glob("#{file}/*")))
+      end
+    end
+    dirs
   end
 
   def get_files_from_directory(files, directories)
     directories.each do |dir|
       files.delete(dir)
-      files << Dir.glob("#{dir}/*")
+      Dir.glob("#{dir}/*").each do |file|
+        files << file unless File.directory? file
+      end
     end
     files
   end

data/lib/rstub.rb

@@ -38,18 +38,18 @@ class RStub
   end
 
   def make_target_directory
-    Dir.mkdir(target) unless Dir.exists? target
+    Dir.mkdir(target) unless Dir.exist? target
   end
 
   def make_new_directories
     directories.each do |d|
       new_dir = "#{target}/#{d}"
-      Dir.mkdir(new_dir) unless d == target && Dir.exists?(new_dir)
+      Dir.mkdir(new_dir) unless Dir.exist?(new_dir)
     end
   end
 
   def make_new_files
-    target_files.each { |file| File.new(file, 'w') }
+    target_files.each { |file| File.new(file, 'w') unless directory? file }
   end
 
   def parse_files_and_directories

data/rstub.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = 'rstub'
-  s.version = '0.1.3'
+  s.version = '0.2.0'
 
   s.date = '2015-05-30'
   s.summary = 'A gem to stub out code'

data/spec/file_parser_spec.rb

@@ -1,91 +1,80 @@
 describe FileParser do
   describe '#stub' do
-    before(:all) do
-      Dir.chdir('spec/fixtures')
-    end
-
-    after(:each) do
-      FileUtils.rm_r('target')
-    end
-
-    after(:all) do
-      File.new('foo.rb', 'w')
-      Dir.chdir('../..')
-    end
-
-    def add_text(text)
-      File.open('foo.rb', 'w') { |f| f.puts text }
+    def add_text(file, text)
+      File.open(file, 'w') { |f| f.puts text }
     end
 
     def stub_all
       RStub.new.start(['*', 'target'])
     end
 
-    def expect_text(text)
-      expect(IO.read('target/foo.rb')).to eql(text)
+    def expect_text(file, text)
+      expect(IO.read("target/#{file}")).to eql(text)
     end
 
-    it 'can add text without stubs' do
-      text = "foo\nbar\n"
-      add_text(text)
-      stub_all
-      expect_text("foo\nbar\n")
-    end
+    describe 'can stub files in the top directory' do
+      it 'can add text without stubs' do
+        text = "foo\nbar\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', "foo\nbar\n")
+      end
 
-    it 'stubs out stub delimiter' do
-      text = "# STUB\n"
-      add_text(text)
-      stub_all
-      expect_text('')
-    end
+      it 'stubs out stub delimiter' do
+        text = "# STUB\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', '')
+      end
 
-    it 'preserves text before stub delimiter' do
-      text = "hello\n# STUB\n"
-      add_text(text)
-      stub_all
-      expect_text("hello\n")
-    end
+      it 'preserves text before stub delimiter' do
+        text = "hello\n# STUB\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', "hello\n")
+      end
 
-    it 'doesn\'t include endstub delimiter' do
-      text = "# STUB\n# ENDSTUB\n"
-      add_text(text)
-      stub_all
-      expect_text('')
-    end
+      it 'doesn\'t include endstub delimiter' do
+        text = "# STUB\n# ENDSTUB\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', '')
+      end
 
-    it 'doesn\'t include text between stub delimiters' do
-      text = "# STUB\n hello # ENDSTUB\n"
-      add_text(text)
-      stub_all
-      expect_text('')
-    end
+      it 'doesn\'t include text between stub delimiters' do
+        text = "# STUB\n hello # ENDSTUB\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', '')
+      end
 
-    it 'doesn\'t include text after the # STUB' do
-      text = "# STUB\n hello\n"
-      add_text(text)
-      stub_all
-      expect_text('')
-    end
+      it 'doesn\'t include text after the # STUB' do
+        text = "# STUB\n hello\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', '')
+      end
 
-    it 'doesn\'t care about other stub starting delimiters after the first' do
-      text = "# STUB\n hello\n # STUB"
-      add_text(text)
-      stub_all
-      expect_text('')
-    end
+      it 'doesn\'t care about other stub starting delimiters after the first' do
+        text = "# STUB\n hello\n # STUB"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', '')
+      end
 
-    it 'includes plain end stub delimiters' do
-      text = "# ENDSTUB\n"
-      add_text(text)
-      stub_all
-      expect_text("# ENDSTUB\n")
-    end
+      it 'includes plain end stub delimiters' do
+        text = "# ENDSTUB\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', "# ENDSTUB\n")
+      end
 
-    it 'resumes putting in text after end delimiter' do
-      text = "hello\n# STUB\nworld\n# ENDSTUB\nfoo\n"
-      add_text(text)
-      stub_all
-      expect_text("hello\nfoo\n")
+      it 'resumes putting in text after end delimiter' do
+        text = "hello\n# STUB\nworld\n# ENDSTUB\nfoo\n"
+        add_text('file1.rb', text)
+        stub_all
+        expect_text('file1.rb', "hello\nfoo\n")
+      end
     end
   end
 end

data/spec/path_parser_spec.rb

@@ -1,37 +1,37 @@
 describe PathParser do
-  before(:all) do
-    Dir.chdir('spec/fixtures')
-  end
+  describe '#get_globs' do
+    before :all do
+      Dir.mkdir 'test_dir/nested_dir'
+      File.new 'test_dir/nested_dir/nested_file2.rb', 'w+'
+    end
 
-  after(:all) do
-    Dir.chdir('../..')
-  end
+    after :all do
+      FileUtils.rm_r 'test_dir/nested_dir' if Dir.exist? 'test_dir/nested_dir'
+    end
 
-  describe '#get_globs' do
     let(:path_parser) { PathParser.new }
 
     it 'returns the same array when there is no match' do
-      expect(path_parser.get_globs(['foo.rb'])[:files])
-        .to match_array(['foo.rb'])
+      expect(path_parser.get_globs(['file1.rb'])[:files])
+        .to match_array(['file1.rb'])
     end
 
     it 'returns glob matches from current directory' do
-      expect(path_parser.get_globs(['*']))
-        .to match_array(directories: ['foobar'],
-                        files: %w(baz.rb foo.rb foobar/foobaz.rb))
-    end
-
-    it 'adds glob matches to the rest of the files' do
-      expect(path_parser.get_globs(['*', 'foo/bar.rb'])[:directories])
-        .to match_array(%w(foo foobar))
-      expect(path_parser.get_globs(['*', 'foo/bar.rb'])[:files])
-        .to match_array(%w(foo/bar.rb baz.rb foo.rb foobar/foobaz.rb))
+      paths = path_parser.get_globs(['*'])
+      expect(paths[:directories])
+        .to match_array(%w(test_dir test_dir/nested_dir))
+      expect(paths[:files])
+        .to match_array(%w(file1.rb file2.rb test_dir/nested_file.rb
+                           test_dir/nested_dir/nested_file2.rb))
     end
 
     it 'returns glob matches from lowel directory' do
-      expect(path_parser.get_globs(['*/*']))
-        .to match_array(directories: ['foobar'],
-                        files: ['foobar/foobaz.rb'])
+      paths = path_parser.get_globs(['*/*'])
+      expect(paths[:directories])
+        .to match_array(%w(test_dir/nested_dir))
+      expect(paths[:files])
+        .to match_array(%w(test_dir/nested_file.rb
+                           test_dir/nested_dir/nested_file2.rb))
     end
   end
 end

data/spec/rstub_spec.rb

@@ -1,36 +1,24 @@
 describe RStub do
   describe '#start' do
-    before(:all) do
-      Dir.chdir('spec/fixtures')
-    end
-
-    after(:each) do
-      FileUtils.rm_r 'target' if Dir.exist?('target')
-    end
-
-    after(:all) do
-      Dir.chdir('../..')
-    end
-
     let(:rstub) { RStub.new }
 
     it 'creates the correct directory and file' do
-      rstub.start(['foo.rb', 'target'])
+      rstub.start(['file1.rb', 'target'])
       expect(Dir.exist?('target')).to be true
-      expect(File.exist?('target/foo.rb')).to be true
+      expect(File.exist?('target/file1.rb')).to be true
     end
 
     it 'raises error with invalid number of arguments' do
       expect { rstub.start }.to raise_error('Not enough arguments')
-      expect { rstub.start(['foo']) }.to raise_error('Not enough arguments')
+      expect { rstub.start(['file1']) }.to raise_error('Not enough arguments')
     end
 
     it 'raises error if the last argument isn\'t a directory' do
-      expect { rstub.start(['foo.rb', 'baz.rb']) }
+      expect { rstub.start(['file1.rb', 'file2.rb']) }
         .to raise_error('The last argument needs to be a directory')
-      expect { rstub.start(['target', 'foo.rb']) }
+      expect { rstub.start(['target', 'file1.rb']) }
         .to raise_error('The last argument needs to be a directory')
-      expect { rstub.start(['foo.rb', '*']) }
+      expect { rstub.start(['file1.rb', '*']) }
         .to raise_error('The last argument needs to be a directory')
     end
 
@@ -41,32 +29,32 @@ describe RStub do
     end
 
     it 'adds multiple files to the directory' do
-      rstub.start(['foo.rb', 'baz.rb', 'target'])
+      rstub.start(['file1.rb', 'file2.rb', 'target'])
       expect(Dir.exist?('target')).to be true
-      expect(File.exist?('target/foo.rb')).to be true
-      expect(File.exist?('target/baz.rb')).to be true
+      expect(File.exist?('target/file1.rb')).to be true
+      expect(File.exist?('target/file2.rb')).to be true
     end
 
     it 'doesn\'t add nonexistent files even when there are multiple files' do
-      rstub.start(['foo.rb', 'bar.rb', 'target'])
+      rstub.start(['file1.rb', 'bar.rb', 'target'])
       expect(Dir.exist?('target')).to be true
-      expect(File.exist?('target/foo.rb')).to be true
+      expect(File.exist?('target/file1.rb')).to be true
       expect(File.exist?('target/bar.rb')).to be false
     end
 
     it 'doesn\'t add nonexistent files even with wildcard' do
-      rstub.start(['foo/*', 'target'])
+      rstub.start(['file1/*', 'target'])
       expect(Dir.exist?('target')).to be true
-      expect(Dir.exist?('target/foo')).to be false
+      expect(Dir.exist?('target/file1')).to be false
     end
 
     it 'all files are added with wildcard' do
       rstub.start(['*', 'target'])
       expect(Dir.exist?('target')).to be true
-      expect(File.exist?('target/foo.rb')).to be true
-      expect(File.exist?('target/baz.rb')).to be true
-      expect(Dir.exist?('target/foobar')).to be true
-      expect(File.exist?('target/foobar/foobaz.rb')).to be true
+      expect(File.exist?('target/file1.rb')).to be true
+      expect(File.exist?('target/file2.rb')).to be true
+      expect(Dir.exist?('target/test_dir')).to be true
+      expect(File.exist?('target/test_dir/nested_file.rb')).to be true
     end
   end
 end

data/spec/spec_helper.rb

@@ -7,3 +7,24 @@ Coveralls.wear!
 
 require 'rstub'
 Dir[File.expand_path('../../lib/rstub/*.rb', __FILE__)].each { |f| require f }
+
+RSpec.configure do |config|
+  config.before :all do
+    Dir.chdir('spec/fixtures')
+    Dir.mkdir 'test_dir' unless Dir.exist? 'test_dir'
+    File.new 'file1.rb', 'w+'
+    File.new 'file2.rb', 'w+'
+    File.new 'test_dir/nested_file.rb', 'w+'
+  end
+
+  config.after :each do
+    FileUtils.rm_r 'target' if Dir.exist? 'target'
+  end
+
+  config.after :all do
+    FileUtils.rm_r 'test_dir' if Dir.exist? 'test_dir'
+    FileUtils.rm_r 'file1.rb'
+    FileUtils.rm_r 'file2.rb'
+    Dir.chdir('../..')
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rstub
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.0
 platform: ruby
 authors:
 - Justin Kim
@@ -59,9 +59,6 @@ files:
 - lib/rstub/path_parser.rb
 - rstub.gemspec
 - spec/file_parser_spec.rb
-- spec/fixtures/baz.rb
-- spec/fixtures/foo.rb
-- spec/fixtures/foobar/foobaz.rb
 - spec/path_parser_spec.rb
 - spec/rstub_spec.rb
 - spec/spec_helper.rb
@@ -91,9 +88,7 @@ specification_version: 4
 summary: A gem to stub out code
 test_files:
 - spec/file_parser_spec.rb
-- spec/fixtures/baz.rb
-- spec/fixtures/foo.rb
-- spec/fixtures/foobar/foobaz.rb
 - spec/path_parser_spec.rb
 - spec/rstub_spec.rb
 - spec/spec_helper.rb
+has_rdoc: 

data/spec/fixtures/baz.rb

@@ -1 +0,0 @@
-