Fixed issue of running kuniri on empty directories

data/end_block_lang.rb

@@ -11,7 +11,7 @@ class EndBlock{LANG} < Languages::EndBlock
 
         # Verify {LANG} end of block.
         # @see EndBlock
-        def has_end_of_block?(pLine)
+        def end_of_block?(pLine)
           # YOUR CODE HERE
         end
 

lib/kuniri/core/configuration/language_available.rb

@@ -5,9 +5,8 @@
 # Version 3.  See the file COPYING for more details
 
 module Configuration
-  #Handling the languages available in the system.
+  # Handling the languages available in the system.
   class LanguageAvailable
-    public
-      LANGUAGES = ["ruby", "python", "cplusplus", "c"] #Put here new language.
+    LANGUAGES = %w(ruby python cplusplus c).freeze # Put here new language.
   end
 end

lib/kuniri/core/configuration/log_available.rb

@@ -6,8 +6,7 @@
 
 module Configuration
   # Configuration for the monitor available.
-  class Log_Available
-    public
-      LOG = ["html", "txt"] #List with log types.
+  class LogAvailable
+    LOG = %w(html txt).freeze # List with log types.
   end
 end

lib/kuniri/core/kuniri.rb

@@ -24,8 +24,9 @@ class Kuniri
     def initialize
       @configurationInfo = {}
       @filesPathProject = []
-      @parserFiles = []
-      @parser = nil
+      @parserFiles = []   # !@attribute Final output from parser.
+      @filesProject = []  # !@attribute Array with object reference of all files
+      @parser = nil       # !@attribute Execute the parser based on settings.
     end
 
     def read_configuration_file(pPath = '.kuniri.yml')
@@ -38,10 +39,10 @@ def read_configuration_file(pPath = '.kuniri.yml')
     def set_configuration(pSource, pLanguage, pOutput, pLevel)
       @settings = Setting.create
       @settings.initializate_settings
-      @configurationInfo = {language: pLanguage,
-                            source: pSource,
-                            output: pOutput,
-                            pLevel: pLevel}
+      @configurationInfo = { language: pLanguage,
+                             source: pSource,
+                             output: pOutput,
+                             pLevel: pLevel }
 
       @settings.set_configuration(@configurationInfo)
     end
@@ -58,7 +59,11 @@ def run_analysis
       @parser = Parser::Parser.new(@filesPathProject,
                                    @configurationInfo[:language])
       Util::LoggerKuniri.info('Start parse...')
-      @parser.start_parser
+      begin
+        @parser.start_parser
+      rescue Error::ConfigurationFileError => e
+        puts e.message
+      end
     end
 
     def get_parser
@@ -67,29 +72,15 @@ def get_parser
 
     private
 
-    @filesProject      # !@attribute Array with object reference of all files
-    @parser            # !@attribute Execute the parser based on settings.
-    @parserFiles       # !@attribute Final output from parser.
-
     # !@param pPath Relative path of the project.
     # !@param pLanguage Language extension for make the parser.
     def get_project_file(pPath = './', pLanguage = '**.rb')
       # Verify if path is a valid directory or file
-      return nil unless File.file?(pPath) or File.directory?(pPath)
-
-      # Handle single file
-      if (File.file?(pPath))
-        @filesProject = [pPath]
-      # Handle multiple files
-      else
-        @filesProject = Dir[File.join(pPath, '**', pLanguage)]
-      end
+      return nil unless File.file?(pPath) || File.directory?(pPath)
 
-      return @filesProject
+      # Handle single file and multiple files
+      @filesProject =
+        File.file?(pPath) ? [pPath] : Dir[File.join(pPath, '**', pLanguage)]
     end
-
-  # Class
-  end
-
-# Kuniri
-end
+  end # Class
+end # Kuniri

lib/kuniri/core/setting.rb

@@ -22,8 +22,8 @@ def initialize
     end
 
     def self.create
-      @@settings = new unless @@settings
-      return @@settings
+      @settings = new unless @settings
+      return @settings
     end
 
     # TODO: Remove parameter and initialize_object. Useless.
@@ -37,7 +37,6 @@ def initializate_settings
     # @return [Hash] Return a Hash with the configurations read in ".kuniri",
     #     otherwise, raise an exception.
     def read_configuration_file(pPath = '.kuniri.yml')
-
       if !(File.exist?(pPath))
         set_default_configuration
       else
@@ -54,62 +53,58 @@ def set_configuration(configurationInfo)
 
     private
 
-      @@settings = nil
-
-      def set_default_configuration
-        @configurationInfo = {:language => "ruby",
-                               :source => "./",
-                               :output => "./"}
-      end
+    @settings = nil
 
-      def verify_syntax
-        unless @configurationInfo.is_a? Hash
-          Util::LoggerKuniri.error('Configuration file has a syntax problem')
-          raise Error::ConfigurationFileError
-        end
+    def set_default_configuration
+      @configurationInfo = { language: 'ruby',
+                             source: './',
+                             output: './' }
+    end
 
-        check_source
-        check_output
-        check_language
+    def verify_syntax
+      unless @configurationInfo.is_a? Hash
+        Util::LoggerKuniri.error('Configuration file has a syntax problem')
+        raise Error::ConfigurationFileError
       end
 
-      def check_source
-        unless @configurationInfo.has_key?:source
-          Util::LoggerKuniri.error('Problem with source parameter')
-          raise Error::ConfigurationFileError
-        else
-          source = @configurationInfo[:source]
-          result = (File.directory?source) || (File.exists?source)
-          unless result
-            Util::LoggerKuniri.error('Wrong path on source')
-            raise Error::ConfigurationFileError
-          end
-        end
-      end
+      check_source
+      check_output
+      check_language
+    end
 
-      def check_output
-        unless @configurationInfo.has_key?:output
-          Util::LoggerKuniri.error('Problem with output field')
+    def check_source
+      if !@configurationInfo.key?:source
+        Util::LoggerKuniri.error('Problem with source parameter')
+        raise Error::ConfigurationFileError
+      else
+        source = @configurationInfo[:source]
+        result = (File.directory?source) || (File.exist?source)
+        unless result
+          Util::LoggerKuniri.error('Wrong path on source')
           raise Error::ConfigurationFileError
         end
       end
+    end
+
+    def check_output
+      return if @configurationInfo.key?:output
+      Util::LoggerKuniri.error('Problem with output field')
+      raise Error::ConfigurationFileError
+    end
 
-      def check_language
-        unless @configurationInfo.has_key?:language
-          Util::LoggerKuniri.error('Problem with language field')
+    def check_language
+      if !@configurationInfo.key?:language
+        Util::LoggerKuniri.error('Problem with language field')
+        raise Error::ConfigurationFileError
+      else
+        result = Configuration::LanguageAvailable::LANGUAGES.include?(
+          @configurationInfo[:language]
+        )
+        unless result
+          Util::LoggerKuniri.error('Problem with specified language')
           raise Error::ConfigurationFileError
-        else
-          result = Configuration::LanguageAvailable::LANGUAGES.include?(
-                                                @configurationInfo[:language])
-          unless result
-            Util::LoggerKuniri.error('Problem with specified language')
-            raise Error::ConfigurationFileError
-          end
         end
       end
-
-  # Class
-  end
-
-# Module
-end
+    end
+  end # Class
+end # Module

lib/kuniri/language/abstract_container/structured_and_oo/aggregation.rb

@@ -8,28 +8,22 @@ module Languages
 
   # @abstract Aggregation is a container for handling aggregation among classes
   class Aggregation
-
-    public
-
-      # Verify if line has an aggregation.
-      # @param pLine Line with the potential class.
-      # @return Return an object AggregationData if it find a class in the
-      #         line, otherwise return nil.
-      def get_aggregation(pLine)
-        raise NotImplementedError
-      end
+    # Verify if line has an aggregation.
+    # @param pLine Line with the potential class.
+    # @return Return an object AggregationData if it find a class in the
+    #         line, otherwise return nil.
+    def get_aggregation(_pLine)
+      raise NotImplementedError
+    end
 
     protected
 
-      # Detect if an aggregation exists.
-      # @param pLine Line to inspect.
-      # @return Returns an row string, otherwise returns nil.
-      def detect_aggregation(pLine)
-        raise NotImplementedError
-      end
-
-  # class
-  end
+    # Detect if an aggregation exists.
+    # @param pLine Line to inspect.
+    # @return Returns an row string, otherwise returns nil.
+    def detect_aggregation(_pLine)
+      raise NotImplementedError
+    end
 
-# module
-end
+  end # class
+end # module

lib/kuniri/language/abstract_container/structured_and_oo/attribute.rb

@@ -10,29 +10,18 @@ module Languages
 
   # @abstract Abstract class for handling attribute.
   class Attribute
-
-    public
-
-      include VariableBehaviourHelpers
-
-      # Verify if a line has an attribute. If it has attribute, firstly, the
-      # function capture all lines and removes specific language declaration
-      # (for instance, in ruby it is:  '@' or ':' and whitespace), finally
-      # it splits the string by an special character and return an object of
-      # AttributeData.
-      # @param pLine An line to be analysed for find attribute.
-      # @return Return on filled object of AttributeData if it find an
-      #         attribute, otherwise it returns nil.
-      def get_attribute(pLine)
-        raise NotImplementedError
-      end
-
-    protected
-
-      @variableBehaviour
-
-  # Class
-  end
-
-# Language
-end
+    include VariableBehaviourHelpers
+
+    # Verify if a line has an attribute. If it has attribute, firstly, the
+    # function capture all lines and removes specific language declaration
+    # (for instance, in ruby it is:  '@' or ':' and whitespace), finally
+    # it splits the string by an special character and return an object of
+    # AttributeData.
+    # @param pLine An line to be analysed for find attribute.
+    # @return Return on filled object of AttributeData if it find an
+    #         attribute, otherwise it returns nil.
+    def get_attribute(_pLine)
+      raise NotImplementedError
+    end
+  end # Class
+end # Language

lib/kuniri/language/abstract_container/structured_and_oo/block.rb

@@ -8,45 +8,38 @@ module Languages
 
   # @abstract Class responsible for handling block.
   class Block
-
-    public
-      # Verify if a line has an block. If it has block, firstly, the
-      # function capture all lines.
-      # @param pLine An line to be analysed for find block.
-      # @return Return on filled object of BlockData if it find an
-      #         block, otherwise it returns nil.
-      def get_block(pLine)
-        raise NotImplementedError
-      end
+    # Verify if a line has an block. If it has block, firstly, the
+    # function capture all lines.
+    # @param pLine An line to be analysed for find block.
+    # @return Return on filled object of BlockData if it find an
+    #         block, otherwise it returns nil.
+    def get_block(_pLine)
+      raise NotImplementedError
+    end
 
     protected
 
-      # Method with a regex to check if line has a block.
-      # @param pLine String with potential block inside.
-      # @return Partial string with detection, otherwise return nil.
-      def detect_block(pLine)
-        raise NotImplementedError
-      end
-
-
-      # Set the name of the block based on the name captured.
-      # Note: If this method is executed, we suppose we already have a block
-      # as a string.
-      # @param pString Partial string to be handled.
-      # @return String with block name in uppercase.
-      def capture_block_name(pString)
-        raise NotImplementedError
-      end
-
-      # Extract expression from the captured string.
-      # @param pString Full string.
-      # @return String with name expression.
-      def capture_expression(pString)
-        raise NotImplementedError
-      end
-
-  # End class
-  end
-
-# module
-end
+    # Method with a regex to check if line has a block.
+    # @param pLine String with potential block inside.
+    # @return Partial string with detection, otherwise return nil.
+    def detect_block(_pLine)
+      raise NotImplementedError
+    end
+
+    # Set the name of the block based on the name captured.
+    # Note: If this method is executed, we suppose we already have a block
+    # as a string.
+    # @param pString Partial string to be handled.
+    # @return String with block name in uppercase.
+    def capture_block_name(_pString)
+      raise NotImplementedError
+    end
+
+    # Extract expression from the captured string.
+    # @param pString Full string.
+    # @return String with name expression.
+    def capture_expression(_pString)
+      raise NotImplementedError
+    end
+  end # End class
+end # module