changing path from absolute to relative, and removing useless code fr…

…om get_commits_modified_file

pydriller/git_repository.py

@@ -271,13 +271,8 @@ def _useless_line(self, line: str):
  line.startswith("'''") or line.startswith('"""') or line.startswith("*")
 
  def get_commits_modified_file(self, filepath: str) -> List[str]:
- all_commits = self.get_list_commits()
+ path = str(Path(filepath))
 
- dict_commits = {}
- for commit in all_commits:
- dict_commits[commit.hash] = commit
-
- path = str(Path(filepath).absolute())
  commits = []
  try:
  commits = self.git.log("--follow", "--format=%H", path).split('\n')

tests/test_git_repository.py

@@ -398,7 +398,7 @@ def test_get_commits_last_modified_lines_with_more_modification():
 def test_get_commits_modified_file():
  gr = GitRepository('test-repos/test1/')
 
- commits = gr.get_commits_modified_file('test-repos/test1/file2.java')
+ commits = gr.get_commits_modified_file('file2.java')
 
  assert len(commits) == 3
  assert '09f6182cef737db02a085e1d018963c7a29bde5a' in commits
@@ -409,6 +409,6 @@ def test_get_commits_modified_file():
 def test_get_commits_modified_file_missing_file():
  gr = GitRepository('test-repos/test1/')
 
- commits = gr.get_commits_modified_file('test-repos/test1/non-existing-file.java')
+ commits = gr.get_commits_modified_file('non-existing-file.java')
 
  assert len(commits) == 0

tests/test_repository_mining.py

@@ -73,14 +73,14 @@ def test_filepath():
  dt = datetime(2018, 6, 6)
  assert 4 == len(list(RepositoryMining(
  path_to_repo='test-repos/test5',
- filepath='test-repos/test5/A.java',
+ filepath='A.java',
  to=dt).traverse_commits()))
 
 def test_filepath_with_rename():
  dt = datetime(2018, 6, 6)
  commits = list(RepositoryMining(
  path_to_repo='test-repos/test1',
- filepath='test-repos/test1/file4.java',
+ filepath='file4.java',
  to=dt).traverse_commits())
  assert 2 == len(commits)