Page MenuHomeSolus
Differential D11621

gajim: Enable tests
ClosedPublic
Actions

Authored by algent on Aug 11 2021, 5:04 PM.
Tags
None
Referenced Files
F10898911: D11621.id.diff
Jul 3 2023, 8:52 AM
F10796808: D11621.id28158.diff
May 28 2023, 3:27 AM
F10730794: D11621.id28158.diff
May 11 2023, 12:11 AM
F10730180: D11621.id28138.diff
May 10 2023, 6:59 PM
F10730167: D11621.diff
May 10 2023, 6:54 PM
F10729907: D11621.id28144.diff
May 10 2023, 4:30 PM
F10727355: D11621.diff
May 9 2023, 8:54 PM
F10696492: D11621.id.diff
Apr 25 2023, 4:44 PM
Subscribers
joebonrichie
livingsilver94

Details

Reviewers
joebonrichie
Group Reviewers
Triage Team
Commits
R865:a3a0cef9017f: gajim: Enable tests
Summary

Per task T9866 enable tests and fix PyGIWarnings.

Signed-off-by: Algent Albrahimi <algent@protonmail.com>

Test Plan

Successfully pass build tests.

Diff Detail

Repository
R865 gajim
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

algent created this revision.Aug 11 2021, 5:04 PM
Herald added a reviewer: Triage Team. · View Herald TranscriptAug 11 2021, 5:04 PM
Harbormaster completed remote builds in B1350: Diff 28138.Aug 11 2021, 5:04 PM
algent requested review of this revision.Aug 11 2021, 5:04 PM
algent updated this revision to Diff 28144.Aug 11 2021, 8:54 PM

Fix patch

Harbormaster completed remote builds in B1356: Diff 28144.Aug 11 2021, 8:54 PM
livingsilver94 added subscribers: joebonrichie, livingsilver94.Aug 12 2021, 11:35 PM

As @joebonrichie said in D11313#188768, there's a way to enable the test that requires a screen.

algent added a comment.Aug 13 2021, 3:57 AM
In D11621#194021, @livingsilver94 wrote:

As @joebonrichie said in D11313#188768, there's a way to enable the test that requires a screen.

It doesn't work. IMO we should complete remove the check section. Didn't find any other distro that had enabled the tests.

joebonrichie added a comment.Aug 13 2021, 6:13 AM

Probably it's due to how the python macros are setup, you can't really append any variables to them.

joebonrichie accepted this revision.Aug 13 2021, 6:13 AM
This revision is now accepted and ready to land.Aug 13 2021, 6:13 AM
Closed by commit R865:a3a0cef9017f: gajim: Enable tests (authored by algent, committed by joebonrichie). · Explain WhyAug 13 2021, 6:13 AM
This revision was automatically updated to reflect the committed changes.
joebonrichie added a commit: R865:a3a0cef9017f: gajim: Enable tests.

Revision Contents
Changeset List

PathSize
files/
41 lines
7 lines
13 lines

Diff 28158

View Options

files/0001-Fix-PyGIWarning.patch

Loading...
View Options

package.yml

Loading...
View Options

pspec_x86_64.xml

Loading...
Copyright © 2015-2021 Solus Project. The Solus logo is Copyright © 2016-2021 Solus Project. All Rights Reserved.