Skip to content

Commit

Permalink
Merge branch 'master' into PyAr#359-pytest-II-tests-cache
Browse files Browse the repository at this point in the history
  • Loading branch information
eduzen committed May 22, 2019
2 parents de3af69 + dc06864 commit 4a597ca
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 27 deletions.
5 changes: 1 addition & 4 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,10 +1,7 @@
flake8==3.5.0
flake8==3.7.7
logassert==2
mccabe==0.6.1
pep257==0.7.0
pep8==1.7.0
pycodestyle==2.3.1
pyflakes==1.6.0
pyuca==1.1.2
pyxdg==0.25
rst2html5==1.9.3
Expand Down
46 changes: 23 additions & 23 deletions tests/test_envbuilder.py
Original file line number Diff line number Diff line change
Expand Up @@ -190,9 +190,9 @@ def test_create_system_site_pkgs_pyvenv(self):
"pyvenv_options": ['--system-site-packages'],
}
with patch.object(EnvBuilder, 'create') as mock_create:
env_builder.create_env(interpreter, is_current, options)
self.assertTrue(env_builder.system_site_packages)
self.assertTrue(mock_create.called)
env_builder.create_env(interpreter, is_current, options)
self.assertTrue(env_builder.system_site_packages)
self.assertTrue(mock_create.called)

def test_create_pyvenv(self):
env_builder = envbuilder._FadesEnvBuilder()
Expand All @@ -202,9 +202,9 @@ def test_create_pyvenv(self):
"pyvenv_options": [],
}
with patch.object(EnvBuilder, 'create') as mock_create:
env_builder.create_env(interpreter, is_current, options)
self.assertFalse(env_builder.system_site_packages)
self.assertTrue(mock_create.called)
env_builder.create_env(interpreter, is_current, options)
self.assertFalse(env_builder.system_site_packages)
self.assertTrue(mock_create.called)

def test_create_system_site_pkgs_virtualenv(self):
env_builder = envbuilder._FadesEnvBuilder()
Expand All @@ -214,8 +214,8 @@ def test_create_system_site_pkgs_virtualenv(self):
"pyvenv_options": [],
}
with patch.object(envbuilder._FadesEnvBuilder, 'create_with_virtualenv') as mock_create:
env_builder.create_env(interpreter, is_current, options)
mock_create.assert_called_with(interpreter, options['virtualenv_options'])
env_builder.create_env(interpreter, is_current, options)
mock_create.assert_called_with(interpreter, options['virtualenv_options'])

def test_create_virtualenv(self):
env_builder = envbuilder._FadesEnvBuilder()
Expand All @@ -225,8 +225,8 @@ def test_create_virtualenv(self):
"pyvenv_options": [],
}
with patch.object(envbuilder._FadesEnvBuilder, 'create_with_virtualenv') as mock_create:
env_builder.create_env(interpreter, is_current, options)
mock_create.assert_called_with(interpreter, options['virtualenv_options'])
env_builder.create_env(interpreter, is_current, options)
mock_create.assert_called_with(interpreter, options['virtualenv_options'])

def test_custom_env_path(self):
builder = envbuilder._FadesEnvBuilder('some-path')
Expand Down Expand Up @@ -361,11 +361,11 @@ def test_filenotfound_exception(self):
"pyvenv_options": ['--system-site-packages'],
}
with patch('fades.envbuilder.helpers.logged_exec') as mock_lexec:
# mock_lexec.side_effect = envbuilder.helpers.ExecutionError('matanga!')
mock_lexec.side_effect = FileNotFoundError('matanga!')
with self.assertRaises(FadesError) as cm:
env_builder.create_env(interpreter, is_current, options)
self.assertEqual(str(cm.exception), 'virtualenv not found')
# mock_lexec.side_effect = envbuilder.helpers.ExecutionError('matanga!')
mock_lexec.side_effect = FileNotFoundError('matanga!')
with self.assertRaises(FadesError) as cm:
env_builder.create_env(interpreter, is_current, options)
self.assertEqual(str(cm.exception), 'virtualenv not found')

def test_executionerror_exception(self):
env_builder = envbuilder._FadesEnvBuilder()
Expand All @@ -375,10 +375,10 @@ def test_executionerror_exception(self):
"pyvenv_options": ['--system-site-packages'],
}
with patch('fades.envbuilder.helpers.logged_exec') as mock_lexec:
mock_lexec.side_effect = envbuilder.helpers.ExecutionError(1, 'cmd', ['stdout'])
with self.assertRaises(FadesError) as cm:
env_builder.create_env(interpreter, is_current, options)
self.assertEqual(str(cm.exception), 'virtualenv could not be run')
mock_lexec.side_effect = envbuilder.helpers.ExecutionError(1, 'cmd', ['stdout'])
with self.assertRaises(FadesError) as cm:
env_builder.create_env(interpreter, is_current, options)
self.assertEqual(str(cm.exception), 'virtualenv could not be run')

def test_general_error_exception(self):
env_builder = envbuilder._FadesEnvBuilder()
Expand All @@ -388,10 +388,10 @@ def test_general_error_exception(self):
"pyvenv_options": ['--system-site-packages'],
}
with patch('fades.envbuilder.helpers.logged_exec') as mock_lexec:
mock_lexec.side_effect = Exception()
with self.assertRaises(FadesError) as cm:
env_builder.create_env(interpreter, is_current, options)
self.assertEqual(str(cm.exception), 'General error while running virtualenv')
mock_lexec.side_effect = Exception()
with self.assertRaises(FadesError) as cm:
env_builder.create_env(interpreter, is_current, options)
self.assertEqual(str(cm.exception), 'General error while running virtualenv')

def test_when_a_venv_is_removed_it_is_removed_from_everywhere(self):
old_date = datetime.utcnow()
Expand Down

0 comments on commit 4a597ca

Please sign in to comment.