delete get_foreign_match_list

It wasn't in use, I'm not sure it's still relevant,
and it probably could have been accomplished more
easily by calling a normal method on "other" instead
of calling a "foreign" method on "self".

authorityspoke/rules.py

@@ -453,33 +453,6 @@ def all_relation_matches(
             matchlist = relation.update_matchlist(matchlist)
         return matchlist
 
-    @staticmethod
-    def get_foreign_match_list(
-        foreign: List[Dict[Factor, Factor]]
-    ) -> List[Dict[Factor, Factor]]:
-        """
-        Compare this to the regular ``matchlist`` objects, in
-        which the indices represent self's entity slots and the
-        values represent other's.
-
-        :returns:
-            a version of ``matchlist`` in which the indices
-            represent ``foreign``'s entity slots and the values
-            represent ``self``'s entity slots.
-        """
-
-        def get_foreign_match(
-            match: Dict[Factor, Factor]
-        ) -> Optional[Dict[Factor, Factor]]:
-            # TODO: write test for multiple keys of match with same value (other than None)
-            return {v: k for k, v in match.items() if v is not None}
-
-        return [
-            get_foreign_match(match)
-            for match in foreign
-            if get_foreign_match(match) is not None
-        ]
-
 
 @dataclass(frozen=True)
 class Rule(Factor):

tests/test_procedures.py

@@ -38,12 +38,6 @@ def test_procedure_string_with_entities(self, make_procedure):
             make_procedure["c2_irrelevant_inputs"]
         )
 
-    def test_foreign_match_list(self, make_procedure, watt_mentioned):
-        w = watt_mentioned
-        assert make_procedure["c2_irrelevant_inputs"].get_foreign_match_list(
-            [{w[2]: w[1], w[3]: w[0]}, {w[1]: w[1], w[2]: w[3]}]
-        ) == [{w[1]: w[2], w[0]: w[3]}, {w[1]: w[1], w[3]: w[2]}]
-
     def test_generic_factors(
         self, watt_factor, make_entity, make_evidence, make_procedure
     ):