Tester: added new section about running Tester with Travis CI #173

klimesf · 2015-01-11T00:13:51Z

Added new section about running tester with Travis CI.

This is just a draft, feedback needed.

After the draft is approved, I will translate the article in English and then squash all the commits.

Majkl578 · 2015-01-11T00:59:28Z

cs/testing-with-travis.texy

+********************************
+
+/--div .[perex]
+Znáte to. Napsali jste testy, které teď procházejí, poplácali se po zádech a váš spánek začal být mnohem klidnější. Máte jistotu, že váš kód spolehlivě funguje. A pak se to stane. Hodiny ukazují 3:15 a do nočního ticha se zakousne nepříjemný zvuk drnčení mobilu na nočním stolku. Volá šéf. Zase to nefunguje. Zalije vás studený pot. Vždyť na to máme testy! V tu chvíli se vynoří vzpomínka na páteční odpoledne - spěcháte domů a zapomínáte spustit testy po poslední úpravě.


Pohádka hezká, ale do dokumentace IMHO nepatří (je moc dlouhá).

klimesf · 2015-01-11T11:01:21Z

@Majkl578 @JanTvrdik díky!

JanTvrdik · 2015-01-11T11:09:02Z

cs/testing-with-travis.texy

+Spouštění testů
+===============
+
+Testy se spouštějí v sekci `script`, kde stačí zavolat tester. Předpokládejeme, že testy jsou umístěny ve složce `tests/` a používáte vlastní `php.ini` umístěné ve složce s testy. Pomocí `-s` si necháme vypsat i přeskočen testy.


přeskočené + „vypsat testy“ nedává česky moc smysl. Lépe asi „ vypsat informace o přeskočených testech“

Předpokládejme místo Předpokládejeme

JanTvrdik · 2015-01-11T11:14:53Z

Wow, hodně dobře napsáno. Ještě mě napadlo – Nedáme to radši do dokumentace Testeru, než do dokumentace Nette? Tj. místo doc.nette.org/en/2.2/testing-with-travis prostě jenom tester.nette.org/en/travis?

JanTvrdik · 2015-01-11T11:19:34Z

Ještě mi došlo, že tam úplně chybí zmínka o nastavení hooku GitHubu.

klimesf · 2015-01-11T11:24:03Z

@JanTvrdik
Ad dev) Podle mě je to dobré spomenout pro jistotu, aby nedošlo k nějakým pomýlením. Nicméně není to nutné, protože se odkazuji na dokumentaci ke composeru.
Ad --no-interaction) Okay, a kvůli vysvětlení --prefer-source to tam můžu nechat?
Ad dokumentace Testeru) jsem pro
Ad GithubHook) to je už nastavení Travisu, které se při použití Testeru neliší, proto mi do dokumentace moc nesedělo. Klidně ho ale doplním.

JanTvrdik · 2015-01-11T11:27:37Z

Ano, to vysvětlení --prefer-source tam dává smysl (sám jsem to třeba nevěděl). Co se GitHub hooku týče, tak už teď tam přece máš dost věcí, které nejsou nijak specifické pro Tester, např. celé kapitoly nastavení jazyka a instalace závislostí.

klimesf · 2015-01-11T11:29:42Z

@JanTvrdik fair enough

JanTvrdik · 2015-01-11T11:33:36Z

Ještě si říkám, jestli něco neudělat s tou děsně plochou strukturou odkazů, zatím mě napadlo akorát

JanTvrdik · 2015-01-11T11:57:57Z

cs/testing-with-travis.texy

+env:
+  - TESTER_PHP_BIN="php"
+  - TESTER_PHP_BIN="hhvm"
+\--


Až teď mi došlo (opravte mě, cc @Majkl578, pokud se mýlím), že tohle rozdělení nedává vůbec smysl, protože když Travis testuje HHVM, tak hhvm binárky je simlinknutá i jako php. Aby tam byl nějaké rozdíl, tak by tam muselo být místo php php-cgi, které není při běhu HHVM dostupné.

Funguje to i pokud použiješ pouze php, ale jak jsem psal výše, nenapadla mě lepší možnost na čem demonstrovat env. Nicméně bych mohl zmínit, že to jde i jinak.

Tak ještě se dá env taky využít na to, že jednou nainstaluješ závislosti normálně a podruhé s --prefer-lowest. Nicméně pokud tam dáš místo TESTER_PHP_BIN="php" radši TESTER_PHP_BIN="php-cgi", tak by to mělo stačit.

Proč je php-cgi lepší možnost než php?
Mimochodem php-cgi na Travisu nefunguje ani pro PHP 5.3.

Protože tam pak bude rozdíl v tom, jestli používáš env nebo ne.

Protože normálně ti webové aplikace taky neběží pod CLI. Není dobré, abys testoval přes jiné SAPI, než přes které aplikaci normálně provozuješ. Nebo tak něco, @dg bude vědět.

Ano, je potřeba použít PHP 5.3.3.

Díky za osvětu :)
Zbytek opraven

milo · 2015-01-11T19:37:55Z

@klimesf Paráda. Napíšu ještě pár návrhů.

milo · 2015-01-11T19:47:58Z

cs/testing-with-travis.texy

+Jak funguje Travis CI
+=====================
+
+Travis CI (dále jen Travis) je velmi populární nástroj pro automatizaci [průběžné integrace](http://cs.wikipedia.org/wiki/Průběžná_integrace). Je plně integrován do prostředí GitHubu.


Travis CI((Continuous Integration)) (dále jen Travis) je velmi populární služba pro... Je poskytována zdarma a nabízí automatizované spouštění testů po zápisu změny do repozitáře. Je plně integrován...

milo · 2015-01-11T20:23:31Z

cs/testing-with-travis.texy

+      env: TESTER_PHP_BIN="php-cgi"
+\--
+
+Aby testování bylo označeno Travisem jako úspěšné, musí projít všechny dílčí testy. Pokud nechceme, aby výsledek některé z nich ovlivňovaly, můžeme to deklarovat v bloku `allow_failures`. V našem případě nám nebude vadit, pokud neprojdou testy na HHVM.


..., musí testy projít ve všech prostředích.

milo · 2015-01-11T20:36:05Z

@klimesf Ode mě vše. Nějak jsem se rozjel... jestli se Ti něco nepozdává, dej vědět. Šlo mi o to to trochu sjednotit s názvoslovím Travisu.

klimesf · 2015-01-11T20:51:05Z

@milo Děkuju moc za feedback a korekturu. :) Text mi teď přijde jasnější.

Jestli už nikdo nebude mít připomínky, pomalu se pustím do překladu.

JanTvrdik · 2015-01-11T21:39:18Z

Mám takový pocit, že tohle bude nejvymakanější stránka, co v dokumentaci máme 😄

klimesf · 2015-01-11T21:47:15Z

@JanTvrdik Whoops, díky.

vojtech-dobes · 2015-01-11T21:51:34Z

cs/testing-with-travis.texy

+Matice testů
+------------
+
+Na základě předchozích údajů vytvořena testovací matice, tedy varianty prostředíve kterých budou testy spuštěny (v našem příkladě 10 variant). Nastavení matice je umožněno v bloku `matrix`.


Na základě předchozích údajů je vytvořena testovací matice...

prostředíve -> prostředí, ve

klimesf · 2015-01-11T23:15:55Z

Feedback for EN version would be appreciated.

klimesf · 2015-01-12T22:26:45Z

Interesting. I have discovered that you can do this:

env:
  - TESTER_PHP_BIN="php-cgi"
  - TESTER_PHP_BIN="hhvm"

matrix:
  allow_failures:
    - php: hhvm

  exclude:
    - env: TESTER_PHP_BIN="hhvm"

  include:
    - php: hhvm
      env: TESTER_PHP_BIN="hhvm"

It excludes all jobs with TESTER_PHP_BIN = 'hhvm' from existing matrix and then includes job with HHVM and TESTER_PHP_BIN = 'hhvm'. Notice that you need to have two rows in env. include will add the env setting for all other versions too.

Should I mention it in the documentation? Do you think it's better than excluding 5 ouf of 10 rows explicitly?
ping @JanTvrdik @Majkl578 @milo

EDIT: output can be seen here

Majkl578 · 2015-01-12T23:18:04Z

Not needed. See recent @dg's commits where he switched to php binary (which is aliased to HHVM eventually).

klimesf · 2015-01-13T13:01:56Z

If you don't have any other objections, we are ready to merge.

dg · 2015-01-13T13:04:12Z

Great work!

Can you squash it?

dg · 2015-01-13T13:13:32Z

cs/testing-with-travis.texy

+Travis CI((Continuous Integration)) (dále jen Travis) je velmi populární služba pro automatizaci [průběžné integrace](http://cs.wikipedia.org/wiki/Průběžná_integrace). Je poskytována zdarma((Pro soukromé repozitáře je zpoplatněna.)) a nabízí automatizované spouštění testů po zápisu změny do repozitáře. Je plně integrována do prostředí GitHubu.
+
+Travis se konfiguruje souborem `.travis.yml` umístěným v kořenovém adresáři projektu. Obsahuje několik bloků, které si podrobněji popíšeme.
+


Ještě mě nepadá, že bych tam dodal, že většina z těch bloků je nepovinná, ať to nevypadá tak složitě.

dg · 2015-01-13T13:19:14Z

Nebude to jednoznakové odsazování málo?

klimesf · 2015-01-13T13:20:23Z

@dg kde přesně?

dg · 2015-01-13T13:22:14Z

Sry, to mi dělal plugin do Chrome.

klimesf · 2015-01-13T13:24:51Z

Btw, co s timhle?

Ještě mě napadlo – Nedáme to radši do dokumentace Testeru, než do dokumentace Nette? Tj. místo doc.nette.org/en/2.2/testing-with-travis prostě jenom tester.nette.org/en/travis?

dg · 2015-01-13T13:28:37Z

Asi bych to, dokud se nepředělá struktura webu, nechal jako součást doc.nette.org.

Tester: added new section about running Tester with Travis CI

klimesf · 2015-01-13T13:30:08Z

👍 díky moc všem zúčastněným!

milo · 2015-01-13T15:01:26Z

@klimesf Díky!

Majkl578 reviewed Jan 11, 2015
View reviewed changes

JanTvrdik reviewed Jan 11, 2015
View reviewed changes

milo reviewed Jan 11, 2015
View reviewed changes

vojtech-dobes reviewed Jan 11, 2015
View reviewed changes

dg reviewed Jan 13, 2015
View reviewed changes

Tester: added new section about running Tester with Travis CI

58a4e6c

dg added a commit that referenced this pull request Jan 13, 2015

Merge pull request #173 from klimesf/doc-2.2

2e39620

Tester: added new section about running Tester with Travis CI

dg merged commit 2e39620 into nette:doc-2.2 Jan 13, 2015

		Travis CI((Continuous Integration)) (dále jen Travis) je velmi populární služba pro automatizaci [průběžné integrace](http://cs.wikipedia.org/wiki/Průběžná_integrace). Je poskytována zdarma((Pro soukromé repozitáře je zpoplatněna.)) a nabízí automatizované spouštění testů po zápisu změny do repozitáře. Je plně integrována do prostředí GitHubu.

		Travis se konfiguruje souborem `.travis.yml` umístěným v kořenovém adresáři projektu. Obsahuje několik bloků, které si podrobněji popíšeme.

Uh oh!

Tester: added new section about running Tester with Travis CI #173

Tester: added new section about running Tester with Travis CI #173

Uh oh!

Conversation

klimesf commented Jan 11, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

klimesf commented Jan 11, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JanTvrdik commented Jan 11, 2015

Uh oh!

JanTvrdik commented Jan 11, 2015

Uh oh!

klimesf commented Jan 11, 2015

Uh oh!

JanTvrdik commented Jan 11, 2015

Uh oh!

klimesf commented Jan 11, 2015

Uh oh!

JanTvrdik commented Jan 11, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milo commented Jan 11, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milo commented Jan 11, 2015

Uh oh!

klimesf commented Jan 11, 2015

Uh oh!

JanTvrdik commented Jan 11, 2015

Uh oh!

klimesf commented Jan 11, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

klimesf commented Jan 11, 2015

Uh oh!

klimesf commented Jan 12, 2015

Uh oh!

Majkl578 commented Jan 12, 2015

Uh oh!

klimesf commented Jan 13, 2015

Uh oh!

dg commented Jan 13, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dg commented Jan 13, 2015

Uh oh!

klimesf commented Jan 13, 2015

Uh oh!

dg commented Jan 13, 2015

Uh oh!

klimesf commented Jan 13, 2015

Uh oh!

dg commented Jan 13, 2015