Skip to content

Draft: unittest #2

Closed
wants to merge 2 commits into from
Closed

Draft: unittest #2

wants to merge 2 commits into from

Conversation

martins0n
Copy link
Contributor

No description provided.

@martins0n martins0n changed the title NEW: unittest start Draft: unittest Jan 8, 2022
@martins0n
Copy link
Contributor Author

martins0n commented Jan 8, 2022

rfc для тестирования grpc клиента.

@daxartio, @solry В таком виде пойдет?



@pytest.fixture()
def grpc_userserceservicer(grpc_users_account):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не могу понять что за словосочетание в названии функции. Так и задумано, там нет ошибки?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь дествительно ошибка.
Я не нашел какого-то канона, мб он есть.
Поэтому собирался называть по следующему правилy: "grpc_{proto_file_name}_{objectname.lower()}"

tests/test_users.py Show resolved Hide resolved
tests/test_users.py Show resolved Hide resolved
@daxartio
Copy link
Contributor

daxartio commented Jan 9, 2022

Я для grpc клиента больше имел в виду интеграционные тесты написать. Здесь получается свой сервер и он же тестируется?

@martins0n
Copy link
Contributor Author

martins0n commented Jan 9, 2022

Я для grpc клиента больше имел в виду интеграционные тесты написать. Здесь получается свой сервер и он же тестируется?

Тут начинал тред .Сначала хотел тоже песочницу дергать.

Наша цель: быть уверенными, что мы актуальный grpc клиент используем для своего верхнеуровнего api, если я правильно понимаю.

Если прям мокать то это совсем плохо, наверно, потому что даже изменение прото клиента мы не заметим.

Текущий подход вроде решает эту проблему: мы постоянно к сервису стороннему не обращаемся и в то же время мы всегда сервер сайд api тестируем ( при условии что proto сгенерированный клиент обновляем постоянно ).

Но в целом можем заменить на реальный сервис -- это должно больше гарантий давать. А если эти тесты будут напрягать, сможем их соптизировать и после мерджа только прогонять например или по расписанию.

Как делать?

@daxartio
Copy link
Contributor

Я думаю, нужно на реальном сервере начать тестировать

@martins0n martins0n closed this Jan 17, 2022
@martins0n martins0n deleted the unit-tests branch January 17, 2022 21:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants