jenkins-bot submitted this change.

View Change


Approvals: jenkins-bot: Verified Xqt: Looks good to me, approved
[tests] Show site if skipping tests due to missing rights

Change-Id: I2541bdc05e98c2fe8fb563d9486e4797ebcdef79
---
M tests/aspects.py
1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/tests/aspects.py b/tests/aspects.py
index 4c3d1fb..4ae285e 100644
--- a/tests/aspects.py
+++ b/tests/aspects.py
@@ -675,9 +675,9 @@

for right in cls.rights.split(','):
if not site.has_right(right):
- raise unittest.SkipTest('User "{}" does not have required '
- 'user right "{}"'
- .format(site.user(), right))
+ raise unittest.SkipTest(
+ f'User "{site.user()}" does not have required user '
+ f'right "{right}" on site "{site}"')


class MetaTestCaseClass(type):

To view, visit change 1004659. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I2541bdc05e98c2fe8fb563d9486e4797ebcdef79
Gerrit-Change-Number: 1004659
Gerrit-PatchSet: 2
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged