Add flake8 config and fix some style issues

Author: Anders Eurenius

graphwalker/gml.py

@@ -1,9 +1,8 @@
 # -*- coding: utf-8 -*-
 # Copyright (c) 2013 Spotify AB
-"""Parser for GML files
+"""Parser for GML files.
 
-# http://www.fim.uni-passau.de/fileadmin/files/lehrstuhl/\
-brandenburg/projekte/gml/gml-technical-report.pdf
+# http://www.fim.uni-passau.de/fileadmin/files/lehrstuhl/brandenburg/projekte/gml/gml-technical-report.pdf
 
 Grammar:
 
@@ -71,7 +70,7 @@ def build_vert(vl):
 
 
 def build_edge(el, serial):
-    e_id, e_name, e_src, e_tgt = 'e%d' % serial,  None, None, None
+    e_id, e_name, e_src, e_tgt = 'e%d' % serial, None, None, None
 
     for key, value in el:
         if key == 'label':

graphwalker/planning.py

@@ -92,6 +92,7 @@ def __iter__(self):
 
 class Random(EvenRandom):
     """Walk through the graph by random edges until done."""
+
     def choose_edge(self, edges):
         naive, weighted = [], []
         for e in edges:
@@ -131,6 +132,7 @@ def choose_edge(self, edges):
 
 class Euler(Planner):
     """Walk through the graph by ordered edges until done."""
+
     def __call__(self, g, stop, start, context):
         self._setup(g, stop, start, context)
 
@@ -186,6 +188,7 @@ def loop(vert):
 
 class Goto(Planner):
     """Plan direct path to goal state(s), repeating [repeat] times."""
+
     def __init__(self, *al, **kw):
         self.al, self.kw = al, kw
         self.goals = self.al
@@ -226,6 +229,7 @@ class Interactive(Planner):
     The protocol between choose and iterplan is deliberately kept simple to
     keep it simple to replace the choose method.
     """
+
     raw_input = raw_input
     out = sys.stderr
     debugger = pdb.Pdb('\t', sys.stdin, sys.stderr)

graphwalker/reporting.py

@@ -12,6 +12,7 @@
 
 class ReportingPlugin(object):
     """Common (optional) base or example for Graphwalker reporting plugins."""
+
     def __init__(self, **kw):
         self.context = kw
 
@@ -51,6 +52,7 @@ def attach_to_step(self, name, data):
 
 class Print(ReportingPlugin):
     """Print report to [output]; 'stdout', 'stderr' or object with write()."""
+
     outputs_map = {'stderr': sys.stderr, 'stdout': sys.stdout}
 
     def initiate(self, test_name):
@@ -88,6 +90,7 @@ def emit(self, message):
 
 class Log(Print):
     """Log report to (name or object) [logger], at [level]."""
+
     getLogger = staticmethod(logging.getLogger)
     levels = dict(
         (n, getattr(logging, n))
@@ -116,6 +119,7 @@ def emit(self, message, *al, **kw):
 
 class PathRecorder(ReportingPlugin):
     """Report steps to a file at [path]/[name], saving attachments by name."""
+
     file = file
 
     def __init__(self, **kw):
@@ -143,6 +147,7 @@ def finalize(self, failure=False):
 
 class Cartographer(ReportingPlugin):
     """Report graph and path steps to a graphviz file and run dot."""
+
     file = file
     system = os.system
     command = 'dot -T%(imgtype)s -o %(imgfname)s %(dotfname)s'
@@ -176,6 +181,7 @@ def step_begin(self, step):
 
 class Attachments(ReportingPlugin):
     """Save attachments to [path]/name."""
+
     file = file
 
     def attach_to_suite(self, name, data):

graphwalker/stopcond.py

@@ -24,11 +24,13 @@ def progress(self):
 
 class Never(StopCond):
     """Never stop."""
+
     pass
 
 
 class Seconds(StopCond):
     """Stop after [timeout] (default: 30) seconds."""
+
     clock = time.time
 
     def __init__(self, *al, **kw):
@@ -50,6 +52,7 @@ def progress(self):
 
 class SeenSteps(StopCond):
     """Stop when all given vertices have been visited."""
+
     def __init__(self, *al, **kw):
         self.targets = set(al)
 
@@ -63,6 +66,7 @@ def progress(self):
 
 class CountSteps(StopCond):
     """Stop after [steps] steps."""
+
     def __init__(self, *al, **kw):
         self.i, self.n = 0, int(al[0]) if al else kw.get('steps', 100)
 
@@ -76,6 +80,7 @@ def add(self, thing):
 
 class Coverage(StopCond):
     """Stop after [vertices] or [edges] number of vertices/edges visited."""
+
     def __init__(self, *al, **kw):
         kw['verts'] = kw.pop('vertices', kw.get('verts', 0))
         self.edge_cov = min(1.0, float(kw.get('edges', 0)) / 100.0)

graphwalker/test/interaction_test.py

@@ -37,12 +37,12 @@ def test_cli_models_with_actor(self):
         self.assertEqual(subprocess.call(argl), 0)
 
     def test_by_interaction(self):
-        """Interaction self-test.
+        r"""Interaction self-test.
 
         For comparison, try this:
 
-            PYTHONPATH=. python graphwalker/cli.py --reporter=Print \\
-              graphwalker/test/examples/selftest.graphml \\
+            PYTHONPATH=. python graphwalker/cli.py --reporter=Print \
+              graphwalker/test/examples/selftest.graphml \
               graphwalker.test.interactor.Interactor
         """
         outer = self

setup.cfg

@@ -0,0 +1,2 @@
+[flake8]
+ignore = I100,Q000,D100,D101,D102,D103,D104,D105