D716.diff
No OneTemporary
Actions

Size

1 KB

Subscribers

None

D716.diff
View Options

	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({