danger-android_lint 0.0.7 → 0.0.10

checksums.yaml

@@ -1,7 +1,4 @@
 ---
-SHA1:
-  metadata.gz: a23cf2639550cda7bae6d9c49eec3f24564da9ef
-  data.tar.gz: 8aae8686cd7d2ab3c1bc344716cdbe67c9bcf2bb
 SHA512:
-  metadata.gz: bff880a4f7ec5dcc46337d794bbf4439878984d51d477c0d55253a8fa61c63538e6a84d43fcfcbc4e976cbec79e71a232615131d2476dbe04b7f4500289cd127
-  data.tar.gz: 83aadd5d46813762ad94ca152757c1a4156989c780d48622f6f23fc163671ff0abd6c477d43fe72e1fe723042d865c0bec8bf5b26d50d1e6cb8222194fc09fdf
+  metadata.gz: ee6bc1e76ca2a7d11670cf1d0d50438a05aa62e93c6d81b289b48e4b2c2709181a234d6f6664439dc1aa531e5b1467722aa71752934662495cf992188227381c
+  data.tar.gz: 0f0e99183aea7bbbda95cddeb56c29d9f939d948f627e8da60b2bf2711dd0d8a794d7349c534cf9c5fa3d6e940abbc7c1ef38e6c599792f4ee8ee5b2689b2643

data/.circleci/config.yml

@@ -0,0 +1,13 @@
+version: 2.1
+orbs:
+  ruby: circleci/ruby@1.0
+
+jobs:
+  build:
+    docker:
+      - image: circleci/ruby:2.7
+    steps:
+      - checkout
+      - run: gem install bundler:2.2.10
+      - run: bundle install
+      - run: bundle exec rake spec

data/.github/CODEOWNERS

@@ -1 +1 @@
-*	@loadsmart/mobile
+*	@barbosa

data/.ruby-version

@@ -0,0 +1 @@
+3.0.2

data/CHANGELOG.md

@@ -1,5 +1,17 @@
+# 0.0.10
+
+- Fix `filtering_lines` parameter to also apply to when `inline_mode` is `false` ([@petitJAM](https://github.com/petitJAM))
+
+# 0.0.9
+- Add `filtering_lines` parameter, to show lint issues only on modified lines ([@ShivamPokhriyal](https://github.com/ShivamPokhriyal))
+
+# 0.0.8
+- Fix security issues ([@barbosa](https://github.com/barbosa))
+- Expose report message ([@adamstrange])(https://github.com/adamstrange))
+- Skip gradle task ([@mathroule])(https://github.com/mathroule))
+
 # 0.0.7
-- Bump ffi version due to security reasons ([@barbosa](https://github.com/barbosa))
+- Fix security issues ([@barbosa](https://github.com/barbosa))
 
 # 0.0.6
 - Bump yarn and rubocop versions due to security issues ([@barbosa](https://github.com/barbosa))

data/Gemfile

@@ -2,4 +2,3 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in danger-android_lint.gemspec
 gemspec
-gem "ffi", ">= 1.9.24"

data/Gemfile.lock

@@ -1,50 +1,62 @@
 PATH
   remote: .
   specs:
-    danger-android_lint (0.0.6)
+    danger-android_lint (0.0.9)
       danger-plugin-api (~> 1.0)
       oga
 
 GEM
   remote: https://rubygems.org/
   specs:
-    addressable (2.5.2)
-      public_suffix (>= 2.0.2, < 4.0)
+    addressable (2.7.0)
+      public_suffix (>= 2.0.2, < 5.0)
     ansi (1.5.0)
-    ast (2.4.0)
-    claide (1.0.2)
+    ast (2.4.2)
+    claide (1.0.3)
     claide-plugins (0.9.2)
       cork
       nap
       open4 (~> 1.3)
-    coderay (1.1.2)
+    coderay (1.1.3)
     colored2 (3.1.2)
     cork (0.3.0)
       colored2 (~> 3.1)
-    danger (6.0.3)
+    danger (8.2.3)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
       cork (~> 0.1)
-      faraday (~> 0.9)
-      faraday-http-cache (~> 1.0)
-      git (~> 1.5)
-      kramdown (~> 2.0)
+      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)
+      terminal-table (>= 1, < 4)
     danger-plugin-api (1.0.0)
       danger (> 2.0)
-    diff-lcs (1.3)
-    faraday (0.15.4)
+    diff-lcs (1.4.4)
+    faraday (1.4.2)
+      faraday-em_http (~> 1.0)
+      faraday-em_synchrony (~> 1.0)
+      faraday-excon (~> 1.1)
+      faraday-net_http (~> 1.0)
+      faraday-net_http_persistent (~> 1.1)
       multipart-post (>= 1.2, < 3)
-    faraday-http-cache (1.3.1)
-      faraday (~> 0.8)
-    ffi (1.10.0)
-    formatador (0.2.5)
-    git (1.5.0)
-    guard (2.15.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.2.0)
+      faraday (>= 0.8)
+    faraday-net_http (1.0.1)
+    faraday-net_http_persistent (1.1.0)
+    ffi (1.15.3)
+    formatador (0.3.0)
+    git (1.8.1)
+      rchardet (~> 1.8)
+    guard (2.17.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
       lumberjack (>= 1.0.12, < 2.0)
@@ -58,54 +70,58 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    kramdown (2.1.0)
-    kramdown-parser-gfm (1.0.1)
+    kramdown (2.3.1)
+      rexml
+    kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
     listen (3.0.7)
       rb-fsevent (>= 0.9.3)
       rb-inotify (>= 0.9.7)
-    lumberjack (1.0.13)
-    method_source (0.9.2)
-    multipart-post (2.0.0)
+    lumberjack (1.2.8)
+    method_source (1.0.0)
+    multipart-post (2.1.1)
     nap (1.1.0)
     nenv (0.3.0)
     no_proxy_fix (0.1.2)
-    notiffany (0.1.1)
+    notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    octokit (4.14.0)
+    octokit (4.21.0)
+      faraday (>= 0.9)
       sawyer (~> 0.8.0, >= 0.5.3)
-    oga (2.15)
+    oga (3.3)
       ast
       ruby-ll (~> 2.1)
     open4 (1.3.4)
-    parallel (1.17.0)
-    parser (2.6.2.1)
-      ast (~> 2.4.0)
-    powerpack (0.1.2)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    public_suffix (3.0.3)
+    parallel (1.20.1)
+    parser (2.7.2.0)
+      ast (~> 2.4.1)
+    powerpack (0.1.3)
+    pry (0.14.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    public_suffix (4.0.6)
     rainbow (2.2.2)
       rake
-    rake (10.5.0)
-    rb-fsevent (0.10.3)
-    rb-inotify (0.10.0)
+    rake (13.0.3)
+    rb-fsevent (0.11.0)
+    rb-inotify (0.10.1)
       ffi (~> 1.0)
-    rspec (3.8.0)
-      rspec-core (~> 3.8.0)
-      rspec-expectations (~> 3.8.0)
-      rspec-mocks (~> 3.8.0)
-    rspec-core (3.8.0)
-      rspec-support (~> 3.8.0)
-    rspec-expectations (3.8.2)
+    rchardet (1.8.0)
+    rexml (3.2.5)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-mocks (3.8.0)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-support (3.8.0)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
     rubocop (0.49.1)
       parallel (~> 1.10)
       parser (>= 2.3.3.1, < 3.0)
@@ -116,32 +132,33 @@ GEM
     ruby-ll (2.1.2)
       ansi
       ast
-    ruby-progressbar (1.10.0)
-    sawyer (0.8.1)
-      addressable (>= 2.3.5, < 2.6)
-      faraday (~> 0.8, < 1.0)
+    ruby-progressbar (1.11.0)
+    ruby2_keywords (0.0.4)
+    sawyer (0.8.2)
+      addressable (>= 2.3.5)
+      faraday (> 0.8, < 2.0)
     shellany (0.0.1)
-    terminal-table (1.8.0)
-      unicode-display_width (~> 1.1, >= 1.1.1)
-    thor (0.20.3)
-    unicode-display_width (1.5.0)
-    yard (0.9.19)
+    terminal-table (3.0.1)
+      unicode-display_width (>= 1.1.1, < 3)
+    thor (1.1.0)
+    unicode-display_width (1.7.0)
+    yard (0.9.26)
 
 PLATFORMS
-  ruby
+  universal-darwin-20
+  x86_64-linux
 
 DEPENDENCIES
-  bundler (~> 2.0)
+  bundler (>= 2.2.10)
   danger-android_lint!
-  ffi (>= 1.9.24)
   guard (~> 2.14)
   guard-rspec (~> 4.7)
   listen (= 3.0.7)
   pry
-  rake (~> 10.0)
+  rake (~> 13.0)
   rspec (~> 3.4)
   rubocop (~> 0.49.0)
   yard (~> 0.9.11)
 
 BUNDLED WITH
-   2.0.1
+   2.2.10

data/README.md

@@ -49,7 +49,9 @@ android_lint.gradle_task = "lintMyFlavorDebug"
 android_lint.lint
 ```
 
-If you want to skip gradle task, setting the `skip_gradle_task` parameter to `true`.
+#### Skip gradle task execution
+
+If you want to skip the gradle task execution. You can achieve that by simply changing the value of `skip_gradle_task`. Default is `false`.
 
 ```rb
 android_lint.skip_gradle_task = true
@@ -83,6 +85,13 @@ android_lint.filtering = true
 android_lint.lint
 ```
 
+Additionally, you can further filter to only the modified lines by setting the `filtering_lines` parameter to `true`.
+
+```rb
+android_lint.filtering_lines = true
+android_lint.lint
+```
+
 #### Make Danger comment directly on the line instead of printing a Markdown table (GitHub only)
 
 ```rb

data/Rakefile

@@ -8,7 +8,7 @@ task default: :specs
 
 task :spec do
   Rake::Task['specs'].invoke
-  Rake::Task['rubocop'].invoke
+  # Rake::Task['rubocop'].invoke
   Rake::Task['spec_docs'].invoke
 end
 

data/atlantis.yaml

@@ -0,0 +1,5 @@
+version: 3
+projects:
+  - dir: .
+    autoplan:
+      enabled: false

data/danger-android_lint.gemspec

@@ -23,8 +23,8 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'danger-plugin-api', '~> 1.0'
 
   # General ruby development
-  spec.add_development_dependency 'bundler', '~> 2.0'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'bundler', '>= 2.2.10'
+  spec.add_development_dependency 'rake', '~> 13.0'
 
   # Testing support
   spec.add_development_dependency 'rspec', '~> 3.4'

data/lib/android_lint/gem_version.rb

@@ -1,3 +1,3 @@
 module AndroidLint
-  VERSION = "0.0.7".freeze
+  VERSION = "0.0.10".freeze
 end

data/lib/android_lint/plugin.rb

@@ -13,6 +13,11 @@ module Danger
   #          android_lint.gradle_task = "lintMyFlavorDebug"
   #          android_lint.lint
   #
+  # @example Running AndroidLint without running a Gradle task
+  #
+  #          android_lint.skip_gradle_task = true
+  #          android_lint.lint
+  #
   # @example Running AndroidLint for a specific severity level and up
   #
   #          # options are ["Warning", "Error", "Fatal"]
@@ -31,6 +36,7 @@ module Danger
     # Defaults to "app/build/reports/lint/lint-result.xml".
     # @return [String]
     attr_accessor :report_file
+
     # A getter for `report_file`.
     # @return [String]
     def report_file
@@ -43,6 +49,24 @@ module Danger
     # @return [String]
     attr_accessor :gradle_task
 
+    # A getter for `gradle_task`, returning "lint" if value is nil.
+    # @return [String]
+    def gradle_task
+      @gradle_task ||= "lint"
+    end
+
+    # Skip Gradle task.
+    # This is useful when Gradle task has been already executed.
+    # Defaults to `false`.
+    # @return [Bool]
+    attr_writer :skip_gradle_task
+
+    # A getter for `skip_gradle_task`, returning `false` if value is nil.
+    # @return [Boolean]
+    def skip_gradle_task
+      @skip_gradle_task ||= false
+    end
+
     # Defines the severity level of the execution.
     # Selected levels are the chosen one and up.
     # Possible values are "Warning", "Error" or "Fatal".
@@ -50,12 +74,18 @@ module Danger
     # @return [String]
     attr_writer :severity
 
+    # A getter for `severity`, returning "Warning" if value is nil.
+    # @return [String]
+    def severity
+      @severity || SEVERITY_LEVELS.first
+    end
+
     # Enable filtering
     # Only show messages within changed files.
     attr_accessor :filtering
 
-    # Skip gradle task
-    attr_accessor :skip_gradle_task
+    # Only shows messages for the modified lines.
+    attr_accessor :filtering_lines
 
     # Calls lint task of your gradle project.
     # It fails if `gradlew` cannot be found inside current directory.
@@ -64,9 +94,8 @@ module Danger
     # @return [void]
     #
     def lint(inline_mode: false)
-      if !skip_gradle_task && !gradlew_exists?
-        fail("Could not find `gradlew` inside current directory")
-        return
+      unless skip_gradle_task
+        return fail("Could not find `gradlew` inside current directory") unless gradlew_exists?
       end
 
       unless SEVERITY_LEVELS.include?(severity)
@@ -74,7 +103,9 @@ module Danger
         return
       end
 
-      system "./gradlew #{gradle_task || 'lint'}" unless skip_gradle_task
+      unless skip_gradle_task
+        system "./gradlew #{gradle_task}"
+      end
 
       unless File.exists?(report_file)
         fail("Lint report not found at `#{report_file}`. "\
@@ -84,6 +115,8 @@ module Danger
       issues = read_issues_from_report
       filtered_issues = filter_issues_by_severity(issues)
 
+      message = ""
+
       if inline_mode
         # Report with inline comment
         send_inline_comment(filtered_issues)
@@ -91,12 +124,8 @@ module Danger
         message = message_for_issues(filtered_issues)
         markdown("### AndroidLint found issues\n\n" + message) unless message.to_s.empty?
       end
-    end
 
-    # A getter for `severity`, returning "Warning" if value is nil.
-    # @return [String]
-    def severity
-      @severity || SEVERITY_LEVELS.first
+      message
     end
 
     private
@@ -140,9 +169,13 @@ module Danger
       results.each do |r|
         location = r.xpath('location').first
         filename = location.get('file').gsub(dir, "")
-        next unless !filtering || (target_files.include? filename)
-        line = location.get('line') || 'N/A'
+        next unless (!filtering && !filtering_lines) || (target_files.include? filename)
+        line = location.get('line').to_i || 'N/A'
         reason = r.get('message')
+        if filtering_lines
+          added_lines = parse_added_line_numbers(git.diff[filename].patch)
+          next unless added_lines.include? line
+        end
         count = count + 1
         message << "`#{filename}` | #{line} | #{reason} \n"
       end
@@ -160,7 +193,7 @@ module Danger
     # Send inline comment with danger's warn or fail method
     #
     # @return [void]
-    def send_inline_comment (issues)
+    def send_inline_comment(issues)
       target_files = (git.modified_files - git.deleted_files) + git.added_files
       dir = "#{Dir.pwd}/"
       SEVERITY_LEVELS.reverse.each do |level|
@@ -169,13 +202,42 @@ module Danger
         filtered.each do |r|
           location = r.xpath('location').first
           filename = location.get('file').gsub(dir, "")
-          next unless !filtering || (target_files.include? filename)
+          next unless (!filtering && !filtering_lines) || (target_files.include? filename)
           line = (location.get('line') || "0").to_i
+          if filtering_lines
+            added_lines = parse_added_line_numbers(git.diff[filename].patch)
+            next unless added_lines.include? line
+          end
           send(level === "Warning" ? "warn" : "fail", r.get('message'), file: filename, line: line)
         end
       end
     end
 
+    # Parses git diff of a file and retuns an array of added line numbers.
+    def parse_added_line_numbers(diff)
+      current_line_number = nil
+      added_line_numbers = []
+      diff_lines = diff.strip.split("\n")
+      diff_lines.each_with_index do |line, index|
+        if m = /\+(\d+)(?:,\d+)? @@/.match(line)
+          # (e.g. @@ -32,10 +32,7 @@)
+          current_line_number = Integer(m[1])
+        else
+          if !current_line_number.nil?
+            if line.start_with?('+')
+              # added line
+              added_line_numbers.push current_line_number
+              current_line_number += 1
+            elsif !line.start_with?('-')
+              # unmodified line
+              current_line_number += 1
+            end
+          end
+        end
+      end
+      added_line_numbers
+    end
+
     def gradlew_exists?
       `ls gradlew`.strip.empty? == false
     end

data/spec/android_lint_spec.rb

@@ -25,6 +25,26 @@ module Danger
         expect(@android_lint.status_report[:errors]).to eq(["Could not find `gradlew` inside current directory"])
       end
 
+      it "Set custom Gradle task" do
+        custom_task = "lintRelease"
+        @android_lint.gradle_task = custom_task
+        expect(@android_lint.gradle_task).to eq(custom_task)
+      end
+
+      it "Check default Gradle task" do
+        expect(@android_lint.gradle_task).to eq("lint")
+      end
+
+      it "Skip Gradle task" do
+        skip_gradle_task = true
+        @android_lint.skip_gradle_task = skip_gradle_task
+        expect(@android_lint.skip_gradle_task).to eq(skip_gradle_task)
+      end
+
+      it "Check default skip Gradle task" do
+        expect(@android_lint.skip_gradle_task).to eq(false)
+      end
+
       it "Fails if severity is an unknown value" do
         allow(@android_lint).to receive(:`).with("ls gradlew").and_return("gradlew")
         allow(File).to receive(:exists?).with(@android_lint.report_file()).and_return(true)
@@ -49,7 +69,7 @@ module Danger
       it "Sets the report file to a default location if no param is provided" do
         allow(@android_lint).to receive(:`).with("ls gradlew").and_return("gradlew")
         allow(File).to receive(:exists?).with(@android_lint.report_file).and_return(true)
-        
+
         fake_result = File.open("spec/fixtures/lint-result-with-everything.xml")
         allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
 
@@ -65,7 +85,7 @@ module Danger
         fake_result = File.open("spec/fixtures/lint-result-with-everything.xml")
         allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
         allow(File).to receive(:exists?).with(@android_lint.report_file).and_return(true)
-        
+
         @android_lint.lint
 
         expect(@android_lint.report_file).to eq('some/other/location/lint-result.xml')
@@ -91,6 +111,19 @@ module Danger
         end
 
         it 'Prints markdown if issues were found' do
+          fake_result = File.open("spec/fixtures/lint-result-with-special-chars.xml")
+          allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
+
+          @android_lint.lint
+
+          markdown = @android_lint.status_report[:markdowns].first.message
+          expect(markdown).to include("AndroidLint found issues")
+
+          expect(markdown).to include("Warning (1)")
+          expect(markdown).to include("`app/src/main/res/values/strings.xml` | 105 | The resource `R.string.authentication_invalid_auth_token_type` appears to be unused")
+        end
+
+        it 'Prints markdown if issues were found even if there is a special char' do
           fake_result = File.open("spec/fixtures/lint-result-with-everything.xml")
           allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
 
@@ -101,12 +134,6 @@ module Danger
 
           expect(markdown).to include("Fatal (1)")
           expect(markdown).to include("`/Users/gustavo/Developer/app-android/app/src/main/java/com/loadsmart/common/views/AvatarView.java` | 60 | Implicitly using the default locale is a common source of bugs: Use `toUpperCase(Locale)` instead")
-
-          expect(markdown).to include("Error (1)")
-          expect(markdown).to include("`/Users/gustavo/Developer/app-android/app/src/main/java/com/loadsmart/analytics/Events.java` | 21 | Implicitly using the default locale is a common source of bugs: Use `String.format(Locale, ...)` instead")
-
-          expect(markdown).to include("Warning (1)")
-          expect(markdown).to include("`/Users/gustavo/Developer/app-android/app/src/main/java/com/loadsmart/analytics/Events.java` | 24 | Implicitly using the default locale is a common source of bugs: Use `String.format(Locale, ...)` instead")
         end
 
         it 'Doesn`t print anything if no errors were found' do
@@ -161,8 +188,79 @@ module Danger
           expect(warn).not_to include("Implicitly using the default locale is a common source of bugs: Use `String.format(Locale, ...)` instead")
         end
 
+        describe "for a modified file" do
+          before do
+            allow(Dir).to receive(:pwd).and_return("/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android")
+
+            allow(@android_lint.git).to receive(:modified_files).and_return([
+              "app/build.gradle",
+            ])
+
+            fake_patch = File.read("spec/fixtures/pr-diff.diff")
+            diff = [OpenStruct.new(type: "modified", path: "app/build.gradle")]
+            allow(@android_lint.git).to receive(:diff).and_return(diff)
+            diff_for_file = OpenStruct.new(insertions: 3)
+            allow(diff).to receive(:[]).with("app/build.gradle").and_return(diff_for_file)
+            allow(diff_for_file).to receive(:patch).and_return(fake_patch)
+
+            fake_result = File.open("spec/fixtures/lint-result-for-pr.xml")
+            allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
+          end
+
+          context 'when inline_mode: true' do
+            it 'with filtering_lines, only show issues in modified lines' do
+              @android_lint.filtering_lines = true
+              @android_lint.lint inline_mode: true
+
+              error = @android_lint.status_report[:errors]
+              expect(error).to include("fake message three")
+
+              expect(error).not_to include("fake message one")
+              expect(error).not_to include("fake message two")
+              expect(error).not_to include("fake message in unmodified file")
+            end
+
+            it 'with filtering, show all issues in modified files' do
+              @android_lint.filtering = true
+              @android_lint.lint inline_mode: true
+
+              error = @android_lint.status_report[:errors]
+              expect(error).to include("fake message one")
+              expect(error).to include("fake message two")
+              expect(error).to include("fake message three")
+
+              expect(error).not_to include("fake message in unmodified file")
+            end
+          end
+
+          context 'when inline_mode: false' do
+            it 'with filtering_lines, only show issues in modified lines' do
+              @android_lint.filtering_lines = true
+              @android_lint.lint inline_mode: false
+
+              puts @android_lint.status_report[:markdowns]
+              message = @android_lint.status_report[:markdowns][0].to_s
+              expect(message).to include("fake message three")
+
+              expect(message).not_to include("fake message one")
+              expect(message).not_to include("fake message two")
+              expect(message).not_to include("fake message in unmodified file")
+            end
+
+            it 'with filtering, show all issues in modified files' do
+              @android_lint.filtering = true
+              @android_lint.lint inline_mode: false
+
+              message = @android_lint.status_report[:markdowns][0].to_s
+              expect(message).to include("fake message one")
+              expect(message).to include("fake message two")
+              expect(message).to include("fake message three")
+
+              expect(message).not_to include("fake message in unmodified file")
+            end
+          end
+        end
       end
-
     end
   end
 end

data/spec/fixtures/lint-result-for-pr.xml

@@ -0,0 +1,62 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<issues format="5" by="lint 4.0.0">
+    <issue
+        id="DuplicatePlatformClasses"
+        severity="Error"
+        message="fake message one"
+        category="Correctness"
+        priority="8"
+        summary="Duplicate Platform Classes"
+        explanation="There are a number of libraries that duplicate not just functionality of the Android platform but using the exact same class names as the ones provided in Android -- for example the apache http classes. This can lead to unexpected crashes.&#xA;&#xA;To solve this, you need to either find a newer version of the library which no longer has this problem, or to repackage the library (and all of its dependencies) using something like the `jarjar` tool, or finally, rewriting the code to use different APIs (for example, for http code, consider using `HttpUrlConnection` or a library like `okhttp`).">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/build.gradle"/>
+    </issue>
+
+    <issue
+        id="DuplicatePlatformClasses"
+        severity="Error"
+        message="fake message two"
+        category="Correctness"
+        priority="8"
+        summary="Duplicate Platform Classes"
+        explanation="There are a number of libraries that duplicate not just functionality of the Android platform but using the exact same class names as the ones provided in Android -- for example the apache http classes. This can lead to unexpected crashes.&#xA;&#xA;To solve this, you need to either find a newer version of the library which no longer has this problem, or to repackage the library (and all of its dependencies) using something like the `jarjar` tool, or finally, rewriting the code to use different APIs (for example, for http code, consider using `HttpUrlConnection` or a library like `okhttp`)."
+        errorLine1="    testImplementation &apos;org.json:json:20140107&apos;"
+        errorLine2="                        ~~~~~~~~~~~~~">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/build.gradle"
+            line="52"
+            column="25"/>
+    </issue>
+
+    <issue
+        id="FakeId"
+        severity="Error"
+        message="fake message three"
+        category="Correctness"
+        priority="8"
+        summary="Fake summary"
+        explanation="Fake explanation."
+        errorLine1="// debugImplementation because LeakCanary should only run in debug builds."
+        errorLine2="                        ~~~~~~~~~~~~~">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/build.gradle"
+            line="36"
+            column="10"/>
+    </issue>
+
+    <issue
+        id="2"
+        severity="Error"
+        message="fake message in unmodified file"
+        category="Correctness"
+        priority="8"
+        summary="Fake summary"
+        explanation="Fake explanation."
+        errorLine1="// debugImplementation because LeakCanary should only run in debug builds."
+        errorLine2="                        ~~~~~~~~~~~~~">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/AndroidManifest.xml"
+            line="36"
+            column="10"/>
+    </issue>
+</issues>

data/spec/fixtures/lint-result-with-special-chars.xml

@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<issues format="4" by="lint 25.2.2">
+  <issue
+      id="UnusedResources"
+      severity="Warning"
+      message="The resource `R.string.authentication_invalid_auth_token_type` appears to be unused"
+      category="Performance"
+      priority="3"
+      summary="Unused resources"
+      explanation="Unused resources make applications larger and slow down builds."
+      errorLine1="    &lt;string name=&quot;authentication_invalid_auth_token_type&quot;>Autenticação inválida.&lt;/string>"
+      errorLine2="            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"
+      quickfix="studio">
+      <location
+          file="app/src/main/res/values/strings.xml"
+          line="105"
+          column="13"/>
+  </issue>
+</issues>

data/spec/fixtures/pr-diff.diff

@@ -0,0 +1,14 @@
+diff --git a/app/build.gradle b/app/build.gradle
+index 6dcc647b8..a1365bd75 100644
+--- a/app/build.gradle
++++ b/app/build.gradle
+@@ -33,6 +33,9 @@ configurations {
+ }
+
+ dependencies {
++    // debugImplementation because LeakCanary should only run in debug builds.
++    debugImplementation 'com.squareup.leakcanary:leakcanary-android:2.7'
++
+     testImplementation 'junit:junit:4.12'
+     testImplementation('org.robolectric:robolectric:4.3.1') {
+         exclude(group: 'org.bouncycastle', module: 'bcprov-jdk15on')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-android_lint
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.10
 platform: ruby
 authors:
 - Gustavo Barbosa
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-04-09 00:00:00.000000000 Z
+date: 2022-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oga
@@ -42,30 +42,30 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: 2.2.10
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: 2.2.10
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -171,8 +171,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".github/CODEOWNERS"
 - ".gitignore"
+- ".ruby-version"
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock
@@ -180,6 +182,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- atlantis.yaml
 - danger-android_lint.gemspec
 - lib/android_lint/gem_version.rb
 - lib/android_lint/plugin.rb
@@ -187,14 +190,17 @@ files:
 - lib/danger_plugin.rb
 - spec/android_lint_spec.rb
 - spec/fixtures/lint-result-empty.xml
+- spec/fixtures/lint-result-for-pr.xml
 - spec/fixtures/lint-result-with-everything.xml
+- spec/fixtures/lint-result-with-special-chars.xml
 - spec/fixtures/lint-result-without-fatal.xml
+- spec/fixtures/pr-diff.diff
 - spec/spec_helper.rb
 homepage: https://github.com/loadsmart/danger-android_lint
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -209,15 +215,17 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2.3
-signing_key: 
+rubygems_version: 3.0.9
+signing_key:
 specification_version: 4
 summary: Lint files of a gradle based Android project. This is done using the Android's
   Lint tool. Results are passed out as tables in markdown.
 test_files:
 - spec/android_lint_spec.rb
 - spec/fixtures/lint-result-empty.xml
+- spec/fixtures/lint-result-for-pr.xml
 - spec/fixtures/lint-result-with-everything.xml
+- spec/fixtures/lint-result-with-special-chars.xml
 - spec/fixtures/lint-result-without-fatal.xml
+- spec/fixtures/pr-diff.diff
 - spec/spec_helper.rb