vendor update #372
vendor update #372
Conversation
tests: use latest transformation image
|
Lgtm, jen si nejsem jistej, jestli nezafixovat ten Python image na novější verzi. Nějaká budoucí změna by mohla rozbít testy. |
|
nerozumim, latest je nejnovejsi verze, ne? |
|
aha, uz asi rozumim, ty myslis na "novejsi verzi nez tam byla" a ne na "novejsi verzi nez je latest" |
|
hm, prijde mi lepsi updatovat testy pri kazdym major releasu debianu, nez poustet testy oproti necemu co uz treba rok neni na produkci |
|
Rozumím - ale pokud to chápu dobře, tak python transformace tam je proto, že se tím dobře mockuje komponenta. Netestuješ ten image, ale nějaký chování. Mohl by tam být místo toho nějakej dummy python executor, a ten bys chtěl imho mít zafixovanej na konkrétní verzi, aby ti to nějakej update nerozbil. |
no, jo a ne. python tranformace tam je proto, ze se pres ni da pustit kod, kterej potrebujem, ne jen kvuli tomu, ze to mockuje komponentu. Ale je pravda, ze to neni primarne urceny k testum toho image, cili bysme si mohli asi udelat dummy python executor. |
to jsem myslel tím mockováním komponenty, takže si rozumíme :-) |
|
no uz jsem na to udelal issue #373 |
|
ono to nebude uplne na vterinu prace, takze bych to kdyztak udelal nekdy separatne az se bude vrtat do testu, toto tedka mergnu |
|
jj určitě. |
replaces #370