Skip to content

Package field update method #222

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JonoYang merged 8 commits into from
Nov 22, 2023
Merged

Package field update method #222

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d50cf71
Create methods to update package fields
JonoYang Nov 13, 2023
1a00a3b
Add tests
JonoYang Nov 14, 2023
23b75fa
Refactor update_fields and related methods
JonoYang Nov 17, 2023
695913f
Add test for update_fields special cases
JonoYang Nov 17, 2023
71c3e4a
Handle related models in update_fields
JonoYang Nov 21, 2023
60c5996
Test updating related models using update_fields
JonoYang Nov 21, 2023
db75be0
Update the logic around updated_fields
JonoYang Nov 22, 2023
bd1adf4
Test exceptions
JonoYang Nov 22, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Test exceptions 
Signed-off-by: Jono Yang <jyang@nexb.com>
  • Loading branch information
@JonoYang
JonoYang committed Nov 22, 2023
commit bd1adf4621636d4ea336a4b97da76b18bdca7ce7
14 changes: 14 additions & 0 deletions packagedb/tests/test_models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -431,3 +431,17 @@ def test_packagedb_package_model_update_fields_related_models(self):
self.assertEqual(1, len(p6.history))
history_message = p6.history[0]['message']
self.assertEqual(expected_message, history_message)

def test_packagedb_package_model_update_fields_exceptions(self):
p1 = Package.objects.create(download_url='http://a.a', name='name', version='1.0')
with self.assertRaises(AttributeError):
p1.update_fields(asdf=123)

with self.assertRaises(ValueError):
p1.update_fields(resources=[1])

with self.assertRaises(ValueError):
p1.update_fields(dependencies=[1])

with self.assertRaises(ValueError):
p1.update_fields(parties=[1])