danger-missing_codeowners 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9e93a3e15840e49e1966b6bfae9e59a8bfff84539b3cc1314e6224fab84464d9
-  data.tar.gz: 9288cb2e4e84f5c5dd09e4d0363f28276e3241d56c509f03075088401124fb4c
+  metadata.gz: fcd83e0d8b50b255205c5ecfbceed72c29db635c92fc27d2f2b9420b6ba5a8a6
+  data.tar.gz: 43a2e6402320a69e456f6c19524c8388b2be03195bf25656157fea8840069f66
 SHA512:
-  metadata.gz: 173a41e2f5fe25c8a953e2bd1dfc78900ee4012604f70692785f8475ffae110e8a7ddb21fc20c0f70dd50093c1102804d46566ad89e401f3fd6198d70c0f50df
-  data.tar.gz: 4e5a963fb0242853a0a0ad88637f05c2e3ae0c8ff69d8b124e403b79cbccecec866b9f23ea37512b0cdd5594e5a1f0ef6e14b79a9dd6c00e2cbc67f6de2ab06f
+  metadata.gz: fac672eff7a5197a07065fcfd8ab2c5b7546d4ed33071e2a6d4e00fa6366f8579d9e4fd9af726ba03ac8776c571af6bc02c8d624c24a3ed5792102567821cc97
+  data.tar.gz: b82d354dcc331dc13abb1958374edb8dc72c55a71eb056bd8f1a4f846682101167a17fdb1af9c6576aea3a3d412e357b23c218beb5fad7797e3367865a7c99c2

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-missing_codeowners (1.1.0)
+    danger-missing_codeowners (1.2.0)
       danger-plugin-api (~> 1.0)
       pathspec (~> 1.0.0)
 
@@ -134,6 +134,7 @@ GEM
 
 PLATFORMS
   arm64-darwin-21
+  ruby
   x86_64-darwin-20
 
 DEPENDENCIES

data/lib/missing_codeowners/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MissingCodeowners
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

data/lib/missing_codeowners/plugin.rb

@@ -59,7 +59,7 @@ module Danger
     #
     # @return  [void]
     #
-    def verify(*opts)
+    def verify(*opts, &block)
       @verify_all_files ||= false
       @max_number_of_files_to_report ||= 100
       @severity ||= "error"
@@ -86,8 +86,12 @@ module Danger
         markdown format_missing_owners_message(@files_missing_codeowners, @max_number_of_files_to_report)
         danger_message = "Add CODEOWNERS rules to match all files."
         @severity == "error" ? (fail danger_message) : (warn danger_message)
+
+        yield @files_missing_codeowners if block
       else
         log "No files missing CODEOWNERS."
+
+        yield [] if block
       end
 
       log "-----"

data/spec/missing_codeowners_spec.rb

@@ -75,6 +75,26 @@ module Danger
           expect(@my_plugin.files_missing_codeowners.length).to eq(0)
         end
 
+        it "calls the block with an empty array if there are no missing files" do
+          allow(@my_plugin).to receive(:git_modified_files).and_return(["any_file.yml", "any_file.go"])
+
+          @my_plugin.verify do |x|
+            expect(x).to eq []
+          end
+
+          expect(@my_plugin.files_missing_codeowners.length).to eq(0)
+        end
+
+        it "calls the block with the list of missing files if there are missing files" do
+          allow(@my_plugin).to receive(:git_modified_files).and_return(["myfile"])
+
+          @my_plugin.verify do |x|
+            expect(x).to eq ["myfile"]
+          end
+
+          expect(@my_plugin.files_missing_codeowners.length).to eq(1)
+        end
+
         it "fails when there are files without CODEOWNERS rules and severity is error" do
           @my_plugin.verify_all_files = true
           allow(@my_plugin).to receive(:git_all_files).and_return(["app/source.swift", ".swiftlint.yml"])

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-missing_codeowners
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - andre-alves
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-25 00:00:00.000000000 Z
+date: 2023-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api