danger-logging_lint 0.0.1 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b4f62fd36901e935296e80d8e6e884604818be116be3731668fd46d9e2f29cf1
-  data.tar.gz: '0518bc5b6aabe4236039303b27e211025458c5870b135789f6311f03918fba5c'
+  metadata.gz: a08e90afc0c447a3f28d9afe54ee806eff0c0f4b4baa203d006c850ee9f59f51
+  data.tar.gz: e4b1797375798f5e2c00bf6c51feec0292b343887d2f16719d238dc9bc0ded2c
 SHA512:
-  metadata.gz: 47ad03a313f30f32a397ff9d0faffa584b758179f8a701b53ed47e78b05657cfcec4d102965f8d045df00c14ada9b34d2750551085970d8d18e312830b197cfa
-  data.tar.gz: 42cd8e9e2872c7f126bb0c6918741b9df60cf171685beb41ee2309c9f3d185b9e444dc31d024281594511cb108e5d199f667785ce709bf88741204d2efda7ac2
+  metadata.gz: 685a9d6243a365cc2098b5f9ad36aeb097d55ed06e08b0a546675f9e9393675e1f01e097c89bfdeebc66dfe11fad1570fe1847eea338fdd927dad385b0202720
+  data.tar.gz: e2b4e4e4883faf5436c814cc5d0b2672ef9d8db7b1202737279aeda490f8f79195882fb839847d1fed93ccae7cf15e32d1f686c999b838842acc261cc6b9f77f

data/.github/workflows/deploy.yml

@@ -0,0 +1,36 @@
+name: Deploy
+
+on:
+  workflow_dispatch:
+    branches: [ master ]
+
+jobs:
+  test:
+    name: Publish gem
+    runs-on: ubuntu-latest
+    environment: production
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '3.0'
+          bundler-cache: true
+      - name: Run tests
+        run: bundle exec rake spec
+      - name: Publish gem
+        uses: dawidd6/action-publish-gem@v1
+        with:
+          api_key: ${{secrets.RUBYGEMS_API_KEY}}
+      - name: Create git tag
+        uses: duderman/gh-gem-tag-action@v1
+        id: release_info
+        with:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+      - uses: xresloader/upload-to-github-release@v1
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          tag_name: ${{ steps.release_info.outputs.tag }}
+          draft: false
+          file: '*.gem'

data/.github/workflows/test.yml

@@ -0,0 +1,24 @@
+name: Test
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  test:
+    name: Test with Ruby ${{ matrix.ruby }}
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: [ '2.5', '2.7', '3.0' ]
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - name: Run tests
+        run: bundle exec rake spec

data/.gitignore

@@ -2,3 +2,7 @@
 pkg
 .idea/
 .yardoc
+
+coverage/
+.coverage
+coverage.xml

data/CHANGELOG.md

@@ -0,0 +1,29 @@
+## Changelog
+
+### Version 0.0.4 (2022-04-28)
+
+- Fixed crash when changed file is a directory (filters them out).
+- Fixed crash when opening missing file (filters them out).
+- Split rspec into multiple files.
+- Added tests for linter with all variables set using Danger file.
+- Variables used in multiple tests are defined as constants in `spec_helper.rb`.
+
+### Version 0.0.3 (2022-04-22)
+
+- Added deploy and test yaml for github workflow.
+- Added deployment to Readme.
+- Added codecov connection and dependency.
+- Added Readme badges.
+- Remove "Check: " hardcoded ext from warning message.
+- Fixed tests.
+- Updated Gemfile.lock.
+
+### Version 0.0.2 (2022-04-21)
+
+- Updated `gemspec` (gem) documentation.
+- Fixed warning call.
+- Changed test checks to use `violation_report` instead of `status_report`.
+
+### Version 0.0.1 (2022-04-20)
+
+- Initial version of the library.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-logging_lint (0.0.1)
+    danger-logging_lint (0.0.4)
       danger-plugin-api (~> 1.0)
 
 GEM
@@ -15,11 +15,13 @@ GEM
       cork
       nap
       open4 (~> 1.3)
+    codecov (0.5.2)
+      simplecov (>= 0.15, < 0.22)
     coderay (1.1.3)
     colored2 (3.1.2)
     cork (0.3.0)
       colored2 (~> 3.1)
-    danger (8.5.0)
+    danger (8.6.0)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
@@ -35,6 +37,7 @@ GEM
     danger-plugin-api (1.0.0)
       danger (> 2.0)
     diff-lcs (1.5.0)
+    docile (1.4.0)
     faraday (1.10.0)
       faraday-em_http (~> 1.0)
       faraday-em_synchrony (~> 1.0)
@@ -62,7 +65,7 @@ GEM
     faraday-retry (1.0.3)
     ffi (1.15.5)
     formatador (1.1.0)
-    git (1.10.2)
+    git (1.11.0)
       rchardet (~> 1.8)
     guard (2.18.0)
       formatador (>= 0.2.4)
@@ -99,14 +102,14 @@ GEM
       sawyer (~> 0.8.0, >= 0.5.3)
     open4 (1.3.4)
     parallel (1.22.1)
-    parser (3.1.1.0)
+    parser (3.1.2.0)
       ast (~> 2.4.1)
     pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    public_suffix (4.0.6)
+    public_suffix (4.0.7)
     rainbow (3.1.1)
-    rake (10.5.0)
+    rake (13.0.6)
     rb-fsevent (0.11.1)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
@@ -126,13 +129,13 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.11.0)
     rspec-support (3.11.0)
-    rubocop (1.27.0)
+    rubocop (1.28.1)
       parallel (~> 1.10)
       parser (>= 3.1.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.16.0, < 2.0)
+      rubocop-ast (>= 1.17.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
     rubocop-ast (1.17.0)
@@ -143,6 +146,12 @@ GEM
       addressable (>= 2.3.5)
       faraday (> 0.8, < 2.0)
     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)
     terminal-table (3.0.2)
       unicode-display_width (>= 1.1.1, < 3)
     thor (1.2.1)
@@ -156,12 +165,13 @@ PLATFORMS
 
 DEPENDENCIES
   bundler (~> 2.0)
+  codecov (~> 0.5.1)
   danger-logging_lint!
   guard (~> 2.14)
   guard-rspec (~> 4.7)
   listen (= 3.0.7)
   pry
-  rake (~> 10.0)
+  rake (>= 12.3.3)
   rspec (~> 3.4)
   rubocop
   yard

data/README.md

@@ -1,4 +1,5 @@
-## logging_lint
+## Logging Lint
+[![Gem Version](https://badge.fury.io/rb/danger-logging_lint.svg)](https://badge.fury.io/rb/danger-logging_lint) [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://github.com/eManPrague/danger-logging_lint/blob/master/LICENSE.txt) [![Test](https://github.com/eManPrague/danger-logging_lint/actions/workflows/test.yml/badge.svg)](https://github.com/eManPrague/danger-logging_lint/actions/workflows/test.yml) [![codecov](https://codecov.io/gh/eManPrague/danger-logging_lint/branch/master/graph/badge.svg?token=Z2RZKYNBVI)](https://codecov.io/gh/eManPrague/danger-logging_lint)
 
 This danger plugin can be used to check log lines in modified (added) files. It heavily relies on regex configuration which can be modified to search all kinds of parts of code in the files. Default configuration is set to support [Kotlin eMan Logger Library](https://github.com/eManPrague/logger-ktx). Ex: logInfo { "Info message $var" }.
 
@@ -72,3 +73,7 @@ If all of these checks pass then it will trigger linter on target files (filtere
 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.
+
+## Deployment
+
+Gem is deployed manually from master branch using [Github Action](https://github.com/eManPrague/danger-logging_lint/actions/workflows/deploy.yml). Make sure you increased the gem version before triggering it.

data/codecov.yml

@@ -0,0 +1,20 @@
+codecov:
+  require_ci_to_pass: yes
+
+coverage:
+  precision: 2
+  round: down
+  range: '70...100'
+
+parsers:
+  gcov:
+    branch_detection:
+      conditional: yes
+      loop: yes
+      method: no
+      macro: no
+
+comment:
+  layout: 'reach,diff,flags,files,footer'
+  behavior: default
+  require_changes: no

data/danger-logging_lint.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |spec|
   spec.authors       = ["David Sucharda"]
   spec.email         = ["david.sucharda@eman.cz"]
   spec.description   = "Checks logging commands in code."
-  spec.summary       = "Logging might be a security issue that is why this plugin checks files for new/changed logs with variables that might be a security issue and warns them using Danger."
+  spec.summary       = "Since logging can be a security issue in some cases (ex: usage in production) this linter checks for logs with variables and triggers warning for them using Danger."
   spec.homepage      = "https://github.com/eManPrague/danger-logging_lint"
   spec.license       = "MIT"
 
@@ -23,9 +23,10 @@ Gem::Specification.new do |spec|
 
   # General ruby development
   spec.add_development_dependency "bundler", "~> 2.0"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rake", ">= 12.3.3"
 
   # Testing support
+  spec.add_development_dependency 'codecov', '~> 0.5.1'
   spec.add_development_dependency "rspec", "~> 3.4"
 
   # Linting code and docs

data/lib/logging_lint/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LoggingLint
-  VERSION = "0.0.1"
+  VERSION = "0.0.4"
 end

data/lib/logging_lint/plugin.rb

@@ -191,28 +191,41 @@ module Danger
     #
     def log_lint
       if log_functions.nil? || log_functions.size <= 0
-        self.fail("No log functions are defined. Please check your Danger file.")
+        self.fail("Logging lint: No log functions are defined. Please check your Danger file.")
         return
       end
 
       if line_variable_regex.nil? || line_variable_regex.size <= 0
-        message("At least one variable index must be defined (using default). Please check your Danger file.")
+        message("Logging lint: At least one variable index must be defined (using default). Please check your Danger file.")
       end
 
       target_files = (git.modified_files - git.deleted_files) + git.added_files
+      target_files = target_files.reject { |filename| invalid_file?(filename) }
       if !file_extensions.nil? && file_extensions.size >= 0
         file_extensions_regex = "(.#{file_extensions.join('|.')})"
         target_files = target_files.grep(/#{file_extensions_regex}/)
       end
 
       if target_files.empty?
-        message("No files to check.")
+        message("Logging lint: No files to check.")
         return
       end
 
       check_files(target_files)
     end
 
+    #
+    # Checks if file is not valid. It will not be valid in two cases:
+    # 1) Files is a directory.
+    # 2) Files does not exist.
+    # In both cases we cannot open it and lint it. There is also no reason to lint them.
+    #
+    # @return [Boolean] true if invalid file
+    #
+    def invalid_file?(filename)
+      File.directory?(filename) || !File.exist?(filename)
+    end
+
     #
     # Checks all files for log violations based on log regex and log function. Each log function id extended by log
     # regex and searched for (format: #log_function#log_regex). Each of such found line is then checked if it contains a
@@ -224,13 +237,15 @@ module Danger
     def check_files(files)
       raw_file = ""
       files.each do |filename|
+        next if invalid_file?(filename)
+
         raw_file = File.read(filename)
         log_functions.each do |log_function|
           raw_file.scan(/#{log_function}#{log_regex}/m) do |c|
             if contains_variable(c)
               char_index = $~.offset(0)[0] + line_offset(c)
               line_index = raw_file[0..char_index].lines.count
-              warn(compose_warning_text(warning_text), true, filename, line_index)
+              warn(compose_warning_text(warning_text), file: filename, line: line_index)
             end
           end
         end
@@ -291,7 +306,7 @@ module Danger
     def compose_warning_text(warning_text)
       return warning_text if warning_description.nil?
 
-      "#{warning_text} Check: #{warning_description}"
+      "#{warning_text} #{warning_description}"
     end
   end
 end

data/spec/logging_lint_no_run_spec.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+require File.expand_path("spec_helper", __dir__)
+
+#
+# Tests for situations when the linter does not run because of either configuration or there are no files to check.
+#
+
+module Danger
+  describe Danger::DangerLoggingLint do
+    #
+    # Defines linter, danger file and other variables used by the linter.
+    #
+    describe "with Dangerfile" do
+      before do
+        @dangerfile = testing_dangerfile
+        @logging_lint = @dangerfile.logging_lint
+
+        mock_variables(@logging_lint)
+      end
+
+      #
+      # Test for logging lines in cases when linter does not run (either by config or file settings).
+      #
+
+      it "Error is printed when log functions are not configured" do
+        allow(@logging_lint).to receive(:log_functions).and_return([])
+        @logging_lint.log_lint
+        expect(@dangerfile.status_report[:errors]).to eq(["Logging lint: No log functions are defined. Please check your Danger file."])
+      end
+
+      it "Error is printed when log variable regex is not configured" do
+        allow(@logging_lint).to receive(:line_variable_regex).and_return([])
+        @logging_lint.log_lint
+        expect(@dangerfile.status_report[:messages][0]).to eq("Logging lint: At least one variable index must be defined (using default). Please check your Danger file.")
+      end
+
+      it "Nothing is printed when there are no files to check" do
+        @logging_lint.log_lint
+        expect(@dangerfile.status_report[:errors]).to eq([])
+        expect(@dangerfile.status_report[:messages][0]).to eq("Logging lint: No files to check.")
+      end
+
+      it "Nothing is printed when there are only folders to check" do
+        allow(@logging_lint).to receive(:file_extensions).and_return([])
+        allow(@logging_lint.git).to receive(:modified_files).and_return(%W(#{DIR_NAME}))
+        @logging_lint.log_lint
+        expect(@dangerfile.status_report[:errors]).to eq([])
+      end
+
+      it "Nothing is printed when there are no files to check (filtered by extensions)" do
+        allow(@logging_lint.git).to receive(:modified_files).and_return(MODIFIED_FILES)
+        allow(@logging_lint).to receive(:file_extensions).and_return(%w(unknownExtension))
+        @logging_lint.log_lint
+        expect(@dangerfile.status_report[:errors]).to eq([])
+      end
+    end
+  end
+end

data/spec/logging_lint_spec.rb

@@ -8,11 +8,6 @@ module Danger
       expect(Danger::DangerLoggingLint.new(nil)).to be_a Danger::Plugin
     end
 
-    dir_name = File.dirname(__FILE__)
-    modified_files = %W(#{dir_name}/fixtures/ModifiedFile.kt #{dir_name}/fixtures/IgnoredModifiedFile.txt)
-    added_files = %W(#{dir_name}/fixtures/NewFile.kt)
-    warning_text = "Does this log comply with security rules?"
-
     #
     # Defines linter, danger file and other variables used by the linter.
     #
@@ -25,48 +20,27 @@ module Danger
         allow(@logging_lint.git).to receive(:added_files).and_return([])
         allow(@logging_lint.git).to receive(:modified_files).and_return([])
         allow(@logging_lint).to receive(:file_extensions).and_return(%w(kt))
+        allow(@logging_lint).to receive(:log_functions).and_call_original
+        allow(@logging_lint).to receive(:warning_text).and_call_original
+        allow(@logging_lint).to receive(:log_regex).and_call_original
+        allow(@logging_lint).to receive(:line_variable_regex).and_call_original
+        allow(@logging_lint).to receive(:line_remove_regex).and_call_original
       end
 
       #
-      # Test for logging lines in cases when linter does not run (either by config or file settings).
+      # Test for logging lines in cases when linter does run.
       #
 
-      it "Error is printed when log functions are not configured" do
-        allow(@logging_lint).to receive(:log_functions).and_return([])
-        @logging_lint.log_lint
-        expect(@dangerfile.status_report[:errors]).to eq(["No log functions are defined. Please check your Danger file."])
-      end
-
-      it "Error is printed when log variable regex is not configured" do
-        allow(@logging_lint).to receive(:line_variable_regex).and_return([])
-        @logging_lint.log_lint
-        expect(@dangerfile.status_report[:messages][0]).to eq("At least one variable index must be defined (using default). Please check your Danger file.")
-      end
-
-      it "Nothing is printed when there are no files to check" do
-        @logging_lint.log_lint
-        expect(@dangerfile.status_report[:errors]).to eq([])
-      end
-
-      it "Nothing is printed when there are no files to check (filtered by extensions)" do
-        allow(@logging_lint.git).to receive(:modified_files).and_return(modified_files)
-        allow(@logging_lint).to receive(:file_extensions).and_return(%w(unknownExtension))
-        @logging_lint.log_lint
-        expect(@dangerfile.status_report[:errors]).to eq([])
-      end
-
-      it "Nothing is printed when log levels are not present" do
-        allow(@logging_lint).to receive(:log_functions).and_return(%w(missingLogLevel))
+      it "Nothing is printed when log functions are not present in files" do
+        allow(@logging_lint.git).to receive(:modified_files).and_return(MODIFIED_FILES)
+        allow(@logging_lint.git).to receive(:added_files).and_return(ADDED_FILES)
+        allow(@logging_lint).to receive(:log_functions).and_return(%w(unknownLogLevel))
         @logging_lint.log_lint
         expect(@dangerfile.status_report[:warnings]).to eq([])
       end
 
-      #
-      # Test for logging lines in cases when linter does run.
-      #
-
       it "Log with variables is warned for modified files (end line index)" do
-        allow(@logging_lint.git).to receive(:modified_files).and_return(modified_files)
+        allow(@logging_lint.git).to receive(:modified_files).and_return(MODIFIED_FILES)
         allow(@logging_lint).to receive(:line_index_position).and_return("end")
         @logging_lint.log_lint
         violation_lines = [63, 64, 73, 76, 88, 92, 97, 98, 101, 106, 107, 110]
@@ -74,7 +48,7 @@ module Danger
       end
 
       it "Log with variables is warned for modified files (start line index)" do
-        allow(@logging_lint.git).to receive(:modified_files).and_return(modified_files)
+        allow(@logging_lint.git).to receive(:modified_files).and_return(MODIFIED_FILES)
         allow(@logging_lint).to receive(:line_index_position).and_return("start")
         @logging_lint.log_lint
         violation_lines = [63, 64, 71, 74, 85, 89, 93, 98, 99, 102, 107, 108]
@@ -82,49 +56,47 @@ module Danger
       end
 
       it "Log with variables is warned for modified files (middle line index)" do
-        allow(@logging_lint.git).to receive(:modified_files).and_return(modified_files)
+        allow(@logging_lint.git).to receive(:modified_files).and_return(MODIFIED_FILES)
         allow(@logging_lint).to receive(:line_index_position).and_return("middle")
         @logging_lint.log_lint
         violation_lines = [63, 64, 73, 76, 88, 92, 97, 98, 101, 106, 107, 110]
         compare_warning_with_lines(violation_lines)
       end
 
-      it "Log with variables is warned for new files" do
-        allow(@logging_lint.git).to receive(:added_files).and_return(added_files)
+      it "Log with variables is warned for modified files without crashing due to missing files" do
+        allow(@logging_lint.git).to receive(:modified_files).and_return(MODIFIED_FILES + %W(#{DIR_NAME}/fixtures/MissingFile.kt))
         @logging_lint.log_lint
-        violation_lines = [47, 48, 57, 60, 72, 76]
+        violation_lines = [63, 64, 73, 76, 88, 92, 97, 98, 101, 106, 107, 110]
         compare_warning_with_lines(violation_lines)
       end
 
-      #
-      # Test for waning texts and links.
-      #
-
-      it "Log with variables is warned with link address" do
-        warning_link = "http://error.io"
-        allow(@logging_lint.git).to receive(:added_files).and_return(added_files)
-        allow(@logging_lint).to receive(:warning_description).and_return(warning_link)
+      it "Log with variables is warned for new files" do
+        allow(@logging_lint.git).to receive(:added_files).and_return(ADDED_FILES)
         @logging_lint.log_lint
-        expect(@dangerfile.status_report[:warnings][0]).to eq("#{warning_text} Check: #{warning_link}")
+        violation_lines = [47, 48, 57, 60, 72, 76]
+        compare_warning_with_lines(violation_lines)
       end
 
-      it "Log with variables is warned without link address" do
-        allow(@logging_lint.git).to receive(:added_files).and_return(added_files)
+      it "Log with variables is warned for new files (with all params)" do
+        custom_warning_text = "Warning text"
+        allow(@logging_lint.git).to receive(:added_files).and_return(ADDED_FILES)
+        @logging_lint.log_functions = %w(logInfo)
+        @logging_lint.warning_text = custom_warning_text
+        @logging_lint.log_regex = '[ ]?[{(](?:\n?|.)["]?(?:\n?|.)["]?(?:\n?|.)+(?:[)}][ ]?\n)'
+        @logging_lint.line_variable_regex = ['[{(](\n| |\+)*([^\"]\w[^\"])+', '(\".*\$.*\")']
+        @logging_lint.line_remove_regex = ['(\+ )?\".*\"']
         @logging_lint.log_lint
-        expect(@dangerfile.status_report[:warnings][0]).to eq(warning_text)
+        violation_lines = [47, 48, 57, 60, 72, 76]
+        compare_warning_with_lines(violation_lines)
+        expect(@dangerfile.violation_report[:warnings][0].message).to eq(custom_warning_text)
       end
 
       #
       # Compares violation lines against danger warning lines. It expects them to be equal.
       #
       def compare_warning_with_lines(violation_lines)
-        warnings = @dangerfile.status_report[:warnings]
-        warning_lines = []
-        warnings.each_with_index do |value, index|
-          if index > 0 && (index + 1) % 4 == 0
-            warning_lines << value
-          end
-        end
+        warnings = @dangerfile.violation_report[:warnings]
+        warning_lines = warnings.map(&:line)
         expect(warning_lines).to eq(violation_lines)
       end
     end

data/spec/logging_lint_text_spec.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require File.expand_path("spec_helper", __dir__)
+
+#
+# Tests for warning text creation. Text can contain description which can be defined in Danger file.
+#
+
+module Danger
+  describe Danger::DangerLoggingLint do
+    #
+    # Defines linter, danger file and other variables used by the linter.
+    #
+    describe "with Dangerfile" do
+      before do
+        @dangerfile = testing_dangerfile
+        @logging_lint = @dangerfile.logging_lint
+
+        mock_variables(@logging_lint)
+      end
+
+      #
+      # Test for waning text and description (optional).
+      #
+
+      it "Log with variables is warned description (link address)" do
+        warning_description = "http://error.io"
+        allow(@logging_lint.git).to receive(:added_files).and_return(ADDED_FILES)
+        allow(@logging_lint).to receive(:warning_description).and_return(warning_description)
+        @logging_lint.log_lint
+        expect(@dangerfile.status_report[:warnings][0]).to eq("#{WARNING_TEXT} #{warning_description}")
+      end
+
+      it "Log with variables is warned without warning description" do
+        allow(@logging_lint.git).to receive(:added_files).and_return(ADDED_FILES)
+        @logging_lint.log_lint
+        expect(@dangerfile.status_report[:warnings][0]).to eq(WARNING_TEXT)
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,5 +1,11 @@
 # frozen_string_literal: true
 
+require "simplecov"
+SimpleCov.start
+
+require "codecov"
+SimpleCov.formatter = SimpleCov::Formatter::Codecov
+
 require "pathname"
 ROOT = Pathname.new(File.expand_path("..", __dir__))
 $:.unshift("#{ROOT}lib".to_s)
@@ -65,3 +71,22 @@ def testing_dangerfile
   env = Danger::EnvironmentManager.new(testing_env)
   Danger::Dangerfile.new(env, testing_ui)
 end
+
+# Mocks linter variables. Should be called in "before" block.
+def mock_variables(logging_lint)
+  allow(logging_lint.git).to receive(:deleted_files).and_return([])
+  allow(logging_lint.git).to receive(:added_files).and_return([])
+  allow(logging_lint.git).to receive(:modified_files).and_return([])
+  allow(logging_lint).to receive(:file_extensions).and_return(%w(kt))
+  allow(logging_lint).to receive(:log_functions).and_call_original
+  allow(logging_lint).to receive(:warning_text).and_call_original
+  allow(logging_lint).to receive(:log_regex).and_call_original
+  allow(logging_lint).to receive(:line_variable_regex).and_call_original
+  allow(logging_lint).to receive(:line_remove_regex).and_call_original
+end
+
+# Defines test variables used in multiple text files.
+DIR_NAME = File.dirname(__FILE__)
+MODIFIED_FILES = %W(#{DIR_NAME}/fixtures/ModifiedFile.kt #{DIR_NAME}/fixtures/IgnoredModifiedFile.txt).freeze
+ADDED_FILES = %W(#{DIR_NAME}/fixtures/NewFile.kt).freeze
+WARNING_TEXT = "Does this log comply with security rules?"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-logging_lint
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.4
 platform: ruby
 authors:
 - David Sucharda
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-04-20 00:00:00.000000000 Z
+date: 2022-04-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -40,18 +40,32 @@ dependencies:
         version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 12.3.3
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 12.3.3
+- !ruby/object:Gem::Dependency
+  name: codecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 0.5.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 0.5.1
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -157,15 +171,19 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/deploy.yml"
+- ".github/workflows/test.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - Gemfile.lock
 - Guardfile
 - LICENSE.txt
 - README.md
 - Rakefile
+- codecov.yml
 - danger-logging_lint.gemspec
 - lib/danger_logging_lint.rb
 - lib/danger_plugin.rb
@@ -174,7 +192,9 @@ files:
 - spec/fixtures/IgnoredModifiedFile.txt
 - spec/fixtures/ModifiedFile.kt
 - spec/fixtures/NewFile.kt
+- spec/logging_lint_no_run_spec.rb
 - spec/logging_lint_spec.rb
+- spec/logging_lint_text_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/eManPrague/danger-logging_lint
 licenses:
@@ -195,15 +215,16 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.33
 signing_key: 
 specification_version: 4
-summary: Logging might be a security issue that is why this plugin checks files for
-  new/changed logs with variables that might be a security issue and warns them using
-  Danger.
+summary: 'Since logging can be a security issue in some cases (ex: usage in production)
+  this linter checks for logs with variables and triggers warning for them using Danger.'
 test_files:
 - spec/fixtures/IgnoredModifiedFile.txt
 - spec/fixtures/ModifiedFile.kt
 - spec/fixtures/NewFile.kt
+- spec/logging_lint_no_run_spec.rb
 - spec/logging_lint_spec.rb
+- spec/logging_lint_text_spec.rb
 - spec/spec_helper.rb