initial commit 2

5 files changed, 223 insertions, 0 deletions

diff --git a/addons/website_blog/tests/__init__.py b/addons/website_blog/tests/__init__.py
new file mode 100644
index 00000000..15f0e603
--- /dev/null
+++ b/addons/website_blog/tests/__init__.py
@@ -0,0 +1,6 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+from . import test_ui
+from . import test_website_blog_flow
+from . import test_performance
diff --git a/addons/website_blog/tests/common.py b/addons/website_blog/tests/common.py
new file mode 100644
index 00000000..d23691e9
--- /dev/null
+++ b/addons/website_blog/tests/common.py
@@ -0,0 +1,46 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+from odoo.tests import common
+
+
+class TestWebsiteBlogCommon(common.TransactionCase):
+    def setUp(self):
+        super(TestWebsiteBlogCommon, self).setUp()
+
+        Users = self.env['res.users']
+
+        group_blog_manager_id = self.ref('website.group_website_designer')
+        group_employee_id = self.ref('base.group_user')
+        group_public_id = self.ref('base.group_public')
+
+        self.user_employee = Users.with_context({'no_reset_password': True}).create({
+            'name': 'Armande Employee',
+            'login': 'armande',
+            'email': 'armande.employee@example.com',
+            'notification_type': 'inbox',
+            'groups_id': [(6, 0, [group_employee_id])]
+        })
+        self.user_blogmanager = Users.with_context({'no_reset_password': True}).create({
+            'name': 'Bastien BlogManager',
+            'login': 'bastien',
+            'email': 'bastien.blogmanager@example.com',
+            'notification_type': 'inbox',
+            'groups_id': [(6, 0, [group_blog_manager_id, group_employee_id])]
+        })
+        self.user_public = Users.with_context({'no_reset_password': True}).create({
+            'name': 'Cedric Public',
+            'login': 'cedric',
+            'email': 'cedric.public@example.com',
+            'notification_type': 'inbox',
+            'groups_id': [(6, 0, [group_public_id])]
+        })
+
+        self.test_blog = self.env['blog.blog'].with_user(self.user_blogmanager).create({
+            'name': 'New Blog',
+        })
+        self.test_blog_post = self.env['blog.post'].with_user(self.user_blogmanager).create({
+            'name': 'New Post',
+            'blog_id': self.test_blog.id,
+            'website_published': True,
+        })
diff --git a/addons/website_blog/tests/test_performance.py b/addons/website_blog/tests/test_performance.py
new file mode 100644
index 00000000..9a9c7e87
--- /dev/null
+++ b/addons/website_blog/tests/test_performance.py
@@ -0,0 +1,42 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+from odoo.addons.website.tests.test_performance import UtilPerf
+import random
+
+
+class TestBlogPerformance(UtilPerf):
+    def setUp(self):
+        super().setUp()
+        # if website_livechat is installed, disable it
+        if 'channel_id' in self.env['website']:
+            self.env['website'].search([]).channel_id = False
+
+    def test_10_perf_sql_blog_standard_data(self):
+        self.assertEqual(self._get_url_hot_query('/blog'), 28)
+
+    def test_20_perf_sql_blog_bigger_data_scaling(self):
+        BlogPost = self.env['blog.post']
+        BlogTag = self.env['blog.tag']
+        blogs = self.env['blog.blog'].search([])
+        blog_tags = BlogTag.create([{'name': 'Blog Tag Test %s' % i} for i in range(1, 20)])
+        BlogPost.create([{'name': 'Blog Post Test %s' % i, 'is_published': True, 'blog_id': blogs[i % 2].id} for i in range(1, 20)])
+        blog_posts = BlogPost.search([])
+        for blog_post in blog_posts:
+            blog_post.tag_ids += blog_tags
+            blog_tags = blog_tags[:-1]
+        self.assertEqual(self._get_url_hot_query('/blog'), 28)
+        self.assertEqual(self._get_url_hot_query(blog_post[0].website_url), 31)
+
+    def test_30_perf_sql_blog_bigger_data_scaling(self):
+        BlogPost = self.env['blog.post']
+        BlogTag = self.env['blog.tag']
+        blogs = self.env['blog.blog'].search([])
+        blog_tags = BlogTag.create([{'name': 'New Blog Tag Test %s' % i} for i in range(1, 50)])
+        BlogPost.create([{'name': 'New Blog Post Test %s' % i, 'is_published': True, 'blog_id': blogs[random.randint(0, 1)].id} for i in range(1, 100)])
+        blog_posts = BlogPost.search([])
+        for blog_post in blog_posts:
+            blog_post.write({'tag_ids': [[6, 0, random.choices(blog_tags.ids, k=random.randint(0, len(blog_tags)))]]})
+
+        self.assertLessEqual(self._get_url_hot_query('/blog'), 28)
+        self.assertLessEqual(self._get_url_hot_query(blog_post[0].website_url), 31)
diff --git a/addons/website_blog/tests/test_ui.py b/addons/website_blog/tests/test_ui.py
new file mode 100644
index 00000000..f4c7b003
--- /dev/null
+++ b/addons/website_blog/tests/test_ui.py
@@ -0,0 +1,19 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+import odoo.tests
+from odoo import tools
+
+
+@odoo.tests.tagged('post_install', '-at_install')
+class TestUi(odoo.tests.HttpCase):
+    def test_admin(self):
+        self.env['blog.blog'].create({'name': 'Travel'})
+        self.env['ir.attachment'].create({
+            'public': True,
+            'type': 'url',
+            'url': '/web/image/123/transparent.png',
+            'name': 'transparent.png',
+            'mimetype': 'image/png',
+        })
+        self.start_tour("/", 'blog', login='admin')
diff --git a/addons/website_blog/tests/test_website_blog_flow.py b/addons/website_blog/tests/test_website_blog_flow.py
new file mode 100644
index 00000000..60d8a7c6
--- /dev/null
+++ b/addons/website_blog/tests/test_website_blog_flow.py
@@ -0,0 +1,110 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+from odoo.exceptions import UserError
+from odoo.tests.common import users
+from odoo.addons.website.tools import MockRequest
+from odoo.addons.website_blog.tests.common import TestWebsiteBlogCommon
+from odoo.addons.portal.controllers.mail import PortalChatter
+
+
+class TestWebsiteBlogFlow(TestWebsiteBlogCommon):
+    def setUp(self):
+        super(TestWebsiteBlogFlow, self).setUp()
+        group_portal = self.env.ref('base.group_portal')
+        self.user_portal = self.env['res.users'].with_context({'no_reset_password': True}).create({
+            'name': 'Dorian Portal',
+            'login': 'portal_user',
+            'email': 'portal_user@example.com',
+            'notification_type': 'inbox',
+            'groups_id': [(6, 0, [group_portal.id])]
+        })
+
+    def test_website_blog_followers(self):
+        """ Test the flow of followers and notifications for blogs. Intended
+        flow :
+
+         - people subscribe to a blog
+         - when creating a new post, nobody except the creator follows it
+         - people subscribed to the blog does not receive comments on posts
+         - when published, a notification is sent to all blog followers
+         - if someone subscribe to the post or comment it, it become follower
+           and receive notification for future comments. """
+
+        # Create a new blog, subscribe the employee to the blog
+        self.assertIn(
+            self.user_blogmanager.partner_id, self.test_blog.message_partner_ids,
+            'website_blog: blog create should be in the blog followers')
+        self.test_blog.message_subscribe([self.user_employee.partner_id.id, self.user_public.partner_id.id])
+
+        # Create a new post, blog followers should not follow the post
+        self.assertNotIn(
+            self.user_employee.partner_id, self.test_blog_post.message_partner_ids,
+            'website_blog: subscribing to a blog should not subscribe to its posts')
+        self.assertNotIn(
+            self.user_public.partner_id, self.test_blog_post.message_partner_ids,
+            'website_blog: subscribing to a blog should not subscribe to its posts')
+
+        # Publish the blog
+        self.test_blog_post.write({'website_published': True})
+
+        # Check publish message has been sent to blog followers
+        publish_message = next((m for m in self.test_blog_post.blog_id.message_ids if m.subtype_id.id == self.ref('website_blog.mt_blog_blog_published')), None)
+        self.assertEqual(
+            publish_message.notified_partner_ids,
+            self.user_employee.partner_id | self.user_public.partner_id,
+            'website_blog: peuple following a blog should be notified of a published post')
+
+        # Armand posts a message -> becomes follower
+        self.test_blog_post.sudo().message_post(
+            body='Armande BlogUser Commented',
+            message_type='comment',
+            author_id=self.user_employee.partner_id.id,
+            subtype_xmlid='mail.mt_comment',
+        )
+        self.assertIn(
+            self.user_employee.partner_id, self.test_blog_post.message_partner_ids,
+            'website_blog: people commenting a post should follow it afterwards')
+
+    @users('portal_user')
+    def test_blog_comment(self):
+        """Test comment on blog post with attachment."""
+        attachment = self.env['ir.attachment'].sudo().create({
+            'name': 'some_attachment.pdf',
+            'res_model': 'mail.compose.message',
+            'datas': 'test',
+            'type': 'binary',
+            'access_token': 'azerty',
+        })
+
+        with MockRequest(self.env):
+            PortalChatter().portal_chatter_post(
+                'blog.post',
+                self.test_blog_post.id,
+                'Test message blog post',
+                attachment_ids=str(attachment.id),
+                attachment_tokens=attachment.access_token
+            )
+
+        self.assertTrue(self.env['mail.message'].sudo().search(
+            [('model', '=', 'blog.post'), ('attachment_ids', 'in', attachment.ids)]))
+
+        second_attachment = self.env['ir.attachment'].sudo().create({
+            'name': 'some_attachment.pdf',
+            'res_model': 'mail.compose.message',
+            'datas': 'test',
+            'type': 'binary',
+            'access_token': 'azerty',
+        })
+
+        with self.assertRaises(UserError), MockRequest(self.env):
+            PortalChatter().portal_chatter_post(
+                'blog.post',
+                self.test_blog_post.id,
+                'Test message blog post',
+                attachment_ids=str(second_attachment.id),
+                attachment_tokens='wrong_token'
+            )
+
+        self.assertFalse(self.env['mail.message'].sudo().search(
+            [('model', '=', 'blog.post'), ('attachment_ids', 'in', second_attachment.ids)]))