flutter_rb 0.8.2 → 1.0.0

data/lib/flutter_rb/project/specs/flutter/platform_plugin.rb

@@ -1,18 +1,18 @@
-module FlutterRb
-  # Flutter plugin, contains platform, package and plugin class
-  class PlatformPlugin
-    def initialize(platform, package, plugin_class)
-      @platform = platform
-      @package = package
-      @plugin_class = plugin_class
-    end
-
-    attr_reader :platform, :package, :plugin_class
-  end
-
-  # Supported platforms for this tool
-  class Platform
-    ANDROID = 'android'.freeze
-    IOS = 'ios'.freeze
-  end
-end
+module FlutterRb
+  # Flutter plugin, contains platform, package and plugin class
+  class PlatformPlugin
+    def initialize(platform, package, plugin_class)
+      @platform = platform
+      @package = package
+      @plugin_class = plugin_class
+    end
+
+    attr_reader :platform, :package, :plugin_class
+  end
+
+  # Supported platforms for this tool
+  class Platform
+    ANDROID = 'android'.freeze
+    IOS = 'ios'.freeze
+  end
+end

data/lib/flutter_rb/project/specs/flutter/pubspec.rb

@@ -1,69 +1,69 @@
-require_relative './pubspec_info'
-require_relative './dev_dependency'
-require_relative './platform_plugin'
-
-module FlutterRb
-  # pubspec.yaml representation
-  class Pubspec
-    def initialize(
-      path,
-      pubspec_info,
-      dev_dependencies,
-      platform_plugins
-    )
-      @path = path
-      @pubspec_info = pubspec_info
-      @dev_dependencies = dev_dependencies
-      @platform_plugins = platform_plugins
-    end
-
-    attr_reader :path, :pubspec_info, :dev_dependencies, :platform_plugins
-  end
-
-  # pubspec.yaml parser
-  class PubspecParser
-    def initialize(path, pubspec)
-      @path = path
-      @pubspec = pubspec
-    end
-
-    def parse
-      Pubspec.new(
-        @path,
-        pubspec_info(@pubspec),
-        dev_dependencies(@pubspec),
-        platform_plugins(@pubspec)
-      )
-    end
-
-    def pubspec_info(pubspec)
-      PubspecInfo.new(
-        pubspec['name'],
-        pubspec['description'],
-        pubspec['version'],
-        pubspec['author'],
-        pubspec['homepage']
-      )
-    end
-
-    def dev_dependencies(pubspec)
-      pubspec['dev_dependencies']&.map do |dev_dependency|
-        DevDependency.new(
-          dev_dependency.first,
-          dev_dependency.last
-        )
-      end
-    end
-
-    def platform_plugins(pubspec)
-      pubspec.dig('flutter', 'plugin', 'platforms')&.map do |platform_plugin|
-        plugin_info = platform_plugin.last
-        PlatformPlugin.new(
-          plugin_info['package'],
-          plugin_info['pluginClass'],
-          platform_plugin.first == Platform::ANDROID ? Platform::ANDROID : Platform::IOS
-        )
-      end
-    end
-  end
-end
+require_relative './pubspec_info'
+require_relative './dev_dependency'
+require_relative './platform_plugin'
+
+module FlutterRb
+  # pubspec.yaml representation
+  class Pubspec
+    def initialize(
+      path,
+      pubspec_info,
+      dev_dependencies,
+      platform_plugins
+    )
+      @path = path
+      @pubspec_info = pubspec_info
+      @dev_dependencies = dev_dependencies
+      @platform_plugins = platform_plugins
+    end
+
+    attr_reader :path, :pubspec_info, :dev_dependencies, :platform_plugins
+  end
+
+  # pubspec.yaml parser
+  class PubspecParser
+    def initialize(path, pubspec)
+      @path = path
+      @pubspec = pubspec
+    end
+
+    def parse
+      Pubspec.new(
+        @path,
+        pubspec_info(@pubspec),
+        dev_dependencies(@pubspec),
+        platform_plugins(@pubspec)
+      )
+    end
+
+    def pubspec_info(pubspec)
+      PubspecInfo.new(
+        pubspec['name'],
+        pubspec['description'],
+        pubspec['version'],
+        pubspec['author'],
+        pubspec['homepage']
+      )
+    end
+
+    def dev_dependencies(pubspec)
+      pubspec['dev_dependencies']&.map do |dev_dependency|
+        DevDependency.new(
+          dev_dependency.first,
+          dev_dependency.last
+        )
+      end
+    end
+
+    def platform_plugins(pubspec)
+      pubspec.dig('flutter', 'plugin', 'platforms')&.map do |platform_plugin|
+        plugin_info = platform_plugin.last
+        PlatformPlugin.new(
+          plugin_info['package'],
+          plugin_info['pluginClass'],
+          platform_plugin.first == Platform::ANDROID ? Platform::ANDROID : Platform::IOS
+        )
+      end
+    end
+  end
+end

data/lib/flutter_rb/project/specs/flutter/pubspec_info.rb

@@ -1,20 +1,20 @@
-module FlutterRb
-  # Flutter plugin info from pubspec.yaml
-  class PubspecInfo
-    def initialize(
-      name,
-      description,
-      version,
-      author,
-      homepage
-    )
-      @name = name
-      @description = description
-      @version = version
-      @author = author
-      @homepage = homepage
-    end
-
-    attr_reader :name, :description, :version, :author, :homepage
-  end
-end
+module FlutterRb
+  # Flutter plugin info from pubspec.yaml
+  class PubspecInfo
+    def initialize(
+      name,
+      description,
+      version,
+      author,
+      homepage
+    )
+      @name = name
+      @description = description
+      @version = version
+      @author = author
+      @homepage = homepage
+    end
+
+    attr_reader :name, :description, :version, :author, :homepage
+  end
+end

data/lib/flutter_rb/project/specs/ios/ios_folder.rb

@@ -1,14 +1,14 @@
-require_relative './podspec'
-
-module FlutterRb
-  # iOS representation
-  class IOSFolder
-    def initialize(path, pubspec)
-      @path = path
-      podspec_path = "#{path}/#{pubspec.pubspec_info.name}.podspec"
-      @podspec = File.exist?(podspec_path) ? PodspecParser.new(podspec_path).parse : nil
-    end
-
-    attr_reader :path, :podspec
-  end
-end
+require_relative './podspec'
+
+module FlutterRb
+  # iOS representation
+  class IOSFolder
+    def initialize(path, pubspec)
+      @path = path
+      podspec_path = "#{path}/#{pubspec.pubspec_info.name}.podspec"
+      @podspec = File.exist?(podspec_path) ? PodspecParser.new(podspec_path).parse : nil
+    end
+
+    attr_reader :path, :podspec
+  end
+end

data/lib/flutter_rb/project/specs/ios/podspec.rb

@@ -1,40 +1,40 @@
-require 'cocoapods'
-
-module FlutterRb
-  # Podspec representation
-  class Podspec
-    def initialize(
-      path,
-      name,
-      version,
-      authors,
-      source
-    )
-      @path = path
-      @name = name
-      @version = version
-      @authors = authors
-      @source = source
-    end
-
-    attr_reader :path, :name, :version, :authors, :source
-  end
-
-  # Podspec parser
-  class PodspecParser
-    def initialize(path)
-      @path = path
-    end
-
-    def parse
-      podspec = Pod::Specification.from_file(@path)
-      @podspec = Podspec.new(
-        @path,
-        podspec.name,
-        podspec.version.version,
-        podspec.authors.nil? ? podspec.author : podspec.authors,
-        podspec.source
-      )
-    end
-  end
-end
+require 'cocoapods'
+
+module FlutterRb
+  # Podspec representation
+  class Podspec
+    def initialize(
+      path,
+      name,
+      version,
+      authors,
+      source
+    )
+      @path = path
+      @name = name
+      @version = version
+      @authors = authors
+      @source = source
+    end
+
+    attr_reader :path, :name, :version, :authors, :source
+  end
+
+  # Podspec parser
+  class PodspecParser
+    def initialize(path)
+      @path = path
+    end
+
+    def parse
+      podspec = Pod::Specification.from_file(@path)
+      @podspec = Podspec.new(
+        @path,
+        podspec.name,
+        podspec.version.version,
+        podspec.authors.nil? ? podspec.author : podspec.authors,
+        podspec.source
+      )
+    end
+  end
+end

data/lib/flutter_rb/report/check_report.rb

@@ -1,47 +1,47 @@
-require 'colorize'
-
-module FlutterRb
-  # Check report
-  class CheckReport
-    def initialize(
-      check_name,
-      check_report_status,
-      message,
-      path
-    )
-      @check_name = check_name
-      @check_report_status = check_report_status
-      @message = message
-      @path = path
-    end
-
-    def print(colorize: true)
-      if colorize
-        status_color = color_for_report_status(@check_report_status)
-        " * [#{@check_report_status.colorize(status_color)}] #{@check_name}: #{@message}"
-      else
-        " * [#{@check_report_status}] #{@check_name}: #{@message}"
-      end
-    end
-
-    def color_for_report_status(check_report_status)
-      case check_report_status
-      when CheckReportStatus::NORMAL
-        :green
-      when CheckReportStatus::WARNING
-        :yellow
-      when CheckReportStatus::ERROR
-        :red
-      end
-    end
-
-    attr_reader :check_name, :check_report_status, :message, :path
-  end
-
-  # Check report status
-  class CheckReportStatus
-    NORMAL = 'normal'.freeze
-    WARNING = 'warning'.freeze
-    ERROR = 'error'.freeze
-  end
-end
+require 'colorize'
+
+module FlutterRb
+  # Check report
+  class CheckReport
+    def initialize(
+      check_name,
+      check_report_status,
+      message,
+      path
+    )
+      @check_name = check_name
+      @check_report_status = check_report_status
+      @message = message
+      @path = path
+    end
+
+    def print(colorize: true)
+      if colorize
+        status_color = color_for_report_status(@check_report_status)
+        " * [#{@check_report_status.colorize(status_color)}] #{@check_name}: #{@message}"
+      else
+        " * [#{@check_report_status}] #{@check_name}: #{@message}"
+      end
+    end
+
+    def color_for_report_status(check_report_status)
+      case check_report_status
+      when CheckReportStatus::NORMAL
+        :green
+      when CheckReportStatus::WARNING
+        :yellow
+      when CheckReportStatus::ERROR
+        :red
+      end
+    end
+
+    attr_reader :check_name, :check_report_status, :message, :path
+  end
+
+  # Check report status
+  class CheckReportStatus
+    NORMAL = 'normal'.freeze
+    WARNING = 'warning'.freeze
+    ERROR = 'error'.freeze
+  end
+end

data/lib/flutter_rb.rb

@@ -1,93 +1,95 @@
-require_relative './flutter_rb/project/project'
-require_relative './flutter_rb/checks/plugin_directories_check'
-require_relative './flutter_rb/checks/plugin_pubspec_check'
-require_relative './flutter_rb/checks/plugin_gradle_check'
-require_relative './flutter_rb/checks/plugin_podspec_check'
-require_relative './flutter_rb/config/flutter_rb_config_initializer'
-
-require_relative './checkstyle_report/checkstyle_report'
-
-module FlutterRb
-  # Start FlutterRb checks
-  class FlutterRb
-    def start(path, with_report)
-      project = ProjectParser.new(path).project
-      if project.nil?
-        exit_with_no_project
-      else
-        check_project(
-          project,
-          path,
-          with_report
-        )
-      end
-    end
-
-    def exit_with_no_project
-      puts 'No project'
-      exit(-1)
-    end
-
-    # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
-    def check_project(project, path, with_report)
-      config_initializer = FlutterRbConfigInitializer.new
-      config_path = "#{path}/.flutter_rb.yaml"
-      config = File.exist?(config_path) ? config_initializer.parse(config_path) : config_initializer.default
-      checks = explore_project(
-        project,
-        config.flutter_checks,
-        config.android_checks,
-        config.ios_checks
-      )
-      checks.each { |check| puts check.print }
-      errors = checks.reject { |check| check.check_report_status == CheckReportStatus::NORMAL }
-      create_report(path, checks) if with_report
-      exit(errors.empty? ? 0 : -1)
-    end
-    # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
-
-    def explore_project(
-      project,
-      flutter_checks,
-      android_checks,
-      ios_checks
-    )
-      result = []
-      result += flutter_checks.map { |check| check.check(project) }
-      result += android_checks.map { |check| check.check(project) } unless project.android_folder.nil?
-      result += ios_checks.map { |check| check.check(project) } unless project.ios_folder.nil?
-      result
-    end
-
-    # rubocop:disable Metrics/MethodLength
-    def create_report(path, checks)
-      errors = checks.map do |check|
-        CheckstyleReport::CheckstyleError.new(
-          level_for_report(check.check_report_status),
-          check.message,
-          check.path,
-          0,
-          0,
-          check.check_name
-        )
-      end
-      CheckstyleReport::CheckstyleReport.new(
-        path,
-        'frb-checkstyle-report',
-        errors
-      ).create_report
-    end
-    # rubocop:enable Metrics/MethodLength
-
-    def level_for_report(check_report_status)
-      case check_report_status
-      when CheckReportStatus::NORMAL
-        CheckstyleReport::CheckstyleError::SAVERITY_NORMAL
-      when CheckReportStatus::WARNING
-        CheckstyleReport::CheckstyleError::SAVERITY_WARNING
-      when CheckReportStatus::ERROR
-        CheckstyleReport::CheckstyleError::SAVERITY_ERROR
-      end
-    end
-  end
-end
+require_relative './flutter_rb/project/project'
+require_relative './flutter_rb/checks/plugin_directories_check'
+require_relative './flutter_rb/checks/plugin_pubspec_check'
+require_relative './flutter_rb/checks/plugin_gradle_check'
+require_relative './flutter_rb/checks/plugin_podspec_check'
+require_relative './flutter_rb/config/flutter_rb_config_initializer'
+
+require_relative './checkstyle_report/checkstyle_report'
+
+module FlutterRb
+  # Start FlutterRb checks
+  class FlutterRb
+    def start(path, with_report)
+      project = ProjectParser.new(path).project
+      if project.nil?
+        exit_with_no_project
+      else
+        check_project(
+          project,
+          path,
+          with_report
+        )
+      end
+    end
+
+    def exit_with_no_project
+      puts 'No project'
+      exit(-1)
+    end
+
+    # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
+    def check_project(project, path, with_report)
+      config_initializer = FlutterRbConfigInitializer.new
+      config_path = "#{path}/.flutter_rb.yaml"
+      config = File.exist?(config_path) ? config_initializer.parse(config_path) : config_initializer.default
+      checks = explore_project(
+        project,
+        config.flutter_checks,
+        config.android_checks,
+        config.ios_checks
+      )
+      checks.each { |check| puts check.print }
+      errors = checks.reject { |check| check.check_report_status == CheckReportStatus::NORMAL }
+      create_report(path, checks) if with_report
+      exit(errors.empty? ? 0 : -1)
+    end
+    # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
+
+    def explore_project(
+      project,
+      flutter_checks,
+      android_checks,
+      ios_checks
+    )
+      result = []
+      result += flutter_checks.map { |check| check.check(project) }
+      result += android_checks.map { |check| check.check(project) } unless project.android_folder.nil?
+      result += ios_checks.map { |check| check.check(project) } unless project.ios_folder.nil?
+      result
+    end
+
+    # rubocop:disable Metrics/MethodLength
+    def create_report(path, checks)
+      errors = checks.map do |check|
+        CheckstyleReport::CheckstyleError.new(
+          level_for_report(check.check_report_status),
+          check.message,
+          check.path,
+          0,
+          0,
+          check.check_name
+        )
+      end
+      CheckstyleReport::CheckstyleReport.new(
+        path,
+        'frb-checkstyle-report',
+        errors
+      ).create_report
+    end
+    # rubocop:enable Metrics/MethodLength
+
+    def level_for_report(check_report_status)
+      case check_report_status
+      when CheckReportStatus::NORMAL
+        CheckstyleReport::CheckstyleError::SEVERITY_NORMAL
+      when CheckReportStatus::WARNING
+        CheckstyleReport::CheckstyleError::SEVERITY_WARNING
+      when CheckReportStatus::ERROR
+        CheckstyleReport::CheckstyleError::SEVERITY_ERROR
+      else
+        throw 'Unknown CheckReportStatus'
+      end
+    end
+  end
+end