commentbox - ocena paczki, co poprawić, laravelowa paczka Opcje

[starf]

Hej,

Ogólnie od jakiegoś czasu pracuję jako programista, ale jako że nie mam nikogo kto by mi zrobił jakikolwiek codereview albo pomógł powiedział co poprawić to stwierdziłem że zapytam tutaj (IMG:style_emoticons/default/smile.gif)
Jak każdy szanujący się programista staram się rozwijać samodzielnie itp. Postanowiłem napisać własną paczkę do Laravela, i bardzo bym prosił nie tyle o ocene bo mam świadomość tego że nie koniecznie pięknie jest napisane, samemu mi się nie podoba to raczej wiadomości co mógłbym zmienić poprawić no i oczywiście jak.

Podsyłam link

https://github.com/19lyczkul91/CommentBox

ponadto zależy mi też żeby ktoś mi doradził (tutorial lub po prostu napisał) jak pisać testy jednostkowe. Pod jak nie mam na myśli coś w stylu "O Panie tu musisz wywołać tą metodę i zawsze ma Ci zwracać to i to" tylko raczej coś w stylu
tu powinieneś napisać testy, tu nie.
Konkretne moje pytanie czy to metod store/update itp powinno się robić testy ? I jeśli tak to jak zrobić np do metody store test żeby nie utworzyło nowego elementu ? nie wiem co jeszcze jak wymyślę to dodam.

[nospor]

Wybacz, wlasnie wrocilem z tzw "welcome drinks" i moge troche niekontakowac (IMG:style_emoticons/default/wink.gif)

Czyli żebym dobrze zrozumiał, nie przejmuję się jakimś wyrzucaniem kodu 404 i message nie ma takiego czegoś tam tylko po prostu jak coś to wywali nam całego html-a a front niech sobie radzi jak chce tak ?

Ale czemu html? Lapales wyjatek po czym zwracales odpowiedni response. Teraz masz robic dokladnie to samo ale w ExceptionListener. Patrzac na twoj kod teraz, to widze ze wywaliles te try catch, ale nie dodales ExceptionListener wiec teraz jest jeszcze gorzej (IMG:style_emoticons/default/wink.gif)
Musisz miec globalby ExceptionListener, ktory bedzie lapal te wyjatki i zwracal to co trzeba. Chyba, ze zaczynam cos przegapiac bo tak jak mowilem troche juz nie kontaktuje (IMG:style_emoticons/default/wink.gif) Za godzine bede ok (IMG:style_emoticons/default/wink.gif)

Tak jak w pierwszym poście jak zrobić testy do store tak żeby nie utworzyć nowego rekordu ?

A to juz musza sie wypowiedziec madrzejsi ode mnie. Sam z checia poslucham (IMG:style_emoticons/default/smile.gif)