EnvironmentModules now correctly set a non-zero exit code

Starting Environment Modules v5.5, a non-zero exit code is set by
modulecmd.tcl when trying to load a non-existing module.

test/framework/modules.py

@@ -122,10 +122,10 @@ def test_run_module(self):
             error_pattern = "Module command '.*thisdoesnotmakesense' failed with exit code [1-9]"
             self.assertErrorRegex(EasyBuildError, error_pattern, self.modtool.run_module, 'thisdoesnotmakesense')
 
-            # we need to use a different error pattern here with Environment Modules,
+            # we need to use a different error pattern here with EnvironmentModulesC,
             # because a load of a non-existing module doesnt' trigger a non-zero exit code...
             # it will still fail though, just differently
-            if isinstance(self.modtool, EnvironmentModulesC) or isinstance(self.modtool, EnvironmentModules):
+            if isinstance(self.modtool, EnvironmentModulesC):
                 error_pattern = "Unable to locate a modulefile for 'nosuchmodule/1.2.3'"
             else:
                 error_pattern = "Module command '.*load nosuchmodule/1.2.3' failed with exit code [1-9]"