Phabricator: Fix loading tasks creation

I thought having easily fix the issue reported in #1997 (closed) but while trying to implement a test to validate the changes, I noticed there was a major issue in the current lister implementation.

Turns out except the first listed repository (whose index is retrieved through a bootstrap process) all other listed ones were filtered out. Consequently, no entries in the listers database and no scheduler loading tasks were created when listing a Phabricator instance.

I now better understand why the Phabricator instances we listed so far in production still do not have any visit (see [here](https://archive.softwareheritage.org/browse/search/?q=forge.softwareheritage.org&with_visit&with_content for instance) for instance).

This critical issue is due to the max_index member overriding of the Phabricator lister when it bootstraps to get the first repository index. Unfortunately, the tests were passing as the conditions were still respected (but with only one repository ingested in listers database).

That diff fixes that nasty bug and also the issue observed in #1997 (closed) (less critical as it will only appear when listing repositories when the max_bound parameter is provided to the run method of the lister, the fact that we encountered it is due to the bug described above). I also fix some typos to make my git pre-commit hook happy.

Test Plan

Tests implementation have been improved and new ones have been added to capture the observed issues.

---

Migrated from D1982 (view on Phabricator)

swh/lister/core/tests/test_lister.py

@@ -163,8 +163,7 @@ class HttpListerTester(HttpListerTesterBase, abc.ABC):
         if m and (len(m.groups()) > 0):
             return self.convert_type(m.group(1))
 
-    @requests_mock.Mocker()
-    def test_fetch_multiple_pages_yesdb(self, http_mocker):
+    def create_fl_with_db(self, http_mocker):
         http_mocker.get(self.test_re, text=self.mock_response)
         db = init_db()
 
@@ -174,10 +173,16 @@ class HttpListerTester(HttpListerTesterBase, abc.ABC):
                 'args': {'db': db.url()}
                 }
             })
+        fl.db = db
         self.init_db(db, fl.MODEL)
 
         self.disable_scheduler(fl)
+        return fl
+
+    @requests_mock.Mocker()
+    def test_fetch_multiple_pages_yesdb(self, http_mocker):
 
+        fl = self.create_fl_with_db(http_mocker)
         fl.run(min_bound=self.first_index)
 
         self.assertEqual(fl.db_last_index(), self.last_index)