Changed FindSuspects to take a Repository factory, rather than mutating it

appengine/findit/crash/loglinear/test/changelist_classifier_test.py

Index: appengine/findit/crash/loglinear/test/changelist_classifier_test.py
diff --git a/appengine/findit/crash/loglinear/test/changelist_classifier_test.py b/appengine/findit/crash/loglinear/test/changelist_classifier_test.py
index 8bc46e485e9c0306a22708c494e80afffc7e2029..a1719ad4fb953233711b176448f9ee5e6df79271 100644
--- a/appengine/findit/crash/loglinear/test/changelist_classifier_test.py
+++ b/appengine/findit/crash/loglinear/test/changelist_classifier_test.py
@@ -117,9 +117,22 @@ class LogLinearChangelistClassifierTest(CrashTestSuite):
         'TopFrameIndex': 1.,
     }
 
+    repository = GitilesRepository(self.GetMockHttpClient())
+
+    def MutateTheRepo(dep_url): # pragma: no cover
+      """A factory function for returning ``Repository`` objects.
+
+      The current definition captures the functionality of before
+      we actored out this factory method. That is, it's not really a
+      "factory" but rather mutates the main repo object in place. In
+      the future this should be changed to do the right thing instead.
+      """
+      repository.repo_url = dep_url
+      return repository
+
     self.changelist_classifier = (
         loglinear_changelist_classifier.LogLinearChangelistClassifier(
-          GitilesRepository(self.GetMockHttpClient()), weights))
+            repository, MutateTheRepo, weights))
 
   def testAggregateChangedFilesAggreegates(self):
     """Test that ``AggregateChangedFiles`` does aggregate reasons per file.