mirror of
https://github.com/OCA/knowledge.git
synced 2025-12-26 06:57:43 -06:00
test bug fixed
test error fixed Lint error improved
This commit is contained in:
committed by
Gabriela Mogollon
parent
ff8e884874
commit
8a6e36be08
@@ -10,19 +10,20 @@ class TestDocumentPageApproval(common.TransactionCase):
|
||||
# Check content of a category
|
||||
category = self.env['document.page'].search([
|
||||
('name', '=', 'OpenERP Features')
|
||||
])
|
||||
])
|
||||
|
||||
self.assertIsNotNone(category.display_content, 'a category')
|
||||
|
||||
# Check content of a page
|
||||
pages = self.env['document.page'].search([
|
||||
('parent_id', '=', category.id)
|
||||
])
|
||||
])
|
||||
page = pages[0]
|
||||
self.assertIsNotNone(page.display_content, 'Page content')
|
||||
|
||||
# Check if approval is required
|
||||
self.assertFalse(page.is_approval_required(page))
|
||||
self.assertTrue(page.is_approval_required(page) ==
|
||||
category.approval_required)
|
||||
|
||||
# Check content of an approval page
|
||||
page.approval_required = True
|
||||
|
||||
@@ -12,17 +12,26 @@ class TestDocumentPageHistoryWorkflow(common.TransactionCase):
|
||||
|
||||
def test_can_user_approve_this_page(self):
|
||||
"""Test if a user can approve this page."""
|
||||
user = self.env.user
|
||||
category = self.env.ref('document_page.demo_category1')
|
||||
category.approval_required = True
|
||||
category.approver_uid = user
|
||||
category.approver_gid = self.env.ref(
|
||||
'document_page_approval.group_document_approver_user')
|
||||
|
||||
pages = self.env['document.page.history'].search([
|
||||
('page_id', '=', category.id)
|
||||
])
|
||||
page = pages[0]
|
||||
page = self.env['document.page'].create({
|
||||
'name': 'Test Page10',
|
||||
'content': 'A difficult test',
|
||||
'parent_id': category.id
|
||||
})
|
||||
|
||||
self.assertTrue(page.can_user_approve)
|
||||
history = self.env['document.page.history'].search(
|
||||
[
|
||||
('page_id', '=', page.id)
|
||||
],
|
||||
limit=1,
|
||||
order='create_date DESC'
|
||||
)
|
||||
|
||||
self.assertTrue(history.can_user_approve_page)
|
||||
|
||||
def test_get_approvers_guids(self):
|
||||
"""Get approver guids."""
|
||||
@@ -30,7 +39,7 @@ class TestDocumentPageHistoryWorkflow(common.TransactionCase):
|
||||
category.approval_required = True
|
||||
pages = self.env['document.page.history'].search([
|
||||
('page_id', '=', category.id)
|
||||
])
|
||||
])
|
||||
page = pages[0]
|
||||
approvers_guid = page.get_approvers_guids()
|
||||
self.assertTrue(len(approvers_guid) > 0)
|
||||
@@ -41,7 +50,7 @@ class TestDocumentPageHistoryWorkflow(common.TransactionCase):
|
||||
category.approval_required = True
|
||||
pages = self.env['document.page.history'].search([
|
||||
('page_id', '=', category.id)
|
||||
])
|
||||
])
|
||||
page = pages[0]
|
||||
_logger.info("Email: " + str(page.get_approvers_email))
|
||||
self.assertIsNotNone(page.get_approvers_email)
|
||||
@@ -52,7 +61,7 @@ class TestDocumentPageHistoryWorkflow(common.TransactionCase):
|
||||
category.approval_required = True
|
||||
pages = self.env['document.page.history'].search([
|
||||
('page_id', '=', category.id)
|
||||
])
|
||||
])
|
||||
page = pages[0]
|
||||
_logger.info("Page: "+str(page.get_page_url))
|
||||
_logger.info("Page: " + str(page.get_page_url))
|
||||
self.assertIsNotNone(page.get_page_url)
|
||||
|
||||
Reference in New Issue
Block a user