danger-flutter_lint-no-filter 1.0.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: 796be253e5b35340688c8a0d2dc84ca85808a1cda4da9cb246c6073a7862ad03
+  data.tar.gz: 2e4eab3fbd1a43b6fa7b75904895d7775a01331de50b2314a5820a59b5eea977
+SHA512:
+  metadata.gz: 3ba8b9b17c1e15f798c762af01259a646cd46fcb46c639314caefac98c3b4b8ef0a40e68af746ec58ca918d1f1cb1a7e0a90884c3f7a5b0ed98de87e0072ae34
+  data.tar.gz: 18357d189bf2634f030fd1f0dac2f846e151c9be791bfec6897156e8f5a054e77886a2b43ec5e0a48bd3e72e777a1ab27b335cb7c1b5a5f41df52088c769150f

data/.gitignore

@@ -0,0 +1,11 @@
+/.bundle/
+/.yardoc
+/_yardoc/
+/coverage/
+/doc/
+/pkg/
+/spec/reports/
+/tmp/
+
+# rspec failure tracking
+.rspec_status

data/.rspec

@@ -0,0 +1,3 @@
+--format documentation
+--color
+--require spec_helper

data/.rubocop.yml

@@ -0,0 +1,120 @@
+AllCops:
+  TargetRubyVersion: 2.2
+
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+  Enabled: true
+
+Style/ClassCheck:
+  EnforcedStyle: kind_of?
+
+Style/BracesAroundHashParameters:
+  Enabled: false
+
+Lint/UselessAssignment:
+  Exclude:
+    - '**/spec/**/*'
+
+Layout/IndentHash:
+  Enabled: false
+
+Layout/AlignHash:
+  Enabled: false
+
+Layout/DotPosition:
+  Enabled: false
+
+Style/DoubleNegation:
+  Enabled: false
+
+Lint/UnusedBlockArgument:
+  Enabled: false
+
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+Metrics/AbcSize:
+  Max: 60
+
+Style/WordArray:
+  MinSize: 19
+
+Style/SignalException:
+  Enabled: false
+
+Style/RedundantReturn:
+  Enabled: false
+
+Style/IfUnlessModifier:
+  Enabled: false
+
+Style/AndOr:
+  Enabled: false
+
+Metrics/ClassLength:
+  Max: 350
+
+Metrics/CyclomaticComplexity:
+  Max: 17
+
+Metrics/LineLength:
+  Max: 370
+
+Metrics/ParameterLists:
+  Max: 10
+
+Metrics/PerceivedComplexity:
+  Max: 18
+
+Style/GuardClause:
+  Enabled: false
+
+Style/ConditionalAssignment:
+  Enabled: false
+
+Style/RedundantSelf:
+  Enabled: false
+
+Metrics/MethodLength:
+  Max: 60
+
+Style/Documentation:
+  Enabled: false
+
+Style/IfInsideElse:
+  Enabled: false
+
+Style/BlockComments:
+  Enabled: false
+
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+
+Metrics/BlockLength:
+  Max: 345
+  Exclude:
+    - "**/*_spec.rb"
+
+Style/MixinGrouping:
+  Enabled: false
+
+Layout/IndentHeredoc:
+  Enabled: false
+
+Style/SpecialGlobalVars:
+  Enabled: false
+
+PercentLiteralDelimiters:
+  PreferredDelimiters:
+    "%":  ()
+    "%i": ()
+    "%q": ()
+    "%Q": ()
+    "%r": "{}"
+    "%s": ()
+    "%w": ()
+    "%W": ()
+    "%x": ()
+
+Security/YAMLLoad:
+  Enabled: false

data/.travis.yml

@@ -0,0 +1,11 @@
+sudo: false
+language: ruby
+os:
+  - osx
+
+cache: bundler
+bundler_args: --without documentation
+rvm:
+  - 2.4.1
+
+script: bundle exec rake spec

data/Gemfile

@@ -0,0 +1,3 @@
+source "https://rubygems.org"
+
+gemspec

data/Gemfile.lock

@@ -0,0 +1,187 @@
+GIT
+  remote: https://github.com/hatt0519/flutter-analyze-parser.git
+  revision: 4778648930dd68cb5d5ee933f16226a087c5493e
+  specs:
+    flutter_analyze_parser (0.1.2)
+
+PATH
+  remote: .
+  specs:
+    danger-flutter_lint (1.0.1)
+      danger-plugin-api (~> 1.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    addressable (2.8.7)
+      public_suffix (>= 2.0.2, < 7.0)
+    ast (2.4.2)
+    claide (1.1.0)
+    claide-plugins (0.9.2)
+      cork
+      nap
+      open4 (~> 1.3)
+    codecov (0.6.0)
+      simplecov (>= 0.15, < 0.22)
+    coderay (1.1.3)
+    colored2 (3.1.2)
+    cork (0.3.0)
+      colored2 (~> 3.1)
+    danger (8.6.1)
+      claide (~> 1.0)
+      claide-plugins (>= 0.9.2)
+      colored2 (~> 3.1)
+      cork (~> 0.1)
+      faraday (>= 0.9.0, < 2.0)
+      faraday-http-cache (~> 2.0)
+      git (~> 1.7)
+      kramdown (~> 2.3)
+      kramdown-parser-gfm (~> 1.0)
+      no_proxy_fix
+      octokit (~> 4.7)
+      terminal-table (>= 1, < 4)
+    danger-plugin-api (1.0.0)
+      danger (> 2.0)
+    diff-lcs (1.5.1)
+    docile (1.4.0)
+    faraday (1.10.3)
+      faraday-em_http (~> 1.0)
+      faraday-em_synchrony (~> 1.0)
+      faraday-excon (~> 1.1)
+      faraday-httpclient (~> 1.0)
+      faraday-multipart (~> 1.0)
+      faraday-net_http (~> 1.0)
+      faraday-net_http_persistent (~> 1.0)
+      faraday-patron (~> 1.0)
+      faraday-rack (~> 1.0)
+      faraday-retry (~> 1.0)
+      ruby2_keywords (>= 0.0.4)
+    faraday-em_http (1.0.0)
+    faraday-em_synchrony (1.0.0)
+    faraday-excon (1.1.0)
+    faraday-http-cache (2.5.1)
+      faraday (>= 0.8)
+    faraday-httpclient (1.0.1)
+    faraday-multipart (1.0.4)
+      multipart-post (~> 2)
+    faraday-net_http (1.0.1)
+    faraday-net_http_persistent (1.2.0)
+    faraday-patron (1.0.0)
+    faraday-rack (1.0.0)
+    faraday-retry (1.0.3)
+    ffi (1.17.0)
+    formatador (1.1.0)
+    git (1.19.1)
+      addressable (~> 2.8)
+      rchardet (~> 1.8)
+    guard (2.18.1)
+      formatador (>= 0.2.4)
+      listen (>= 2.7, < 4.0)
+      lumberjack (>= 1.0.12, < 2.0)
+      nenv (~> 0.1)
+      notiffany (~> 0.0)
+      pry (>= 0.13.0)
+      shellany (~> 0.0)
+      thor (>= 0.18.1)
+    guard-compat (1.2.1)
+    guard-rspec (4.7.3)
+      guard (~> 2.1)
+      guard-compat (~> 1.1)
+      rspec (>= 2.99.0, < 4.0)
+    kramdown (2.4.0)
+      rexml
+    kramdown-parser-gfm (1.1.0)
+      kramdown (~> 2.0)
+    listen (3.9.0)
+      rb-fsevent (~> 0.10, >= 0.10.3)
+      rb-inotify (~> 0.9, >= 0.9.10)
+    lumberjack (1.2.10)
+    method_source (1.1.0)
+    multipart-post (2.4.1)
+    nap (1.1.0)
+    nenv (0.3.0)
+    no_proxy_fix (0.1.2)
+    notiffany (0.1.3)
+      nenv (~> 0.1)
+      shellany (~> 0.0)
+    octokit (4.25.1)
+      faraday (>= 1, < 3)
+      sawyer (~> 0.9)
+    open4 (1.3.4)
+    parallel (1.24.0)
+    parser (3.3.4.0)
+      ast (~> 2.4.1)
+      racc
+    pry (0.14.2)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    public_suffix (5.1.1)
+    racc (1.8.0)
+    rainbow (3.1.1)
+    rake (10.5.0)
+    rb-fsevent (0.11.2)
+    rb-inotify (0.11.1)
+      ffi (~> 1.0)
+    rb-readline (0.5.5)
+    rchardet (1.8.0)
+    regexp_parser (2.9.2)
+    rexml (3.3.2)
+      strscan
+    rspec (3.13.0)
+      rspec-core (~> 3.13.0)
+      rspec-expectations (~> 3.13.0)
+      rspec-mocks (~> 3.13.0)
+    rspec-core (3.13.0)
+      rspec-support (~> 3.13.0)
+    rspec-expectations (3.13.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.13.0)
+    rspec-mocks (3.13.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.13.0)
+    rspec-support (3.13.1)
+    rubocop (0.93.1)
+      parallel (~> 1.10)
+      parser (>= 2.7.1.5)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8)
+      rexml
+      rubocop-ast (>= 0.6.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 2.0)
+    rubocop-ast (1.30.0)
+      parser (>= 3.2.1.0)
+    ruby-progressbar (1.13.0)
+    ruby2_keywords (0.0.5)
+    sawyer (0.9.2)
+      addressable (>= 2.3.5)
+      faraday (>= 0.17.3, < 3)
+    shellany (0.0.1)
+    simplecov (0.21.2)
+      docile (~> 1.1)
+      simplecov-html (~> 0.11)
+      simplecov_json_formatter (~> 0.1)
+    simplecov-html (0.12.3)
+    simplecov_json_formatter (0.1.4)
+    strscan (3.1.0)
+    terminal-table (3.0.2)
+      unicode-display_width (>= 1.1.1, < 3)
+    thor (1.3.1)
+    unicode-display_width (1.8.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  codecov (~> 0.1)
+  danger-flutter_lint!
+  flutter_analyze_parser!
+  guard-rspec (~> 4.7)
+  rake (~> 10.0)
+  rb-readline (~> 0.5)
+  rspec (~> 3.0)
+  rubocop (~> 0.6)
+  simplecov (~> 0.16)
+
+BUNDLED WITH
+   2.1.4

data/Guardfile

@@ -0,0 +1,14 @@
+guard :rspec, cmd: "bundle exec rspec" do
+  require "guard/rspec/dsl"
+  dsl = Guard::RSpec::Dsl.new(self)
+
+  # RSpec files
+  rspec = dsl.rspec
+  watch(rspec.spec_helper) { rspec.spec_dir }
+  watch(rspec.spec_support) { rspec.spec_dir }
+  watch(rspec.spec_files)
+
+  # Ruby files
+  ruby = dsl.ruby
+  dsl.watch_spec_files_for(ruby.lib_files)
+end

data/LICENSE.txt

@@ -0,0 +1,21 @@
+The MIT License (MIT)
+
+Copyright (c) 2019 Mateusz Szklarek
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.

data/README.md

@@ -0,0 +1,70 @@
+# danger-flutter_lint
+
+[![Gem](https://img.shields.io/gem/v/danger-flutter_lint.svg)](https://rubygems.org/gems/danger-flutter_lint)
+[![Build Status](https://travis-ci.org/mateuszszklarek/danger-flutter_lint.svg?branch=master)](https://travis-ci.org/mateuszszklarek/danger-flutter_lint)
+[![codecov](https://codecov.io/gh/mateuszszklarek/danger-flutter_lint/branch/master/graph/badge.svg)](https://codecov.io/gh/mateuszszklarek/danger-flutter_lint)
+
+A Danger Plugin to lint dart files using `flutter analyze` command line interface.
+
+## Installation
+
+Add this line to your application's Gemfile:
+
+	$ gem 'danger-flutter_lint'
+
+Or install it yourself as:
+
+    $ gem install danger-flutter_lint
+
+## Usage
+
+Flutter Analyze doesn't give an option to generate report but you can achieve this easily using regular shell command (locally or on CI):
+
+```sh
+$ flutter analyze > flutter_analyze_report.txt
+```
+
+It will add output from `flutter analyze` command to `flutter_analyze_report.txt`.
+
+Now you need to set `report_path` and invoke `lint` in your Dangerfile.
+
+```ruby
+flutter_lint.report_path = "flutter_analyze_report.txt"
+flutter_lint.lint
+```
+
+This will add markdown table with summary into your PR.
+
+Or make Danger comment directly on the line instead of printing a Markdown table (GitHub only)
+
+```ruby
+flutter_lint.lint(inline_mode: true)
+```
+
+Default value for `inline_mode` parameter is false.
+
+#### Lint only added/modified files
+
+If you're dealing with a legacy project, with tons of warnings, you may want to lint only new/modified files. You can easily achieve that, setting the `only_modified_files` parameter to `true`.
+
+```ruby
+flutter_lint.only_modified_files = true
+flutter_lint.report_path = "flutter_analyze_report.txt"
+flutter_lint.lint
+```
+
+## Development
+
+1. Clone this repo
+2. Run `bundle install` to setup dependencies.
+3. Run `bundle exec rake spec` to run the tests.
+4. Use `bundle exec guard` to automatically have tests run as you make changes.
+5. Make your changes.
+
+## Contributing
+
+Bug reports and pull requests are welcome on GitHub at https://github.com/mateuszszklarek/danger-flutter_lint.
+
+## License
+
+The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/Rakefile

@@ -0,0 +1,17 @@
+require "bundler/gem_tasks"
+require "rspec/core/rake_task"
+require "rubocop/rake_task"
+
+RSpec::Core::RakeTask.new(:specs)
+
+task default: :specs
+
+task :spec do
+  Rake::Task["specs"].invoke
+  Rake::Task["rubocop"].invoke
+end
+
+desc "Run RuboCop on the lib/specs directory"
+RuboCop::RakeTask.new(:rubocop) do |task|
+  task.patterns = ["lib/**/*.rb"]
+end

data/danger-flutter_lint-no-filter.gemspec

@@ -0,0 +1,28 @@
+lib = File.expand_path("lib", __dir__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require "flutter_lint/gem_version.rb"
+
+Gem::Specification.new do |spec|
+  spec.name          = "danger-flutter_lint-no-filter"
+  spec.version       = FlutterLint::VERSION
+  spec.authors       = ["hatt0519"]
+  spec.email         = ["moroku0519@gmail.com"]
+  spec.summary       = "original: https://github.com/mateuszszklarek/danger-flutterlint This is the fixed danger-flutterlint not to filter flutter analyze result"
+  spec.homepage      = "https://github.com/hatt0519/danger-flutter_lint"
+  spec.license       = "MIT"
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ["lib"]
+
+  spec.add_runtime_dependency "danger-plugin-api", "~> 1.0"
+
+  spec.add_development_dependency "codecov", "~> 0.1"
+  spec.add_development_dependency "guard-rspec", "~> 4.7"
+  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rb-readline", "~> 0.5"
+  spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency "rubocop", "~> 0.6"
+  spec.add_development_dependency "simplecov", "~> 0.16"
+end

data/lib/danger_flutter_lint.rb

@@ -0,0 +1 @@
+require "flutter_lint/gem_version"

data/lib/danger_plugin.rb

@@ -0,0 +1 @@
+require "flutter_lint/plugin"

data/lib/flutter_lint/gem_version.rb

@@ -0,0 +1,3 @@
+module FlutterLint
+  VERSION = "1.0.0".freeze
+end

data/lib/flutter_lint/plugin.rb

@@ -0,0 +1,111 @@
+module Danger
+  class DangerFlutterLint < Plugin
+    # Enable only_modified_files
+    # Only show messages within changed files.
+    attr_accessor :only_modified_files
+
+    # Report path
+    # You should set output from `flutter analyze` here
+    attr_accessor :report_path
+
+    def lint(inline_mode: false)
+      if flutter_exists?
+        lint_if_report_exists(inline_mode: inline_mode)
+      else
+        fail("Could not find `flutter` inside current directory")
+      end
+    end
+
+    private
+
+    def lint_if_report_exists(inline_mode:)
+      if !report_path.nil? && File.exist?(report_path)
+        report = File.open(report_path)
+        violations = FlutterAnalyzeParser.violations(report)
+        lint_mode(inline_mode: inline_mode, violations: violations)
+      else
+        fail("Could not run lint without setting report path or report file doesn't exists")
+      end
+    end
+
+    def lint_mode(inline_mode:, violations:)
+      if inline_mode
+        send_inline_comments(violations)
+      else
+        markdown(summary_table(violations))
+      end
+    end
+
+    def send_inline_comments(violations)
+      filtered_violations = filtered_violations(violations)
+
+      filtered_violations.each do |violation|
+        send("warn", violation.description, file: violation.file, line: violation.line)
+      end
+    end
+
+    def summary_table(violations)
+      filtered_violations = filtered_violations(violations)
+
+      if filtered_violations.empty?
+        return "### Flutter Analyze found #{filtered_violations.length} issues ✅"
+      else
+        return markdown_table(filtered_violations)
+      end
+    end
+
+    def markdown_table(violations)
+      table = "### Flutter Analyze found #{violations.length} issues ❌\n\n"
+      table << "| File | Line | Rule |\n"
+      table << "| ---- | ---- | ---- |\n"
+
+      return violations.reduce(table) { |acc, violation| acc << table_row(violation) }
+    end
+
+    def table_row(violation)
+      "| `#{violation.file}` | #{violation.line} | #{violation.rule} |\n"
+    end
+
+    def filtered_violations(violations)
+      target_files = (git.modified_files - git.deleted_files) + git.added_files
+      filtered_violations = violations.select { |violation| target_files.include? violation.file }
+
+      return only_modified_files ? filtered_violations : violations
+    end
+
+    def flutter_exists?
+      `which flutter`.strip.empty? == false
+    end
+  end
+  FlutterViolation = Struct.new(:rule, :description, :file, :line)
+
+  class FlutterAnalyzeParser
+    class << self
+      def violations(input)
+        filtered_input = filter_input(input)
+
+        return [] if filtered_input.detect { |element| element.include? "No issues found!" }
+
+        filtered_input
+          .select { |line| line.start_with?("info", "warning", "error") }
+          .map(&method(:parse_line))
+      end
+
+      private
+
+      def filter_input(input)
+        input.each_line
+          .map(&:strip)
+          .reject(&:empty?)
+      end
+
+      def parse_line(line)
+        _, description, file_with_line_number, rule = line.split(" • ")
+        file, line = file_with_line_number.split(":")
+
+        FlutterViolation.new(rule, description, file, line.to_i)
+      end
+    end
+  end
+
+end

data/spec/fixtures/flutter_analyze_with_violations.txt

@@ -0,0 +1,6 @@
+
+Analyzing my_flutter_project...
+
+      info • Name types using UpperCamelCase • lib/main.dart:5:7 • camel_case_types
+      info • Prefer const with constant constructors • lib/home/home_page.dart:13:13 • prefer_const_constructors
+      info • AVOID catches without on clauses • lib/profile/user/phone_widget.dart:19:7 • avoid_catches_without_on_clauses

data/spec/fixtures/flutter_analyze_without_violations.txt

@@ -0,0 +1,5 @@
+
+Analyzing my_flutter_project...
+
+
+No issues found! (ran in 1.8s)

data/spec/flutter_lint_spec.rb

@@ -0,0 +1,155 @@
+require File.expand_path("spec_helper", __dir__)
+
+module Danger
+  describe Danger::DangerFlutterLint do
+    it "should be a danger plugin" do
+      expect(Danger::DangerFlutterLint.new(nil)).to be_a Danger::Plugin
+    end
+
+    describe "with a Dangerfile" do
+      before do
+        @dangerfile = testing_dangerfile
+        @flutter_lint = @dangerfile.flutter_lint
+        allow(@flutter_lint.git).to receive(:deleted_files).and_return([])
+        allow(@flutter_lint.git).to receive(:added_files).and_return([])
+        allow(@flutter_lint.git).to receive(:modified_files).and_return([
+          "lib/home/home_page.dart",
+          "lib/profile/user/phone_widget.dart"
+        ])
+      end
+
+      context "when flutter is not installed" do
+        before do
+          allow(@flutter_lint).to receive(:`).with("which flutter").and_return("")
+        end
+
+        it "should fail when lint" do
+          @flutter_lint.lint
+
+          expect(@flutter_lint.status_report[:errors]).to eq(["Could not find `flutter` inside current directory"])
+        end
+      end
+
+      context "when flutter is installed and report is not set" do
+        before do
+          allow(@flutter_lint).to receive(:`).with("which flutter").and_return("/Users/johndoe/.flutter/bin/flutter")
+        end
+
+        it "should fail when lint" do
+          @flutter_lint.lint
+
+          expect(@flutter_lint.status_report[:errors]).to eq(["Could not run lint without setting report path or report file doesn't exists"])
+        end
+
+        context "when report is set but file not exists" do
+          before do
+          allow(@flutter_lint).to receive(:`).with("which flutter").and_return("/Users/johndoe/.flutter/bin/flutter")
+          end
+
+          it "should fail when lint" do
+            @flutter_lint.report_path = "users/johndoe/invalid_path/report.txt"
+            @flutter_lint.lint
+
+            expect(@flutter_lint.status_report[:errors]).to eq(["Could not run lint without setting report path or report file doesn't exists"])
+          end
+        end
+
+        context "when set `flutter analyze` report path without violations" do
+          before do
+            @flutter_lint.report_path = "spec/fixtures/flutter_analyze_without_violations.txt"
+          end
+
+          it "should NOT fail when lint" do
+            @flutter_lint.lint
+
+            expect(@flutter_lint.status_report[:errors]).to be_empty
+          end
+
+          it "should add markdown message with 0 violations when inline mode is off" do
+            @flutter_lint.lint(inline_mode: false)
+
+            markdown = @flutter_lint.status_report[:markdowns].first.message
+            expect(markdown).to eq("### Flutter Analyze found 0 issues ✅")
+          end
+
+          it "should NOT print markdown message when inline mode is on" do
+            @flutter_lint.lint(inline_mode: true)
+
+            markdown = @flutter_lint.status_report[:markdowns]
+            expect(markdown).to be_empty
+          end
+        end
+
+        context "when set `flutter analyze` report with some violations" do
+          before do
+            @flutter_lint.report_path = "spec/fixtures/flutter_analyze_with_violations.txt"
+          end
+
+          it "should NOT fail when lint" do
+            @flutter_lint.lint
+
+            expect(@flutter_lint.status_report[:errors]).to be_empty
+          end
+
+          it "should print markdown message with 3 violations when inline mode is off & only_modified_files not set" do
+            @flutter_lint.lint(inline_mode: false)
+
+            expected = <<~MESSAGE
+            ### Flutter Analyze found 3 issues ❌\n
+            | File | Line | Rule |
+            | ---- | ---- | ---- |
+            | `lib/main.dart` | 5 | camel_case_types |
+            | `lib/home/home_page.dart` | 13 | prefer_const_constructors |
+            | `lib/profile/user/phone_widget.dart` | 19 | avoid_catches_without_on_clauses |
+            MESSAGE
+
+            expect(@flutter_lint.status_report[:markdowns].first.message).to eq(expected)
+          end
+
+          it "should send 3 inline comment instead of markdown when inline mode is on & only_modified_files not set" do
+            @flutter_lint.lint(inline_mode: true)
+
+            warnings = @flutter_lint.status_report[:warnings]
+
+            exepcted_warnings = [
+              "Name types using UpperCamelCase", 
+              "Prefer const with constant constructors", 
+              "AVOID catches without on clauses"
+            ]
+
+            expect(warnings).to eq(exepcted_warnings)
+          end
+
+          it "should print markdown message with 2 violations when inline mode is off & only_modified_files set to true" do
+            @flutter_lint.only_modified_files = true
+            @flutter_lint.lint(inline_mode: false)
+
+            expected = <<~MESSAGE
+            ### Flutter Analyze found 2 issues ❌\n
+            | File | Line | Rule |
+            | ---- | ---- | ---- |
+            | `lib/home/home_page.dart` | 13 | prefer_const_constructors |
+            | `lib/profile/user/phone_widget.dart` | 19 | avoid_catches_without_on_clauses |
+            MESSAGE
+
+            expect(@flutter_lint.status_report[:markdowns].first.message).to eq(expected)
+          end
+
+          it "should send 2 inline comment instead of markdown when inline mode is on & only_modified_files set to true" do
+            @flutter_lint.only_modified_files = true
+            @flutter_lint.lint(inline_mode: true)
+
+            warnings = @flutter_lint.status_report[:warnings]
+
+            exepcted_warnings = [
+              "Prefer const with constant constructors", 
+              "AVOID catches without on clauses"
+            ]
+
+            expect(warnings).to eq(exepcted_warnings)
+          end
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,58 @@
+require "pathname"
+ROOT = Pathname.new(File.expand_path("..", __dir__))
+$:.unshift((ROOT + "lib").to_s)
+$:.unshift((ROOT + "spec").to_s)
+
+require "simplecov"
+SimpleCov.start
+
+if ENV["CI"] == "true"
+  require "codecov"
+  SimpleCov.formatter = SimpleCov::Formatter::Codecov
+end
+
+require "bundler/setup"
+
+require "rspec"
+require "danger"
+require "danger_plugin"
+require "flutter_analyze_parser"
+
+RSpec.configure do |config|
+  config.example_status_persistence_file_path = ".rspec_status"
+  config.filter_gems_from_backtrace "bundler"
+  config.expect_with :rspec do |c|
+    c.syntax = :expect
+  end
+  config.color = true
+  config.tty = true
+end
+
+# Testing
+def testing_env
+  {
+    "HAS_JOSH_K_SEAL_OF_APPROVAL" => "true",
+    "TRAVIS_PULL_REQUEST" => "800",
+    "TRAVIS_REPO_SLUG" => "artsy/eigen",
+    "TRAVIS_COMMIT_RANGE" => "759adcbd0d8f...13c4dc8bb61d",
+    "DANGER_GITHUB_API_TOKEN" => "123sbdq54erfsd3422gdfio"
+  }
+end
+
+def testing_ui
+  @output = StringIO.new
+  def @output.winsize
+    [20, 9999]
+  end
+
+  cork = Cork::Board.new(out: @output)
+  def cork.string
+    out.string.gsub(/\e\[([;\d]+)?m/, "")
+  end
+  cork
+end
+
+def testing_dangerfile
+  env = Danger::EnvironmentManager.new(testing_env)
+  Danger::Dangerfile.new(env, testing_ui)
+end

metadata

@@ -0,0 +1,179 @@
+--- !ruby/object:Gem::Specification
+name: danger-flutter_lint-no-filter
+version: !ruby/object:Gem::Version
+  version: 1.0.0
+platform: ruby
+authors:
+- hatt0519
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2024-07-17 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: danger-plugin-api
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: codecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: rb-readline
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.16'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.16'
+description: 
+email:
+- moroku0519@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".rspec"
+- ".rubocop.yml"
+- ".travis.yml"
+- Gemfile
+- Gemfile.lock
+- Guardfile
+- LICENSE.txt
+- README.md
+- Rakefile
+- danger-flutter_lint-no-filter.gemspec
+- lib/danger_flutter_lint.rb
+- lib/danger_plugin.rb
+- lib/flutter_lint/gem_version.rb
+- lib/flutter_lint/plugin.rb
+- spec/fixtures/flutter_analyze_with_violations.txt
+- spec/fixtures/flutter_analyze_without_violations.txt
+- spec/flutter_lint_spec.rb
+- spec/spec_helper.rb
+homepage: https://github.com/hatt0519/danger-flutter_lint
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubygems_version: 3.0.3
+signing_key: 
+specification_version: 4
+summary: 'original: https://github.com/mateuszszklarek/danger-flutterlint This is
+  the fixed danger-flutterlint not to filter flutter analyze result'
+test_files:
+- spec/fixtures/flutter_analyze_with_violations.txt
+- spec/fixtures/flutter_analyze_without_violations.txt
+- spec/flutter_lint_spec.rb
+- spec/spec_helper.rb