Merge branch 'hierarchyfix' into improvements

* hierarchyfix:
  Added remaining tests for new base classes and removed some methods whose existance was doubtful or unsafe
  Fixed remaining tests to deal with the changes
  commit: fixed failing commit tests as the mocked git command would always return the same thing which does not work anymore - re-implemented it in a more dynamic manner, but in the end tests will have to be revised anyway
  mode-only change for test system - this should be in a separate repository in fact so that changes are a little more self-contained and not depending on the actual source repository
  fixed issue in Ref.name implementation which would not handle components properly
  lazymixin system now supports per-attribute baking, it is up to the class whether it bakes more. This also leads to more efficient use of memory as values are only cached and set when required - the baking system does not require an own tracking variable anymore, and values are only to be cached once - then python will natively find the cache without involving any additional overhead. This works by using __getattr__ instead of __get_attribute__ which would always be called
  put Tree and Blob onto a new base class suitable to deal with IndexObjects
  blob tests fixed to deal with changes to the Blob type
  converted all spaces to tabs ( 4 spaces = 1 tab ) just to allow me and my editor to work with the files properly. Can convert it back for releaes
  Re-designed the tag testing - it does not use fixtures anymore but dyamically checks the existance of tags within the repository - it basically tests the interface and checks that expected return types are actually returned
  Intermediate commit: commit,tree and blob objects now derive from object - test is in place which still fails on purpose. Need to integrate tags which can be objects or just a special form of a ref
  Renamed lazy.py to base.py to have a file for base classes - lazy not yet changed to allow proper rename tracking

1 files changed, 80 insertions, 79 deletions

diff --git a/test/git/test_blob.py b/test/git/test_blob.py
index 8741fc1d..94d3a33b 100644
--- a/test/git/test_blob.py
+++ b/test/git/test_blob.py
@@ -9,89 +9,90 @@ from test.testlib import *
 from git import *
 
 class TestBlob(object):
-    def setup(self):
-        self.repo = Repo(GIT_REPO)
-    
-    @patch_object(Git, '_call_process')
-    def test_should_return_blob_contents(self, git):
-        git.return_value = fixture('cat_file_blob')
-        blob = Blob(self.repo, **{'id': 'abc'})
-        assert_equal("Hello world", blob.data)
-        assert_true(git.called)
-        assert_equal(git.call_args, (('cat_file', 'abc'), {'p': True, 'with_raw_output': True}))
+	def setup(self):
+		self.repo = Repo(GIT_REPO)
+	
+	@patch_object(Git, '_call_process')
+	def test_should_return_blob_contents(self, git):
+		git.return_value = fixture('cat_file_blob')
+		blob = Blob(self.repo, **{'id': 'abc'})
+		assert_equal("Hello world", blob.data)
+		assert_true(git.called)
+		assert_equal(git.call_args, (('cat_file', 'abc'), {'p': True, 'with_raw_output': True}))
 
-    @patch_object(Git, '_call_process')
-    def test_should_return_blob_contents_with_newline(self, git):
-        git.return_value = fixture('cat_file_blob_nl')
-        blob = Blob(self.repo, **{'id': 'abc'})
-        assert_equal("Hello world\n", blob.data)
-        assert_true(git.called)
-        assert_equal(git.call_args, (('cat_file', 'abc'), {'p': True, 'with_raw_output': True}))
-    
-    @patch_object(Git, '_call_process')
-    def test_should_cache_data(self, git):
-        git.return_value = fixture('cat_file_blob')
-        blob = Blob(self.repo, **{'id': 'abc'})
-        blob.data
-        blob.data
-        assert_true(git.called)
-        assert_equal(git.call_count, 1)
-        assert_equal(git.call_args, (('cat_file', 'abc'), {'p': True, 'with_raw_output': True}))
+	@patch_object(Git, '_call_process')
+	def test_should_return_blob_contents_with_newline(self, git):
+		git.return_value = fixture('cat_file_blob_nl')
+		blob = Blob(self.repo, **{'id': 'abc'})
+		assert_equal("Hello world\n", blob.data)
+		assert_true(git.called)
+		assert_equal(git.call_args, (('cat_file', 'abc'), {'p': True, 'with_raw_output': True}))
+	
+	@patch_object(Git, '_call_process')
+	def test_should_cache_data(self, git):
+		git.return_value = fixture('cat_file_blob')
+		bid = '787b92b63f629398f3d2ceb20f7f0c2578259e84'
+		blob = Blob(self.repo, bid)
+		blob.data
+		blob.data
+		assert_true(git.called)
+		assert_equal(git.call_count, 1)
+		assert_equal(git.call_args, (('cat_file', bid), {'p': True, 'with_raw_output': True}))
 
-    @patch_object(Git, '_call_process')
-    def test_should_return_file_size(self, git):
-        git.return_value = fixture('cat_file_blob_size')
-        blob = Blob(self.repo, **{'id': 'abc'})
-        assert_equal(11, blob.size)
-        assert_true(git.called)
-        assert_equal(git.call_args, (('cat_file', 'abc'), {'s': True}))
+	@patch_object(Git, '_call_process')
+	def test_should_return_file_size(self, git):
+		git.return_value = fixture('cat_file_blob_size')
+		blob = Blob(self.repo, **{'id': 'abc'})
+		assert_equal(11, blob.size)
+		assert_true(git.called)
+		assert_equal(git.call_args, (('cat_file', 'abc'), {'s': True}))
 
-    @patch_object(Git, '_call_process')
-    def test_should_cache_file_size(self, git):
-        git.return_value = fixture('cat_file_blob_size')
-        blob = Blob(self.repo, **{'id': 'abc'})
-        assert_equal(11, blob.size)
-        assert_equal(11, blob.size)
-        assert_true(git.called)
-        assert_equal(git.call_count, 1)
-        assert_equal(git.call_args, (('cat_file', 'abc'), {'s': True}))
+	@patch_object(Git, '_call_process')
+	def test_should_cache_file_size(self, git):
+		git.return_value = fixture('cat_file_blob_size')
+		blob = Blob(self.repo, **{'id': 'abc'})
+		assert_equal(11, blob.size)
+		assert_equal(11, blob.size)
+		assert_true(git.called)
+		assert_equal(git.call_count, 1)
+		assert_equal(git.call_args, (('cat_file', 'abc'), {'s': True}))
   
-    def test_mime_type_should_return_mime_type_for_known_types(self):
-        blob = Blob(self.repo, **{'id': 'abc', 'path': 'foo.png'})
-        assert_equal("image/png", blob.mime_type)
+	def test_mime_type_should_return_mime_type_for_known_types(self):
+		blob = Blob(self.repo, **{'id': 'abc', 'path': 'foo.png'})
+		assert_equal("image/png", blob.mime_type)
   
-    def test_mime_type_should_return_text_plain_for_unknown_types(self):
-        blob = Blob(self.repo, **{'id': 'abc'})
-        assert_equal("text/plain", blob.mime_type)
+	def test_mime_type_should_return_text_plain_for_unknown_types(self):
+		blob = Blob(self.repo, **{'id': 'abc','path': 'something'})
+		assert_equal("text/plain", blob.mime_type)
   
-    @patch_object(Git, '_call_process')
-    def test_should_display_blame_information(self, git):
-        git.return_value = fixture('blame')
-        b = Blob.blame(self.repo, 'master', 'lib/git.py')
-        assert_equal(13, len(b))
-        assert_equal( 2, len(b[0]) )
-        # assert_equal(25, reduce(lambda acc, x: acc + len(x[-1]), b))
-        assert_equal(hash(b[0][0]), hash(b[9][0]))
-        c = b[0][0]
-        assert_true(git.called)
-        assert_equal(git.call_args, (('blame', 'master', '--', 'lib/git.py'), {'p': True}))
-        
-        assert_equal('634396b2f541a9f2d58b00be1a07f0c358b999b3', c.id)
-        assert_equal('Tom Preston-Werner', c.author.name)
-        assert_equal('tom@mojombo.com', c.author.email)
-        assert_equal(time.gmtime(1191997100), c.authored_date)
-        assert_equal('Tom Preston-Werner', c.committer.name)
-        assert_equal('tom@mojombo.com', c.committer.email)
-        assert_equal(time.gmtime(1191997100), c.committed_date)
-        assert_equal('initial grit setup', c.message)
-        
-        # test the 'lines per commit' entries
-        tlist = b[0][1]
-        assert_true( tlist )
-        assert_true( isinstance( tlist[0], basestring ) )
-        assert_true( len( tlist ) < sum( len(t) for t in tlist ) )				 # test for single-char bug
-        
+	@patch_object(Git, '_call_process')
+	def test_should_display_blame_information(self, git):
+		git.return_value = fixture('blame')
+		b = Blob.blame(self.repo, 'master', 'lib/git.py')
+		assert_equal(13, len(b))
+		assert_equal( 2, len(b[0]) )
+		# assert_equal(25, reduce(lambda acc, x: acc + len(x[-1]), b))
+		assert_equal(hash(b[0][0]), hash(b[9][0]))
+		c = b[0][0]
+		assert_true(git.called)
+		assert_equal(git.call_args, (('blame', 'master', '--', 'lib/git.py'), {'p': True}))
+		
+		assert_equal('634396b2f541a9f2d58b00be1a07f0c358b999b3', c.id)
+		assert_equal('Tom Preston-Werner', c.author.name)
+		assert_equal('tom@mojombo.com', c.author.email)
+		assert_equal(time.gmtime(1191997100), c.authored_date)
+		assert_equal('Tom Preston-Werner', c.committer.name)
+		assert_equal('tom@mojombo.com', c.committer.email)
+		assert_equal(time.gmtime(1191997100), c.committed_date)
+		assert_equal('initial grit setup', c.message)
+		
+		# test the 'lines per commit' entries
+		tlist = b[0][1]
+		assert_true( tlist )
+		assert_true( isinstance( tlist[0], basestring ) )
+		assert_true( len( tlist ) < sum( len(t) for t in tlist ) )				 # test for single-char bug
+		
   
-    def test_should_return_appropriate_representation(self):
-        blob = Blob(self.repo, **{'id': 'abc'})
-        assert_equal('<git.Blob "abc">', repr(blob))
+	def test_should_return_appropriate_representation(self):
+		blob = Blob(self.repo, **{'id': 'abc'})
+		assert_equal('<git.Blob "abc">', repr(blob))