Bug 44037 - Test cases 20_appcenter/7*_umc* fail in UCS 4.2
Test cases 20_appcenter/7*_umc* fail in UCS 4.2
Status: CLOSED FIXED
Product: UCS Test
Classification: Unclassified
Component: App Center
unspecified
Other Linux
: P5 normal (vote)
: ---
Assigned To: Philipp Hahn
Florian Best
:
Depends on: 44346
Blocks:
  Show dependency treegraph
 
Reported: 2017-03-23 06:24 CET by Stefan Gohmann
Modified: 2017-06-23 18:38 CEST (History)
1 user (show)

See Also:
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:


Attachments
ucs-test-appcenter.log (44.61 KB, text/x-log)
2017-03-23 06:24 CET, Stefan Gohmann
Details

Note You need to log in before you can comment on or make changes to this bug.
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!