Did a level 1_5 tests. by Beloded1 · Pull Request #2 · Beloded1/testing_exercises

Beloded1 · 2023-05-26T12:55:44Z

I didn't like the test for two_square_equation. I am sure, that I could do much more, but this didn't work. That is why I wrote just two, because others didn't work properly. Also I spent much time for one_median tests, but it conclude bag in themself.

vppuzakov · 2023-05-26T13:05:32Z

@@ -0,0 +1,10 @@
+from functions.level_1_5.two_square_equation import solve_square_equation
+
+def test_solve_square_equation_if_discriminant_is_negatibe():


💡 нужно еще потестить вот эту ветку:

https://github.com/Beloded1/testing_exercises/blob/main/functions/level_1_5/two_square_equation.py#L14-L18

vppuzakov · 2023-05-26T13:06:14Z

+def test_solve_square_equation_if_discriminant_is_negatibe():
+    assert solve_square_equation(1.0, 1.0, 1.0) == (None, None)
+
+def test_solve_square_equation_if_discriminant_is_positive():


💡 наименования у тестов должны быть test__method__behavior

vppuzakov · 2023-05-26T13:07:53Z

+
+def test_get_median_value_with_odd_len_of_items():
+    items = [1, 3, 6, 8, 9, 10]
+    assert get_median_value(items) == 9


💡 лучше сделать этот тест ванлайнером, и передать параметры items=[1, 3, ...] как kwargs

vppuzakov · 2023-05-26T13:08:48Z

@@ -0,0 +1,13 @@
+from functions.level_1_5.one_median import get_median_value
+
+def test_get_median_value_with_empty_items():


👍 тестируем кейс на пустой входной список элементов

vppuzakov · 2023-05-26T14:39:55Z

+    items = [5, 7, 1, 15]
+    assert first(items) == 5
+
+def test_first_with_empty_items_and_default_is_NOT_SET():


🔴 в названиях функциях нельзя CAPS

vppuzakov · 2023-05-26T14:40:12Z

+
+def test_first_with_empty_items_and_default_is_NOT_SET():
+    items = []
+    default = "NOT_SET"


🔴 default есть, но не используется, мне кажется тут one liner бы зашел

vppuzakov · 2023-05-26T14:42:24Z

+    with pytest.raises(AttributeError):
+        first(items)
+
+def test_first_with_empty_items_and_default_is_None():


🔴 капс в названии

vppuzakov · 2023-05-26T14:42:35Z

+def test_first_with_empty_items_and_default_is_None():
+    items = []
+    default = None
+    assert first(items, default) == None


💡 oneliner заходит отлично

vppuzakov · 2023-05-26T14:42:46Z

+def test_first_with_empty_items_and_default_is_int():
+    items = []
+    default = 0
+    assert first(items, default) == 0


💡 замени на ванлайнер

vppuzakov

👍 в целом сами тесты оформлены отлично, тест кейсы найдены хорошие. Считаю что можно смержить и в следующих ПРах вернутся порефачить.

💡 во всех тестах как будто не хватает глагола в названии описывающего поведение, есть только факты того что передается на вход.

Это уже неплохо, но вот мне чего то не хватило из названий чтобы понять тестируемое поведение. То есть да вот мы тестим функцию когда на входе такие-то значения, а какое поведение ожидаем то - в названии не видно.

Типо функция вернет true, выкинет ошибку, вернет первый элемент из списка, строку без замен, строку с заменненным словом.

vppuzakov · 2023-06-02T21:22:43Z

+from functions.level_1_5.five_replace_word import replace_word
+
+def test__replace_word__starts_with_case_irrelevant():
+    assert replace_word('the big black car', 'big', 'Black') == 'the Black black car'


💡 если case irrelevant - то это replace_word('the big black car', 'Big', 'Black'), искомое слово ищется без учета регистра букв

vppuzakov · 2023-06-02T21:24:29Z

+    assert replace_word('the big black car', 'big', 'Black') == 'the Black black car'
+
+
+def test__replace_word__if_replace_word_not_find_in_text():


💡 Можно еще явнее описать тест в названии, test__replace_word__unchanged_when_replace_word_not_found_in_text

vppuzakov · 2023-06-02T21:26:41Z

+    assert check_tweet_sentiment(text, good_words, bad_words) == None
+
+
+def test__check_tweet_sentiment__without_good_and_bad_words():


💡 во всех тестах этого модуля не хватает глагола в названии, описывающего поведение, описывается факт, что на входе, но не поведение

test__check_tweet_sentiment__return_none_when_no_good_and_bad_words_found

vppuzakov · 2023-06-02T21:29:11Z

+
+
+@pytest.mark.xfail(reason='Incorrect calculation')
+def test__get_median_value__with_odd_len_of_items():


💡 не хватает глагола, описывающего поведение

vppuzakov · 2023-06-02T21:30:09Z

+    assert first([], None) == None
+
+
+def test__first__with_empty_items_and_default_is_int():


💡 есть то что на входе в названии, но не отражено поведение, что мы ожидаем увидеть

Did a level 1_5 tests.

62b9a13

vppuzakov suggested changes May 26, 2023

View reviewed changes

Beloded1 added 2 commits May 26, 2023 20:17

Add mark.xfails and fixed some bugs

8c93360

Fix more tests.

ebea2d1

Beloded1 merged commit 468d621 into main May 29, 2023

Beloded1 deleted the feature/do_level_1_5 branch May 29, 2023 10:09

vppuzakov reviewed Jun 2, 2023

View reviewed changes

		@@ -0,0 +1,10 @@
		from functions.level_1_5.two_square_equation import solve_square_equation

		def test_solve_square_equation_if_discriminant_is_negatibe():

		@@ -0,0 +1,13 @@
		from functions.level_1_5.one_median import get_median_value

		def test_get_median_value_with_empty_items():

		assert replace_word('the big black car', 'big', 'Black') == 'the Black black car'


		def test__replace_word__if_replace_word_not_find_in_text():

		assert check_tweet_sentiment(text, good_words, bad_words) == None


		def test__check_tweet_sentiment__without_good_and_bad_words():



		@pytest.mark.xfail(reason='Incorrect calculation')
		def test__get_median_value__with_odd_len_of_items():

		assert first([], None) == None


		def test__first__with_empty_items_and_default_is_int():

Conversation

Beloded1 commented May 26, 2023

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vppuzakov left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants