diff --git a/lib/spack/spack/environment.py b/lib/spack/spack/environment.py
index 9cb3f2575de..41136ab2eb5 100644
--- a/lib/spack/spack/environment.py
+++ b/lib/spack/spack/environment.py
@@ -41,6 +41,7 @@ def update_args(self, **kwargs):
         self.__dict__.update(kwargs)
         self.args.update(kwargs)
 
+
 class NameValueModifier(object):
 
     def __init__(self, name, value, **kwargs):
diff --git a/lib/spack/spack/test/modules.py b/lib/spack/spack/test/modules.py
index 0d33627be34..5e280d8e438 100644
--- a/lib/spack/spack/test/modules.py
+++ b/lib/spack/spack/test/modules.py
@@ -251,7 +251,8 @@ def test_alter_environment(self):
         self.assertEqual(
             len([x for x in content if 'setenv FOO "foo"' in x]), 1)
         self.assertEqual(len([x for x in content if 'unsetenv BAR' in x]), 1)
-        self.assertEqual(len([x for x in content if 'setenv MPILEAKS_ROOT' in x]), 1)
+        self.assertEqual(
+            len([x for x in content if 'setenv MPILEAKS_ROOT' in x]), 1)
 
         spec = spack.spec.Spec('libdwarf %clang platform=test target=x86_32')
         content = self.get_modulefile_content(spec)
@@ -266,7 +267,8 @@ def test_alter_environment(self):
             len([x for x in content if 'is-loaded foo/bar' in x]), 1)
         self.assertEqual(
             len([x for x in content if 'module load foo/bar' in x]), 1)
-        self.assertEqual(len([x for x in content if 'setenv LIBDWARF_ROOT' in x]), 1)
+        self.assertEqual(
+            len([x for x in content if 'setenv LIBDWARF_ROOT' in x]), 1)
 
     def test_blacklist(self):
         spack.modules.CONFIGURATION = configuration_blacklist