Tests #24
@@ -13,26 +13,69 @@ from django.contrib.auth.models import User
|
|||||||
|
|
|||||||
from backend.api import router
|
from backend.api import router
|
||||||
from collections import OrderedDict
|
from collections import OrderedDict
|
||||||
|
|
||||||
class EasyTest(APITestCase):
|
class EasyTest1(APITestCase):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
def get_todo(self, expected_titles):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
def test_list(self):
|
response = self.client.get(self.url, {}, format='json')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
"""
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
The first test.
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
"""
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
user = User.objects.create_user('test_user', 'test@test.com', 'test_password')
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
url = reverse('ToDoLists-list')
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
self.client.force_authenticate(user=user)
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
response = self.client.get(url, {}, format='json')
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
self.assertEqual(response.status_code, status.HTTP_200_OK)
|
self.assertEqual(response.status_code, status.HTTP_200_OK)
|
||||||
self.assertEqual(response.data, OrderedDict([('count', 0), ('next', None), ('previous', None), ('results', [])]))
|
real_titles = [data['title'] for data in response.data['results']]
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.assertEqual((response.data['count'], real_titles), \
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
(len(expected_titles), expected_titles))
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
response = self.client.post(url, {"title": "ToDoList1"}, format='json')
|
def post_todo(self, to_do_title):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
response = self.client.post(self.url, {"title": to_do_title}, format='json')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
self.assertEqual(response.status_code, status.HTTP_201_CREATED)
|
self.assertEqual(response.status_code, status.HTTP_201_CREATED)
|
||||||
self.assertEqual(response.data['title'], "ToDoList1")
|
self.assertEqual(response.data['title'], to_do_title)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
return response.data['id']
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
response = self.client.get(url, {}, format='json')
|
def get_todo_by_id(self, id, expected_title):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
url_with_id = reverse('ToDoLists-detail', args=(id,))
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
response = self.client.get(url_with_id, {id: id}, format='json')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
self.assertEqual(response.status_code, status.HTTP_200_OK)
|
self.assertEqual(response.status_code, status.HTTP_200_OK)
|
||||||
|
self.assertEqual(response.data['title'], expected_title)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
def put_todo(self, id, new_title):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
url_with_id = reverse('ToDoLists-detail', args=(id,))
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
response = self.client.put(url_with_id, {"title": new_title}, format='json')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.assertEqual(response.status_code, status.HTTP_200_OK)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.assertEqual(response.data['title'], new_title)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
def patch_todo(self, id, new_title):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
url_with_id = reverse('ToDoLists-detail', args=(id,))
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
response = self.client.patch(url_with_id, {"title": new_title}, format='json')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.assertEqual(response.status_code, status.HTTP_200_OK)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.assertEqual(response.data['title'], new_title)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
def delete_todo(self, id):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
url_with_id = reverse('ToDoLists-detail', args=(id,))
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
response = self.client.delete(url_with_id, {}, format='json')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
def test_todo(self):
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
"""
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
Tests API for todo.
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
lists/: get, post
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
lists/{id}/: get, put (update), patch (partial_update), delete
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
"""
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
user = User.objects.create_user('test_user', 'test@test.com', 'test_password')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.client.force_authenticate(user=user)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.url = reverse('ToDoLists-list')
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.get_todo([])
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
to_do_title_1, to_do_title_2 = "ToDoList1", "ToDoList2"
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
to_do_id1 = self.post_todo(to_do_title_1)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.get_todo([to_do_title_1])
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
to_do_id2 = self.post_todo(to_do_title_2)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.get_todo([to_do_title_1, to_do_title_2])
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.get_todo_by_id(to_do_id1, to_do_title_1)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.get_todo_by_id(to_do_id2, to_do_title_2)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.delete_todo(to_do_id2)
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.get_todo([to_do_title_1])
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.put_todo(to_do_id1, "ToDoList11")
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
self.patch_todo(to_do_id1, "ToDoList12")
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
|
|
||||||
self.assertEqual((response.data['count'], response.data['next'], response.data['previous'], \
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
response.data['results'][0]['title']), (1, None, None, "ToDoList1"))
|
|
||||||
|
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
1. Сначала проверить, что объект есть
2. Потом проверить, что объект удалился (204) и его нет.
Тоже лучше много проверок, чем одна сложная Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Порядок импортов принят таким:
1. Python модули
2. Библиотеки
3. Пользовательские модули.
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
|
|||||||
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Лучше разбить на 3 проверки, т.к. тогда при фейле сообщение будет более информативным
Тут желательно проверить предусловие, а потом постусловие:
Тут желательно проверить предусловие, а потом постусловие:
Тоже лучше много проверок, чем одна сложная
Тоже лучше много проверок, чем одна сложная
Порядок импортов принят таким:
Т.е. collections -> django -> backend.api
Порядок импортов принят таким:
Т.е. collections -> django -> backend.api
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Вот этот вызов выглядит очень непонятно. Лучше использовать именованные переменные, мне кажется
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась
Может быть можно этот кейс разбить на несколько более простых: типа просто проверка, что всё ок, проверка, что создание + модификация ок, разные элементы создаются и т.п. Сейчас падение этого теста просто показывает, что что-то сломалось, а несколько тестов показали бы, какая именно часть логики сломалась