From cd6bb608d40b2ce964cb5a0c189e8aad1b6b917c Mon Sep 17 00:00:00 2001 From: lauvwbk Date: Wed, 10 May 2023 10:12:25 +0200 Subject: [PATCH] #101 test model page --- aircox/models/page.py | 12 +- aircox/tests/conftest.py | 15 ++ aircox/tests/models/test_page.py | 336 +++++++++++++++---------------- 3 files changed, 184 insertions(+), 179 deletions(-) diff --git a/aircox/models/page.py b/aircox/models/page.py index a19f1c7..3eaabd9 100644 --- a/aircox/models/page.py +++ b/aircox/models/page.py @@ -123,9 +123,14 @@ class BasePage(models.Model): def save(self, *args, **kwargs): if not self.slug: self.slug = slugify(self.title)[:100] - count = Page.objects.filter(slug__startswith=self.slug).count() - if count: - self.slug += "-" + str(count) + pagecount = Page.objects.filter(slug__startswith=self.slug).count() + staticpagecount = StaticPage.objects.filter( + slug__startswith=self.slug + ).count() + if pagecount: + self.slug += "-" + str(pagecount) + if staticpagecount: + self.slug += "-" + str(staticpagecount) if self.parent and not self.cover: self.cover = self.parent.cover @@ -150,7 +155,6 @@ class BasePage(models.Model): def is_trash(self): return self.status == self.STATUS_TRASH - # for the following property, as we call the properties is_published and display_title as property, we need to call them as property (without parenthesis) and not as method (with parenthesis) @property def display_title(self): if self.is_published: diff --git a/aircox/tests/conftest.py b/aircox/tests/conftest.py index bb93cca..e6d2708 100644 --- a/aircox/tests/conftest.py +++ b/aircox/tests/conftest.py @@ -12,6 +12,21 @@ def stations(): return baker.make(models.Station, _quantity=2) +@pytest.fixture +def staticpages(): + return baker.make(models.StaticPage, _quantity=3) + + +@pytest.fixture +def pages(): + return baker.make(models.Page, _quantity=3) + + +@pytest.fixture +def navitem(): + return baker.make(models.NavItem) + + @pytest.fixture def programs(stations): items = list( diff --git a/aircox/tests/models/test_page.py b/aircox/tests/models/test_page.py index 8c7b30c..d304d95 100644 --- a/aircox/tests/models/test_page.py +++ b/aircox/tests/models/test_page.py @@ -2,269 +2,255 @@ import pytest from model_bakery import baker -from aircox.models import Category, PageQuerySet, Page, StaticPage, Comment, NavItem +from aircox.models import Category, Page, StaticPage, NavItem @pytest.mark.django_db class TestCategory: - def test__str__(self): - # return category title field. category = baker.make(Category) title = category.__str__() + assert title == category.title -# poupée russe du modèle page. -# BasePage > Page -# BasePage > StaticPage -# Cannot create BasePage fake instance in fake db -# ? Do I need to create fake page and staticpage instances in order to test each method from BasePage ? - - -class TestBasePageQuerySet(): +# BasePAgeQuerySet is queryset, we use as fake db instances of the model StaticPage in the test. We do not use instances of the Page Model because it's use also as an abstract class. +class TestBasePageQuerySet: @pytest.mark.django_db - def test_draft(self): - # return an array of all draft pages - baker.make(Page, _quantity=5) - draft_pages = Page.objects.draft() + def test_draft(self, staticpages): + draft_pages = StaticPage.objects.draft() + for page in draft_pages: - assert page.status == Page.STATUS_DRAFT + assert page.status == StaticPage.STATUS_DRAFT @pytest.mark.django_db - def test_published(self): - # retrun an array of all published pages - baker.make(Page, _quantity=5) - published_pages = Page.objects.published() + def test_published(self, staticpages): + published_pages = StaticPage.objects.published() + for page in published_pages: - assert page.status == Page.STATUS_PUBLISHED + assert page.status == StaticPage.STATUS_PUBLISHED @pytest.mark.django_db - def test_trash(self): - # return an array of all trash pages - baker.make(Page, _quantity=5) - trashed_pages = Page.objects.trash() + def test_trash(self, staticpages): + trashed_pages = StaticPage.objects.trash() + for page in trashed_pages: - assert page.status == Page.STATUS_TRASH - + assert page.status == StaticPage.STATUS_TRASH @pytest.mark.django_db - def test_parent_with_page(self): - parent = baker.make(Page) - child1 = baker.make(Page, parent=parent) - child2 = baker.make(Page, parent=parent) - child3 = baker.make(Page, parent=parent) - nochild = baker.make(Page, parent=None) + def test_parent_return_childs_from_parent_object(self, staticpages): + parent = baker.make(StaticPage) + childs = baker.make(StaticPage, _quantity=3, parent=parent) + nochild = baker.make(StaticPage, parent=None) + parent_childs = StaticPage.objects.parent(parent) - ''' retrieve all child page from same parent instance ''' - parent_childs = Page.objects.parent(parent) - - assert child1 in parent_childs - assert child2 in parent_childs - assert child3 in parent_childs + for child in childs: + assert child in parent_childs assert nochild not in parent_childs - ''' retrieve all child page from same parent id''' - parentid_childs = Page.objects.parent(parent.id) + @pytest.mark.django_db + def test_parent_return_childs_from_parent_id(self, staticpages): + parent = baker.make(StaticPage) + childs = baker.make(StaticPage, _quantity=3, parent=parent) + nochild = baker.make(StaticPage, parent=None) + parent_childs = StaticPage.objects.parent(parent.id) - assert child1 in parentid_childs - assert child2 in parentid_childs - assert child3 in parentid_childs - assert nochild not in parentid_childs + for child in childs: + assert child in parent_childs + assert nochild not in parent_childs @pytest.mark.django_db - def test_search_with_searchcontent(self): - baker.make(Page, _quantity=4) - page1 = baker.make(Page, title='test') - page2 = baker.make(Page, content='test') - q = 'test' + def test_search_with_searchcontent(self, staticpages): + page1 = baker.make(StaticPage, title="test") + page2 = baker.make(StaticPage, content="test") + q = "test" + results = StaticPage.objects.search(q=q) - results = Page.objects.search(q=q) - - # check : title and content data was use to determine if pages are in result from search 'test' assert page1 in results assert page2 in results @pytest.mark.django_db - def test_search_without_searchcontent(self): - baker.make(Page, _quantity=4) - page1 = baker.make(Page, title='test') - page2 = baker.make(Page, content='test') - q = 'test' + def test_search_without_searchcontent(self, staticpages): + page1 = baker.make(StaticPage, title="test") + page2 = baker.make(StaticPage, content="test") + q = "test" + results = StaticPage.objects.search(q=q, search_content=False) - results = Page.objects.search(q=q, search_content=False) - - # check : as we search not in content, only the page with the data 'test' in its title should be return in the results assert page1 in results assert page2 not in results + +# BasePage is abstract, we use as fake db instances of the model StaticPage in the test. We do not use instances of the Page Model because it's use as an abstract class also. class TestBasePage: + @pytest.mark.django_db + def test__str__(self, staticpages): + page = baker.make(StaticPage, title="Test") + + assert page.__str__() == "Test" @pytest.mark.django_db - def test__str__(self): - # return page title - page = baker.make(Page, title='Test') - assert page.__str__() == 'Test' - - @pytest.mark.django_db - def test_save_case1(self): - ''' save case 1: - - no slug - - the slug generate with .save() is unique in db''' - - page = baker.make(Page, title='Title with spaces', slug=None) + def test_save_generate_unique_slug(self, staticpages): + page = baker.make(StaticPage, title="Title with spaces", slug=None) page.save() - assert page.slug == 'title-with-spaces' + + assert page.slug == "title-with-spaces" @pytest.mark.django_db - def test_save_case2(self): - ''' save case 2: - - no slug - - the slug generate with .save() is not unique in db''' - - baker.make(Page, slug='title') - page = baker.make(Page, title='Title', slug=None) + def test_save_generate_slug_that_already_exist(self, staticpages): + baker.make(StaticPage, slug="title") + page = baker.make(StaticPage, title="Title", slug=None) page.save() - assert page.slug == 'title-1' + + assert page.slug == "title-1" @pytest.mark.django_db - def test_save_case3(self): - ''' save case 3: - - save a page without cover but with a parent with a cover - ''' - - parent = baker.make(Page) - child = baker.make(Page, cover=None, parent=parent) + def test_save_without_cover_with_parent(self, staticpages): + parent = baker.make(StaticPage) + child = baker.make(StaticPage, cover=None, parent=parent) child.save() + assert child.cover == parent.cover @pytest.mark.django_db - def test_get_absolute_url(self): + def test_get_absolute_url_from_unpublished_page(self, staticpages): + unpublished_page = baker.make( + StaticPage, slug="page-slug", status=StaticPage.STATUS_DRAFT + ) - unpublished_page = baker.make(Page, slug='page-slug', status=Page.STATUS_DRAFT) - assert unpublished_page.get_absolute_url() == '#' + assert unpublished_page.get_absolute_url() == "#" - ''' !!! only work with staticpage instance, not page instance. error when using page : TestBasePage::test_get_absolute_url - django.urls.exceptions.NoReverseMatch: Reverse for 'None' not found. 'None' is not a valid view function or pattern name''' - published_page = baker.make(StaticPage, slug='another-page-slug', status=Page.STATUS_PUBLISHED) - assert published_page.get_absolute_url() == '/pages/another-page-slug/' - - # as the following are property from page.py, we test them without the parenthesis at the end. For example: page.is_draft and not page.is_draft() - #property @pytest.mark.django_db - def test_is_draft(self): - #return true if the page/staticapge is draft. - page = baker.make(Page, status=Page.STATUS_DRAFT) + def test_get_absolute_url_from_published_page(self, staticpages): + published_page = baker.make( + StaticPage, + slug="another-page-slug", + status=StaticPage.STATUS_PUBLISHED, + ) + + assert published_page.get_absolute_url() == "/pages/another-page-slug/" + + # as the following test methods are property from page.py, we call them without the parenthesis. + @pytest.mark.django_db + def test_is_draft(self, staticpages): + page = baker.make(StaticPage, status=StaticPage.STATUS_DRAFT) + assert page.is_draft == True - - #property + @pytest.mark.django_db - def test_is_published(self): - #return true if the page/staticapge is published. - page = baker.make(Page, status=Page.STATUS_PUBLISHED) + def test_is_published(self, staticpages): + page = baker.make(StaticPage, status=StaticPage.STATUS_PUBLISHED) + assert page.is_published == True - #property @pytest.mark.django_db - def test_is_draft(self): - #return true if the page/staticapge is published. - page = baker.make(Page, status=Page.STATUS_TRASH) + def test_is_draft(self, staticpages): + page = baker.make(StaticPage, status=StaticPage.STATUS_TRASH) + assert page.is_trash == True - ''' !! in display_title property from page.py, we need to remove the parenthesis of .is_published() and display_title(), because they are property and not proper method, in order to work. otherwhise we get an error : "TypeError: 'bool' object is not callable" ''' - #property @pytest.mark.django_db - def test_display_title(self): - parent_page = baker.make(Page, title='Parent page title', status=Page.STATUS_PUBLISHED) - child_unpublished_page = baker.make(Page, title='Child page title', parent=parent_page, status=Page.STATUS_DRAFT) - - # display title of a published page with title - assert parent_page.display_title == 'Parent page title' - # display title of the parent page if child is draft - assert child_unpublished_page.display_title == 'Parent page title' - - #cached_property - @pytest.mark.django_db - def test_headline(self): - page = baker.make(Page, content='

My headline

My content.

') - empty_content_page = baker.make(Page, content='') + def test_display_title(self, staticpages): + parent_page = baker.make( + StaticPage, + title="Parent page title", + status=StaticPage.STATUS_PUBLISHED, + ) + child_unpublished_page = baker.make( + StaticPage, + title="Child page title", + parent=parent_page, + status=StaticPage.STATUS_DRAFT, + ) - assert page.headline == 'My headline\n' + assert parent_page.display_title == "Parent page title" + assert child_unpublished_page.display_title == "Parent page title" + + @pytest.mark.django_db + def test_headline(self, staticpages): + page = baker.make( + StaticPage, content="

My headline

My content.

" + ) + empty_content_page = baker.make(StaticPage, content="") + + assert page.headline == "My headline\n" assert empty_content_page.headline == "" - #classmethod : is called on the class itself. Exemple below : Page.classmethod(arg) + + #this classmethod is not suitable for StaticPage because staticpage don't have category argument. It's intend only for Page. @pytest.mark.django_db - def test_get_init_kwargs_from(self): + def test_get_init_kwargs_from(self, staticpages): page = baker.make(Page) kwargs = Page.get_init_kwargs_from(page) assert kwargs == {"cover": page.cover, "category": page.category} - - #classmethod - # from_page classmethod create a new instance of the class (page or staticpage) from a page object. It's a fast way to create a new instance without having to manually extract all the informations from the page object in order to create the new instance. - # Only for informations: below are some theorical explanation of the tools use in the method from_page. - # - cls from the classmethod from_page from page.py refers to the class itself - # - The **something syntax in Python is used to pass a variable number of keyword arguments (from the 'something') to a function or method - # - It (**something) is a shorthand way of passing the dictionary as individual arguments without having to unpack it explicitly. (example : unpack this dictionary "something = {'key1' : 'value1', 'key2' : 'value2'}" and "**something" return this arguments "{key1='value1', key2='value2'}") + #this classmethod is not suitable for StaticPage because staticpage don't have category argument. It's intend only for Page. @pytest.mark.django_db - def test_from_page(self): + def test_from_page(self, pages): page_object = baker.make(Page) new_page_instance = Page.from_page(page_object) - # assert that after creating new instance with the methodclass "from_page", the new page instance have the same arguments than the page object (we only test here with cover and category arguments) + assert new_page_instance.cover == page_object.cover assert new_page_instance.category == page_object.category -class TestPageQuerySet(): + +class TestPageQuerySet: @pytest.mark.django_db - def test_published(self): + def test_published(self, pages): baker.make(Page, _quantity=5) published_pages_list = Page.objects.published() for page in published_pages_list: assert page.status == Page.STATUS_PUBLISHED -#class TestPage(): -## TODO en cours par laurent - - - - +class TestPage: + @pytest.mark.django_db + def test_save_published_page_pubdate(self, pages): + page = baker.make(Page, status=Page.STATUS_PUBLISHED, pub_date=None) + page.save() + assert page.pub_date is not None + + @pytest.mark.django_db + def test_save_unpublished_page_pubdate(self, pages): + page = baker.make(Page, status=Page.STATUS_DRAFT, pub_date=None) + page.save() + + assert page.pub_date == None + + @pytest.mark.django_db + def test_save_child_page_category(self, pages): + parent = baker.make(Page) + child = baker.make(Page, parent=parent, category=None) + child.save() + + assert child.category == parent.category +class TestStaticPage: + @pytest.mark.django_db + def test_get_absolute_url(self, staticpages): + page = baker.make( + StaticPage, attach_to=StaticPage.ATTACH_TO_DIFFUSIONS + ) - + assert page.get_absolute_url() == "/week/" +class TestNavItem: + @pytest.mark.django_db + def test_get_url_from_selfurl(self, navitem): + assert navitem.url == navitem.get_url() + @pytest.mark.django_db + def test_get_url_from_pageurl(self, navitem): + page = baker.make(StaticPage) + item = baker.make(NavItem, url=None, page=page) + assert item.get_url() == page.get_absolute_url() + @pytest.mark.django_db + def test_get_url_without_selfurl_page(self, navitem): + item = baker.make(NavItem, url=None, page=None) - - - - - - - - - - - - - - - - - - - - - - - - - - - - + assert item.get_url() == None