Merge pull request #5 from X140Yu/cocoapods-plugin

Closes #2

Author: X140Yu

cocoapods-dependency.gemspec

@@ -1,10 +1,10 @@
 lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'cocoapods/dependency/version'
+require 'cocoapods-dependency/version'
 
 Gem::Specification.new do |spec|
   spec.name          = 'cocoapods-dependency'
-  spec.version       = Cocoapods::Dependency::VERSION
+  spec.version       = CocoapodsDependency::VERSION
   spec.authors       = ['Xinyu Zhao']
   spec.email         = ['[email protected]']
 

lib/cocoapods-dependency.rb

@@ -0,0 +1 @@
+require 'cocoapods-dependency/version'

lib/cocoapods/analyze.rb renamed to lib/cocoapods-dependency/analyze.rb

@@ -1,14 +1,15 @@
 require 'cocoapods'
 require 'pathname'
 
-module Cocoapods
+module CocoapodsDependency
   #
   # Analyze the project using cocoapods
   #
-  class DependencyAnalyzer
+  class Analyzer
     def self.analyze(podfile_dir_path)
       path = Pathname.new(podfile_dir_path)
       raise 'absolute path is needed' unless path.absolute?
+
       Dir.chdir(podfile_dir_path) do
         analyze_with_podfile(
           path,
@@ -24,9 +25,11 @@ def self.podfile_dependencies(podfile)
         children_definitions = td.recursive_children
         children_definitions.each do |cd|
           dependencies_hash_array = cd.send(:get_hash_value, 'dependencies')
-          next if dependencies_hash_array.count == 0
+          next if dependencies_hash_array.count.zero?
+
           dependencies_hash_array.each do |item|
             next if item.class.name != 'Hash'
+
             item.each do |name, _|
               res.push name
             end
@@ -78,6 +81,7 @@ def self.analyze_with_podfile(_podfile_dir_path, podfile, lockfile = nil)
 
     def self.find_dependencies(name, map, res, dependencies_map, root_name)
       return unless map[name]
+
       map[name].each do |k|
         find_dependencies(k.name, map, res, dependencies_map, root_name)
         dependency = dependencies_map[k.name.split('/')[0]]

lib/cocoapods-dependency/command.rb

@@ -0,0 +1 @@
+require 'cocoapods-dependency/command/dependency'

lib/cocoapods-dependency/command/dependency.rb

@@ -0,0 +1,30 @@
+require 'cocoapods-dependency/analyze'
+require 'pp'
+module Pod
+  class Command
+
+    class Dependency < Command
+      self.summary = 'Analyzes the dependencies of any cocoapods projects.'
+
+      self.description = <<-DESC
+      Analyzes the dependencies of any cocoapods projects. Subspecs are properly handled.
+      DESC
+
+      # self.arguments = 'NAME'
+
+      def initialize(argv)
+        @name = argv.shift_argument
+        super
+      end
+
+      def validate!
+        super
+        verify_podfile_exists!
+      end
+
+      def run
+        pp CocoapodsDependency::Analyzer.analyze_with_podfile(nil, config.podfile)
+      end
+    end
+  end
+end

lib/cocoapods-dependency/version.rb

@@ -0,0 +1,4 @@
+# :nocov:
+module CocoapodsDependency
+  VERSION = "0.1.0"
+end

lib/cocoapods/dependency/version.rb

@@ -0,0 +1 @@
+require 'cocoapods-dependency/command'

lib/cocoapods_plugin.rb

@@ -1,10 +1,10 @@
-RSpec.describe Cocoapods::Dependency do
+RSpec.describe 'CocoapodsDependency' do
   it 'has a version number' do
-    expect(Cocoapods::Dependency::VERSION).not_to be nil
+    expect(CocoapodsDependency::VERSION).not_to be nil
   end
 
   it 'no podfile exists in path will raise error' do
-    expect { Cocoapods::DependencyAnalyzer.analyze(Pathname.new('/')) }.to raise_error(/No Podfile exists/)
+    expect { CocoapodsDependency::Analyzer.analyze(Pathname.new('/')) }.to raise_error(/No Podfile exists/)
   end
 
   it 'generate custom podfile' do
@@ -27,7 +27,7 @@
     end
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq('Masonry' => [])
   end
 
@@ -46,7 +46,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 
@@ -64,7 +64,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 
@@ -82,7 +82,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 
@@ -101,7 +101,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 
@@ -122,7 +122,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 
@@ -146,7 +146,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 
@@ -167,7 +167,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 
@@ -190,7 +190,7 @@
     }
 
     expect(
-      Cocoapods::DependencyAnalyzer.analyze_with_podfile(Pathname.new('/'), podfile)
+      CocoapodsDependency::Analyzer.analyze_with_podfile(Pathname.new('/'), podfile)
     ).to eq(res_map)
   end
 end

spec/cocoapods/dependency_spec.rb

@@ -2,7 +2,7 @@
 Coveralls.wear!
 
 require 'bundler/setup'
-require 'cocoapods/analyze'
+require 'cocoapods-dependency/analyze'
 
 RSpec.configure do |config|
   # Enable flags like --only-failures and --next-failure