looks like .codeowner files can override each other (#97)

Author: perryqh

.ruby-version

@@ -0,0 +1 @@
+3.2.2

lib/code_ownership.rb

@@ -39,7 +39,7 @@ def for_file(file)
 
     Mapper.all.each do |mapper|
       owner = mapper.map_file_to_owner(file)
-      break if owner
+      break if owner # TODO: what if there are multiple owners? Should we respond with an error instead of the first match?
     end
 
     @for_file[file] = owner

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

@@ -90,6 +90,37 @@ module CodeOwnership
           end
         end
       end
+
+      context 'with mutliple directory ownership files' do
+        before do
+          write_configuration
+
+          write_file('config/teams/bar.yml', <<~CONTENTS)
+            name: Bar
+          CONTENTS
+
+          write_file('config/teams/foo.yml', <<~CONTENTS)
+            name: Foo
+          CONTENTS
+
+          write_file('app/services/exciting/some_other_file.rb', <<~YML)
+            class Exciting::SomeOtherFile; end
+          YML
+
+          write_file('app/services/exciting/.codeowner', <<~YML)
+            Bar
+          YML
+
+          write_file('app/services/.codeowner', <<~YML)
+            Foo
+          YML
+        end
+
+        it 'allows multiple .codeowner ancestor files' do
+          expect(CodeOwnership.for_file('app/services/exciting/some_other_file.rb').name).to eq 'Bar'
+          expect { CodeOwnership.validate! }.to_not raise_error
+        end
+      end
     end
   end
 end