describe_custom_font() - Code Metrics - Inspection of "BREAKING: Nest APIs under /api" - jacebrowning/memegen - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#244)

by Jace

created 2016-09-24 19:54 UTC

describe_custom_font() A

↳ Parent: describe_get()

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
cc	4
c	3
b	0
f	0
dl	0
loc	21
rs	9.0534

# pylint: disable=unused-variable,misplaced-comparison-constant,expression-not-assigned


import os

import pytest
from expecter import expect
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

from .conftest import load

TESTS = os.path.dirname(__file__)
ROOT = os.path.dirname(TESTS)
IMAGES = os.path.join(ROOT, 'data', 'images')


def describe_get():

    def describe_visible():

        def with_nominal_text(client):
            path = os.path.join(IMAGES, 'iw', 'hello', 'world' + '.img')
            if os.path.exists(path):
                os.remove(path)

            response = client.get("/iw/hello/world.jpg")

            assert 200 == response.status_code
            assert 'image/jpeg' == response.mimetype
            assert os.path.isfile(path)

        def with_only_1_line(client):
            response = client.get("/iw/hello.jpg")

            assert 200 == response.status_code
            assert 'image/jpeg' == response.mimetype

        @pytest.mark.xfail(os.name == 'nt', reason="Windows has a path limit")
        def with_lots_of_text(client):
            top = "-".join(["hello"] * 20)
            bottom = "-".join(["world"] * 20)
            response = client.get("/iw/" + top + "/" + bottom + ".jpg")

            assert 200 == response.status_code
            assert 'image/jpeg' == response.mimetype

    def describe_hidden():

        def when_jpg(client):
            response = client.get("/_aXcJaGVsbG8vd29ybGQJ.jpg")

            assert 200 == response.status_code
            assert 'image/jpeg' == response.mimetype

    def describe_custom_style():

        def when_provided(client):
            response = client.get("/sad-biden/hello.jpg?alt=scowl")

            assert 200 == response.status_code
            assert 'image/jpeg' == response.mimetype

        def it_redirects_to_lose_alt_when_default_style(client):
            response = client.get("/sad-biden/hello.jpg?alt=default")

            assert 302 == response.status_code
            assert '<a href="/sad-biden/hello.jpg">' in \
                load(response, as_json=False)

        def it_redirects_to_lose_alt_when_unknown_style(client):
            response = client.get("/sad-biden/hello.jpg?alt=__unknown__")

            assert 302 == response.status_code
            assert '<a href="/sad-biden/hello.jpg">' in \
                load(response, as_json=False)

        def it_keeps_alt_after_template_redirect(client):
            response = client.get("/sad-joe/hello.jpg?alt=scowl")

            assert 302 == response.status_code
            assert '<a href="/sad-biden/hello.jpg?alt=scowl">' in \
                load(response, as_json=False)

        def it_keeps_alt_after_text_redirect(client):
            response = client.get("/sad-biden.jpg?alt=scowl")

            assert 302 == response.status_code
            assert '-vote.jpg?alt=scowl">' in \
                load(response, as_json=False)

        def when_url(client):
            url = "http://www.gstatic.com/webp/gallery/1.jpg"
            response = client.get("/sad-biden/hello.jpg?alt=" + url)

            expect(response.status_code) == 200
            expect(response.mimetype) == 'image/jpeg'

        def it_returns_an_error_with_non_image_urls(client):
            url = "http://example.com"
            response = client.get("/sad-biden/hello.jpg?alt=" + url)

            expect(response.status_code) == 415

        def it_redirects_to_lose_alt_when_unknown_url(client):
            url = "http://example.com/not/a/real/image.jpg"
            response = client.get("/sad-biden/hello.jpg?alt=" + url)

            expect(response.status_code) == 302
            expect(load(response, as_json=False)).contains(
                '<a href="/sad-biden/hello.jpg">')

        def it_redirects_to_lose_alt_when_bad_url(client):
            url = "http:invalid"
            response = client.get("/sad-biden/hello.jpg?alt=" + url)

            expect(response.status_code) == 302
            expect(load(response, as_json=False)).contains(
                '<a href="/sad-biden/hello.jpg">')

    def describe_custom_font():

        def when_provided(client):
            response = client.get("/iw/hello.jpg?font=impact")

            expect(response.status_code) == 200
            expect(response.mimetype) == 'image/jpeg'

        def it_redirects_on_unknown_fonts(client):
            response = client.get("/iw/hello.jpg?font=__unknown__")

            expect(response.status_code) == 302
            expect(load(response, as_json=False)).contains(
                '<a href="/iw/hello.jpg">')

        def it_keeps_font_after_redirect(client):
            response = client.get("/iw/what%3F.jpg?font=impact")

            expect(response.status_code) == 302
            expect(load(response, as_json=False)).contains(
                '<a href="/iw/what~q.jpg?font=impact">')

    def describe_latest():

        def when_existing(client):
            client.get("/iw/my-first-meme.jpg")

            response = client.get("/latest.jpg")

            expect(response.status_code) == 302
            expect(load(response, as_json=False)).contains(
                '<a href="http://localhost/iw/my-first-meme.jpg">')

        def when_missing(client):
            with open(os.path.join(IMAGES, 'cache.yml'), 'w') as cache:
                cache.write("items: []")

            response = client.get("/latest.jpg")

            expect(response.status_code) == 302
            expect(load(response, as_json=False)).contains(
                '<a href="http://localhost/custom/your-meme/goes-here.jpg'
                '?alt=https://raw.githubusercontent.com/jacebrowning/memegen/'
                'master/memegen/static/images/missing.png">')

    def describe_redirects():

        def when_missing_dashes(client):
            response = client.get("/iw/HelloThere_World/How-areYOU.jpg")

            assert 302 == response.status_code
            assert '<a href="/iw/hello-there-world/how-are-you.jpg">' in \
                load(response, as_json=False)

        def when_no_text(client):
            response = client.get("/live.jpg")

            assert 302 == response.status_code
            assert '<a href="/live/_/do-it-live!.jpg">' in \
                load(response, as_json=False)

        def when_aliased_template(client):
            response = client.get("/insanity-wolf/hello/world.jpg")

            assert 302 == response.status_code
            assert '<a href="/iw/hello/world.jpg">' in \
                load(response, as_json=False)

        def when_jpeg_extension_without_text(client):
            response = client.get("/iw.jpeg")

            assert 302 == response.status_code
            assert '<a href="/iw.jpg">' in \
                load(response, as_json=False)

        def when_jpeg_extension_with_text(client):
            response = client.get("/iw/hello/world.jpeg")

            assert 302 == response.status_code
            assert '<a href="/iw/hello/world.jpg">' in \
                load(response, as_json=False)

    def describe_errors():

        def when_unknown_template(client):
            response = client.get("/make/sudo/give.me.jpg")

            assert 200 == response.status_code
            assert 'image/jpeg' == response.mimetype
            # unit tests ensure this is a placeholder image

        @pytest.mark.xfail(os.name == 'nt', reason="Windows has a path limit")
        def when_too_much_text_for_a_filename(client):
            top = "hello"
            bottom = "-".join(["world"] * 50)
            response = client.get("/iw/" + top + "/" + bottom + ".jpg")

            assert 414 == response.status_code
            assert {
                'message': "Filename too long."
            } == load(response)


Pull Request — master (#244)

describe_custom_font() A

Complexity

Size

Duplication

Importance

1. Missing Dependencies

2. Missing init.py files

1			# pylint: disable=unused-variable,misplaced-comparison-constant,expression-not-assigned
			0 ignored issues – show introduced 2016-09-24 20:02 UTC by Report Bug Copy Issue Report Bad option value 'misplaced-comparison-constant' Loading history...
2
3			import os
4
5			import pytest
6			from expecter import expect
			0 ignored issues – show Configuration introduced 2016-09-24 20:02 UTC by Report Bug Copy Issue Report The import `expecter` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
7
8			from .conftest import load
9
10			TESTS = os.path.dirname(__file__)
11			ROOT = os.path.dirname(TESTS)
12			IMAGES = os.path.join(ROOT, 'data', 'images')
13
14
15			def describe_get():
16
17			def describe_visible():
18
19			def with_nominal_text(client):
20			path = os.path.join(IMAGES, 'iw', 'hello', 'world' + '.img')
21			if os.path.exists(path):
22			os.remove(path)
23
24			response = client.get("/iw/hello/world.jpg")
25
26			assert 200 == response.status_code
27			assert 'image/jpeg' == response.mimetype
28			assert os.path.isfile(path)
29
30			def with_only_1_line(client):
31			response = client.get("/iw/hello.jpg")
32
33			assert 200 == response.status_code
34			assert 'image/jpeg' == response.mimetype
35
36			@pytest.mark.xfail(os.name == 'nt', reason="Windows has a path limit")
37			def with_lots_of_text(client):
38			top = "-".join(["hello"] * 20)
39			bottom = "-".join(["world"] * 20)
40			response = client.get("/iw/" + top + "/" + bottom + ".jpg")
41
42			assert 200 == response.status_code
43			assert 'image/jpeg' == response.mimetype
44
45			def describe_hidden():
46
47			def when_jpg(client):
48			response = client.get("/_aXcJaGVsbG8vd29ybGQJ.jpg")
49
50			assert 200 == response.status_code
51			assert 'image/jpeg' == response.mimetype
52
53			def describe_custom_style():
54
55			def when_provided(client):
56			response = client.get("/sad-biden/hello.jpg?alt=scowl")
57
58			assert 200 == response.status_code
59			assert 'image/jpeg' == response.mimetype
60
61			def it_redirects_to_lose_alt_when_default_style(client):
62			response = client.get("/sad-biden/hello.jpg?alt=default")
63
64			assert 302 == response.status_code
65			assert '<a href="/sad-biden/hello.jpg">' in \
66			load(response, as_json=False)
67
68			def it_redirects_to_lose_alt_when_unknown_style(client):
69			response = client.get("/sad-biden/hello.jpg?alt=__unknown__")
70
71			assert 302 == response.status_code
72			assert '<a href="/sad-biden/hello.jpg">' in \
73			load(response, as_json=False)
74
75			def it_keeps_alt_after_template_redirect(client):
76			response = client.get("/sad-joe/hello.jpg?alt=scowl")
77
78			assert 302 == response.status_code
79			assert '<a href="/sad-biden/hello.jpg?alt=scowl">' in \
80			load(response, as_json=False)
81
82			def it_keeps_alt_after_text_redirect(client):
83			response = client.get("/sad-biden.jpg?alt=scowl")
84
85			assert 302 == response.status_code
86			assert '-vote.jpg?alt=scowl">' in \
87			load(response, as_json=False)
88
89			def when_url(client):
90			url = "http://www.gstatic.com/webp/gallery/1.jpg"
91			response = client.get("/sad-biden/hello.jpg?alt=" + url)
92
93			expect(response.status_code) == 200
94			expect(response.mimetype) == 'image/jpeg'
95
96			def it_returns_an_error_with_non_image_urls(client):
97			url = "http://example.com"
98			response = client.get("/sad-biden/hello.jpg?alt=" + url)
99
100			expect(response.status_code) == 415
101
102			def it_redirects_to_lose_alt_when_unknown_url(client):
103			url = "http://example.com/not/a/real/image.jpg"
104			response = client.get("/sad-biden/hello.jpg?alt=" + url)
105
106			expect(response.status_code) == 302
107			expect(load(response, as_json=False)).contains(
108			'<a href="/sad-biden/hello.jpg">')
109
110			def it_redirects_to_lose_alt_when_bad_url(client):
111			url = "http:invalid"
112			response = client.get("/sad-biden/hello.jpg?alt=" + url)
113
114			expect(response.status_code) == 302
115			expect(load(response, as_json=False)).contains(
116			'<a href="/sad-biden/hello.jpg">')
117
118			def describe_custom_font():
119
120			def when_provided(client):
121			response = client.get("/iw/hello.jpg?font=impact")
122
123			expect(response.status_code) == 200
124			expect(response.mimetype) == 'image/jpeg'
125
126			def it_redirects_on_unknown_fonts(client):
127			response = client.get("/iw/hello.jpg?font=__unknown__")
128
129			expect(response.status_code) == 302
130			expect(load(response, as_json=False)).contains(
131			'<a href="/iw/hello.jpg">')
132
133			def it_keeps_font_after_redirect(client):
134			response = client.get("/iw/what%3F.jpg?font=impact")
135
136			expect(response.status_code) == 302
137			expect(load(response, as_json=False)).contains(
138			'<a href="/iw/what~q.jpg?font=impact">')
139
140			def describe_latest():
141
142			def when_existing(client):
143			client.get("/iw/my-first-meme.jpg")
144
145			response = client.get("/latest.jpg")
146
147			expect(response.status_code) == 302
148			expect(load(response, as_json=False)).contains(
149			'<a href="http://localhost/iw/my-first-meme.jpg">')
150
151			def when_missing(client):
152			with open(os.path.join(IMAGES, 'cache.yml'), 'w') as cache:
153			cache.write("items: []")
154
155			response = client.get("/latest.jpg")
156
157			expect(response.status_code) == 302
158			expect(load(response, as_json=False)).contains(
159			'<a href="http://localhost/custom/your-meme/goes-here.jpg'
160			'?alt=https://raw.githubusercontent.com/jacebrowning/memegen/'
161			'master/memegen/static/images/missing.png">')
162
163			def describe_redirects():
164
165			def when_missing_dashes(client):
166			response = client.get("/iw/HelloThere_World/How-areYOU.jpg")
167
168			assert 302 == response.status_code
169			assert '<a href="/iw/hello-there-world/how-are-you.jpg">' in \
170			load(response, as_json=False)
171
172			def when_no_text(client):
173			response = client.get("/live.jpg")
174
175			assert 302 == response.status_code
176			assert '<a href="/live/_/do-it-live!.jpg">' in \
177			load(response, as_json=False)
178
179			def when_aliased_template(client):
180			response = client.get("/insanity-wolf/hello/world.jpg")
181
182			assert 302 == response.status_code
183			assert '<a href="/iw/hello/world.jpg">' in \
184			load(response, as_json=False)
185
186			def when_jpeg_extension_without_text(client):
187			response = client.get("/iw.jpeg")
188
189			assert 302 == response.status_code
190			assert '<a href="/iw.jpg">' in \
191			load(response, as_json=False)
192
193			def when_jpeg_extension_with_text(client):
194			response = client.get("/iw/hello/world.jpeg")
195
196			assert 302 == response.status_code
197			assert '<a href="/iw/hello/world.jpg">' in \
198			load(response, as_json=False)
199
200			def describe_errors():
201
202			def when_unknown_template(client):
203			response = client.get("/make/sudo/give.me.jpg")
204
205			assert 200 == response.status_code
206			assert 'image/jpeg' == response.mimetype
207			# unit tests ensure this is a placeholder image
208
209			@pytest.mark.xfail(os.name == 'nt', reason="Windows has a path limit")
210			def when_too_much_text_for_a_filename(client):
211			top = "hello"
212			bottom = "-".join(["world"] * 50)
213			response = client.get("/iw/" + top + "/" + bottom + ".jpg")
214
215			assert 414 == response.status_code
216			assert {
217			'message': "Filename too long."
218			} == load(response)
219

jacebrowning / memegen

Pull Request — master (#244)

describe_custom_font() A

Complexity

Size

Duplication

Importance

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files