reduced usage of force_bytes as clients are expected to pass bytes.

It was useful for debugging though, maybe an explicit type assertions would
help others ?
As 'others' will be gitpython, I suppose I can handle it myself

Author: Byron

gitdb/stream.py

@@ -543,9 +543,9 @@ def __init__(self):
 
     def write(self, data):
         """:raise IOError: If not all bytes could be written
+        :param data: byte object
         :return: length of incoming data"""
 
-        data = force_bytes(data)
         self.sha1.update(data)
 
         return len(data)
@@ -590,8 +590,6 @@ def __getattr__(self, attr):
         return getattr(self.buf, attr)
 
     def write(self, data):
-        data = force_bytes(data)
-
         alen = Sha1Writer.write(self, data)
         self.buf.write(self.zip.compress(data))
 
@@ -634,8 +632,6 @@ def __init__(self, fd):
     def write(self, data):
         """:raise IOError: If not all bytes could be written
         :return: lenght of incoming data"""
-        data = force_bytes(data)
-
         self.sha1.update(data)
         cdata = self.zip.compress(data)
         bytes_written = write(self.fd, cdata)

gitdb/test/db/lib.py

@@ -23,7 +23,6 @@
 
 from gitdb.exc import BadObject
 from gitdb.typ import str_blob_type
-from gitdb.utils.encoding import force_bytes
 from gitdb.utils.compat import xrange
 
 from io import BytesIO
@@ -37,7 +36,7 @@ class TestDBBase(TestBase):
     """Base class providing testing routines on databases"""
 
     # data
-    two_lines = "1234\nhello world"
+    two_lines = b'1234\nhello world'
     all_data = (two_lines, )
 
     def _assert_object_writing_simple(self, db):
@@ -83,7 +82,7 @@ def _assert_object_writing(self, db):
 
                 prev_ostream = db.set_ostream(ostream)
                 assert type(prev_ostream) in ostreams or prev_ostream in ostreams
-                istream = IStream(str_blob_type, len(data), BytesIO(data.encode("ascii")))
+                istream = IStream(str_blob_type, len(data), BytesIO(data))
 
                 # store returns same istream instance, with new sha set
                 my_istream = db.store(istream)
@@ -99,7 +98,7 @@ def _assert_object_writing(self, db):
                     assert info.size == len(data)
 
                     ostream = db.stream(sha)
-                    assert ostream.read() == force_bytes(data)
+                    assert ostream.read() == data
                     assert ostream.type == str_blob_type
                     assert ostream.size == len(data)
                 else:

gitdb/test/db/test_ref.py

@@ -2,7 +2,11 @@
 #
 # This module is part of GitDB and is released under
 # the New BSD License: http://www.opensource.org/licenses/bsd-license.php
-from gitdb.test.db.lib import *
+from gitdb.test.db.lib import (
+    TestDBBase, 
+    with_rw_directory, 
+    fixture_path
+)
 from gitdb.db import ReferenceDB
 
 from gitdb.util import (
@@ -24,8 +28,6 @@ def make_alt_file(self, alt_path, alt_list):
 
     @with_rw_directory
     def test_writing(self, path):
-        NULL_BIN_SHA = '\0'.encode("ascii") * 20
-
         alt_path = os.path.join(path, 'alternates')
         rdb = ReferenceDB(alt_path)
         assert len(rdb.databases()) == 0

gitdb/utils/encoding.py

@@ -7,7 +7,7 @@
     string_types = (basestring, )
     text_type = unicode
 
-def force_bytes(data, encoding="utf-8"):
+def force_bytes(data, encoding="ascii"):
     if isinstance(data, bytes):
         return data
 
@@ -23,10 +23,6 @@ def force_text(data, encoding="utf-8"):
     if isinstance(data, string_types):
         return data.decode(encoding)
 
-    if not isinstance(data, bytes):
-        assert False, "Shouldn't be here"
-        data = force_bytes(data, encoding)
-
     if compat.PY3:
         return text_type(data, encoding)
     else: