Fix issue where codeowners file references deleted file (#61)

* add failing test

* Fix test

* Bump version

Author: Alex Evanczuk

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    code_ownership (1.32.13)
+    code_ownership (1.32.14)
       code_teams (~> 1.0)
       packs
       sorbet-runtime

code_ownership.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = 'code_ownership'
-  spec.version       = '1.32.13'
+  spec.version       = '1.32.14'
   spec.authors       = ['Gusto Engineers']
   spec.email         = ['[email protected]']
   spec.summary       = 'A gem to help engineering teams declare ownership of code'

lib/code_ownership/private.rb

@@ -96,7 +96,7 @@ def self.file_tracked?(file)
         File.fnmatch?(unowned_glob, file, File::FNM_PATHNAME | File::FNM_EXTGLOB)
       end
 
-      in_owned_globs && !in_unowned_globs
+      in_owned_globs && !in_unowned_globs && File.exist?(file)
     end
 
     sig { params(team_name: String, location_of_reference: String).returns(CodeTeams::Team) }

spec/lib/code_ownership/private/validations/github_codeowners_up_to_date_spec.rb

@@ -526,6 +526,57 @@ module CodeOwnership
             end
           end
         end
+
+        context 'in an application with a CODEOWNERS file that has a reference to a file that no longer exists' do
+          before do
+            write_configuration
+
+            write_file('packs/my_pack/owned_file.rb', <<~CONTENTS)
+              # @team Bar
+            CONTENTS
+
+            write_file('config/teams/bar.yml', <<~CONTENTS)
+              name: Bar
+              github:
+                team: '@MyOrg/bar-team'
+            CONTENTS
+          end
+
+          it 'prints out the diff' do
+            FileUtils.mkdir('.github')
+            codeowners_path.write <<~CODEOWNERS
+              # STOP! - DO NOT EDIT THIS FILE MANUALLY
+              # This file was automatically generated by "bin/codeownership validate".
+              #
+              # CODEOWNERS is used for GitHub to suggest code/file owners to various GitHub
+              # teams. This is useful when developers create Pull Requests since the
+              # code/file owner is notified. Reference GitHub docs for more details:
+              # https://help.github.com/en/articles/about-code-owners
+
+
+              # Annotations at the top of file
+              /packs/my_pack/owned_file.rb @MyOrg/bar-team
+              /packs/my_pack/deleted_file.rb @MyOrg/bar-team
+            CODEOWNERS
+
+            expect_any_instance_of(codeowners_validation).to_not receive(:`) # rubocop:disable RSpec/AnyInstance
+            expect { CodeOwnership.validate!(autocorrect: false) }.to raise_error do |e|
+              expect(e).to be_a CodeOwnership::InvalidCodeOwnershipConfigurationError
+              expect(e.message).to eq <<~EXPECTED.chomp
+                CODEOWNERS out of date. Run `bin/codeownership validate` to update the CODEOWNERS file
+
+                CODEOWNERS should contain the following lines, but does not:
+                - "# Team YML ownership"
+                - "/config/teams/bar.yml @MyOrg/bar-team"
+
+                CODEOWNERS should not contain the following lines, but it does:
+                - "/packs/my_pack/deleted_file.rb @MyOrg/bar-team"
+
+                See https://github.com/rubyatscale/code_ownership#README.md for more details
+              EXPECTED
+            end
+          end
+        end
       end
 
       context 'code_ownership.yml has skip_codeowners_validation set' do