123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395139613971398139914001401140214031404140514061407140814091410141114121413141414151416141714181419142014211422142314241425142614271428142914301431143214331434143514361437143814391440144114421443144414451446144714481449145014511452145314541455145614571458145914601461146214631464146514661467146814691470147114721473147414751476147714781479148014811482148314841485148614871488148914901491149214931494149514961497149814991500150115021503150415051506150715081509151015111512151315141515151615171518151915201521152215231524152515261527152815291530153115321533153415351536153715381539154015411542154315441545154615471548154915501551155215531554155515561557155815591560156115621563156415651566156715681569157015711572157315741575157615771578157915801581158215831584158515861587158815891590159115921593159415951596159715981599160016011602160316041605160616071608160916101611161216131614161516161617 |
- from django.core.checks import Error, Warning as DjangoWarning
- from django.db import models
- from django.db.models.fields.related import ForeignObject
- from django.test.testcases import SimpleTestCase, skipIfDBFeature
- from django.test.utils import isolate_apps, override_settings
- @isolate_apps('invalid_models_tests')
- class RelativeFieldTests(SimpleTestCase):
- def test_valid_foreign_key_without_accessor(self):
- class Target(models.Model):
- # There would be a clash if Model.field installed an accessor.
- model = models.IntegerField()
- class Model(models.Model):
- field = models.ForeignKey(Target, models.CASCADE, related_name='+')
- field = Model._meta.get_field('field')
- errors = field.check()
- self.assertEqual(errors, [])
- def test_foreign_key_to_missing_model(self):
- # Model names are resolved when a model is being created, so we cannot
- # test relative fields in isolation and we need to attach them to a
- # model.
- class Model(models.Model):
- foreign_key = models.ForeignKey('Rel1', models.CASCADE)
- field = Model._meta.get_field('foreign_key')
- errors = field.check()
- expected = [
- Error(
- "Field defines a relation with model 'Rel1', "
- "which is either not installed, or is abstract.",
- obj=field,
- id='fields.E300',
- ),
- ]
- self.assertEqual(errors, expected)
- @isolate_apps('invalid_models_tests')
- def test_foreign_key_to_isolate_apps_model(self):
- """
- #25723 - Referenced model registration lookup should be run against the
- field's model registry.
- """
- class OtherModel(models.Model):
- pass
- class Model(models.Model):
- foreign_key = models.ForeignKey('OtherModel', models.CASCADE)
- field = Model._meta.get_field('foreign_key')
- self.assertEqual(field.check(from_model=Model), [])
- def test_many_to_many_to_missing_model(self):
- class Model(models.Model):
- m2m = models.ManyToManyField("Rel2")
- field = Model._meta.get_field('m2m')
- errors = field.check(from_model=Model)
- expected = [
- Error(
- "Field defines a relation with model 'Rel2', "
- "which is either not installed, or is abstract.",
- obj=field,
- id='fields.E300',
- ),
- ]
- self.assertEqual(errors, expected)
- @isolate_apps('invalid_models_tests')
- def test_many_to_many_to_isolate_apps_model(self):
- """
- #25723 - Referenced model registration lookup should be run against the
- field's model registry.
- """
- class OtherModel(models.Model):
- pass
- class Model(models.Model):
- m2m = models.ManyToManyField('OtherModel')
- field = Model._meta.get_field('m2m')
- self.assertEqual(field.check(from_model=Model), [])
- def test_many_to_many_with_limit_choices_auto_created_no_warning(self):
- class Model(models.Model):
- name = models.CharField(max_length=20)
- class ModelM2M(models.Model):
- m2m = models.ManyToManyField(Model, limit_choices_to={'name': 'test_name'})
- self.assertEqual(ModelM2M.check(), [])
- def test_many_to_many_with_useless_options(self):
- class Model(models.Model):
- name = models.CharField(max_length=20)
- class ModelM2M(models.Model):
- m2m = models.ManyToManyField(
- Model,
- null=True,
- validators=[lambda x: x],
- limit_choices_to={'name': 'test_name'},
- through='ThroughModel',
- through_fields=('modelm2m', 'model'),
- )
- class ThroughModel(models.Model):
- model = models.ForeignKey('Model', models.CASCADE)
- modelm2m = models.ForeignKey('ModelM2M', models.CASCADE)
- errors = ModelM2M.check()
- field = ModelM2M._meta.get_field('m2m')
- expected = [
- DjangoWarning(
- 'null has no effect on ManyToManyField.',
- obj=field,
- id='fields.W340',
- ),
- DjangoWarning(
- 'ManyToManyField does not support validators.',
- obj=field,
- id='fields.W341',
- ),
- DjangoWarning(
- 'limit_choices_to has no effect on ManyToManyField '
- 'with a through model.',
- obj=field,
- id='fields.W343',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_ambiguous_relationship_model(self):
- class Person(models.Model):
- pass
- class Group(models.Model):
- field = models.ManyToManyField('Person', through="AmbiguousRelationship", related_name='tertiary')
- class AmbiguousRelationship(models.Model):
- # Too much foreign keys to Person.
- first_person = models.ForeignKey(Person, models.CASCADE, related_name="first")
- second_person = models.ForeignKey(Person, models.CASCADE, related_name="second")
- second_model = models.ForeignKey(Group, models.CASCADE)
- field = Group._meta.get_field('field')
- errors = field.check(from_model=Group)
- expected = [
- Error(
- "The model is used as an intermediate model by "
- "'invalid_models_tests.Group.field', but it has more than one "
- "foreign key to 'Person', which is ambiguous. You must specify "
- "which foreign key Django should use via the through_fields "
- "keyword argument.",
- hint=(
- 'If you want to create a recursive relationship, use '
- 'ForeignKey("self", symmetrical=False, through="AmbiguousRelationship").'
- ),
- obj=field,
- id='fields.E335',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_relationship_model_with_foreign_key_to_wrong_model(self):
- class WrongModel(models.Model):
- pass
- class Person(models.Model):
- pass
- class Group(models.Model):
- members = models.ManyToManyField('Person', through="InvalidRelationship")
- class InvalidRelationship(models.Model):
- person = models.ForeignKey(Person, models.CASCADE)
- wrong_foreign_key = models.ForeignKey(WrongModel, models.CASCADE)
- # The last foreign key should point to Group model.
- field = Group._meta.get_field('members')
- errors = field.check(from_model=Group)
- expected = [
- Error(
- "The model is used as an intermediate model by "
- "'invalid_models_tests.Group.members', but it does not "
- "have a foreign key to 'Group' or 'Person'.",
- obj=InvalidRelationship,
- id='fields.E336',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_relationship_model_missing_foreign_key(self):
- class Person(models.Model):
- pass
- class Group(models.Model):
- members = models.ManyToManyField('Person', through="InvalidRelationship")
- class InvalidRelationship(models.Model):
- group = models.ForeignKey(Group, models.CASCADE)
- # No foreign key to Person
- field = Group._meta.get_field('members')
- errors = field.check(from_model=Group)
- expected = [
- Error(
- "The model is used as an intermediate model by "
- "'invalid_models_tests.Group.members', but it does not have "
- "a foreign key to 'Group' or 'Person'.",
- obj=InvalidRelationship,
- id='fields.E336',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_missing_relationship_model(self):
- class Person(models.Model):
- pass
- class Group(models.Model):
- members = models.ManyToManyField('Person', through="MissingM2MModel")
- field = Group._meta.get_field('members')
- errors = field.check(from_model=Group)
- expected = [
- Error(
- "Field specifies a many-to-many relation through model "
- "'MissingM2MModel', which has not been installed.",
- obj=field,
- id='fields.E331',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_missing_relationship_model_on_model_check(self):
- class Person(models.Model):
- pass
- class Group(models.Model):
- members = models.ManyToManyField('Person', through='MissingM2MModel')
- self.assertEqual(Group.check(), [
- Error(
- "Field specifies a many-to-many relation through model "
- "'MissingM2MModel', which has not been installed.",
- obj=Group._meta.get_field('members'),
- id='fields.E331',
- ),
- ])
- @isolate_apps('invalid_models_tests')
- def test_many_to_many_through_isolate_apps_model(self):
- """
- #25723 - Through model registration lookup should be run against the
- field's model registry.
- """
- class GroupMember(models.Model):
- person = models.ForeignKey('Person', models.CASCADE)
- group = models.ForeignKey('Group', models.CASCADE)
- class Person(models.Model):
- pass
- class Group(models.Model):
- members = models.ManyToManyField('Person', through='GroupMember')
- field = Group._meta.get_field('members')
- self.assertEqual(field.check(from_model=Group), [])
- def test_symmetrical_self_referential_field(self):
- class Person(models.Model):
- # Implicit symmetrical=False.
- friends = models.ManyToManyField('self', through="Relationship")
- class Relationship(models.Model):
- first = models.ForeignKey(Person, models.CASCADE, related_name="rel_from_set")
- second = models.ForeignKey(Person, models.CASCADE, related_name="rel_to_set")
- field = Person._meta.get_field('friends')
- errors = field.check(from_model=Person)
- expected = [
- Error(
- 'Many-to-many fields with intermediate tables must not be symmetrical.',
- obj=field,
- id='fields.E332',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_too_many_foreign_keys_in_self_referential_model(self):
- class Person(models.Model):
- friends = models.ManyToManyField('self', through="InvalidRelationship", symmetrical=False)
- class InvalidRelationship(models.Model):
- first = models.ForeignKey(Person, models.CASCADE, related_name="rel_from_set_2")
- second = models.ForeignKey(Person, models.CASCADE, related_name="rel_to_set_2")
- third = models.ForeignKey(Person, models.CASCADE, related_name="too_many_by_far")
- field = Person._meta.get_field('friends')
- errors = field.check(from_model=Person)
- expected = [
- Error(
- "The model is used as an intermediate model by "
- "'invalid_models_tests.Person.friends', but it has more than two "
- "foreign keys to 'Person', which is ambiguous. You must specify "
- "which two foreign keys Django should use via the through_fields "
- "keyword argument.",
- hint='Use through_fields to specify which two foreign keys Django should use.',
- obj=InvalidRelationship,
- id='fields.E333',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_symmetric_self_reference_with_intermediate_table(self):
- class Person(models.Model):
- # Explicit symmetrical=True.
- friends = models.ManyToManyField('self', through="Relationship", symmetrical=True)
- class Relationship(models.Model):
- first = models.ForeignKey(Person, models.CASCADE, related_name="rel_from_set")
- second = models.ForeignKey(Person, models.CASCADE, related_name="rel_to_set")
- field = Person._meta.get_field('friends')
- errors = field.check(from_model=Person)
- expected = [
- Error(
- 'Many-to-many fields with intermediate tables must not be symmetrical.',
- obj=field,
- id='fields.E332',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_symmetric_self_reference_with_intermediate_table_and_through_fields(self):
- """
- Using through_fields in a m2m with an intermediate model shouldn't
- mask its incompatibility with symmetry.
- """
- class Person(models.Model):
- # Explicit symmetrical=True.
- friends = models.ManyToManyField(
- 'self',
- symmetrical=True,
- through="Relationship",
- through_fields=('first', 'second'),
- )
- class Relationship(models.Model):
- first = models.ForeignKey(Person, models.CASCADE, related_name="rel_from_set")
- second = models.ForeignKey(Person, models.CASCADE, related_name="rel_to_set")
- referee = models.ForeignKey(Person, models.CASCADE, related_name="referred")
- field = Person._meta.get_field('friends')
- errors = field.check(from_model=Person)
- expected = [
- Error(
- 'Many-to-many fields with intermediate tables must not be symmetrical.',
- obj=field,
- id='fields.E332',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_foreign_key_to_abstract_model(self):
- class AbstractModel(models.Model):
- class Meta:
- abstract = True
- class Model(models.Model):
- rel_string_foreign_key = models.ForeignKey('AbstractModel', models.CASCADE)
- rel_class_foreign_key = models.ForeignKey(AbstractModel, models.CASCADE)
- fields = [
- Model._meta.get_field('rel_string_foreign_key'),
- Model._meta.get_field('rel_class_foreign_key'),
- ]
- expected_error = Error(
- "Field defines a relation with model 'AbstractModel', "
- "which is either not installed, or is abstract.",
- id='fields.E300',
- )
- for field in fields:
- expected_error.obj = field
- errors = field.check()
- self.assertEqual(errors, [expected_error])
- def test_m2m_to_abstract_model(self):
- class AbstractModel(models.Model):
- class Meta:
- abstract = True
- class Model(models.Model):
- rel_string_m2m = models.ManyToManyField('AbstractModel')
- rel_class_m2m = models.ManyToManyField(AbstractModel)
- fields = [
- Model._meta.get_field('rel_string_m2m'),
- Model._meta.get_field('rel_class_m2m'),
- ]
- expected_error = Error(
- "Field defines a relation with model 'AbstractModel', "
- "which is either not installed, or is abstract.",
- id='fields.E300',
- )
- for field in fields:
- expected_error.obj = field
- errors = field.check(from_model=Model)
- self.assertEqual(errors, [expected_error])
- def test_unique_m2m(self):
- class Person(models.Model):
- name = models.CharField(max_length=5)
- class Group(models.Model):
- members = models.ManyToManyField('Person', unique=True)
- field = Group._meta.get_field('members')
- errors = field.check(from_model=Group)
- expected = [
- Error(
- 'ManyToManyFields cannot be unique.',
- obj=field,
- id='fields.E330',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_foreign_key_to_non_unique_field(self):
- class Target(models.Model):
- bad = models.IntegerField() # No unique=True
- class Model(models.Model):
- foreign_key = models.ForeignKey('Target', models.CASCADE, to_field='bad')
- field = Model._meta.get_field('foreign_key')
- errors = field.check()
- expected = [
- Error(
- "'Target.bad' must set unique=True because it is referenced by a foreign key.",
- obj=field,
- id='fields.E311',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_foreign_key_to_non_unique_field_under_explicit_model(self):
- class Target(models.Model):
- bad = models.IntegerField()
- class Model(models.Model):
- field = models.ForeignKey(Target, models.CASCADE, to_field='bad')
- field = Model._meta.get_field('field')
- errors = field.check()
- expected = [
- Error(
- "'Target.bad' must set unique=True because it is referenced by a foreign key.",
- obj=field,
- id='fields.E311',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_foreign_object_to_non_unique_fields(self):
- class Person(models.Model):
- # Note that both fields are not unique.
- country_id = models.IntegerField()
- city_id = models.IntegerField()
- class MMembership(models.Model):
- person_country_id = models.IntegerField()
- person_city_id = models.IntegerField()
- person = models.ForeignObject(
- Person,
- on_delete=models.CASCADE,
- from_fields=['person_country_id', 'person_city_id'],
- to_fields=['country_id', 'city_id'],
- )
- field = MMembership._meta.get_field('person')
- errors = field.check()
- expected = [
- Error(
- "No subset of the fields 'country_id', 'city_id' on model 'Person' is unique.",
- hint=(
- "Add unique=True on any of those fields or add at least "
- "a subset of them to a unique_together constraint."
- ),
- obj=field,
- id='fields.E310',
- )
- ]
- self.assertEqual(errors, expected)
- def test_on_delete_set_null_on_non_nullable_field(self):
- class Person(models.Model):
- pass
- class Model(models.Model):
- foreign_key = models.ForeignKey('Person', models.SET_NULL)
- field = Model._meta.get_field('foreign_key')
- errors = field.check()
- expected = [
- Error(
- 'Field specifies on_delete=SET_NULL, but cannot be null.',
- hint='Set null=True argument on the field, or change the on_delete rule.',
- obj=field,
- id='fields.E320',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_on_delete_set_default_without_default_value(self):
- class Person(models.Model):
- pass
- class Model(models.Model):
- foreign_key = models.ForeignKey('Person', models.SET_DEFAULT)
- field = Model._meta.get_field('foreign_key')
- errors = field.check()
- expected = [
- Error(
- 'Field specifies on_delete=SET_DEFAULT, but has no default value.',
- hint='Set a default value, or change the on_delete rule.',
- obj=field,
- id='fields.E321',
- ),
- ]
- self.assertEqual(errors, expected)
- @skipIfDBFeature('interprets_empty_strings_as_nulls')
- def test_nullable_primary_key(self):
- class Model(models.Model):
- field = models.IntegerField(primary_key=True, null=True)
- field = Model._meta.get_field('field')
- errors = field.check()
- expected = [
- Error(
- 'Primary keys must not have null=True.',
- hint='Set null=False on the field, or remove primary_key=True argument.',
- obj=field,
- id='fields.E007',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_not_swapped_model(self):
- class SwappableModel(models.Model):
- # A model that can be, but isn't swapped out. References to this
- # model should *not* raise any validation error.
- class Meta:
- swappable = 'TEST_SWAPPABLE_MODEL'
- class Model(models.Model):
- explicit_fk = models.ForeignKey(
- SwappableModel,
- models.CASCADE,
- related_name='explicit_fk',
- )
- implicit_fk = models.ForeignKey(
- 'invalid_models_tests.SwappableModel',
- models.CASCADE,
- related_name='implicit_fk',
- )
- explicit_m2m = models.ManyToManyField(SwappableModel, related_name='explicit_m2m')
- implicit_m2m = models.ManyToManyField(
- 'invalid_models_tests.SwappableModel',
- related_name='implicit_m2m',
- )
- explicit_fk = Model._meta.get_field('explicit_fk')
- self.assertEqual(explicit_fk.check(), [])
- implicit_fk = Model._meta.get_field('implicit_fk')
- self.assertEqual(implicit_fk.check(), [])
- explicit_m2m = Model._meta.get_field('explicit_m2m')
- self.assertEqual(explicit_m2m.check(from_model=Model), [])
- implicit_m2m = Model._meta.get_field('implicit_m2m')
- self.assertEqual(implicit_m2m.check(from_model=Model), [])
- @override_settings(TEST_SWAPPED_MODEL='invalid_models_tests.Replacement')
- def test_referencing_to_swapped_model(self):
- class Replacement(models.Model):
- pass
- class SwappedModel(models.Model):
- class Meta:
- swappable = 'TEST_SWAPPED_MODEL'
- class Model(models.Model):
- explicit_fk = models.ForeignKey(
- SwappedModel,
- models.CASCADE,
- related_name='explicit_fk',
- )
- implicit_fk = models.ForeignKey(
- 'invalid_models_tests.SwappedModel',
- models.CASCADE,
- related_name='implicit_fk',
- )
- explicit_m2m = models.ManyToManyField(SwappedModel, related_name='explicit_m2m')
- implicit_m2m = models.ManyToManyField(
- 'invalid_models_tests.SwappedModel',
- related_name='implicit_m2m',
- )
- fields = [
- Model._meta.get_field('explicit_fk'),
- Model._meta.get_field('implicit_fk'),
- Model._meta.get_field('explicit_m2m'),
- Model._meta.get_field('implicit_m2m'),
- ]
- expected_error = Error(
- ("Field defines a relation with the model "
- "'invalid_models_tests.SwappedModel', which has been swapped out."),
- hint="Update the relation to point at 'settings.TEST_SWAPPED_MODEL'.",
- id='fields.E301',
- )
- for field in fields:
- expected_error.obj = field
- errors = field.check(from_model=Model)
- self.assertEqual(errors, [expected_error])
- def test_related_field_has_invalid_related_name(self):
- digit = 0
- illegal_non_alphanumeric = '!'
- whitespace = '\t'
- invalid_related_names = [
- '%s_begins_with_digit' % digit,
- '%s_begins_with_illegal_non_alphanumeric' % illegal_non_alphanumeric,
- '%s_begins_with_whitespace' % whitespace,
- 'contains_%s_illegal_non_alphanumeric' % illegal_non_alphanumeric,
- 'contains_%s_whitespace' % whitespace,
- 'ends_with_with_illegal_non_alphanumeric_%s' % illegal_non_alphanumeric,
- 'ends_with_whitespace_%s' % whitespace,
- 'with', # a Python keyword
- 'related_name\n',
- '',
- ',', # non-ASCII
- ]
- class Parent(models.Model):
- pass
- for invalid_related_name in invalid_related_names:
- Child = type('Child%s' % invalid_related_name, (models.Model,), {
- 'parent': models.ForeignKey('Parent', models.CASCADE, related_name=invalid_related_name),
- '__module__': Parent.__module__,
- })
- field = Child._meta.get_field('parent')
- errors = Child.check()
- expected = [
- Error(
- "The name '%s' is invalid related_name for field Child%s.parent"
- % (invalid_related_name, invalid_related_name),
- hint="Related name must be a valid Python identifier or end with a '+'",
- obj=field,
- id='fields.E306',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_related_field_has_valid_related_name(self):
- lowercase = 'a'
- uppercase = 'A'
- digit = 0
- related_names = [
- '%s_starts_with_lowercase' % lowercase,
- '%s_tarts_with_uppercase' % uppercase,
- '_starts_with_underscore',
- 'contains_%s_digit' % digit,
- 'ends_with_plus+',
- '_+',
- '+',
- '試',
- '試驗+',
- ]
- class Parent(models.Model):
- pass
- for related_name in related_names:
- Child = type('Child%s' % related_name, (models.Model,), {
- 'parent': models.ForeignKey('Parent', models.CASCADE, related_name=related_name),
- '__module__': Parent.__module__,
- })
- errors = Child.check()
- self.assertFalse(errors)
- def test_to_fields_exist(self):
- class Parent(models.Model):
- pass
- class Child(models.Model):
- a = models.PositiveIntegerField()
- b = models.PositiveIntegerField()
- parent = ForeignObject(
- Parent,
- on_delete=models.SET_NULL,
- from_fields=('a', 'b'),
- to_fields=('a', 'b'),
- )
- field = Child._meta.get_field('parent')
- expected = [
- Error(
- "The to_field 'a' doesn't exist on the related model 'invalid_models_tests.Parent'.",
- obj=field,
- id='fields.E312',
- ),
- Error(
- "The to_field 'b' doesn't exist on the related model 'invalid_models_tests.Parent'.",
- obj=field,
- id='fields.E312',
- ),
- ]
- self.assertEqual(field.check(), expected)
- def test_to_fields_not_checked_if_related_model_doesnt_exist(self):
- class Child(models.Model):
- a = models.PositiveIntegerField()
- b = models.PositiveIntegerField()
- parent = ForeignObject(
- 'invalid_models_tests.Parent',
- on_delete=models.SET_NULL,
- from_fields=('a', 'b'),
- to_fields=('a', 'b'),
- )
- field = Child._meta.get_field('parent')
- self.assertEqual(field.check(), [
- Error(
- "Field defines a relation with model 'invalid_models_tests.Parent', "
- "which is either not installed, or is abstract.",
- id='fields.E300',
- obj=field,
- ),
- ])
- def test_invalid_related_query_name(self):
- class Target(models.Model):
- pass
- class Model(models.Model):
- first = models.ForeignKey(Target, models.CASCADE, related_name='contains__double')
- second = models.ForeignKey(Target, models.CASCADE, related_query_name='ends_underscore_')
- self.assertEqual(Model.check(), [
- Error(
- "Reverse query name 'contains__double' must not contain '__'.",
- hint=("Add or change a related_name or related_query_name "
- "argument for this field."),
- obj=Model._meta.get_field('first'),
- id='fields.E309',
- ),
- Error(
- "Reverse query name 'ends_underscore_' must not end with an "
- "underscore.",
- hint=("Add or change a related_name or related_query_name "
- "argument for this field."),
- obj=Model._meta.get_field('second'),
- id='fields.E308',
- ),
- ])
- @isolate_apps('invalid_models_tests')
- class AccessorClashTests(SimpleTestCase):
- def test_fk_to_integer(self):
- self._test_accessor_clash(
- target=models.IntegerField(),
- relative=models.ForeignKey('Target', models.CASCADE))
- def test_fk_to_fk(self):
- self._test_accessor_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ForeignKey('Target', models.CASCADE))
- def test_fk_to_m2m(self):
- self._test_accessor_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ForeignKey('Target', models.CASCADE))
- def test_m2m_to_integer(self):
- self._test_accessor_clash(
- target=models.IntegerField(),
- relative=models.ManyToManyField('Target'))
- def test_m2m_to_fk(self):
- self._test_accessor_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ManyToManyField('Target'))
- def test_m2m_to_m2m(self):
- self._test_accessor_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ManyToManyField('Target'))
- def _test_accessor_clash(self, target, relative):
- class Another(models.Model):
- pass
- class Target(models.Model):
- model_set = target
- class Model(models.Model):
- rel = relative
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.rel' clashes with field name 'Target.model_set'.",
- hint=("Rename field 'Target.model_set', or add/change "
- "a related_name argument to the definition "
- "for field 'Model.rel'."),
- obj=Model._meta.get_field('rel'),
- id='fields.E302',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_clash_between_accessors(self):
- class Target(models.Model):
- pass
- class Model(models.Model):
- foreign = models.ForeignKey(Target, models.CASCADE)
- m2m = models.ManyToManyField(Target)
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.foreign' clashes with reverse accessor for 'Model.m2m'.",
- hint=(
- "Add or change a related_name argument to the definition "
- "for 'Model.foreign' or 'Model.m2m'."
- ),
- obj=Model._meta.get_field('foreign'),
- id='fields.E304',
- ),
- Error(
- "Reverse accessor for 'Model.m2m' clashes with reverse accessor for 'Model.foreign'.",
- hint=(
- "Add or change a related_name argument to the definition "
- "for 'Model.m2m' or 'Model.foreign'."
- ),
- obj=Model._meta.get_field('m2m'),
- id='fields.E304',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_m2m_to_m2m_with_inheritance(self):
- """ Ref #22047. """
- class Target(models.Model):
- pass
- class Model(models.Model):
- children = models.ManyToManyField('Child', related_name="m2m_clash", related_query_name="no_clash")
- class Parent(models.Model):
- m2m_clash = models.ManyToManyField('Target')
- class Child(Parent):
- pass
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.children' clashes with field name 'Child.m2m_clash'.",
- hint=(
- "Rename field 'Child.m2m_clash', or add/change a related_name "
- "argument to the definition for field 'Model.children'."
- ),
- obj=Model._meta.get_field('children'),
- id='fields.E302',
- )
- ]
- self.assertEqual(errors, expected)
- def test_no_clash_for_hidden_related_name(self):
- class Stub(models.Model):
- pass
- class ManyToManyRel(models.Model):
- thing1 = models.ManyToManyField(Stub, related_name='+')
- thing2 = models.ManyToManyField(Stub, related_name='+')
- class FKRel(models.Model):
- thing1 = models.ForeignKey(Stub, models.CASCADE, related_name='+')
- thing2 = models.ForeignKey(Stub, models.CASCADE, related_name='+')
- self.assertEqual(ManyToManyRel.check(), [])
- self.assertEqual(FKRel.check(), [])
- @isolate_apps('invalid_models_tests')
- class ReverseQueryNameClashTests(SimpleTestCase):
- def test_fk_to_integer(self):
- self._test_reverse_query_name_clash(
- target=models.IntegerField(),
- relative=models.ForeignKey('Target', models.CASCADE))
- def test_fk_to_fk(self):
- self._test_reverse_query_name_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ForeignKey('Target', models.CASCADE))
- def test_fk_to_m2m(self):
- self._test_reverse_query_name_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ForeignKey('Target', models.CASCADE))
- def test_m2m_to_integer(self):
- self._test_reverse_query_name_clash(
- target=models.IntegerField(),
- relative=models.ManyToManyField('Target'))
- def test_m2m_to_fk(self):
- self._test_reverse_query_name_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ManyToManyField('Target'))
- def test_m2m_to_m2m(self):
- self._test_reverse_query_name_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ManyToManyField('Target'))
- def _test_reverse_query_name_clash(self, target, relative):
- class Another(models.Model):
- pass
- class Target(models.Model):
- model = target
- class Model(models.Model):
- rel = relative
- errors = Model.check()
- expected = [
- Error(
- "Reverse query name for 'Model.rel' clashes with field name 'Target.model'.",
- hint=(
- "Rename field 'Target.model', or add/change a related_name "
- "argument to the definition for field 'Model.rel'."
- ),
- obj=Model._meta.get_field('rel'),
- id='fields.E303',
- ),
- ]
- self.assertEqual(errors, expected)
- @isolate_apps('invalid_models_tests')
- class ExplicitRelatedNameClashTests(SimpleTestCase):
- def test_fk_to_integer(self):
- self._test_explicit_related_name_clash(
- target=models.IntegerField(),
- relative=models.ForeignKey('Target', models.CASCADE, related_name='clash'))
- def test_fk_to_fk(self):
- self._test_explicit_related_name_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ForeignKey('Target', models.CASCADE, related_name='clash'))
- def test_fk_to_m2m(self):
- self._test_explicit_related_name_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ForeignKey('Target', models.CASCADE, related_name='clash'))
- def test_m2m_to_integer(self):
- self._test_explicit_related_name_clash(
- target=models.IntegerField(),
- relative=models.ManyToManyField('Target', related_name='clash'))
- def test_m2m_to_fk(self):
- self._test_explicit_related_name_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ManyToManyField('Target', related_name='clash'))
- def test_m2m_to_m2m(self):
- self._test_explicit_related_name_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ManyToManyField('Target', related_name='clash'))
- def _test_explicit_related_name_clash(self, target, relative):
- class Another(models.Model):
- pass
- class Target(models.Model):
- clash = target
- class Model(models.Model):
- rel = relative
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.rel' clashes with field name 'Target.clash'.",
- hint=(
- "Rename field 'Target.clash', or add/change a related_name "
- "argument to the definition for field 'Model.rel'."
- ),
- obj=Model._meta.get_field('rel'),
- id='fields.E302',
- ),
- Error(
- "Reverse query name for 'Model.rel' clashes with field name 'Target.clash'.",
- hint=(
- "Rename field 'Target.clash', or add/change a related_name "
- "argument to the definition for field 'Model.rel'."
- ),
- obj=Model._meta.get_field('rel'),
- id='fields.E303',
- ),
- ]
- self.assertEqual(errors, expected)
- @isolate_apps('invalid_models_tests')
- class ExplicitRelatedQueryNameClashTests(SimpleTestCase):
- def test_fk_to_integer(self, related_name=None):
- self._test_explicit_related_query_name_clash(
- target=models.IntegerField(),
- relative=models.ForeignKey(
- 'Target',
- models.CASCADE,
- related_name=related_name,
- related_query_name='clash',
- )
- )
- def test_hidden_fk_to_integer(self, related_name=None):
- self.test_fk_to_integer(related_name='+')
- def test_fk_to_fk(self, related_name=None):
- self._test_explicit_related_query_name_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ForeignKey(
- 'Target',
- models.CASCADE,
- related_name=related_name,
- related_query_name='clash',
- )
- )
- def test_hidden_fk_to_fk(self):
- self.test_fk_to_fk(related_name='+')
- def test_fk_to_m2m(self, related_name=None):
- self._test_explicit_related_query_name_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ForeignKey(
- 'Target',
- models.CASCADE,
- related_name=related_name,
- related_query_name='clash',
- )
- )
- def test_hidden_fk_to_m2m(self):
- self.test_fk_to_m2m(related_name='+')
- def test_m2m_to_integer(self, related_name=None):
- self._test_explicit_related_query_name_clash(
- target=models.IntegerField(),
- relative=models.ManyToManyField('Target', related_name=related_name, related_query_name='clash'))
- def test_hidden_m2m_to_integer(self):
- self.test_m2m_to_integer(related_name='+')
- def test_m2m_to_fk(self, related_name=None):
- self._test_explicit_related_query_name_clash(
- target=models.ForeignKey('Another', models.CASCADE),
- relative=models.ManyToManyField('Target', related_name=related_name, related_query_name='clash'))
- def test_hidden_m2m_to_fk(self):
- self.test_m2m_to_fk(related_name='+')
- def test_m2m_to_m2m(self, related_name=None):
- self._test_explicit_related_query_name_clash(
- target=models.ManyToManyField('Another'),
- relative=models.ManyToManyField(
- 'Target',
- related_name=related_name,
- related_query_name='clash',
- )
- )
- def test_hidden_m2m_to_m2m(self):
- self.test_m2m_to_m2m(related_name='+')
- def _test_explicit_related_query_name_clash(self, target, relative):
- class Another(models.Model):
- pass
- class Target(models.Model):
- clash = target
- class Model(models.Model):
- rel = relative
- errors = Model.check()
- expected = [
- Error(
- "Reverse query name for 'Model.rel' clashes with field name 'Target.clash'.",
- hint=(
- "Rename field 'Target.clash', or add/change a related_name "
- "argument to the definition for field 'Model.rel'."
- ),
- obj=Model._meta.get_field('rel'),
- id='fields.E303',
- ),
- ]
- self.assertEqual(errors, expected)
- @isolate_apps('invalid_models_tests')
- class SelfReferentialM2MClashTests(SimpleTestCase):
- def test_clash_between_accessors(self):
- class Model(models.Model):
- first_m2m = models.ManyToManyField('self', symmetrical=False)
- second_m2m = models.ManyToManyField('self', symmetrical=False)
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.first_m2m' clashes with reverse accessor for 'Model.second_m2m'.",
- hint=(
- "Add or change a related_name argument to the definition "
- "for 'Model.first_m2m' or 'Model.second_m2m'."
- ),
- obj=Model._meta.get_field('first_m2m'),
- id='fields.E304',
- ),
- Error(
- "Reverse accessor for 'Model.second_m2m' clashes with reverse accessor for 'Model.first_m2m'.",
- hint=(
- "Add or change a related_name argument to the definition "
- "for 'Model.second_m2m' or 'Model.first_m2m'."
- ),
- obj=Model._meta.get_field('second_m2m'),
- id='fields.E304',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_accessor_clash(self):
- class Model(models.Model):
- model_set = models.ManyToManyField("self", symmetrical=False)
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.model_set' clashes with field name 'Model.model_set'.",
- hint=(
- "Rename field 'Model.model_set', or add/change a related_name "
- "argument to the definition for field 'Model.model_set'."
- ),
- obj=Model._meta.get_field('model_set'),
- id='fields.E302',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_reverse_query_name_clash(self):
- class Model(models.Model):
- model = models.ManyToManyField("self", symmetrical=False)
- errors = Model.check()
- expected = [
- Error(
- "Reverse query name for 'Model.model' clashes with field name 'Model.model'.",
- hint=(
- "Rename field 'Model.model', or add/change a related_name "
- "argument to the definition for field 'Model.model'."
- ),
- obj=Model._meta.get_field('model'),
- id='fields.E303',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_clash_under_explicit_related_name(self):
- class Model(models.Model):
- clash = models.IntegerField()
- m2m = models.ManyToManyField("self", symmetrical=False, related_name='clash')
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.m2m' clashes with field name 'Model.clash'.",
- hint=(
- "Rename field 'Model.clash', or add/change a related_name "
- "argument to the definition for field 'Model.m2m'."
- ),
- obj=Model._meta.get_field('m2m'),
- id='fields.E302',
- ),
- Error(
- "Reverse query name for 'Model.m2m' clashes with field name 'Model.clash'.",
- hint=(
- "Rename field 'Model.clash', or add/change a related_name "
- "argument to the definition for field 'Model.m2m'."
- ),
- obj=Model._meta.get_field('m2m'),
- id='fields.E303',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_valid_model(self):
- class Model(models.Model):
- first = models.ManyToManyField("self", symmetrical=False, related_name='first_accessor')
- second = models.ManyToManyField("self", symmetrical=False, related_name='second_accessor')
- errors = Model.check()
- self.assertEqual(errors, [])
- @isolate_apps('invalid_models_tests')
- class SelfReferentialFKClashTests(SimpleTestCase):
- def test_accessor_clash(self):
- class Model(models.Model):
- model_set = models.ForeignKey("Model", models.CASCADE)
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.model_set' clashes with field name 'Model.model_set'.",
- hint=(
- "Rename field 'Model.model_set', or add/change "
- "a related_name argument to the definition "
- "for field 'Model.model_set'."
- ),
- obj=Model._meta.get_field('model_set'),
- id='fields.E302',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_reverse_query_name_clash(self):
- class Model(models.Model):
- model = models.ForeignKey("Model", models.CASCADE)
- errors = Model.check()
- expected = [
- Error(
- "Reverse query name for 'Model.model' clashes with field name 'Model.model'.",
- hint=(
- "Rename field 'Model.model', or add/change a related_name "
- "argument to the definition for field 'Model.model'."
- ),
- obj=Model._meta.get_field('model'),
- id='fields.E303',
- ),
- ]
- self.assertEqual(errors, expected)
- def test_clash_under_explicit_related_name(self):
- class Model(models.Model):
- clash = models.CharField(max_length=10)
- foreign = models.ForeignKey("Model", models.CASCADE, related_name='clash')
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.foreign' clashes with field name 'Model.clash'.",
- hint=(
- "Rename field 'Model.clash', or add/change a related_name "
- "argument to the definition for field 'Model.foreign'."
- ),
- obj=Model._meta.get_field('foreign'),
- id='fields.E302',
- ),
- Error(
- "Reverse query name for 'Model.foreign' clashes with field name 'Model.clash'.",
- hint=(
- "Rename field 'Model.clash', or add/change a related_name "
- "argument to the definition for field 'Model.foreign'."
- ),
- obj=Model._meta.get_field('foreign'),
- id='fields.E303',
- ),
- ]
- self.assertEqual(errors, expected)
- @isolate_apps('invalid_models_tests')
- class ComplexClashTests(SimpleTestCase):
- # New tests should not be included here, because this is a single,
- # self-contained sanity check, not a test of everything.
- def test_complex_clash(self):
- class Target(models.Model):
- tgt_safe = models.CharField(max_length=10)
- clash = models.CharField(max_length=10)
- model = models.CharField(max_length=10)
- clash1_set = models.CharField(max_length=10)
- class Model(models.Model):
- src_safe = models.CharField(max_length=10)
- foreign_1 = models.ForeignKey(Target, models.CASCADE, related_name='id')
- foreign_2 = models.ForeignKey(Target, models.CASCADE, related_name='src_safe')
- m2m_1 = models.ManyToManyField(Target, related_name='id')
- m2m_2 = models.ManyToManyField(Target, related_name='src_safe')
- errors = Model.check()
- expected = [
- Error(
- "Reverse accessor for 'Model.foreign_1' clashes with field name 'Target.id'.",
- hint=("Rename field 'Target.id', or add/change a related_name "
- "argument to the definition for field 'Model.foreign_1'."),
- obj=Model._meta.get_field('foreign_1'),
- id='fields.E302',
- ),
- Error(
- "Reverse query name for 'Model.foreign_1' clashes with field name 'Target.id'.",
- hint=("Rename field 'Target.id', or add/change a related_name "
- "argument to the definition for field 'Model.foreign_1'."),
- obj=Model._meta.get_field('foreign_1'),
- id='fields.E303',
- ),
- Error(
- "Reverse accessor for 'Model.foreign_1' clashes with reverse accessor for 'Model.m2m_1'.",
- hint=("Add or change a related_name argument to "
- "the definition for 'Model.foreign_1' or 'Model.m2m_1'."),
- obj=Model._meta.get_field('foreign_1'),
- id='fields.E304',
- ),
- Error(
- "Reverse query name for 'Model.foreign_1' clashes with reverse query name for 'Model.m2m_1'.",
- hint=("Add or change a related_name argument to "
- "the definition for 'Model.foreign_1' or 'Model.m2m_1'."),
- obj=Model._meta.get_field('foreign_1'),
- id='fields.E305',
- ),
- Error(
- "Reverse accessor for 'Model.foreign_2' clashes with reverse accessor for 'Model.m2m_2'.",
- hint=("Add or change a related_name argument "
- "to the definition for 'Model.foreign_2' or 'Model.m2m_2'."),
- obj=Model._meta.get_field('foreign_2'),
- id='fields.E304',
- ),
- Error(
- "Reverse query name for 'Model.foreign_2' clashes with reverse query name for 'Model.m2m_2'.",
- hint=("Add or change a related_name argument to "
- "the definition for 'Model.foreign_2' or 'Model.m2m_2'."),
- obj=Model._meta.get_field('foreign_2'),
- id='fields.E305',
- ),
- Error(
- "Reverse accessor for 'Model.m2m_1' clashes with field name 'Target.id'.",
- hint=("Rename field 'Target.id', or add/change a related_name "
- "argument to the definition for field 'Model.m2m_1'."),
- obj=Model._meta.get_field('m2m_1'),
- id='fields.E302',
- ),
- Error(
- "Reverse query name for 'Model.m2m_1' clashes with field name 'Target.id'.",
- hint=("Rename field 'Target.id', or add/change a related_name "
- "argument to the definition for field 'Model.m2m_1'."),
- obj=Model._meta.get_field('m2m_1'),
- id='fields.E303',
- ),
- Error(
- "Reverse accessor for 'Model.m2m_1' clashes with reverse accessor for 'Model.foreign_1'.",
- hint=("Add or change a related_name argument to the definition "
- "for 'Model.m2m_1' or 'Model.foreign_1'."),
- obj=Model._meta.get_field('m2m_1'),
- id='fields.E304',
- ),
- Error(
- "Reverse query name for 'Model.m2m_1' clashes with reverse query name for 'Model.foreign_1'.",
- hint=("Add or change a related_name argument to "
- "the definition for 'Model.m2m_1' or 'Model.foreign_1'."),
- obj=Model._meta.get_field('m2m_1'),
- id='fields.E305',
- ),
- Error(
- "Reverse accessor for 'Model.m2m_2' clashes with reverse accessor for 'Model.foreign_2'.",
- hint=("Add or change a related_name argument to the definition "
- "for 'Model.m2m_2' or 'Model.foreign_2'."),
- obj=Model._meta.get_field('m2m_2'),
- id='fields.E304',
- ),
- Error(
- "Reverse query name for 'Model.m2m_2' clashes with reverse query name for 'Model.foreign_2'.",
- hint=("Add or change a related_name argument to the definition "
- "for 'Model.m2m_2' or 'Model.foreign_2'."),
- obj=Model._meta.get_field('m2m_2'),
- id='fields.E305',
- ),
- ]
- self.assertEqual(errors, expected)
- @isolate_apps('invalid_models_tests')
- class M2mThroughFieldsTests(SimpleTestCase):
- def test_m2m_field_argument_validation(self):
- """
- ManyToManyField accepts the ``through_fields`` kwarg
- only if an intermediary table is specified.
- """
- class Fan(models.Model):
- pass
- with self.assertRaisesMessage(ValueError, 'Cannot specify through_fields without a through model'):
- models.ManyToManyField(Fan, through_fields=('f1', 'f2'))
- def test_invalid_order(self):
- """
- Mixing up the order of link fields to ManyToManyField.through_fields
- triggers validation errors.
- """
- class Fan(models.Model):
- pass
- class Event(models.Model):
- invitees = models.ManyToManyField(Fan, through='Invitation', through_fields=('invitee', 'event'))
- class Invitation(models.Model):
- event = models.ForeignKey(Event, models.CASCADE)
- invitee = models.ForeignKey(Fan, models.CASCADE)
- inviter = models.ForeignKey(Fan, models.CASCADE, related_name='+')
- field = Event._meta.get_field('invitees')
- errors = field.check(from_model=Event)
- expected = [
- Error(
- "'Invitation.invitee' is not a foreign key to 'Event'.",
- hint="Did you mean one of the following foreign keys to 'Event': event?",
- obj=field,
- id='fields.E339',
- ),
- Error(
- "'Invitation.event' is not a foreign key to 'Fan'.",
- hint="Did you mean one of the following foreign keys to 'Fan': invitee, inviter?",
- obj=field,
- id='fields.E339',
- ),
- ]
- self.assertEqual(expected, errors)
- def test_invalid_field(self):
- """
- Providing invalid field names to ManyToManyField.through_fields
- triggers validation errors.
- """
- class Fan(models.Model):
- pass
- class Event(models.Model):
- invitees = models.ManyToManyField(
- Fan,
- through='Invitation',
- through_fields=('invalid_field_1', 'invalid_field_2'),
- )
- class Invitation(models.Model):
- event = models.ForeignKey(Event, models.CASCADE)
- invitee = models.ForeignKey(Fan, models.CASCADE)
- inviter = models.ForeignKey(Fan, models.CASCADE, related_name='+')
- field = Event._meta.get_field('invitees')
- errors = field.check(from_model=Event)
- expected = [
- Error(
- "The intermediary model 'invalid_models_tests.Invitation' has no field 'invalid_field_1'.",
- hint="Did you mean one of the following foreign keys to 'Event': event?",
- obj=field,
- id='fields.E338',
- ),
- Error(
- "The intermediary model 'invalid_models_tests.Invitation' has no field 'invalid_field_2'.",
- hint="Did you mean one of the following foreign keys to 'Fan': invitee, inviter?",
- obj=field,
- id='fields.E338',
- ),
- ]
- self.assertEqual(expected, errors)
- def test_explicit_field_names(self):
- """
- If ``through_fields`` kwarg is given, it must specify both
- link fields of the intermediary table.
- """
- class Fan(models.Model):
- pass
- class Event(models.Model):
- invitees = models.ManyToManyField(Fan, through='Invitation', through_fields=(None, 'invitee'))
- class Invitation(models.Model):
- event = models.ForeignKey(Event, models.CASCADE)
- invitee = models.ForeignKey(Fan, models.CASCADE)
- inviter = models.ForeignKey(Fan, models.CASCADE, related_name='+')
- field = Event._meta.get_field('invitees')
- errors = field.check(from_model=Event)
- expected = [
- Error(
- "Field specifies 'through_fields' but does not provide the names "
- "of the two link fields that should be used for the relation "
- "through model 'invalid_models_tests.Invitation'.",
- hint="Make sure you specify 'through_fields' as through_fields=('field1', 'field2')",
- obj=field,
- id='fields.E337')]
- self.assertEqual(expected, errors)
- def test_superset_foreign_object(self):
- class Parent(models.Model):
- a = models.PositiveIntegerField()
- b = models.PositiveIntegerField()
- c = models.PositiveIntegerField()
- class Meta:
- unique_together = (('a', 'b', 'c'),)
- class Child(models.Model):
- a = models.PositiveIntegerField()
- b = models.PositiveIntegerField()
- value = models.CharField(max_length=255)
- parent = ForeignObject(
- Parent,
- on_delete=models.SET_NULL,
- from_fields=('a', 'b'),
- to_fields=('a', 'b'),
- related_name='children',
- )
- field = Child._meta.get_field('parent')
- errors = field.check(from_model=Child)
- expected = [
- Error(
- "No subset of the fields 'a', 'b' on model 'Parent' is unique.",
- hint=(
- "Add unique=True on any of those fields or add at least "
- "a subset of them to a unique_together constraint."
- ),
- obj=field,
- id='fields.E310',
- ),
- ]
- self.assertEqual(expected, errors)
- def test_intersection_foreign_object(self):
- class Parent(models.Model):
- a = models.PositiveIntegerField()
- b = models.PositiveIntegerField()
- c = models.PositiveIntegerField()
- d = models.PositiveIntegerField()
- class Meta:
- unique_together = (('a', 'b', 'c'),)
- class Child(models.Model):
- a = models.PositiveIntegerField()
- b = models.PositiveIntegerField()
- d = models.PositiveIntegerField()
- value = models.CharField(max_length=255)
- parent = ForeignObject(
- Parent,
- on_delete=models.SET_NULL,
- from_fields=('a', 'b', 'd'),
- to_fields=('a', 'b', 'd'),
- related_name='children',
- )
- field = Child._meta.get_field('parent')
- errors = field.check(from_model=Child)
- expected = [
- Error(
- "No subset of the fields 'a', 'b', 'd' on model 'Parent' is unique.",
- hint=(
- "Add unique=True on any of those fields or add at least "
- "a subset of them to a unique_together constraint."
- ),
- obj=field,
- id='fields.E310',
- ),
- ]
- self.assertEqual(expected, errors)
|