Testy zrypane po zmianach w script.js

[kelner]

Wygląda na to, że ostatnie zmiany spowodowały błędna obsługę funkcji testowych (nie wziąłem ich pod uwage ...) - jeśli masz czas i chce Ci się możesz spróbować je skorygować w stosunku do zmian lub pogrzebać w samym skrypcie.

[kelner]

Prawdopodobnie dlatego, że objąłem cześć skryptu w onload (bo na razie musiałem) ...

[lwiechec]

ale widzê ¿e - testy wci±¿ przechodz±?...

https://travis-ci.org/TexasRangers/jabberwocky

Lukasz

On Fri, Jan 18, 2013 at 12:53 PM, kelner notifications@github.com wrote:

Prawdopodobnie dlatego, ¿e obj±³em cze¶æ skryptu w onload (bo na razie musia³em) ...

Reply to this email directly or view it on GitHubhttps://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12418857.

[kelner]

no gdy wyswietlam plik html z testami to nic nie dziala :) a travis chyba sprawdza tylko brancze master wiec pewnie dlatego przechodzi - ja na razie zmiany robilem w workcopy.

[lwiechec]

fakt - testy są z mastera...

OK, rzućmy okiem...

Lukasz

2013/1/18 kelner notifications@github.com

no gdy wyswietlam plik html z testami to nic nie dziala :) a travis chyba sprawdza tylko brancze master wiec pewnie dlatego przechodzi - ja na razie zmiany robilem w workcopy.

Reply to this email directly or view it on GitHubhttps://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12419119.

[kelner]

aha - calosc po zmianach testowalem w Chromium 22 i poprawonosc i plynnosc animacji okna wyboru zestawu byla ok - czy u ciebie tez wszystko wyglada i dziala cacy?

[lwiechec]

na pierwszy rzut oka - bardzo cacy. Ładne!

Ale - mam problem ze zrozumieniem dlaczego cały kod dżawaskrypta wrzuciłeś do wnętrza window.onload() ?...

Lukasz

2013/1/18 kelner notifications@github.com

aha - calosc po zmianach testowalem w Chromium 22 i poprawonosc i plynnosc animacji okna wyboru zestawu byla ok - czy u ciebie tez wszystko wyglada i dziala cacy?

Reply to this email directly or view it on GitHubhttps://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12419318.

[kelner]

Bo byly problemy z parsowaniem dokumentu: chodzilo o to, czesc skryptu ladowala sie za wczesnie (np. zanim caly html byl wyswietlony), co powodowalo problemy w przypadku dodawania tymczasowych elementow html-a poprzez skrypt. Także przez to burzyly sie niektore funkcje (blędny scoping, czesc zmiennych globalnych nie byla widziana od samego początku, a byc powinna). Teoretycznie moglem po prostu dac odnosnik do skryptu na koncu dokumentu HTML (co nie jest dobra praktyka), jednak na razie wybralem rozwiazanie z onload.

[kelner]

Teoretycznie da sie to jakos przegrzebac, bo jak rozumiem przez to testy nie moga dobrac sie do funkcji , bo nie sa globalne. Na razie jednak chciałbym pozostawić ta działającą wersje do czasu zatwierdzenia projektu jako beta. Wiec jesli przez to jest zbyt duzo grzebania w testach lub po prosut niemozliwe to na razie zostaw. W tym absolutnym momencie szkoda na to czasu jesli mialoby to strasznie wyginac Ci neurony...

[lwiechec]

OK. Chyba nie wiem za wiele o JS bo nie mam jak na razie pojęcia jak dostać się do funkcji z poziomu testu... spróbuję jeszcze coś wymyśleć! :D

Lukasz

2013/1/18 kelner notifications@github.com

Teoretycznie da sie to jakos przegrzebac, bo jak rozumiem przez to testy nie moga dobrac sie do funkcji , bo nie sa globalne. Na razie jednak chciałbym pozostawić ta działającą wersje do czasu zatwierdzenia projektu jako beta. Wiec jesli przez to jest zbyt duzo grzebania w testach lub po prosut niemozliwe to na razie zostaw. W tym absolutnym momencie szkoda na to czasu jesli mialoby to strasznie wyginac Ci neurony...

Reply to this email directly or view it on GitHubhttps://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12419676.

[kelner]

jak masz czas i checi :)) a tak poza - dales/dasz rade kajknąć do sklepu?

[lwiechec]

hej, do sklepu bêdê móg³ podjechaæ w przysz³ym tyogdniu... wytrzymasz? :)

Lukasz

On Fri, Jan 18, 2013 at 4:13 PM, kelner notifications@github.com wrote:

jak masz czas i checi :)) a tak poza - dales/dasz rade kajkn±æ do sklepu?

Reply to this email directly or view it on GitHubhttps://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12425880.

[kelner]

no tak bardzo srednio, teoretycznie w niedziele juz raczej musialbym kupic sam ...

[lwiechec]

sorki - nie dam rady inaczej, w weekend sklep jest zamkniêty, z tego co wiem.

Lukasz

On Fri, Jan 18, 2013 at 4:39 PM, kelner notifications@github.com wrote:

no tak bardzo srednio, teoretycznie w niedziele juz raczej musialbym kupic sam ...

Reply to this email directly or view it on GitHubhttps://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12427202.

[kelner]

poniedzialek?

[lwiechec]

postaram siê! :)

Lukasz

On Fri, Jan 18, 2013 at 4:49 PM, kelner notifications@github.com wrote:

poniedzialek?

Reply to this email directly or view it on GitHubhttps://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12427718.

[kelner]

dzienks!

[kelner]

ok, sprobuje przeanalizowac skrypt jeszcze raz i dac do onload tylko to co jest niezbedne oraz nie wplywa na testy, lub wymysle jakis workaround bez onload wogole..

[kelner]

testy juz wszystkie dzialaja - teraz trzeba praktycznie przetestowac calosc w roznych wariantach rozgrywki.

[lwiechec]

Ok - dobra robota. Cala idea testow wydaje sie troche idee fixe ale wierze ze jest konieczna :)

-- Lukasz

-----Original Message----- From: kelner notifications@github.com Date: Sat, 19 Jan 2013 03:10:45 To: TexasRangers/jabberwockyjabberwocky@noreply.github.com Reply-To: TexasRangers/jabberwocky reply@reply.github.com Cc: Lukasz Wiecheccoquelicot408@gmail.com Subject: Re: [jabberwocky] Testy zrypane po zmianach w script.js (#24)

testy juz wszystkie dzialaja - teraz trzeba praktycznie przetestowac calosc w roznych wariantach rozgrywki.

---

Reply to this email directly or view it on GitHub: https://github.com/TexasRangers/jabberwocky/issues/24#issuecomment-12453439