Index: univention/testing/udm_extensions.py =================================================================== --- univention/testing/udm_extensions.py (Revision 44858) +++ univention/testing/udm_extensions.py (Arbeitskopie) @@ -222,7 +222,7 @@ if not app_id: app_id = '' else: - app_id = 'UNIVENTION_APP_IDENTIFIER="%s"' % app_id + app_id = 'export UNIVENTION_APP_IDENTIFIER="%s"' % app_id if not version_start: version_start = '' else: @@ -269,7 +269,7 @@ if not app_id: app_id = '' else: - app_id = 'UNIVENTION_APP_IDENTIFIER="%s"' % app_id + app_id = 'export UNIVENTION_APP_IDENTIFIER="%s"' % app_id if not version_start: version_start = '' else: Index: tests/72_udm-extensions/10_update_extension_via_package =================================================================== --- tests/72_udm-extensions/10_update_extension_via_package (Revision 44858) +++ tests/72_udm-extensions/10_update_extension_via_package (Arbeitskopie) @@ -31,8 +31,9 @@ version_start = random_ucs_version(max_major=2) version_end = random_ucs_version(min_major=5) - # create special extension name - extension_identifier = '%s_%s' % (extension_name, package_version.replace('.','_')) + # extension name is the one finally appearing in the "name" attribute of the UDM module + # extension_identifier = '%s_%s' % (extension_name, package_version.replace('.','_')) + extension_identifier = '%s' % (extension_name,) extension_buffer = get_extension_buffer(extension_type, extension_identifier) joinscript_buffer = get_join_script_buffer(extension_type, '/usr/share/%s/%s' % (package_name, extension_filename), Index: tests/72_udm-extensions/12_update_extension_via_other_packagename =================================================================== --- tests/72_udm-extensions/12_update_extension_via_other_packagename (Revision 44858) +++ tests/72_udm-extensions/12_update_extension_via_other_packagename (Arbeitskopie) @@ -31,8 +31,9 @@ version_start = random_ucs_version(max_major=2) version_end = random_ucs_version(min_major=5) - # create special extension name - extension_identifier = '%s_%s' % (extension_name, package_version.replace('.','_')) + # extension name is the one finally appearing in the "name" attribute of the UDM module + # extension_identifier = '%s_%s' % (extension_name, package_version.replace('.','_')) + extension_identifier = '%s' % (extension_name,) extension_buffer = get_extension_buffer(extension_type, extension_identifier) joinscript_buffer = get_join_script_buffer(extension_type, '/usr/share/%s/%s' % (package_name, extension_filename), Index: tests/72_udm-extensions/41_test_udm_module =================================================================== --- tests/72_udm-extensions/41_test_udm_module (Revision 44858) +++ tests/72_udm-extensions/41_test_udm_module (Arbeitskopie) @@ -8,7 +8,7 @@ from univention.config_registry import ConfigRegistry import univention.testing.udm as udm_test -from univention.testing.utils import verify_ldap_object, wait_for_replication_and_postrun, fail +from univention.testing.utils import verify_ldap_object, wait_for_replication, fail from univention.testing.strings import random_name, random_version, random_ucs_version from univention.testing.udm_extensions import * import subprocess @@ -43,7 +43,7 @@ ucsversionend = version_end, active = 'FALSE') - wait_for_replication_and_postrun() + wait_for_replication() udm.stop_cli_server() verify_ldap_object(dn, { @@ -57,14 +57,14 @@ }) - output = subprocess.Popen(['udm', 'modules'], stdout=subprocess.PIPE).communicate() + output, stderr = subprocess.Popen(['udm', 'modules'], stdout=subprocess.PIPE).communicate() if not extension_name in output: fail('ERROR: udm cli server has not been reloaded yet or module registration failed') extension_dn = udm.create_object(extension_name, position=ucr.get('ldap/base'), name=object_name) udm.remove_object(extension_name, dn=extension_dn) - wait_for_replication_and_postrun() + wait_for_replication() udm.stop_cli_server() with udm_test.UCSTestUDM() as udm: Index: tests/72_udm-extensions/11_update_extension_via_package_expected_fail =================================================================== --- tests/72_udm-extensions/11_update_extension_via_package_expected_fail (Revision 44858) +++ tests/72_udm-extensions/11_update_extension_via_package_expected_fail (Arbeitskopie) @@ -27,8 +27,9 @@ packages = [] try: for package_version in (package_version_HIGH, package_version_LOW): - # create special extension name - extension_identifier = '%s_%s' % (extension_name, package_version.replace('.','_')) + # extension name is the one finally appearing in the "name" attribute of the UDM module + # extension_identifier = '%s_%s' % (extension_name, package_version.replace('.','_')) + extension_identifier = '%s' % (extension_name,) extension_buffer = get_extension_buffer(extension_type, extension_identifier) joinscript_buffer = get_join_script_buffer(extension_type, '/usr/share/%s/%s' % (package_name, extension_filename), @@ -58,7 +59,7 @@ content = open(get_absolute_extension_filename(extension_type, extension_filename)).read() if package_version == package_version_HIGH and not extension_identifier in content: fail('ERROR: UDM extension of package %d has not been written to disk (%s)' % (len(packages), extension_filename,)) - if package_version == package_version_LOW and not extension_identifier in content: + if package_version == package_version_LOW and extension_identifier in content: fail('ERROR: the extension update has been performed but should not (old version=%s ; new version=%s)' % (package_version_HIGH, package_version_LOW)) finally: