Skip to content

Add more tests for first challenge#1

Open
SashoStoichkov wants to merge 1 commit intod0ivanov:masterfrom
SashoStoichkov:master
Open

Add more tests for first challenge#1
SashoStoichkov wants to merge 1 commit intod0ivanov:masterfrom
SashoStoichkov:master

Conversation

@SashoStoichkov
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Owner

@d0ivanov d0ivanov left a comment

Choose a reason for hiding this comment

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

Браво за тестовете!
В случая няма нужда да тестваш всички аритметични операции. Конкретно за тази задача това няма да направи тестовете ти по-пълни. Подадената като първи аргумент фунцкия може да е всякаква и не можеш (а и не трябва) да се опитваш да тесваш всички възможни извиквания. Разликата между двете функции е начинът, по който изчисляват резултата си. Това се вижда най-добре, когато подадеш операция, която не е асоциативна, като първи аргумент. Тестовете с функции, които не са асоциативни са ти най-ценни. По един такъв тест според мен е достатъчен. За пълнота по-скоро можеш да тесваш извиквания върху елементи, различни от числа. Например

accumulate_left(lambda a, b: a + b, [], [[1, 2, 3], [4, 5, 6]])
>> [1, 2, 3, 4, 5, 6]

Супер е, че си се сетил да тестваш и със списък, и с тупъл.

mzdravkov added a commit that referenced this pull request Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants