[noelo_cohelo]

#java #programowanie

Ma ktoś pomysł jak przetestować wyrzucanie tego wyjątku?
AirborneStatus ma 2 pola, jedno jest stringiem a drugie booleanem, nie ma opcji żeby podać złe dane - o ile nie ma błędu w oprogramowaniu, wiadomość zawsze utworzy się dobra, a wtedy też dobrze się zaszyfruje do XMLa...

Chciałabym uniknąć używania PowerMocka (obie metody są statyczne, klasy Utils...), bo słyszałam że to świadczy o złym designie... W takim razie jak przeształcić ten kod żeby był dobry i testowalny?
Myślę nad tym i myślę i nie wiem jak to wymyślić ( ͡° ͜ʖ ͡°)

[noelo_cohelo]

@SmallCaracan: to po c--j robisz nadzieję wołając przy tym wpisie? (⌐ ͡■ ͜ʖ ͡■)

[nfan]

spróbuj coś w tym stylu

[noelo_cohelo]

@nfan: nie zawołałeś...
mógłbyś mi to wyjaśnić bardziej merytorycznie? ten kod w 85% składa się z mojego przerobionego + np new JAXBException()nie zrobię bo nie ma takiej metody. topicAirborneStatusto też nie jest metoda, spyto jak rozumiem to metoda z Mockito ale nie ma nawet nawiasów... to jakiś trolling?

[nfan]

@noelo_cohelo bo to tylko podpowiedź, a nie działający kod, ale od początku

1. statyczne wywołania metod na utilach zamykasz w metodach testowanej klasy, dzięki temu masz możliwość symulowania co te metody zwracają bez używania PowerMocka (pierwsze dwie metody w kodzie, który przysłałem)
2. w testach jednostkowch obiekt klasy, która posiada te metody robisz jako spy
3. określasz, że gdy na obiekcie stworzonym w punkcie 2 (w przykładzie nazwałem go spy) zostanie wywołana metoda marschal() to ma zostać rzucony wyjątek

[Godziu73]

@interface: o, konkretny tip ( ͡° ͜ʖ ͡°)

[noelo_cohelo]

@nfan: super, dzięki!
1. głupio wyglądają te pierwsze dwie metody ;) ale może tak trzeba.
2. no właśnie tego mi brakowało - co to ten spy ;)
4. tego to już nie rozumiem. jakie powiadomienie i co masz na myśli przez .topicAirborneStatus(...)?
5. nie pisałam że nie ma żadnego tylko takiego bez parametrów

Gdzieś tam wyszperałam też i wymyśliłam że mogłabym te metody zrobić niestatyczne (w sumie to moja klasa),

[noelo_cohelo]

@Godziu73: szkoda że go nie rozumiem :/
@interface: bardzo descriptive nazwa, co to jest d--a? :P

klamry {} zastąpią podane później po przecinkach wartości w kolejności w jakiej zostały podane.

nie czaję. gdzie te wartości będą podawane? O_o

[Godziu73]

@noelo_cohelo: d--a to zmienna, tak samo np

LOG.warn("var1 value {}, var2 value {}", firstvariable, secondvariable)

[Godziu73]

@interface: bo chodzi o to żeby urwać te milisekundy! ( ͡º ͜ʖ͡º)

[nfan]

@noelo_cohelo
4. a widzisz, zamieszałem. Żle popatrzyłem i byłem pewien, że topicAirborneStatus to metoda w klasie, którą testujesz, a w rzeczywistości to jest obiekt jakiejś innej klasy z metodą publish. W takim razie musisz zrobić mock'a dla topicAirborneStatus i na nim zrobić verify, składniowo podobnie do tego co wrzuciłem :)

Możesz tak zrobić, tylko wtedy żeby to było fajnie testowalne to musiałabyś instancję XmlHelper np. wrzucać jako argument do

[MacDada]

@noelo_cohelo: Nie wiem co to jest ten cały AirbornService, ale jak masz problem z testowaniem, to często jest to problem z dizjanem.

Twoim problemem jest ścisłe przywiązanie do obcego kodu (tight coupling).

Po pierwsze, komunikujesz się z zewnętrzną usługą statycznie – jest to wyraźne naruszenie zasady odwracania zależności (dependency inversion principle). Powinieneś dostawać obiekt tej usługi w konstruktorze – wtedy nie miałbyś problemu z jego podmianą (na inaczej skonfigurowany egzemplarz, np w celach

[noelo_cohelo]

@nfan: a to protected - zwiększamy widoczność metody tylko pod testy? średnio mi to wygląda

[noelo_cohelo]

jak masz problem z testowaniem, to często jest to problem z dizjanem.

@MacDada: mnóstwo razy to słyszałam, tylko nigdy nie wiem jak zrobić dobry dizajn ( ͡° ͜ʖ ͡°)
co to jest zewnętrzna usługa? XmlUtils to moja klasa.

a problem w ogóle nie dotyczy logera tylko samego wyrzucania wyjątku - nie wiem jak to zasymulować

[MacDada]

mnóstwo razy to słyszałam, tylko nigdy nie wiem jak zrobić dobry dizajn ( ͡° ͜ʖ ͡°)

@noelo_cohelo: https://en.wikipedia.org/wiki/SOLID_(object-oriented_design) + czytaj wszystko od Wujka Boba

[noelo_cohelo]

@MacDada: marshal.

Potraktować test funkcjonalnie i przekazać taki message, żeby to wybuchło (zauważ, że samo message możesz zmockować)

Myślałam o tym, wtedy wybuchnie na createFlightStatusMessage i też nie będzie to JAXBException.

Potraktować test jednostkowo i zmockować usługę tak,