Comment 1 Alexander Kläser 2017-01-31 12:28:07 CET

Eduard, could you have a look at this?

Comment 2 Alexander Kläser 2017-01-31 12:29:19 CET

Created attachment 8387 [details]
Adjustments to apply tests on 4.2-0 code

Here are the changes that I applied to both i18n tests.

Comment 3 Stefan Gohmann 2017-03-20 08:46:53 CET

For me, it is not release critical. So, my suggestion disable the test case and move it to 4.2-0-errata.

Comment 4 Eduard Mai 2017-03-22 10:39:00 CET

AFAIS the test ist skipped since r75906. The WIP tag is checked for: http://jenkins.knut.univention.de:8080/job/UCS-4.2/job/UCS-4.2-0/job/AutotestUpgrade/72/SambaVersion=s3,Systemrolle=master/testReport/85_ucs-translation-template/01_merge_new_module/test/

Comment 5 Stefan Gohmann 2017-03-22 10:40:52 CET

OK, leaving as open and move to 4.2-0-errata.

Comment 6 Eduard Mai 2017-03-22 11:00:18 CET

Thoughts on this: both tests 00 and 01 in 85_ucs-translation-template test suite are flawed in that they checkout the public Univention SVN repository during runtime. This will (theoretically) broaden the test surface. The downside is, that it will not play well in some of our existing testing environments and adds additional maintenance overhead when those change. Additionally its not necessary for the goal: test the univention-ucs-translation-* tools and the translationhelper module. For this, static testing data is sufficient.

Comment 7 Philipp Hahn 2019-09-20 15:21:21 CEST

The test still references SVN: <http://forge.univention.org/svn/dev/branches/ucs-4.1/ucs-4.1-1/management>
We switched to GIT: <https://github.com/univention/univention-corporate-server>