tests.test_cli.test_watch_complex() - Code Metrics - Inspection of "Merge pull request #23 from danbradham/compile_dir..." - spyder-ide/qtsass - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7e9855...54cf91 )

by Gonzalo

created 2018-05-21 13:04 UTC

tests.test_cli.test_watch_complex() F

↳ Parent: tests.test_cli

Complexity

Conditions

Size

Total Lines	46
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	29
dl	0
loc	46
rs	2.6955
c	0
b	0
f	0
cc	14
nop	1

How to fix Complexity

# -*- coding: utf-8 -*-
# -----------------------------------------------------------------------------
# Copyright (c) 2015 Yann Lanthony
# Copyright (c) 2017-2018 Spyder Project Contributors
#
# Licensed under the terms of the MIT License
# (See LICENSE.txt for details)
# -----------------------------------------------------------------------------
"""Test qtsass cli."""

# Standard library imports
from __future__ import absolute_import
import os
import time
import signal

import sys
from os.path import join, dirname, normpath, exists, basename

from textwrap import dedent

from subprocess import Popen, PIPE
from collections import namedtuple


PY3 = sys.version_info.major == 3
PROJECT_DIR = normpath(dirname(dirname(__file__)))
Result = namedtuple('Result', "code stdout stderr")


def example(*paths):
    """Get path to an example."""

    return normpath(join(dirname(__file__), '..', 'examples', *paths))


def touch(file):
    """Touch a file."""

    with open(file, 'a') as f:

        os.utime(file, None)


def await_condition(condition, timeout=2000):
    """Return True if a condition is met in the given timeout period"""

    for _ in range(timeout):
        if condition():
            return True
        time.sleep(0.001)
    return False


def invoke(args):
    """Invoke qtsass cli with specified args"""

    kwargs = dict(
        stdout=PIPE,
        stderr=PIPE,
        cwd=PROJECT_DIR
    )
    proc = Popen(['python', '-m', 'qtsass'] + args, **kwargs)
    return proc


def invoke_with_result(args):
    """Invoke qtsass cli and return a Result obj"""

    proc = invoke(args)
    out, err = proc.communicate()
    out = out.decode('ascii', errors="ignore")
    err = err.decode('ascii', errors="ignore")
    return Result(proc.returncode, out, err)


def test_compile_dummy_to_stdout():
    """CLI compile dummy example to stdout."""

    args = [example('dummy.scss')]
    result = invoke_with_result(args)

    assert result.code == 0
    assert result.stdout


def test_compile_dummy_to_file(tmpdir):
    """CLI compile dummy example to file."""

    input = example('dummy.scss')

    output = tmpdir.join('dummy.css')
    args = [input, '-o', output.strpath]
    result = invoke_with_result(args)

    assert result.code == 0
    assert exists(output.strpath)


def test_watch_dummy(tmpdir):
    """CLI watch dummy example."""

    input = example('dummy.scss')

    output = tmpdir.join('dummy.css')
    args = [input, '-o', output.strpath, '-w']
    proc = invoke(args)

    # Wait for initial compile
    output_exists = lambda: exists(output.strpath)
    if not await_condition(output_exists):
        proc.terminate()
        assert False, "Failed to compile dummy.scss"

    # Ensure subprocess is still alive
    assert proc.poll() is None

    # Touch input file, triggering a recompile
    created = output.mtime()
    file_modified = lambda: output.mtime() > created
    time.sleep(0.1)
    touch(input)

    if not await_condition(file_modified):
        proc.terminate()
        assert False, 'Output file has not been recompiled...'

    proc.terminate()


def test_compile_complex(tmpdir):
    """CLI compile complex example."""

    input = example('complex')

    output = tmpdir.mkdir('output')
    args = [input, '-o', output.strpath]
    result = invoke_with_result(args)

    assert result.code == 0

    expected_files = [output.join('light.css'), output.join('dark.css')]
    for file in expected_files:
        assert exists(file.strpath)


def test_watch_complex(tmpdir):
    """CLI watch complex example."""

    input = example('complex')

    output = tmpdir.mkdir('output')
    args = [input, '-o', output.strpath, '-w']
    proc = invoke(args)

    expected_files = [output.join('light.css'), output.join('dark.css')]

    # Wait for initial compile
    files_created = lambda: all([exists(f.strpath) for f in expected_files])
    if not await_condition(files_created):
        assert False, 'All expected files have not been created...'

    # Ensure subprocess is still alive
    assert proc.poll() is None

    # Input files to touch
    input_full = example('complex', 'light.scss')
    input_partial = example('complex', '_base.scss')
    input_nested = example('complex', 'widgets', '_qwidget.scss')

    def touch_and_wait(input_file, timeout=2000):
        """Touch a file, triggering a recompile"""

        filename = basename(input_file)
        old_mtimes = [f.mtime() for f in expected_files]
        files_modified = lambda: all(
            [f.mtime() > old_mtimes[i] for i, f in enumerate(expected_files)]
        )
        time.sleep(0.1)
        touch(input_file)

        if not await_condition(files_modified, timeout):
            proc.terminate()
            err = 'Modifying %s did not trigger recompile.' % filename
            assert False, err

        return True

    assert touch_and_wait(input_full)
    assert touch_and_wait(input_partial)
    assert touch_and_wait(input_nested)

    proc.terminate()


def test_invalid_input():
    """CLI input is not a file or dir."""

    proc = invoke_with_result(['file_does_not_exist.scss'])
    assert proc.code == 1
    assert 'Error: input must be' in proc.stdout

    proc = invoke_with_result(['./dir/does/not/exist'])
    assert proc.code == 1
    assert 'Error: input must be' in proc.stdout


def test_dir_missing_output():
    """CLI dir missing output option"""

    proc = invoke_with_result([example('complex')])
    assert proc.code == 1
    assert 'Error: missing required option' in proc.stdout


1			# -- coding: utf-8 --
2			# -----------------------------------------------------------------------------
3			# Copyright (c) 2015 Yann Lanthony
4			# Copyright (c) 2017-2018 Spyder Project Contributors
5			#
6			# Licensed under the terms of the MIT License
7			# (See LICENSE.txt for details)
8			# -----------------------------------------------------------------------------
9			"""Test qtsass cli."""
10
11			# Standard library imports
12			from __future__ import absolute_import
13			import os
14			import time
15			import signal
			0 ignored issues – show Unused Code introduced 2018-05-20 19:08 UTC by Report Bug Copy Issue Report The import `signal` seems to be unused. Loading history...
16			import sys
17			from os.path import join, dirname, normpath, exists, basename
			0 ignored issues – show introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report Imports from package os are not grouped Loading history...
18			from textwrap import dedent
			0 ignored issues – show Unused Code introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report Unused dedent imported from textwrap Loading history...
19			from subprocess import Popen, PIPE
20			from collections import namedtuple
21
22
23			PY3 = sys.version_info.major == 3
24			PROJECT_DIR = normpath(dirname(dirname(__file__)))
25			Result = namedtuple('Result', "code stdout stderr")
26
27
28			def example(*paths):
29			"""Get path to an example."""
30
31			return normpath(join(dirname(__file__), '..', 'examples', *paths))
32
33
34			def touch(file):
35			"""Touch a file."""
36
37			with open(file, 'a') as f:
			0 ignored issues – show Coding Style Naming introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report The name `f` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Unused Code introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report The variable `f` seems to be unused. Loading history...
38			os.utime(file, None)
39
40
41			def await_condition(condition, timeout=2000):
42			"""Return True if a condition is met in the given timeout period"""
43
44			for _ in range(timeout):
45			if condition():
46			return True
47			time.sleep(0.001)
48			return False
49
50
51			def invoke(args):
52			"""Invoke qtsass cli with specified args"""
53
54			kwargs = dict(
55			stdout=PIPE,
56			stderr=PIPE,
57			cwd=PROJECT_DIR
58			)
59			proc = Popen(['python', '-m', 'qtsass'] + args, **kwargs)
60			return proc
61
62
63			def invoke_with_result(args):
64			"""Invoke qtsass cli and return a Result obj"""
65
66			proc = invoke(args)
67			out, err = proc.communicate()
68			out = out.decode('ascii', errors="ignore")
69			err = err.decode('ascii', errors="ignore")
70			return Result(proc.returncode, out, err)
71
72
73			def test_compile_dummy_to_stdout():
74			"""CLI compile dummy example to stdout."""
75
76			args = [example('dummy.scss')]
77			result = invoke_with_result(args)
78
79			assert result.code == 0
80			assert result.stdout
81
82
83			def test_compile_dummy_to_file(tmpdir):
84			"""CLI compile dummy example to file."""
85
86			input = example('dummy.scss')
			0 ignored issues – show Bug Best Practice introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report This seems to re-define the built-in `input`. It is generally discouraged to redefine built-ins as this makes code very hard to read. Loading history...
87			output = tmpdir.join('dummy.css')
88			args = [input, '-o', output.strpath]
89			result = invoke_with_result(args)
90
91			assert result.code == 0
92			assert exists(output.strpath)
93
94
95			def test_watch_dummy(tmpdir):
96			"""CLI watch dummy example."""
97
98			input = example('dummy.scss')
			0 ignored issues – show Bug Best Practice introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report This seems to re-define the built-in `input`. It is generally discouraged to redefine built-ins as this makes code very hard to read. Loading history...
99			output = tmpdir.join('dummy.css')
100			args = [input, '-o', output.strpath, '-w']
101			proc = invoke(args)
102
103			# Wait for initial compile
104			output_exists = lambda: exists(output.strpath)
105			if not await_condition(output_exists):
106			proc.terminate()
107			assert False, "Failed to compile dummy.scss"
108
109			# Ensure subprocess is still alive
110			assert proc.poll() is None
111
112			# Touch input file, triggering a recompile
113			created = output.mtime()
114			file_modified = lambda: output.mtime() > created
115			time.sleep(0.1)
116			touch(input)
117
118			if not await_condition(file_modified):
119			proc.terminate()
120			assert False, 'Output file has not been recompiled...'
121
122			proc.terminate()
123
124
125			def test_compile_complex(tmpdir):
126			"""CLI compile complex example."""
127
128			input = example('complex')
			0 ignored issues – show Bug Best Practice introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report This seems to re-define the built-in `input`. It is generally discouraged to redefine built-ins as this makes code very hard to read. Loading history...
129			output = tmpdir.mkdir('output')
130			args = [input, '-o', output.strpath]
131			result = invoke_with_result(args)
132
133			assert result.code == 0
134
135			expected_files = [output.join('light.css'), output.join('dark.css')]
136			for file in expected_files:
137			assert exists(file.strpath)
138
139
140			def test_watch_complex(tmpdir):
141			"""CLI watch complex example."""
142
143			input = example('complex')
			0 ignored issues – show Bug Best Practice introduced 2018-05-20 18:57 UTC by Report Bug Copy Issue Report This seems to re-define the built-in `input`. It is generally discouraged to redefine built-ins as this makes code very hard to read. Loading history...
144			output = tmpdir.mkdir('output')
145			args = [input, '-o', output.strpath, '-w']
146			proc = invoke(args)
147
148			expected_files = [output.join('light.css'), output.join('dark.css')]
149
150			# Wait for initial compile
151			files_created = lambda: all([exists(f.strpath) for f in expected_files])
152			if not await_condition(files_created):
153			assert False, 'All expected files have not been created...'
154
155			# Ensure subprocess is still alive
156			assert proc.poll() is None
157
158			# Input files to touch
159			input_full = example('complex', 'light.scss')
160			input_partial = example('complex', '_base.scss')
161			input_nested = example('complex', 'widgets', '_qwidget.scss')
162
163			def touch_and_wait(input_file, timeout=2000):
164			"""Touch a file, triggering a recompile"""
165
166			filename = basename(input_file)
167			old_mtimes = [f.mtime() for f in expected_files]
168			files_modified = lambda: all(
169			[f.mtime() > old_mtimes[i] for i, f in enumerate(expected_files)]
170			)
171			time.sleep(0.1)
172			touch(input_file)
173
174			if not await_condition(files_modified, timeout):
175			proc.terminate()
176			err = 'Modifying %s did not trigger recompile.' % filename
177			assert False, err
178
179			return True
180
181			assert touch_and_wait(input_full)
182			assert touch_and_wait(input_partial)
183			assert touch_and_wait(input_nested)
184
185			proc.terminate()
186
187
188			def test_invalid_input():
189			"""CLI input is not a file or dir."""
190
191			proc = invoke_with_result(['file_does_not_exist.scss'])
192			assert proc.code == 1
193			assert 'Error: input must be' in proc.stdout
194
195			proc = invoke_with_result(['./dir/does/not/exist'])
196			assert proc.code == 1
197			assert 'Error: input must be' in proc.stdout
198
199
200			def test_dir_missing_output():
201			"""CLI dir missing output option"""
202
203			proc = invoke_with_result([example('complex')])
204			assert proc.code == 1
205			assert 'Error: missing required option' in proc.stdout
206

spyder-ide / qtsass

GitHub Access Token became invalid

Push — master ( 7e9855...54cf91 )

tests.test_cli.test_watch_complex() F

Complexity

Size

Duplication

Importance

How to fix Complexity

Complexity

Duplication Side-by-Side

Filter issues like