2019-12-12 17:26:37 +00:00
|
|
|
# Copyright 2019 Google LLC
|
|
|
|
#
|
|
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
|
|
# you may not use this file except in compliance with the License.
|
|
|
|
# You may obtain a copy of the License at
|
|
|
|
#
|
|
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
#
|
|
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
|
|
# See the License for the specific language governing perepo_managerissions and
|
|
|
|
# limitations under the License.
|
2020-01-31 23:31:18 +00:00
|
|
|
"""Test the functionality of the RepoManager class."""
|
2019-12-12 17:26:37 +00:00
|
|
|
|
|
|
|
import os
|
|
|
|
import unittest
|
2020-02-28 16:30:42 +00:00
|
|
|
from unittest import mock
|
2020-01-06 20:17:26 +00:00
|
|
|
import tempfile
|
2019-12-12 17:26:37 +00:00
|
|
|
|
2019-12-17 00:48:49 +00:00
|
|
|
import repo_manager
|
2020-02-28 16:30:42 +00:00
|
|
|
import utils
|
2019-12-12 17:26:37 +00:00
|
|
|
|
2020-01-30 18:27:56 +00:00
|
|
|
OSS_FUZZ_REPO = 'https://github.com/google/oss-fuzz'
|
|
|
|
|
2019-12-12 17:26:37 +00:00
|
|
|
|
|
|
|
class TestRepoManager(unittest.TestCase):
|
|
|
|
"""Class to test the functionality of the RepoManager class."""
|
|
|
|
|
|
|
|
|
2020-02-28 16:30:42 +00:00
|
|
|
class RepoManagerCloneUnitTest(unittest.TestCase):
|
2020-01-30 18:27:56 +00:00
|
|
|
"""Class to test the functionality of clone of the RepoManager class."""
|
|
|
|
|
|
|
|
def test_clone_valid_repo(self):
|
2019-12-12 17:26:37 +00:00
|
|
|
"""Tests the correct location of the git repo."""
|
2020-01-06 20:17:26 +00:00
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
2020-01-30 18:27:56 +00:00
|
|
|
test_repo_manager = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
2020-01-06 20:17:26 +00:00
|
|
|
git_path = os.path.join(test_repo_manager.base_dir,
|
|
|
|
test_repo_manager.repo_name, '.git')
|
|
|
|
self.assertTrue(os.path.isdir(git_path))
|
|
|
|
test_repo_manager.remove_repo()
|
2020-01-30 18:27:56 +00:00
|
|
|
|
|
|
|
def test_clone_invalid_repo(self):
|
|
|
|
"""Test that constructing RepoManager with an invalid repo will fail."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-30 18:27:56 +00:00
|
|
|
repo_manager.RepoManager(' ', tmp_dir)
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-30 18:27:56 +00:00
|
|
|
repo_manager.RepoManager('not_a_valid_repo', tmp_dir)
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-30 18:27:56 +00:00
|
|
|
repo_manager.RepoManager('https://github.com/oss-fuzz-not-real.git',
|
|
|
|
tmp_dir)
|
|
|
|
|
|
|
|
|
2020-02-28 16:30:42 +00:00
|
|
|
class RepoManagerCheckoutUnitTest(unittest.TestCase):
|
2020-01-30 18:27:56 +00:00
|
|
|
"""Class to test the functionality of checkout of the RepoManager class."""
|
2019-12-12 17:26:37 +00:00
|
|
|
|
2020-01-30 18:27:56 +00:00
|
|
|
def test_checkout_valid_commit(self):
|
2019-12-12 17:26:37 +00:00
|
|
|
"""Tests that the git checkout command works."""
|
2020-01-06 20:17:26 +00:00
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
2020-01-30 18:27:56 +00:00
|
|
|
test_repo_manager = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
commit_to_test = '04ea24ee15bbe46a19e5da6c5f022a2ffdfbdb3b'
|
2020-01-06 20:17:26 +00:00
|
|
|
test_repo_manager.checkout_commit(commit_to_test)
|
|
|
|
self.assertEqual(commit_to_test, test_repo_manager.get_current_commit())
|
2020-01-30 18:27:56 +00:00
|
|
|
|
|
|
|
def test_checkout_invalid_commit(self):
|
|
|
|
"""Tests that the git checkout invalid commit fails."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
|
|
test_repo_manager = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-06 20:17:26 +00:00
|
|
|
test_repo_manager.checkout_commit(' ')
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-06 20:17:26 +00:00
|
|
|
test_repo_manager.checkout_commit(
|
|
|
|
'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa')
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-30 18:27:56 +00:00
|
|
|
test_repo_manager.checkout_commit('not-a-valid-commit')
|
2019-12-12 17:26:37 +00:00
|
|
|
|
2020-01-30 18:27:56 +00:00
|
|
|
|
2020-02-28 16:30:42 +00:00
|
|
|
class RepoManagerGetCommitListUnitTest(unittest.TestCase):
|
2020-01-30 18:27:56 +00:00
|
|
|
"""Class to test the functionality of get commit list in the
|
|
|
|
RepoManager class."""
|
|
|
|
|
|
|
|
def test_get_valid_commit_list(self):
|
|
|
|
"""Tests an accurate commit list can be retrieved from the repo manager."""
|
2020-01-06 20:17:26 +00:00
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
2020-01-30 18:27:56 +00:00
|
|
|
test_repo_manager = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
old_commit = '04ea24ee15bbe46a19e5da6c5f022a2ffdfbdb3b'
|
|
|
|
new_commit = 'fa662173bfeb3ba08d2e84cefc363be11e6c8463'
|
2020-01-06 20:17:26 +00:00
|
|
|
commit_list = [
|
2020-01-30 18:27:56 +00:00
|
|
|
'fa662173bfeb3ba08d2e84cefc363be11e6c8463',
|
|
|
|
'17035317a44fa89d22fe6846d868d4bf57def78b',
|
|
|
|
'97dee00a3c4ce95071c3e061592f5fd577dea886',
|
|
|
|
'04ea24ee15bbe46a19e5da6c5f022a2ffdfbdb3b'
|
2020-01-06 20:17:26 +00:00
|
|
|
]
|
2020-04-14 22:59:33 +00:00
|
|
|
result_list = test_repo_manager.get_commit_list(new_commit, old_commit)
|
2020-01-06 20:17:26 +00:00
|
|
|
self.assertListEqual(commit_list, result_list)
|
2020-01-30 18:27:56 +00:00
|
|
|
|
|
|
|
def test_invalid_commit_list(self):
|
|
|
|
"""Tests that the propper Errors are thrown when invalid commits are
|
|
|
|
passed."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
|
|
old_commit = '04ea24ee15bbe46a19e5da6c5f022a2ffdfbdb3b'
|
|
|
|
new_commit = 'fa662173bfeb3ba08d2e84cefc363be11e6c8463'
|
|
|
|
test_repo_manager = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-30 18:27:56 +00:00
|
|
|
test_repo_manager.get_commit_list('fakecommit', new_commit)
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(ValueError):
|
2020-01-30 18:27:56 +00:00
|
|
|
test_repo_manager.get_commit_list(new_commit, 'fakecommit')
|
2020-01-31 23:31:18 +00:00
|
|
|
with self.assertRaises(RuntimeError):
|
2020-01-29 19:03:43 +00:00
|
|
|
# pylint: disable=arguments-out-of-order
|
2020-04-14 22:59:33 +00:00
|
|
|
test_repo_manager.get_commit_list(old_commit, new_commit)
|
2019-12-12 17:26:37 +00:00
|
|
|
|
|
|
|
|
2020-02-28 16:30:42 +00:00
|
|
|
class GitDiffUnitTest(unittest.TestCase):
|
|
|
|
"""Class testing functionality of get_git_diff in the repo_manager module."""
|
2020-01-31 23:31:18 +00:00
|
|
|
|
2020-02-28 16:30:42 +00:00
|
|
|
def test_diff_exists(self):
|
|
|
|
"""Tests that a real diff is returned when a valid repo manager exists."""
|
2020-01-31 23:31:18 +00:00
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
2020-02-28 16:30:42 +00:00
|
|
|
repo_man = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
with mock.patch.object(utils,
|
|
|
|
'execute',
|
|
|
|
return_value=('test.py\ndiff.py', None, 0)):
|
|
|
|
diff = repo_man.get_git_diff()
|
|
|
|
self.assertCountEqual(diff, ['test.py', 'diff.py'])
|
|
|
|
|
|
|
|
def test_diff_empty(self):
|
|
|
|
"""Tests that None is returned when there is no difference between repos."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
|
|
repo_man = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
with mock.patch.object(utils, 'execute', return_value=('', None, 0)):
|
|
|
|
diff = repo_man.get_git_diff()
|
|
|
|
self.assertIsNone(diff)
|
|
|
|
|
|
|
|
def test_error_on_command(self):
|
|
|
|
"""Tests that None is returned when the command errors out."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
|
|
repo_man = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
with mock.patch.object(utils,
|
|
|
|
'execute',
|
|
|
|
return_value=('', 'Test error.', 1)):
|
|
|
|
diff = repo_man.get_git_diff()
|
|
|
|
self.assertIsNone(diff)
|
|
|
|
|
|
|
|
def test_diff_no_change(self):
|
|
|
|
"""Tests that None is returned when there is no difference between repos."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
|
|
repo_man = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
diff = repo_man.get_git_diff()
|
|
|
|
self.assertIsNone(diff)
|
|
|
|
|
|
|
|
|
|
|
|
class CheckoutPRIntegrationTest(unittest.TestCase):
|
|
|
|
"""Class testing functionality of checkout_pr in the repo_manager module."""
|
|
|
|
|
|
|
|
def test_pull_request_exists(self):
|
|
|
|
"""Tests that a diff is returned when a valid PR is checked out."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
|
|
repo_man = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
repo_man.checkout_pr('refs/pull/3415/merge')
|
|
|
|
diff = repo_man.get_git_diff()
|
|
|
|
print(diff)
|
|
|
|
self.assertCountEqual(diff, ['README.md'])
|
2020-01-31 23:31:18 +00:00
|
|
|
|
|
|
|
def test_checkout_invalid_pull_request(self):
|
|
|
|
"""Tests that the git checkout invalid pull request fails."""
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
|
|
test_repo_manager = repo_manager.RepoManager(OSS_FUZZ_REPO, tmp_dir)
|
|
|
|
with self.assertRaises(RuntimeError):
|
|
|
|
test_repo_manager.checkout_pr(' ')
|
|
|
|
with self.assertRaises(RuntimeError):
|
|
|
|
test_repo_manager.checkout_pr(
|
|
|
|
'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa')
|
|
|
|
with self.assertRaises(RuntimeError):
|
|
|
|
test_repo_manager.checkout_pr('not/a/valid/pr')
|
|
|
|
|
|
|
|
|
2019-12-12 17:26:37 +00:00
|
|
|
if __name__ == '__main__':
|
|
|
|
unittest.main()
|