diff --git a/swh/loader/dir/converters.py b/swh/loader/dir/converters.py index dd5173a..da6012c 100644 --- a/swh/loader/dir/converters.py +++ b/swh/loader/dir/converters.py @@ -1,148 +1,150 @@ # Copyright (C) 2015 The Software Heritage developers # See the AUTHORS file at the top-level directory of this distribution # License: GNU General Public License version 3, or any later version # See top-level LICENSE file for more information """Convert dir objects to dictionaries suitable for swh.storage""" import datetime +import os from swh.loader.dir.git.git import GitType from swh.loader.dir.git import git, utils def to_datetime(ts): """Convert a timestamp to utc datetime. """ return datetime.datetime.utcfromtimestamp(ts).replace( tzinfo=datetime.timezone.utc) def format_to_minutes(offset_str): """Convert a git string timezone format string (e.g +0200, -0310) to minutes. Args: offset_str: a string representing an offset. Returns: A positive or negative number of minutes of such input """ sign = offset_str[0] hours = int(offset_str[1:3]) minutes = int(offset_str[3:]) + (hours * 60) return minutes if sign == '+' else -1 * minutes def blob_to_content(obj, log=None, max_content_size=None, origin_id=None): - if 'data' not in obj: - filepath = obj['path'] - content_raw = open(filepath, 'rb').read() - obj.update({'data': content_raw, - 'length': len(content_raw)}) - return _blob_to_content(obj, log, max_content_size, origin_id) + """Convert obj to a swh storage content. + + """ + filepath = obj['path'] + obj['length'] = os.path.getsize(filepath) + blob = _blob_to_content(obj, log, max_content_size, origin_id) + + # performance reason, we load only the content now + if blob['status'] == 'visible': + blob['data'] = open(filepath, 'rb').read() + + return blob def _blob_to_content(obj, log=None, max_content_size=None, origin_id=None): """Convert to a compliant swh content. """ size = obj['length'] ret = { 'sha1': obj['sha1'], 'sha256': obj['sha256'], 'sha1_git': obj['sha1_git'], 'length': size, + 'status': 'visible', 'perms': obj['perms'].value, 'type': obj['type'].value, } if max_content_size and size > max_content_size: if log: log.info('Skipping content %s, too large (%s > %s)' % (utils.hash_to_hex(obj['sha1_git']), size, max_content_size)) ret.update({'status': 'absent', 'reason': 'Content too large', 'origin': origin_id}) return ret - - ret.update({ - 'status': 'visible', - 'data': obj['data'], - }) - return ret # Map of type to swh types _entry_type_map = { GitType.TREE: 'dir', GitType.BLOB: 'file', GitType.COMM: 'rev', } def tree_to_directory(tree, objects, log=None): """Format a tree as a directory """ entries = [] for entry in objects[tree['path']]: entries.append({ 'type': _entry_type_map[entry['type']], 'perms': int(entry['perms'].value), 'name': entry['name'], 'target': entry['sha1_git'] }) return { 'id': tree['sha1_git'], 'entries': entries } def commit_to_revision(commit, objects, log=None): """Format a commit as a revision. """ upper_directory = objects[git.ROOT_TREE_KEY][0] return { 'id': commit['sha1_git'], 'date': to_datetime(commit['author_date']), 'date_offset': format_to_minutes(commit['author_offset']), 'committer_date': to_datetime(commit['committer_date']), 'committer_date_offset': format_to_minutes(commit['committer_offset']), 'type': commit['type'], 'directory': upper_directory['sha1_git'], 'message': commit['message'], 'author_name': commit['author_name'], 'author_email': commit['author_email'], 'committer_name': commit['committer_name'], 'committer_email': commit['committer_email'], 'synthetic': True, 'metadata': commit['metadata'], 'parents': [], } def annotated_tag_to_release(release, log=None): """Format a swh release. """ return { 'id': release['sha1_git'], 'revision': release['revision'], 'name': release['name'], 'comment': release['comment'], 'date': to_datetime(release['date']), 'date_offset': format_to_minutes(release['offset']), 'author_name': release['author_name'], 'author_email': release['author_email'], 'synthetic': True, } diff --git a/swh/loader/dir/tests/test_converters.py b/swh/loader/dir/tests/test_converters.py index f071365..8740ff3 100644 --- a/swh/loader/dir/tests/test_converters.py +++ b/swh/loader/dir/tests/test_converters.py @@ -1,204 +1,284 @@ # Copyright (C) 2015 The Software Heritage developers # See the AUTHORS file at the top-level directory of this distribution # License: GNU General Public License version 3, or any later version # See top-level LICENSE file for more information -import unittest import datetime +import os +import tempfile +import unittest from nose.tools import istest from swh.loader.dir import converters from swh.loader.dir.git import git from swh.loader.dir.git.git import GitType, GitPerm class TestConverters(unittest.TestCase): + @istest def format_to_minutes(self): self.assertEquals(converters.format_to_minutes('+0100'), 60) self.assertEquals(converters.format_to_minutes('-0200'), -120) self.assertEquals(converters.format_to_minutes('+1250'), 12*60+50) self.assertEquals(converters.format_to_minutes('+0000'), 0) self.assertEquals(converters.format_to_minutes('-0000'), 0) @istest def annotated_tag_to_release(self): # given release = { 'sha1_git': '123', 'revision': '456', 'name': 'some-release', 'comment': 'some-comment-on-release', 'date': 1444054085, 'offset': '-0300', 'author_name': 'someone', 'author_email': 'someone@whatelse.eu' } expected_release = { 'id': '123', 'revision': '456', 'name': 'some-release', 'comment': 'some-comment-on-release', 'date': datetime.datetime.fromtimestamp( 1444054085, tz=datetime.timezone.utc), 'date_offset': -180, 'author_name': 'someone', 'author_email': 'someone@whatelse.eu', 'synthetic': True, } # when actual_release = converters.annotated_tag_to_release(release) # then self.assertDictEqual(actual_release, expected_release) + @istest + def blob_to_content(self): + # given + contentfile = b'temp file for testing blob to content convertion' + tmpfilepath = tempfile.mktemp( + suffix='.swh', + prefix='tmp-file-check-converter-visible', + dir='/tmp') + with open(tmpfilepath, 'wb') as f: + f.write(contentfile) + + obj = { + 'path': tmpfilepath, + 'perms': GitPerm.BLOB, + 'type': GitType.BLOB, + 'sha1': 'some-sha1', + 'sha256': 'some-sha256', + 'sha1_git': 'some-sha1git', + } + + expected_blob = { + 'data': contentfile, + 'length': len(contentfile), + 'status': 'visible', + 'sha1': 'some-sha1', + 'sha256': 'some-sha256', + 'sha1_git': 'some-sha1git', + 'perms': GitPerm.BLOB.value, + 'type': GitType.BLOB.value, + } + + # when + actual_blob = converters.blob_to_content(obj) + + # then + self.assertEqual(actual_blob, expected_blob) + + os.remove(tmpfilepath) + + @istest + def blob_to_content_absent2(self): + # given + contentfile = b'temp file for testing blob to content convertion' + tmpfilepath = tempfile.mktemp(suffix='.swh', + prefix='tmp-file-check-converter-absent', + dir='/tmp') + with open(tmpfilepath, 'wb') as f: + f.write(contentfile) + + obj = { + 'path': tmpfilepath, + 'perms': GitPerm.BLOB, + 'type': GitType.BLOB, + 'sha1': 'some-sha1', + 'sha256': 'some-sha256', + 'sha1_git': 'some-sha1git', + } + + expected_blob = { + 'length': len(contentfile), + 'status': 'absent', + 'sha1': 'some-sha1', + 'sha256': 'some-sha256', + 'sha1_git': 'some-sha1git', + 'perms': GitPerm.BLOB.value, + 'type': GitType.BLOB.value, + 'reason': 'Content too large', + 'origin': 190 + } + + # when + actual_blob = converters.blob_to_content(obj, None, + max_content_size=10, + origin_id=190) + + # then + self.assertEqual(actual_blob, expected_blob) + + os.remove(tmpfilepath) + @istest def blob_to_content_visible(self): obj = { 'length': 9, - 'data': b'some-data', 'sha1': b'sha1', 'sha1_git': b'sha1-git', 'sha256': b'sha256', 'perms': GitPerm.BLOB, 'type': GitType.BLOB } expected_content = { 'length': 9, - 'data': b'some-data', 'sha1': b'sha1', 'sha1_git': b'sha1-git', 'sha256': b'sha256', 'perms': GitPerm.BLOB.value, 'type': GitType.BLOB.value, 'status': 'visible' } # when actual_content = converters._blob_to_content(obj) # then self.assertEqual(actual_content, expected_content) @istest def blob_to_content_absent(self): obj = { 'length': 9, - 'data': b'some-data', 'sha1': b'sha1', 'sha1_git': b'sha1-git', 'sha256': b'sha256', 'perms': GitPerm.BLOB, 'type': GitType.BLOB } expected_content = { 'length': 9, 'sha1': b'sha1', 'sha1_git': b'sha1-git', 'sha256': b'sha256', 'perms': GitPerm.BLOB.value, 'type': GitType.BLOB.value, 'status': 'absent', 'reason': 'Content too large', 'origin': 3} # when actual_content = converters._blob_to_content(obj, max_content_size=5, origin_id=3) # then self.assertDictEqual(actual_content, expected_content) @istest def tree_to_directory_no_entries(self): # given tree = { 'path': 'foo', 'sha1_git': b'tree_sha1_git' } objects = { 'foo': [{'type': GitType.TREE, 'perms': GitPerm.TREE, 'name': 'bar', 'sha1_git': b'sha1-target'}, {'type': GitType.BLOB, 'perms': GitPerm.BLOB, 'name': 'file-foo', 'sha1_git': b'file-foo-sha1-target'}] } expected_directory = { 'id': b'tree_sha1_git', 'entries': [{'type': 'dir', 'perms': int(GitPerm.TREE.value), 'name': 'bar', 'target': b'sha1-target'}, {'type': 'file', 'perms': int(GitPerm.BLOB.value), 'name': 'file-foo', 'target': b'file-foo-sha1-target'}] } # when actual_directory = converters.tree_to_directory(tree, objects) # then self.assertEqual(actual_directory, expected_directory) @istest def commit_to_revision(self): # given commit = { 'sha1_git': 'commit-git-sha1', 'author_date': 1444054085, 'author_offset': '+0000', 'committer_date': 1444054085, 'committer_offset': '-0000', 'type': 'tar', 'message': 'synthetic-message-input', 'author_name': 'author-name', 'author_email': 'author-email', 'committer_name': 'committer-name', 'committer_email': 'committer-email', 'metadata': {'checksums': {'sha1': b'sha1-as-bytes'}}, 'directory': 'targeted-tree-sha1', } objects = { git.ROOT_TREE_KEY: [{'sha1_git': 'targeted-tree-sha1'}] } expected_revision = { 'id': 'commit-git-sha1', 'date': datetime.datetime.fromtimestamp( 1444054085, tz=datetime.timezone.utc), 'date_offset': 0, 'committer_date': datetime.datetime.fromtimestamp( 1444054085, tz=datetime.timezone.utc), 'committer_date_offset': 0, 'type': 'tar', 'directory': 'targeted-tree-sha1', 'message': 'synthetic-message-input', 'author_name': 'author-name', 'author_email': 'author-email', 'committer_name': 'committer-name', 'committer_email': 'committer-email', 'synthetic': True, 'metadata': {'checksums': {'sha1': b'sha1-as-bytes'}}, 'parents': [], } # when actual_revision = converters.commit_to_revision(commit, objects) # then self.assertEquals(actual_revision, expected_revision)