0001-matomo-add-services-urls-on-re-configuration-37614.patch

Nicolas Roche, 12 novembre 2019 21:19

Voir les différences: en ligne côte à côte

Subject: [PATCH] matomo: add services urls on re-configuration (#37614)

 hobo/matomo/utils.py       | 17 +++++++++--
 tests/test_matomo_utils.py | 60 ++++++++++++++++++++++++++++++++------
 tests/test_matomo_views.py | 10 +++++--
 3 files changed, 72 insertions(+), 15 deletions(-)

                 raise MatomoException('get_site_id_from_site_url fails')
             return tag.text
         def add_site(self, site_name, site_urls):
         def add_site(self, site_name):
             data = {'method': 'SitesManager.addSite',
                     'siteName': site_name}
             tree = self.call(data)
             try:
                 tag = tree.xpath('/result')[0]
             except IndexError:
                 raise MatomoException('add_site fails')
             return tag.text
         def add_site_alias_urls(self, id_site, site_urls):
             data = {'method': 'SitesManager.addSiteAliasUrls',
                     'idSite': id_site}
             cpt = 0
             for url in site_urls:
                 key = 'urls[%i]' % cpt
-...
             try:
                 tag = tree.xpath('/result')[0]
             except IndexError:
                 raise MatomoException('add_site fails')
                 raise MatomoException('add_site_alias_urls fails')
             return tag.text
         def add_user(self, user_login, password, initial_id_site):
-...
             id_site = matomo.get_site_id_from_site_url(tenant_name)
         except MatomoError as exc:
             if str(exc) == 'url not found':
                 id_site = matomo.add_site(tenant_name, site_urls)
                 id_site = matomo.add_site(tenant_name)
             else:
                 raise exc
         matomo.add_site_alias_urls(id_site, site_urls)
         return id_site
     def upgrade_user(matomo, user_login, id_site):

     <not_result>no result tag</not_result>
     """
     ADD_SITE_ALIAS_URLS_SUCCESS = """<?xml version="1.0" encoding="utf-8" ?>
     <result>1</result>
     """
     ADD_SITE_ALIAS_URLS_ERROR = """<?xml version="1.0" encoding="utf-8" ?>
     <result>
             <error message="Merci de specifier une valeur pour idSite" />
     </result>
     """
     USER_ALREADY_THERE = """<?xml version="1.0" encoding="utf-8" ?>
     <result>
         <error message="Username 'hobo.dev.publik.love' already exists." />
-...
     def test_add_site(mocked_post):
         """webservice to add a new site"""
         mocked_post.return_value.status_code = 200
         urls = ['https://combo.dev.publik.love',
                 'https://wcs.dev.publik.love']
         with override_settings(MATOMO_SERVER=CONFIG):
             matomo = MatomoWS()
             # success
             content = ADD_SITE_SUCCESS
             mocked_post.return_value.content = content
             site_id = matomo.add_site("hobo.dev.publik.love", urls)
             site_id = matomo.add_site("hobo.dev.publik.love")
             assert site_id == '42'
             # error
-...
             mocked_post.return_value.content = content
             with pytest.raises(MatomoError,
                                match="Please specify a value for 'siteName'."):
                 site_id = matomo.add_site("hobo.dev.publik.love", urls)
                 site_id = matomo.add_site("hobo.dev.publik.love")
             # strange message
             content = ADD_SITE_BAD_RESPONSE
             mocked_post.return_value.content = content
             with pytest.raises(MatomoException, match='add_site fails'):
                 site_id = matomo.add_site("hobo.dev.publik.love", urls)
                 site_id = matomo.add_site("hobo.dev.publik.love")
     @mock.patch('requests.post')
     def test_add_site_alias_url(mocked_post):
         """webservice to add a new site"""
         mocked_post.return_value.status_code = 200
         urls = ['https://combo.dev.publik.love',
                 'https://wcs.dev.publik.love']
         with override_settings(MATOMO_SERVER=CONFIG):
             matomo = MatomoWS()
             # success
             content = ADD_SITE_ALIAS_URLS_SUCCESS
             mocked_post.return_value.content = content
             site_id = matomo.add_site_alias_urls('42', urls)
             assert site_id == '1'
             # error
             content = ADD_SITE_ALIAS_URLS_ERROR
             mocked_post.return_value.content = content
             with pytest.raises(MatomoError,
                                match="Merci de specifier une valeur pour idSite"):
                 site_id = matomo.add_site_alias_urls(None, urls)
             # strange message
             content = ADD_SITE_BAD_RESPONSE
             mocked_post.return_value.content = content
             with pytest.raises(MatomoException, match='add_site_alias_urls fails'):
                 site_id = matomo.add_site_alias_urls('42', urls)
     @mock.patch('requests.post')
     def test_add_user(mocked_post):
-...
             matomo = MatomoWS()
             # site not already here
             contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS]
             contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS, ADD_SITE_ALIAS_URLS_SUCCESS]
             mocked_post.side_effect = requests_post_mocked_replies(contents)
             site_id = upgrade_site(matomo, "hobo.dev.publik.love", urls)
             assert site_id == '42'
             # site already here
             contents = [GET_SITE_42_FROM_URL]
             contents = [GET_SITE_42_FROM_URL, ADD_SITE_ALIAS_URLS_SUCCESS]
             mocked_post.side_effect = requests_post_mocked_replies(contents)
             site_id = upgrade_site(matomo, "hobo.dev.publik.love", urls)
             assert site_id == '42'
             # error while updating urls
             contents = [GET_SITE_42_FROM_URL, ADD_SITE_ALIAS_URLS_ERROR]
             mocked_post.side_effect = requests_post_mocked_replies(contents)
             with pytest.raises(MatomoException):
                 upgrade_site(matomo, "hobo.dev.publik.love", urls)
             # error while adding new site
             contents = [GET_NO_SITE_FROM_URL, MATOMO_ERROR]
             mocked_post.side_effect = requests_post_mocked_replies(contents)
-...
         Fargo.objects.create(base_url='https://fargo.dev.publik.love')
         with override_settings(MATOMO_SERVER=CONFIG):
             contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS,
             contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS, ADD_SITE_ALIAS_URLS_SUCCESS,
                         DEL_UNKNOWN_USER, MATOMO_SUCCESS,
                         JAVASCRIPT_TAG, PING_SUCCESS]
             mocked_post.side_effect = requests_post_mocked_replies(contents)
-...
         Fargo.objects.create(base_url='https://fargo.dev.publik.love')
         with override_settings(MATOMO_SERVER=CONFIG):
             contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS,
             contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS, ADD_SITE_ALIAS_URLS_SUCCESS,
                         DEL_UNKNOWN_USER, MATOMO_SUCCESS,
                         JAVASCRIPT_TAG_BAD_RESPONSE]
             mocked_post.side_effect = requests_post_mocked_replies(contents)

     <result>42</result>
     """
     ADD_SITE_ALIAS_URLS_SUCCESS = """<?xml version="1.0" encoding="utf-8" ?>
     <result>1</result>
     """
     DEL_UNKNOWN_USER = """<?xml version="1.0" encoding="utf-8" ?>
     <result>
             <error message="User 'hobo.dev.publik.love' doesn't exist." />
-...
         Wcs.objects.create(base_url='https://wcs.dev.publik.love')
         Fargo.objects.create(base_url='https://fargo.dev.publik.love')
         contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS,
         contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS, ADD_SITE_ALIAS_URLS_SUCCESS,
                     DEL_UNKNOWN_USER, MATOMO_SUCCESS,
                     JAVASCRIPT_TAG, PING_SUCCESS]
-...
         Wcs.objects.create(base_url='https://wcs.dev.publik.love')
         Fargo.objects.create(base_url='https://fargo.dev.publik.love')
         contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS,
         contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS, ADD_SITE_ALIAS_URLS_SUCCESS,
                     DEL_UNKNOWN_USER, MATOMO_SUCCESS,
                     JAVASCRIPT_TAG, PING_ERROR]
-...
         Wcs.objects.create(base_url='https://wcs.dev.publik.love')
         Fargo.objects.create(base_url='https://fargo.dev.publik.love')
         contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS,
         contents = [GET_NO_SITE_FROM_URL, ADD_SITE_SUCCESS, ADD_SITE_ALIAS_URLS_SUCCESS,
                     DEL_UNKNOWN_USER, MATOMO_SUCCESS,
                     JAVASCRIPT_TAG_BAD_RESPONSE]
+    -

Projet

Général

Profil

Produits Entr'ouvert » Hobo

0001-matomo-add-services-urls-on-re-configuration-37614.patch