hjwp / hjwp/Book-TDD-Web-Dev-Python

Page 226: `lists/tests/test_model.py`

Offen
#115 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
514
Forks
175
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

It's a bit confusing when you write:
```python
item.save()
item.full_clean()
```
while the name of the tests is `test_cannot_save_empty_list_item`
Because in the end... _the item is saved_. It can be verified by putting the following right after the with block:
```python
self.assertEqual(Item.objects.count(), 0)
```
**Two potential solutions**
- Maybe just swapping the order of the 2 lines might make things less confusing :)

```python
item.full_clean()
item.save()
```

- Another solution would be to simply rename the tests from `test_cannot_save_empty_list_item` to `test_cannot_validate_empty_list_item`
And removing the call to `save()` alltogether.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Öffne lists/tests/test_model.py und untersuche die Tests mit dem Namen test_cannot_save_empty_list_item sowie die Aufrufe item.save() und item.full_clean(). Vergleiche den vorgeschlagenen Ansatz zur Anordnung oder Umbenennung und überprüfe anschließend die gewählte Formulierung und das Verhalten mit der Testsuite; abgeschlossen ist die Aufgabe, wenn der Testname die Validierung oder das Speichern korrekt beschreibt und bestätigt, ob ein Element persistiert wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
testing
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.