Changeset View
Changeset View
Standalone View
Standalone View
swh/storage/tests/test_storage.py
| Show First 20 Lines • Show All 3,554 Lines • ▼ Show 20 Lines | def test_origin_metadata_get(self): | ||||
| 'tool_id': tool['id'] | 'tool_id': tool['id'] | ||||
| }] | }] | ||||
| # then | # then | ||||
| self.assertEqual(len(all_metadatas), 2) | self.assertEqual(len(all_metadatas), 2) | ||||
| self.assertEqual(len(metadatas_for_origin2), 1) | self.assertEqual(len(metadatas_for_origin2), 1) | ||||
| self.assertCountEqual(all_metadatas, expected_results) | self.assertCountEqual(all_metadatas, expected_results) | ||||
| def test_metadata_provider_add(self): | |||||
| provider = { | |||||
| 'provider_name': 'swMATH', | |||||
| 'provider_type': 'registry', | |||||
| 'provider_url': 'http://www.swmath.org/', | |||||
| 'metadata': { | |||||
| 'email': 'contact@swmath.org', | |||||
| 'license': 'All rights reserved' | |||||
| } | |||||
| } | |||||
douardda: why not call it with `**provider`? | |||||
Done Inline ActionsBecause the argument names are provider_name, provider_type, and provider_url. I plan on changing that when D1509 is accepted. vlorentz: Because the argument names are `provider_name`, `provider_type`, and `provider_url`. I plan on… | |||||
| provider['id'] = provider_id = self.storage.metadata_provider_add( | |||||
| **provider) | |||||
| self.assertEqual( | |||||
| provider, | |||||
| self.storage.metadata_provider_get_by({ | |||||
| 'provider_name': 'swMATH', | |||||
| 'provider_url': 'http://www.swmath.org/' | |||||
| })) | |||||
| self.assertEqual( | |||||
| provider, | |||||
| self.storage.metadata_provider_get(provider_id)) | |||||
| def test_origin_metadata_get_by_provider_type(self): | def test_origin_metadata_get_by_provider_type(self): | ||||
| # given | # given | ||||
| origin_id = self.storage.origin_add([self.origin])[0]['id'] | origin_id = self.storage.origin_add([self.origin])[0]['id'] | ||||
| origin_id2 = self.storage.origin_add([self.origin2])[0]['id'] | origin_id2 = self.storage.origin_add([self.origin2])[0]['id'] | ||||
| provider1_id = self.storage.metadata_provider_add( | provider1_id = self.storage.metadata_provider_add( | ||||
| self.provider['name'], | self.provider['name'], | ||||
| self.provider['type'], | self.provider['type'], | ||||
| self.provider['url'], | self.provider['url'], | ||||
| ▲ Show 20 Lines • Show All 413 Lines • Show Last 20 Lines | |||||
why not call it with **provider?