Xqt has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/911255 )
Change subject: [fix] Fix make.bat file to show duration time
......................................................................
[fix] Fix make.bat file to show duration time
STARTTIME and ENDTIME is given without preleading "0".
Calculate hours respecting this fact.
use "nature" as sample theme
Change-Id: Ic298c74c9107a93fba87400278ecd55a8007e66e
---
M docs/make.bat
1 file changed, 19 insertions(+), 5 deletions(-)
Approvals:
jenkins-bot: Verified
Xqt: Looks good to me, approved
diff --git a/docs/make.bat b/docs/make.bat
index 7819bc3..bcc8d53 100644
--- a/docs/make.bat
+++ b/docs/make.bat
@@ -39,14 +39,14 @@
echo Start: %STARTTIME%
echo End: %ENDTIME%
-rem calculate elapsed time, code modified from
+rem calculate elapsed time, modified code from
rem https://stackoverflow.com/questions/4487100/how-can-i-use-a-windows-batch-f…
rem convert STARTTIME and ENDTIME to seconds
-set /A STARTTIME=(1%STARTTIME:~0,2%-100)*3600 + (1%STARTTIME:~3,2%-100)*60 + (1%STARTTIME:~6,2%-100)
-set /A ENDTIME=(1%ENDTIME:~0,2%-100)*3600 + (1%ENDTIME:~3,2%-100)*60 + (1%ENDTIME:~6,2%-100)
+set /A STARTTIME=(%STARTTIME:~0,2%)*3600 + (1%STARTTIME:~3,2%-100)*60 + (1%STARTTIME:~6,2%-100)
+set /A ENDTIME=(%ENDTIME:~0,2%)*3600 + (1%ENDTIME:~3,2%-100)*60 + (1%ENDTIME:~6,2%-100)
-rem calculating the duratyion is easy
+rem calculating the duration is easy
set /A DURATION=%ENDTIME%-%STARTTIME%
rem we might have measured the time inbetween days
@@ -70,7 +70,7 @@
echo.
echo make has the following options:
echo make ^<target^> [^<theme^>]
-echo make html basic
+echo make html nature
echo.
%SPHINXBUILD% -M help %SOURCEDIR% %BUILDDIR%
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/911255
To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ic298c74c9107a93fba87400278ecd55a8007e66e
Gerrit-Change-Number: 911255
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/i18n/+/912825 )
Change subject: Localisation updates from https://translatewiki.net.
......................................................................
Localisation updates from https://translatewiki.net.
Change-Id: I1541ab6a987bc8a4cd978b5d24c89d89331ad6c6
---
M redirect/ia.json
A replicate_wiki/ia.json
M protect/ia.json
A replicate_wiki/fr.json
M delinker/ia.json
A replicate_wiki/ko.json
A replicate_wiki/sl.json
M unprotect/ia.json
A replicate_wiki/he.json
A replicate_wiki/hi.json
A replicate_wiki/ru.json
M redirect/he.json
12 files changed, 92 insertions(+), 9 deletions(-)
Approvals:
L10n-bot: Looks good to me, approved
jenkins-bot: Verified
diff --git a/delinker/ia.json b/delinker/ia.json
index 070a87c..da03f95 100644
--- a/delinker/ia.json
+++ b/delinker/ia.json
@@ -4,5 +4,5 @@
"McDutchie"
]
},
- "delinker-delink": "Robot: Le file [[%(title)s]] ha essite removite perque illo ha essite delite per [[:User:%(user)s]]: ''%(comment)s''."
+ "delinker-delink": "Robot: Le file [[%(title)s]] ha essite removite perque illo ha essite delite per [[:User:%(user)s]]: ''%(comment)s''"
}
diff --git a/protect/ia.json b/protect/ia.json
index 0bc9c7f..daa0f4b 100644
--- a/protect/ia.json
+++ b/protect/ia.json
@@ -7,6 +7,6 @@
"protect-category": "Robot: Protege tote le paginas in categoria %(cat)s",
"protect-images": "Robot: Protege tote le imagines in le pagina %(page)s",
"protect-links": "Robot: Protege tote le paginas ligate ab %(page)s",
- "protect-ref": "Robot: Protege tote le paginas que refere a %(page)s",
+ "protect-ref": "Robot: Protection de tote le paginas que refere a %(page)s",
"protect-simple": "Robot: Protege un lista de files"
}
diff --git a/redirect/he.json b/redirect/he.json
index 128d57d..de47e62 100644
--- a/redirect/he.json
+++ b/redirect/he.json
@@ -8,8 +8,8 @@
]
},
"redirect-fix-broken-moved": "תיקון הפניה שבורה לדף היעד %(to)s, שהועבר",
- "redirect-fix-double": "מתקן הפניה כפולה → %(to)s",
- "redirect-fix-loop": "תיקון הפניה מעגלית ל%(to)s",
+ "redirect-fix-double": "מתקן הפניה כפולה לקישור %(to)s",
+ "redirect-fix-loop": "תיקון הפניה מעגלית לקישור %(to)s",
"redirect-remove-broken": "יעד ההפניה נמחק או שאינו קיים",
- "redirect-remove-loop": "הפניה זו גורמת ללולאה אין־סופית של הפניות"
+ "redirect-remove-loop": "ההפניה הזאת גורמת ללולאה אין־סופית של הפניות"
}
diff --git a/redirect/ia.json b/redirect/ia.json
index 2492fa5..f07d957 100644
--- a/redirect/ia.json
+++ b/redirect/ia.json
@@ -6,9 +6,9 @@
]
},
"redirect-broken-redirect-template": "{{eliminar|Redirection a un pagina delite o non existente}}",
- "redirect-fix-broken-moved": "Repara un redirection rupte verso un pagina renominate: %(to)s",
- "redirect-fix-double": "reparation de duple redirection → %(to)s",
- "redirect-fix-loop": "Repara redirection circular a %(to)s",
+ "redirect-fix-broken-moved": "Reparation de un redirection rupte verso le pagina renominate %(to)s",
+ "redirect-fix-double": "Reparation de un duple redirection a %(to)s",
+ "redirect-fix-loop": "Reparation de un bucla de redirectiones a %(to)s",
"redirect-remove-broken": "Redirection a un pagina delite o non existente",
"redirect-remove-loop": "Le destination del redirection forma un circulo de redirectiones"
}
diff --git a/replicate_wiki/fr.json b/replicate_wiki/fr.json
new file mode 100644
index 0000000..7128ec2
--- /dev/null
+++ b/replicate_wiki/fr.json
@@ -0,0 +1,12 @@
+{
+ "@metadata": {
+ "authors": [
+ "Verdy p"
+ ]
+ },
+ "replicate_wiki-headline": "Pages différentes de l’original",
+ "replicate_wiki-missing-users": "Administrateurs de l’original qui manquent ici",
+ "replicate_wiki-same-pages": "Toutes les pages importantes sont identiques",
+ "replicate_wiki-same-users": "Tous les utilisateurs de l’original sont également présents sur ce wiki",
+ "replicate_wiki-summary": "Bot : synchronisation du wiki à partir de %(source)s"
+}
diff --git a/replicate_wiki/he.json b/replicate_wiki/he.json
new file mode 100644
index 0000000..e6e3b1f
--- /dev/null
+++ b/replicate_wiki/he.json
@@ -0,0 +1,12 @@
+{
+ "@metadata": {
+ "authors": [
+ "Amire80"
+ ]
+ },
+ "replicate_wiki-headline": "דפים שונים מהמקור",
+ "replicate_wiki-missing-users": "מפעילים מהמקור שחסרים כאן",
+ "replicate_wiki-same-pages": "כל הדפים החשובים זהים",
+ "replicate_wiki-same-users": "כל המשתמשים מהמקור נמצאים גם בוויקי הזה",
+ "replicate_wiki-summary": "סנכרון ויקי מאתר %(source)s"
+}
diff --git a/replicate_wiki/hi.json b/replicate_wiki/hi.json
new file mode 100644
index 0000000..61aa36f
--- /dev/null
+++ b/replicate_wiki/hi.json
@@ -0,0 +1,9 @@
+{
+ "@metadata": {
+ "authors": [
+ "Abijeet Patro"
+ ]
+ },
+ "replicate_wiki-headline": "पृष्ठ जो मूल से भिन्न हैं",
+ "replicate_wiki-same-pages": "सभी महत्वपूर्ण पृष्ठ समान हैं"
+}
diff --git a/replicate_wiki/ia.json b/replicate_wiki/ia.json
new file mode 100644
index 0000000..2cc3f8d
--- /dev/null
+++ b/replicate_wiki/ia.json
@@ -0,0 +1,12 @@
+{
+ "@metadata": {
+ "authors": [
+ "McDutchie"
+ ]
+ },
+ "replicate_wiki-headline": "Paginas que differe del original",
+ "replicate_wiki-missing-users": "Administratores del original que manca hic",
+ "replicate_wiki-same-pages": "Tote le paginas importante es identic",
+ "replicate_wiki-same-users": "Tote le usatores del original es tamben presente sur iste wiki",
+ "replicate_wiki-summary": "Robot: Synchronisation wiki desde %(source)s"
+}
diff --git a/replicate_wiki/ko.json b/replicate_wiki/ko.json
new file mode 100644
index 0000000..aef96f8
--- /dev/null
+++ b/replicate_wiki/ko.json
@@ -0,0 +1,8 @@
+{
+ "@metadata": {
+ "authors": [
+ "Ykhwong"
+ ]
+ },
+ "replicate_wiki-same-pages": "중요한 모든 문서가 동일합니다"
+}
diff --git a/replicate_wiki/ru.json b/replicate_wiki/ru.json
new file mode 100644
index 0000000..63e01c5
--- /dev/null
+++ b/replicate_wiki/ru.json
@@ -0,0 +1,9 @@
+{
+ "@metadata": {
+ "authors": [
+ "Okras"
+ ]
+ },
+ "replicate_wiki-headline": "Страницы, отличающиеся от оригинала",
+ "replicate_wiki-same-pages": "Все важные страницы одинаковы"
+}
diff --git a/replicate_wiki/sl.json b/replicate_wiki/sl.json
new file mode 100644
index 0000000..812b895
--- /dev/null
+++ b/replicate_wiki/sl.json
@@ -0,0 +1,12 @@
+{
+ "@metadata": {
+ "authors": [
+ "Eleassar"
+ ]
+ },
+ "replicate_wiki-headline": "Strani, ki se razlikujejo od izvirnika",
+ "replicate_wiki-missing-users": "Administratorji iz izvirnika, ki tukaj manjkajo",
+ "replicate_wiki-same-pages": "Vse pomembne strani so enake",
+ "replicate_wiki-same-users": "Vsi uporabniki iz izvirnika so prisotni tudi v tem vikiju",
+ "replicate_wiki-summary": "Bot: Vikisinhronizacija iz %(source)s"
+}
diff --git a/unprotect/ia.json b/unprotect/ia.json
index c2b9e75..a49b620 100644
--- a/unprotect/ia.json
+++ b/unprotect/ia.json
@@ -7,6 +7,6 @@
"unprotect-category": "Robot: Disprotege tote le paginas del categoria %(cat)s",
"unprotect-images": "Robot: Disprotege tote le files sur le pagina %(page)s",
"unprotect-links": "Robot: Disprotege tote le paginas ligate ab %(page)s",
- "unprotect-ref": "Robot: Disprotege tote le paginas que refere a %(page)s",
+ "unprotect-ref": "Robot: Disprotection de tote le paginas que refere a %(page)s",
"unprotect-simple": "Robot: Disprotege un lista de files"
}
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/i18n/+/912825
To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/i18n
Gerrit-Branch: master
Gerrit-Change-Id: I1541ab6a987bc8a4cd978b5d24c89d89331ad6c6
Gerrit-Change-Number: 912825
Gerrit-PatchSet: 1
Gerrit-Owner: L10n-bot <l10n-bot(a)translatewiki.net>
Gerrit-Reviewer: L10n-bot <l10n-bot(a)translatewiki.net>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/i18n/+/904891 )
Change subject: [i18n] Add i18n support for replicate_wiki
......................................................................
[i18n] Add i18n support for replicate_wiki
Bug: T333759
Change-Id: I2325a4a416a6e9d6e741c0356b5d21c10984ab62
---
A replicate_wiki/qqq.json
A replicate_wiki/en.json
2 files changed, 35 insertions(+), 0 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/replicate_wiki/en.json b/replicate_wiki/en.json
new file mode 100644
index 0000000..e248c02
--- /dev/null
+++ b/replicate_wiki/en.json
@@ -0,0 +1,13 @@
+{
+ "@metadata": {
+ "authors": [
+ "Valhallasw",
+ "Xqt"
+ ]
+ },
+ "replicate_wiki-headline": "Pages that differ from original",
+ "replicate_wiki-missing-users": "Admins from original that are missing here",
+ "replicate_wiki-same-pages": "All important pages are the same",
+ "replicate_wiki-same-users": "All users from original are also present on this wiki",
+ "replicate_wiki-summary": "Bot: Wiki synchronization from %(source)s"
+}
diff --git a/replicate_wiki/qqq.json b/replicate_wiki/qqq.json
new file mode 100644
index 0000000..84123c7
--- /dev/null
+++ b/replicate_wiki/qqq.json
@@ -0,0 +1,12 @@
+{
+ "@metadata": {
+ "authors": [
+ "Xqt"
+ ]
+ },
+ "replicate_wiki-headline": "Headline for replication overview for pages that differ from source wiki",
+ "replicate_wiki-missing-users": "Headline for replication overview for users missing on target wiki",
+ "replicate_wiki-same-pages": "Text if no important pages differ from source",
+ "replicate_wiki-same-users": "Text if no user from original wiki is missing on target wiki",
+ "replicate_wiki-summary": "Edit summary when the bot transfers pages from a source wiki to target. Parameters:\n* %(source)s - name of the source wiki"
+}
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/i18n/+/904891
To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/i18n
Gerrit-Branch: master
Gerrit-Change-Id: I2325a4a416a6e9d6e741c0356b5d21c10984ab62
Gerrit-Change-Number: 904891
Gerrit-PatchSet: 3
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
jenkins-bot has submitted this change. ( https://gerrit.wikimedia.org/r/c/pywikibot/core/+/909314 )
Change subject: [IMPR] Revise Family class
......................................................................
[IMPR] Revise Family class
- remove obsolete.setter which is only used for tests
- move interwiki_replacements and interwiki_removals class
properties from WikimediaFamily to Family class to have
a unique result for all families
- add empty closed_wikis, removed_wikis and code_aliases to
Family class which are used by interwiki_replacements and
interwiki_removals; remove closed_wikis and removed_wikis
from WikimediaFamily.
- simplify SingleSiteFamily because closed_wikis is always present
- codes must be a List not a Tuple in Family classes because SubdomainFamily
modifies codes; update wikihow and wowwiki families
- update tests accordingly
This changes enables to change the interface for removed or closed sites
with wowwiki and others.
Bug: T334834
Change-Id: Ib22cbb964670471a60b47d89734fb4d63cea5710
---
M pywikibot/families/wikihow_family.py
M pywikibot/family.py
M pywikibot/families/wowwiki_family.py
M tests/family_tests.py
4 files changed, 95 insertions(+), 61 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/families/wikihow_family.py b/pywikibot/families/wikihow_family.py
index d72fe38..d49b642 100644
--- a/pywikibot/families/wikihow_family.py
+++ b/pywikibot/families/wikihow_family.py
@@ -3,7 +3,7 @@
.. versionadded:: 3.0
"""
#
-# (C) Pywikibot team, 2020-2022
+# (C) Pywikibot team, 2020-2023
#
# Distributed under the terms of the MIT license.
#
@@ -21,10 +21,10 @@
name = 'wikihow'
domain = 'wikihow.com'
- codes = (
+ codes = [
'ar', 'cs', 'de', 'en', 'es', 'fr', 'hi', 'id', 'it', 'ja', 'ko', 'nl',
'pt', 'ru', 'th', 'tr', 'vi', 'zh',
- )
+ ]
removed_wikis = ['ca', 'cy', 'fa', 'he', 'pl', 'ur']
diff --git a/pywikibot/families/wowwiki_family.py b/pywikibot/families/wowwiki_family.py
index b683c71..fdb5871 100644
--- a/pywikibot/families/wowwiki_family.py
+++ b/pywikibot/families/wowwiki_family.py
@@ -15,11 +15,11 @@
name = 'wowwiki'
domain = 'wowwiki.fandom.com'
- codes = (
+ codes = [
'ar', 'cs', 'da', 'de', 'el', 'en', 'es', 'et', 'fa', 'fi', 'fr', 'he',
'hu', 'it', 'ja', 'ko', 'nl', 'no', 'pl', 'pt', 'pt-br', 'ru', 'uk',
'zh', 'zh-tw',
- )
+ ]
removed_wikis = ['is', 'hr', 'lt', 'lv', 'ro', 'sk', 'sr', 'sv', 'tr']
diff --git a/pywikibot/family.py b/pywikibot/family.py
index 9966afe..1d9d0b8 100644
--- a/pywikibot/family.py
+++ b/pywikibot/family.py
@@ -1,6 +1,6 @@
"""Objects representing MediaWiki families."""
#
-# (C) Pywikibot team, 2004-2022
+# (C) Pywikibot team, 2004-2023
#
# Distributed under the terms of the MIT license.
#
@@ -18,9 +18,11 @@
import pywikibot
from pywikibot import config
-from pywikibot.backports import ( # skipcq: PY-W2000
+from pywikibot.backports import (
Dict,
+ FrozenSet,
List,
+ Mapping,
Set,
Tuple,
removesuffix,
@@ -45,6 +47,8 @@
.. versionchanged:: 8.0
``alphabetic``, ``alphabetic_revised`` and ``fyinterwiki``
attributes where removed.
+ .. versionchanged:: 8.2
+ :attr:`obsolete` setter was removed.
"""
def __new__(cls):
@@ -82,6 +86,19 @@
name = None
+ #: Not open for edits; stewards can still edit.
+ closed_wikis: List[str] = []
+
+ #: Completely removed sites
+ removed_wikis: List[str] = []
+
+ code_aliases: Dict[str, str] = {}
+ """Code mappings which are only an alias, and there is no 'old' wiki.
+
+ For all except 'nl_nds', subdomains do exist as a redirect, but that
+ should not be relied upon.
+ """
+
langs: Dict[str, str] = {}
# A list of category redirect template names in different languages
@@ -161,15 +178,6 @@
# family.
interwiki_forward = None
- # Which language codes no longer exist and by which language code
- # should they be replaced. If for example the language with code xx:
- # now should get code yy:, add {'xx':'yy'} to obsolete.
- interwiki_replacements: Dict[str, str] = {}
-
- # Codes that should be removed, usually because the site has been
- # taken down.
- interwiki_removals: List[str] = []
-
# Language codes of the largest wikis. They should be roughly sorted
# by size.
languages_by_size: List[str] = []
@@ -528,7 +536,7 @@
"""Return the path to title using index.php with redirects disabled."""
return f'{self.path(code)}?title={title}&redirect=no'
- def interface(self, code) -> str:
+ def interface(self, code: str) -> str:
"""Return interface to use for code."""
if code in self.interwiki_removals:
if code in self.codes:
@@ -691,16 +699,6 @@
data.update(self.interwiki_replacements)
return types.MappingProxyType(data)
- @obsolete.setter
- def obsolete(self, data) -> None:
- """Split obsolete dict into constituent parts."""
- self.interwiki_removals[:] = [old for (old, new) in data.items()
- if new is None]
- self.interwiki_replacements.clear()
- self.interwiki_replacements.update((old, new)
- for (old, new) in data.items()
- if new is not None)
-
@classproperty
def domains(cls) -> Set[str]:
"""
@@ -719,6 +717,32 @@
"""
return set(cls.langs.keys())
+ @classproperty
+ def interwiki_replacements(cls) -> Mapping[str, str]:
+ """Return an interwiki code replacement mapping.
+
+ Which language codes no longer exist and by which language code
+ should they be replaced. If for example the language with code
+ xx: now should get code yy:, add {'xx':'yy'} to
+ :attr:`code_aliases`.
+
+ .. versionchanged:: 8.2
+ changed from dict to invariant mapping.
+ """
+ return types.MappingProxyType(cls.code_aliases)
+
+ @classproperty
+ def interwiki_removals(cls) -> FrozenSet[str]:
+ """Return a list of interwiki codes to be removed from wiki pages.
+
+ Codes that should be removed, usually because the site has been
+ taken down.
+
+ .. versionchanged:: 8.2
+ changed from list to invariant frozenset.
+ """
+ return frozenset(cls.removed_wikis + cls.closed_wikis)
+
class SingleSiteFamily(Family):
@@ -761,16 +785,13 @@
if hasattr(cls, 'test_codes'):
codes += cls.test_codes
- if hasattr(cls, 'closed_wikis'):
- codes += cls.closed_wikis
+
+ codes += cls.closed_wikis
# shortcut this classproperty
- cls.langs = {code: f'{code}.{cls.domain}'
- for code in codes}
-
- if hasattr(cls, 'code_aliases'):
- cls.langs.update({alias: f'{code}.{cls.domain}'
- for alias, code in cls.code_aliases.items()})
+ cls.langs = {code: f'{code}.{cls.domain}' for code in codes}
+ cls.langs.update({alias: f'{code}.{cls.domain}'
+ for alias, code in cls.code_aliases.items()})
return cls.langs
@@ -934,11 +955,6 @@
'be-x-old': 'be-tarask',
}
- # Not open for edits; stewards can still edit.
- closed_wikis: List[str] = []
- # Completely removed
- removed_wikis: List[str] = []
-
# WikimediaFamily uses Wikibase for the category name containing
# disambiguation pages for the various languages. We need the
# Wikibase code and item number:
@@ -960,16 +976,6 @@
raise NotImplementedError(
f"Family {cls.name} needs to define property 'domain'")
- @classproperty
- def interwiki_removals(cls):
- """Return a list of interwiki codes to be removed from wiki pages."""
- return frozenset(cls.removed_wikis + cls.closed_wikis)
-
- @classproperty
- def interwiki_replacements(cls):
- """Return an interwiki code replacement mapping."""
- return types.MappingProxyType(cls.code_aliases)
-
def shared_image_repository(self, code):
"""Return Wikimedia Commons as the shared image repository."""
return ('commons', 'commons')
diff --git a/tests/family_tests.py b/tests/family_tests.py
index 2e65095..3ca109e 100755
--- a/tests/family_tests.py
+++ b/tests/family_tests.py
@@ -1,7 +1,7 @@
#!/usr/bin/env python3
"""Tests for the family module."""
#
-# (C) Pywikibot team, 2014-2022
+# (C) Pywikibot team, 2014-2023
#
# Distributed under the terms of the MIT license.
#
@@ -103,6 +103,8 @@
family = Family.load('wikipedia')
self.assertIsInstance(family.obsolete, Mapping)
# redirected code (see site tests test_alias_code_site)
+ self.assertEqual(family.code_aliases['dk'], 'da')
+ self.assertEqual(family.interwiki_replacements['dk'], 'da')
self.assertEqual(family.obsolete['dk'], 'da')
# closed/locked site (see site tests test_locked_site)
self.assertIsNone(family.obsolete['mh'])
@@ -110,19 +112,21 @@
self.assertIsNone(family.obsolete['ru-sib'])
self.assertIn('dk', family.interwiki_replacements)
- def test_set_obsolete(self):
- """Test obsolete can be set."""
+ def test_obsolete_from_attributes(self):
+ """Test obsolete property for given class attributes."""
# Construct a temporary family and instantiate it
family = type('TempFamily', (Family,), {})()
self.assertEqual(family.obsolete, {})
self.assertEqual(family.interwiki_replacements, {})
- self.assertEqual(family.interwiki_removals, [])
+ self.assertEqual(family.interwiki_removals, frozenset())
- family.obsolete = {'a': 'b', 'c': None}
+ # Construct a temporary family with other attributes and instantiate it
+ family = type('TempFamily', (Family,),
+ {'code_aliases': {'a': 'b'}, 'closed_wikis': ['c']})()
self.assertEqual(family.obsolete, {'a': 'b', 'c': None})
self.assertEqual(family.interwiki_replacements, {'a': 'b'})
- self.assertEqual(family.interwiki_removals, ['c'])
+ self.assertEqual(family.interwiki_removals, frozenset('c'))
def test_obsolete_readonly(self):
"""Test obsolete result not updatable."""
@@ -137,12 +141,10 @@
"'mappingproxy' object does not support item assignment"):
family.obsolete['a'] = 'b'
- def test_WikimediaFamily_obsolete_readonly(self):
- """Test WikimediaFamily obsolete is readonly."""
- family = Family.load('wikipedia')
with self.assertRaisesRegex(
- TypeError,
- "'frozenset' object does not support item assignment"):
+ AttributeError,
+ "property 'obsolete' of 'Family' object has no setter|"
+ "can't set attribute"):
family.obsolete = {'a': 'b', 'c': None}
--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/909314
To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ib22cbb964670471a60b47d89734fb4d63cea5710
Gerrit-Change-Number: 909314
Gerrit-PatchSet: 9
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged