Bug 44037

Summary: Test cases 20_appcenter/7*_umc* fail in UCS 4.2
Product: UCS Test Reporter: Stefan Gohmann <gohmann>
Component: App CenterAssignee: Philipp Hahn <hahn>
Status: CLOSED FIXED QA Contact: Florian Best <best>
Severity: normal    
Priority: P5 CC: hahn
Version: unspecified   
Target Milestone: ---   
Hardware: Other   
OS: Linux   
What kind of report is it?: Development Internal What type of bug is this?: ---
Who will be affected by this bug?: --- How will those affected feel about the bug?: ---
User Pain: Enterprise Customer affected?:
School Customer affected?: ISV affected?:
Waiting Support: Flags outvoted (downgraded) after PO Review:
Ticket number: Bug group (optional):
Max CVSS v3 score:
Bug Depends on: 44346    
Bug Blocks:    
Attachments: ucs-test-appcenter.log

Description Stefan Gohmann univentionstaff 2017-03-23 06:24:23 CET
Created attachment 8624 [details]
ucs-test-appcenter.log

The test cases 20_appcenter/7*_umc* fail in UCS 4.2 due to changes in UCS 4.2 App Center.

I've disabled the test cases.
Comment 1 Stefan Gohmann univentionstaff 2017-03-23 06:25:47 CET
(In reply to Stefan Gohmann from comment #0)
> I've disabled the test cases.

r78171:
* 20_appcenter/7*umc*: Temporary skip test case (Bug #44037)
Comment 2 Florian Best univentionstaff 2017-05-09 19:57:43 CEST
r79255 | Bug #44037: fix some typos

I found that it makes requests to meta-inf/4.1/ instead of /meta-inf/4.2/.
The tests pass if I remove 4.1 from:
80_docker/dockertest.py:»   »   »   self._write_ucs_ini('[4.2]\nSupportedUCSVersions=4.2, 4.1\n')
→ but the file is the same on http://appcenter-test.software-univention.de/meta-inf/ucs.ini so we shouldn't change this in the test case.
Maybe the order of supported_ucs_versions is checked wrong.
Comment 3 Florian Best univentionstaff 2017-05-10 13:08:41 CEST
Fixed by building the local appcenter for all UCS versions which are supported.

ucs-test (7.0.21-23):
r79270 | Bug #44037: fix UMC appcenter tests
Comment 4 Florian Best univentionstaff 2017-05-12 13:42:40 CEST
The test cases are still failing on EC2 (not on my VM :/).
Comment 5 Florian Best univentionstaff 2017-05-29 16:06:17 CEST
The tests somehow think they are running on UCS 4.3:

Warnung: Fehlschlag beim Holen von http://localhost/univention-repository/4.3/maintained/component/test32549/all/Packages  404  Not Found [IP: ::1 80].
29.05.17 02:27:16.740  MODULE      ( WARN    ) : Warnung: Fehlschlag beim Holen von http://localhost/univention-repository/4.3/maintained/4.3-0/all/Packages  404  Not Found [IP: ::1 80].
29.05.17 02:27:56.140  MODULE      ( PROCESS ) : Ign  http://localhost 4.3-0/all/ InRelease
29.05.17 02:27:56.142  MODULE      ( PROCESS ) : Ign  http://localhost 4.3-0/all/ Release.gpg
29.05.17 02:27:56.143  MODULE      ( PROCESS ) : Ign  http://localhost 4.3-0/all/ Release
29.05.17 02:27:56.144  MODULE      ( PROCESS ) : Fehl http://localhost 4.3-0/all/ Packages
29.05.17 02:27:56.146  MODULE      ( PROCESS ) : Fehl http://localhost 4.3-0/all/ Packages
29.05.17 02:27:56.147  MODULE      ( PROCESS ) : Fehl http://localhost 4.3-0/all/ Packages
29.05.17 02:27:56.160  MODULE      ( PROCESS ) : Fehl http://localhost 4.3-0/all/ Packages
29.05.17 02:27:56.204  MODULE      ( PROCESS ) : Fehl http://localhost 4.3-0/all/ Packages
29.05.17 02:27:56.205  MODULE      ( PROCESS ) : Fehl http://localhost 4.3-0/all/ Packages
Comment 7 Philipp Hahn univentionstaff 2017-06-02 13:51:28 CEST
r80019 | Bug #44037 test/up: Clean "update in process APT source"
 Bug #44346 introduced a work-around to copy the "temporary update
 sources list" to "update in process list", as otherwise after a failed
 update the new packages are no longer available.
    
 This introduces a regression in ucs-test for the updater, as test 54
 contains such a test case, where the update fails. This leaves behind a
 temporary source entry, which is no longer valid afterwards.
    
 Remove any "update in process list" after each test.

Package: ucs-test
Version: 7.0.21-36A~4.2.0.201706021350
Branch: ucs_4.2-0
Scope: errata4.2-0
Comment 8 Florian Best univentionstaff 2017-06-12 15:17:45 CEST
Thanks for the help!