diff --git a/swh/storage/db.py b/swh/storage/db.py
--- a/swh/storage/db.py
+++ b/swh/storage/db.py
@@ -1024,7 +1024,7 @@
         cur = self._cursor(cur)
         cur.execute('''select %s
                        from metadata_provider
-                       where provider_id=%%s ''' % (
+                       where id=%%s ''' % (
                                  ','.join(self.metadata_provider_cols)),
                     (provider_id, ))
 
diff --git a/swh/storage/tests/test_storage.py b/swh/storage/tests/test_storage.py
--- a/swh/storage/tests/test_storage.py
+++ b/swh/storage/tests/test_storage.py
@@ -1694,6 +1694,26 @@
         # then
         self.assertEqual(expected_tool, actual_tool)
 
+    def test_metadata_provider_get(self):
+        # given
+        no_provider = self.storage.metadata_provider_get(6459456445615)
+        self.assertIsNone(no_provider)
+        # when
+        provider_id = self.storage.metadata_provider_add(
+            self.provider['name'],
+            self.provider['type'],
+            self.provider['url'],
+            self.provider['metadata'])
+
+        actual_provider = self.storage.metadata_provider_get(provider_id)
+        expected_provider = {
+            'provider_name': self.provider['name'],
+            'provider_url': self.provider['url']
+        }
+        # then
+        del actual_provider['id']
+        self.assertTrue(actual_provider, expected_provider)
+
     def test_metadata_provider_get_by(self):
         # given
         no_provider = self.storage.metadata_provider_get_by({