52
78%
develop: 77%

Ran 12 Nov 2019 12:10PM UTC

Jobs 1

Files 25

Run time 5s

Badge

Embed ▾

pending completion

Build # 52

Build Type

push

travis-ci-com

Committed by

FinalAngel

Commit Message

Fixed #6737 -- Fix 'urls.W001' warning with custom apphook urls (#6738)

* Refs #6737 -- Add test for apphooks urls config check

* Fixed #6737 -- Fix 'urls.W001' warning with custom apphook urls

When rewriting the apphooks urls in appresolver.recurse_patterns,
the original RegexPattern.is_endpoint flag is lost, so urls with a callback
view throws an "include with a route ending with a '$'" warning ('urls.W001').

    @apphook_pool.register
    class MyApp(CMSApp):
        name = "MyApp"

        def get_urls(self, page=None, language=None, **kwargs):
            return [
                re_path(r"^$", views.MyAppView.as_view(), name="my-app-index")
                # or this
                # path(r"", views.MyAppView.as_view(), name="my-app-index")
            ]

    $ ./manage.py check
    System check identified some issues:

    WARNINGS:
    ?: (urls.W001) Your URL pattern '^my-app-url/my-view-url/$' [name='my-app-index']
    uses include with a route ending with a '$'. Remove the dollar from the
    route to avoid problems including URLs.

This fix set the RegexPattern flag is_endpoint to True for callback views
in appresolver.recurse_patterns as in
https://github.com/django/django/blob/stable/2.2.x/django/urls/conf.py#L70

Run Details

2542 of 3250 relevant lines covered (78.22%)

34.0 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	52.1 (FRONTEND=1 UNIT=1)	12 Nov 2019 12:10PM UTC	0	78.22	Travis Job 52.1

Source Files on build 52

Detailed source file information is not available for this build.

divio / django-cms / 52
78%
develop: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 52

divio / django-cms / 52 78% develop: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 52

divio / django-cms / 52
78%
develop: 77%

README BADGES
x