From a5e5a3f4646d9a173c679e5a8128a4209b78e89d Mon Sep 17 00:00:00 2001 From: Mpaa Date: Mon, 29 May 2017 22:57:51 +0200 Subject: [PATCH] FilePage: remove deprecated use of fileUrl Use get_file_url in favor of deprecated fileUrl. Change-Id: I06ab79183b13918b89abfb3b7e7f3a27becfe1cb --- pywikibot/page.py | 4 ++-- tests/file_tests.py | 28 +++++++++++++++++++--------- 2 files changed, 21 insertions(+), 11 deletions(-) diff --git a/pywikibot/page.py b/pywikibot/page.py index 49f282e751..3dbd1d8641 100644 --- a/pywikibot/page.py +++ b/pywikibot/page.py @@ -2495,10 +2495,10 @@ def fileIsShared(self): if not self.site.has_image_repository: return False elif 'wikitravel_shared' in self.site.shared_image_repository(): - return self.fileUrl().startswith( + return self.latest_file_info.url.startswith( u'http://wikitravel.org/upload/shared/') else: - return self.fileUrl().startswith( + return self.latest_file_info.url.startswith( 'https://upload.wikimedia.org/wikipedia/commons/') @deprecated("FilePage.latest_file_info.sha1") diff --git a/tests/file_tests.py b/tests/file_tests.py index 483b4847e1..324f2c15c0 100644 --- a/tests/file_tests.py +++ b/tests/file_tests.py @@ -20,7 +20,7 @@ class TestShareFiles(TestCase): - """Test methods fileIsShared, exists and fileUrl with shared files.""" + """Test fileIsShared, exists and fileUrl/get_file_url with shared files.""" sites = { 'enwiki': { @@ -43,6 +43,16 @@ class TestShareFiles(TestCase): cached = True + def test_fileUrl_versus_get_file_url(self): + """Test fileUrl() is equivalent to get_file_url().""" + title = 'File:Sepp Maier 1.JPG' + commons = self.get_site('commons') + commons_file = pywikibot.FilePage(commons, title) + self.assertEqual(commons_file.fileUrl(), commons_file.get_file_url()) + itwp = self.get_site('itwiki') + itwp_file = pywikibot.FilePage(itwp, title) + self.assertEqual(itwp_file.fileUrl(), itwp_file.get_file_url()) + def testSharedOnly(self): """Test fileIsShared() on file page with shared file only.""" title = 'File:Sepp Maier 1.JPG' @@ -60,9 +70,9 @@ def testSharedOnly(self): self.assertTrue(itwp_file.fileIsShared()) self.assertTrue(commons_file.fileIsShared()) - self.assertTrue(commons_file.fileUrl()) + self.assertTrue(commons_file.get_file_url()) - self.assertIn('/wikipedia/commons/', itwp_file.fileUrl()) + self.assertIn('/wikipedia/commons/', itwp_file.get_file_url()) self.assertRaises(pywikibot.NoPage, itwp_file.get) def testLocalOnly(self): @@ -78,14 +88,14 @@ def testLocalOnly(self): commons_file = pywikibot.FilePage(commons, title) - self.assertTrue(enwp_file.fileUrl()) + self.assertTrue(enwp_file.latest_file_info.url) self.assertTrue(enwp_file.exists()) self.assertFalse(commons_file.exists()) self.assertFalse(enwp_file.fileIsShared()) self.assertRaises(pywikibot.NoPage, commons_file.fileIsShared) - self.assertRaises(pywikibot.NoPage, commons_file.fileUrl) + self.assertRaises(pywikibot.NoPage, commons_file.get_file_url) self.assertRaises(pywikibot.NoPage, commons_file.get) def testOnBoth(self): @@ -100,7 +110,7 @@ def testOnBoth(self): commons_file = pywikibot.FilePage(commons, title) - self.assertTrue(itwp_file.fileUrl()) + self.assertTrue(itwp_file.get_file_url()) self.assertTrue(itwp_file.exists()) self.assertTrue(commons_file.exists()) @@ -115,13 +125,13 @@ def testNonFileLocal(self): testwp = self.get_site('testwiki') testwp_file = pywikibot.FilePage(testwp, title) - self.assertTrue(testwp_file.fileUrl()) + self.assertTrue(testwp_file.latest_file_info.url) self.assertTrue(testwp_file.exists()) self.assertTrue(testwp_file.fileIsShared()) commons_file = pywikibot.FilePage(commons, title) - self.assertEqual(testwp_file.fileUrl(), - commons_file.fileUrl()) + self.assertEqual(testwp_file.get_file_url(), + commons_file.get_file_url()) class TestFilePage(TestCase):