New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mysql tests - WIP #157

Closed
wants to merge 19 commits into
base: master
from

Conversation

5 participants
@chemix
Copy link

chemix commented Dec 14, 2014

Ahoj,
mam nad dibi napsano par webu a stale si rikam, ze me bavi v ni pracovat vice nez v Nette\Database. Po dotazu na Davida jak to s dibi je, mi rekl, ze je skoda ze nema testy. Tak jsem si rekl ze bych mohl zkusit nejake napsat. Abych netestoval zda mi funguje MySQL server, ale zda dibi pripravi spravny dotaz, zakoukal jsem se do zdrojaku a jedine reseni co me napadlo je pridani metody co vrati pripraveny SQL dotaz.

Q: je o toto zajem?
Q: jak nejlepe toto zacit delat?

@JanTvrdik

This comment has been minimized.

Copy link
Contributor

JanTvrdik commented Dec 14, 2014

Tu metodu vůbec přidávat nepotřebuješ, DibiTranslator::translate() je public.

@chemix

This comment has been minimized.

Copy link

chemix commented Dec 14, 2014

@JanTvrdik diky, takze myslis ze toto reseni je spravne?

chemix@13bbed3#diff-649223062ebe14ec767a25560fac8b67R20

@chemix

This comment has been minimized.

Copy link

chemix commented Dec 14, 2014

jak poresit trevis, aby se "naoko" pripojil k mysql databazi?

@dg

This comment has been minimized.

Copy link
Owner

dg commented Dec 14, 2014

Naoko to připojit nepůjde, escapovací funkce vyžadují aktivní connection. Travis by se dal asi použít z Nette Database.

@chemix

This comment has been minimized.

Copy link

chemix commented Dec 14, 2014

@dg to jde zatim mimo me :( ... co koukam na https://github.com/nette/database/blob/master/.travis.yml tak tam asi nenajdu uplne vsechno ... vecer se mrknu co to vlastne ten Travis je

@chemix

This comment has been minimized.

Copy link

chemix commented Dec 14, 2014

narazil jsem na Martinuv travis configurak, ale jeste na to koukam jak suva z nudli (samej matrix a neo nidke) https://github.com/mrtnzlml/www.zeminem.cz/blob/develop/.travis.yml

@mrtnzlml nepomohl by jsi s tim?

@mrtnzlml

This comment has been minimized.

Copy link

mrtnzlml commented Dec 14, 2014

Mám minimálně ještě následující týden hodně povinností při studiu, ale zatím alespoň na vysvětlenou. Jde o to, že se napíšou testy nezávisle na Travis-CI a ten se pak je propojí s GitHubem a při každém commitu se automaticky spustí testy podle osnovy, která je napsána právě v tom .travis.yml. Fakticky tak spouštíš to samé jako u sebe, ale v nějakém jiném prostředí s tím, že je díky tomu možné nastavit a otestovat více situací. Nemělo by to tedy být tak složité...

@chemix

This comment has been minimized.

Copy link

chemix commented Dec 14, 2014

slozite to asi neni :) ale hlasi mi to ted chybu ve stylu "DibiDriverException: No such file or directory" a to uz je mimo moji jurisdikci

@chemix

This comment has been minimized.

Copy link

chemix commented Dec 15, 2014

debian ve VM uz je zeleny ... ale trevis stale ne :(

@chemix

This comment has been minimized.

Copy link

chemix commented Dec 15, 2014

zelena :)

tak ja jdu psat testy a pak se pokusim o dalsi squash

extension=mysql.so

[MySQL]
mysql.default_socket = "/var/run/mysqld/mysqld.sock"

This comment has been minimized.

@Majkl578

Majkl578 Dec 15, 2014

Tohle tu nemá co dělat, pokud to potřebuje Travis, měl by mít vlastní config.

This comment has been minimized.

@chemix

chemix Dec 15, 2014

mel jsem za to ze tenhle soubor tu je prave kvuli travis

@@ -4,8 +4,8 @@
'mysql' => array(
'driver' => 'mysql',
'host' => 'localhost',
'username' => 'root',
'password' => 'xxx',
'username' => 'travis',

This comment has been minimized.

@Majkl578

Majkl578 Dec 15, 2014

Totéž jako u socketu.

This comment has been minimized.

@chemix

chemix Dec 15, 2014

proc je tu tedy root/xxx ?

@dg dg force-pushed the dg:master branch 8 times, most recently from 601dbd2 to 26b167f Jan 12, 2015

@dg

This comment has been minimized.

Copy link
Owner

dg commented Jan 13, 2015

see f5f4f78

@chemix

This comment has been minimized.

Copy link

chemix commented Jan 13, 2015

@dg koukam ze konfigurace databazovych pripojeni sla do ini souboru a objevila se funkce reformat() ktera by mohla volat ty testy ne jen nad mysql ale nad vsema db? Ale stejne asi nektere veci jsou db specific, jako treba limit...

zaktualizuji to k masteru a uvidim co s tim

@dg

This comment has been minimized.

Copy link
Owner

dg commented Jan 13, 2015

Jde o to, že tam máš nějaké testy na Fluent a translator, ktere by asi bylo skoda zahodit, teda pokud nemáš pocit, že duplikují existující.

@chemix

This comment has been minimized.

Copy link

chemix commented Jan 13, 2015

Mrknu na to poradne

@chemix

This comment has been minimized.

Copy link

chemix commented Jan 14, 2015

@dg koukam ze kdyz to obalim funkci reformat tak se to da spustit, mozna spis ale bude fajn ty testy premistit do tech tvych souboru nez to mit takhle ve slozce

@dg dg force-pushed the dg:master branch 2 times, most recently from 38b7f81 to ca99b0b Jun 15, 2015

@dg dg force-pushed the dg:master branch 18 times, most recently from 2eb5a2d to a29d204 Oct 6, 2015

@dg

This comment has been minimized.

Copy link
Owner

dg commented Oct 8, 2015

Is it still relevant?

@dg dg force-pushed the dg:master branch from 8e9b13b to c4de007 Oct 8, 2015

@chemix chemix closed this Oct 9, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment