Skip to content

Commit

Permalink
Fix a broken test case after the Django 4.1 update. (#1093)
Browse files Browse the repository at this point in the history
This fixes the CI starting occur on #1086 and following PRs due to the release
of Django 4.1 (https://docs.djangoproject.com/en/4.1/releases/4.1/) which
shipped the change

    # Even if this relation is not to pk, we require still pk value.
    # The wish is that the instance has been already saved to DB,
    # although having a pk value isn't a guarantee of that.
    if self.instance.pk is None:
        raise ValueError(
            f"{instance.__class__.__name__!r} instance needs to have a primary "
            f"key value before this relationship can be used."
        )

in django/django#15318.
  • Loading branch information
PIG208 authored Aug 8, 2022
1 parent 589f315 commit db1edee
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tests/typecheck/fields/test_related.yml
Original file line number Diff line number Diff line change
Expand Up @@ -654,6 +654,7 @@
pass
TransactionManager = BaseManager.from_queryset(TransactionQuerySet)
class Transaction(models.Model):
pk = 0
objects = TransactionManager()
def test(self) -> None:
self.transactionlog_set
Expand Down

0 comments on commit db1edee

Please sign in to comment.