[Hopsiup-siup]

Hej programistyczne świry, chciałby ktoś zrobić mi małe #codereview? Napisałem kilka testów w #nunit i #csharp (#dotnet dla zasięgu), ale nie wiem czy poziom refaktoryzacji jest zadowalający :C

https://pastebin.com/sJFhi1qa
( ͡° ͜ʖ ͡°)

[azetka]

@Hopsiup-siup: chyba chodziło Ci o "expectation" a nie "expection" ( ͡° ͜ʖ ͡°)
Poza tym śmieszna konwencja nazw metod z tym 'when' I 'given', skąd ją wziąłeś? (Nie znam się więc pytam z ciekawości xD)

[Hopsiup-siup]

@azetka: dzięki, jak coś działa to nie jest śmieszne

[azetka]

@Hopsiup-siup: dzięki za rozbudowaną odpowiedź.
Nie wiem czy działa, nie ma tej metody w pliku który wkleiłeś, więc zakładając że nie jest to metoda napisana pod testy, to musiałabym zajrzeć do metody żeby dowiedzieć się co w końcu się dzieje "when coś tam" bo sama nazwa tego nie mówi.

[Hopsiup-siup]

@azetka: sorki, niewiele spałem, kolokwium dziś i jestem trochę rozdrażniony. Tak w skrócie to metoda whenserverwinsapoint inkrementuje wartość przechowywaną w zmiennej, do której przypisany jest enum, dla gracza pierwszego, analogicznie metoda whenreceiverwinsapoint. Wiem że bez kodu niewiele da się powiedzieć, ale zakładam, że metody są poprawne ponieważ miałem je wykorzystać i głównym celem było napisanie poszczególnych asercji a potem refaktoryzacja. Nie wiem czy mogę udostępnić kod, dlatego wrzuciłem tylko testy napisane

[ppawel]

śmieszna konwencja nazw

@azetka: OP chyba sam nie wie czemu tak nazwał (⌐ ͡■ ͜ʖ ͡■), więc proponuję poczytaj o BDD, np. tu https://en.wikipedia.org/wiki/Behavior-driven_development#Behavioral_specifications , albo o specification by examble, a jak się wciągniesz to i o języku Gherkin, i możesz zacząć pisać swoje ogórkowe testy z np. Cucumberem ( ͡° ͜ʖ ͡°)

[azetka]

@Hopsiup-siup: ja też przepraszam, bo chciałam zamieszkować a mogło to zabrzmieć agresywnie xD
Same testy moim zdaniem są ok (tzn ewentualnie tak jak zauważyłeś, te wszystkie testcasy mógłbyś zamknąć w jednego testcasesourca, byłoby przejrzyściej), jak dobrze rozumiem z tego co napisałeś, te metody "when..." i "given..." nie są napisane przez Ciebie? (bo ja bym pewnie połączyła te "when" w jedną "PlayerWinsAPoint(argument PlayerType)" )

[azetka]

@ppawel: interesting, poczytam

[Hopsiup-siup]

@azetka: uuu dobre, może jeszcze poprawie :3 dzięki :***

[ja_tu_czytam]

@azetka: jest to dość popularne podejście (GWT)
//given
//when
//then

ewentualnie bardziej znane (AAA)
//arrange
//act
//assert

[ja_tu_czytam]

@Hopsiup-siup: ja bym zmienił test case na TestCaseSource albowiem sporo masz tych TC
Możesz poczytać tutaj
https://github.com/nunit/docs/wiki/TestCaseSource-Attribute

[Hopsiup-siup]

@ja_tu_czytam: @azetka: https://pastebin.com/yB2LMq1Z better? ( ͡° ͜ʖ ͡°)

[Hopsiup-siup]

@ppawel: @ja_tu_czytam: @azetka: w sumie to wam nie podziękowałem, ale dzięki waszym sugestiom udalo mi się dostać robotę i mogę być w elitarnym gronie #programista15k pijąc San Miguela w Las Palmas. Serdecznie dziękuje, następny drineczek to wasze zdrowie!

[azetka]

@Hopsiup-siup: popatrzyłam co tam pisałam te trzy lata temu i widzę że coś #!$%@?łam bez składu, ale nie ma za co ( ͡° ͜ʖ ͡°)

[Hopsiup-siup]

@azetka: wazne ze dziala ( ͡° ͜ʖ ͡°)