aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMaximilian Hils <git@maximilianhils.com>2019-09-05 23:20:54 +0200
committerGitHub <noreply@github.com>2019-09-05 23:20:54 +0200
commita08c22dcdd5535dcef453f7c1c2445c11f35beda (patch)
tree5f031138dc9f0ab8f32df1b3d5f261788b5785db
parentddaf5a1f1ba9e4607c7cdc9378276ef2ddc2e236 (diff)
parente77f375186403cee05dc66c069ed1155059b2d63 (diff)
downloadmitmproxy-a08c22dcdd5535dcef453f7c1c2445c11f35beda.tar.gz
mitmproxy-a08c22dcdd5535dcef453f7c1c2445c11f35beda.tar.bz2
mitmproxy-a08c22dcdd5535dcef453f7c1c2445c11f35beda.zip
Merge pull request #3631 from mhils/lint
Fix CI
-rw-r--r--.travis.yml1
-rw-r--r--mitmproxy/contentviews/base.py2
-rw-r--r--mitmproxy/contentviews/image/image_parser.py2
-rw-r--r--setup.py2
-rw-r--r--test/mitmproxy/addons/test_view.py2
-rw-r--r--test/mitmproxy/script/test_concurrent.py26
6 files changed, 17 insertions, 18 deletions
diff --git a/.travis.yml b/.travis.yml
index 20afc279..44d452c2 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -52,7 +52,6 @@ matrix:
- wget https://github.com/gohugoio/hugo/releases/download/v0.41/hugo_0.41_Linux-64bit.deb
- sudo dpkg -i hugo*.deb
- pip install tox virtualenv setuptools
- - pyenv global system 3.6
script:
- tox
after_success:
diff --git a/mitmproxy/contentviews/base.py b/mitmproxy/contentviews/base.py
index 6072dfb7..9b34f3d4 100644
--- a/mitmproxy/contentviews/base.py
+++ b/mitmproxy/contentviews/base.py
@@ -37,7 +37,7 @@ class View:
def format_pairs(
items: typing.Iterable[typing.Tuple[TTextType, TTextType]]
-)-> typing.Iterator[TViewLine]:
+) -> typing.Iterator[TViewLine]:
"""
Helper function that accepts a list of (k,v) pairs into a list of
diff --git a/mitmproxy/contentviews/image/image_parser.py b/mitmproxy/contentviews/image/image_parser.py
index fcc50cb5..d5bb404f 100644
--- a/mitmproxy/contentviews/image/image_parser.py
+++ b/mitmproxy/contentviews/image/image_parser.py
@@ -54,7 +54,7 @@ def parse_gif(data: bytes) -> Metadata:
entries = block.body.body.entries
for entry in entries:
comment = entry.bytes
- if comment is not b'':
+ if comment != b'':
parts.append(('comment', str(comment)))
return parts
diff --git a/setup.py b/setup.py
index 12439f4e..2571cfd8 100644
--- a/setup.py
+++ b/setup.py
@@ -89,7 +89,7 @@ setup(
],
'dev': [
"asynctest>=0.12.0",
- "flake8>=3.5,<3.7",
+ "flake8>=3.5,<=3.7.8",
"Flask>=1.0,<1.1",
"mypy>=0.590,<0.591",
"parver>=0.1,<2.0",
diff --git a/test/mitmproxy/addons/test_view.py b/test/mitmproxy/addons/test_view.py
index 976c14b7..f5088a68 100644
--- a/test/mitmproxy/addons/test_view.py
+++ b/test/mitmproxy/addons/test_view.py
@@ -471,7 +471,7 @@ def test_focus():
v = view.View()
v.add([tft()])
f = view.Focus(v)
- assert f.index is 0
+ assert f.index == 0
assert f.flow is v[0]
# Start empty
diff --git a/test/mitmproxy/script/test_concurrent.py b/test/mitmproxy/script/test_concurrent.py
index 3ec58760..70d41511 100644
--- a/test/mitmproxy/script/test_concurrent.py
+++ b/test/mitmproxy/script/test_concurrent.py
@@ -43,17 +43,17 @@ class TestConcurrent(tservers.MasterTest):
assert await tctx.master.await_log("decorator not supported")
def test_concurrent_class(self, tdata):
- with taddons.context() as tctx:
- sc = tctx.script(
- tdata.path(
- "mitmproxy/data/addonscripts/concurrent_decorator_class.py"
- )
+ with taddons.context() as tctx:
+ sc = tctx.script(
+ tdata.path(
+ "mitmproxy/data/addonscripts/concurrent_decorator_class.py"
)
- f1, f2 = tflow.tflow(), tflow.tflow()
- tctx.cycle(sc, f1)
- tctx.cycle(sc, f2)
- start = time.time()
- while time.time() - start < 5:
- if f1.reply.state == f2.reply.state == "committed":
- return
- raise ValueError("Script never acked")
+ )
+ f1, f2 = tflow.tflow(), tflow.tflow()
+ tctx.cycle(sc, f1)
+ tctx.cycle(sc, f2)
+ start = time.time()
+ while time.time() - start < 5:
+ if f1.reply.state == f2.reply.state == "committed":
+ return
+ raise ValueError("Script never acked")