Imported Upstream version 0.7
[pysam.git] / tests / pysam_test.py
index d2bf4150de58551accb107f1822e85912c7c58d5..34071822e350237802cf99a50715a9d807debeb2 100755 (executable)
@@ -8,11 +8,22 @@ and data files located there.
 import pysam
 import unittest
 import os, re, sys
-import itertools, collections
+import itertools
+import collections
 import subprocess
 import shutil
 import logging
+
+IS_PYTHON3 = sys.version_info[0] >= 3
+
+if IS_PYTHON3:
+    from itertools import zip_longest
+else:
+    from itertools import izip as zip_longest
+
+
 SAMTOOLS="samtools"
+WORKDIR="pysam_test_work"
 
 def checkBinaryEqual( filename1, filename2 ):
     '''return true if the two files are binary equal.'''
@@ -25,11 +36,11 @@ def checkBinaryEqual( filename1, filename2 ):
     def chariter( infile ):
         while 1:
             c = infile.read(1)
-            if c == "": break
+            if c == b"": break
             yield c
 
     found = False
-    for c1,c2 in itertools.izip( chariter( infile1), chariter( infile2) ):
+    for c1,c2 in zip_longest( chariter( infile1), chariter( infile2) ):
         if c1 != c2: break
     else:
         found = True
@@ -44,15 +55,18 @@ def runSamtools( cmd ):
     try:
         retcode = subprocess.call(cmd, shell=True)
         if retcode < 0:
-            print >>sys.stderr, "Child was terminated by signal", -retcode
-    except OSError, e:
-        print >>sys.stderr, "Execution failed:", e
+            print("Child was terminated by signal", -retcode)
+    except OSError as e:
+        print("Execution failed:", e)
 
 def getSamtoolsVersion():
     '''return samtools version'''
 
-    pipe = subprocess.Popen(SAMTOOLS, shell=True, stderr=subprocess.PIPE).stderr
-    lines = "".join(pipe.readlines())
+    with subprocess.Popen(SAMTOOLS, shell=True, stderr=subprocess.PIPE).stderr as pipe:
+        lines = b"".join(pipe.readlines())
+
+    if IS_PYTHON3:
+        lines = lines.decode('ascii')
     return re.search( "Version:\s+(\S+)", lines).groups()[0]
 
 class BinaryTest(unittest.TestCase):
@@ -65,56 +79,131 @@ class BinaryTest(unittest.TestCase):
     first_time = True
 
     # a list of commands to test
-    mCommands = \
-        { "faidx" : \
-        ( 
-            ("ex1.fa.fai", "faidx ex1.fa"), 
-            ("pysam_ex1.fa.fai", (pysam.faidx, "ex1.fa") ),
-            ),
-          "import" :
+    commands = \
+        { 
+          "view" :
               (
-                ("ex1.bam", "import ex1.fa.fai ex1.sam.gz ex1.bam" ),
-                ("pysam_ex1.bam", (pysam.samimport, "ex1.fa.fai ex1.sam.gz pysam_ex1.bam") ),
+                ("ex1.view", "view ex1.bam > ex1.view"),
+                ("pysam_ex1.view", (pysam.view, "ex1.bam" ) ),
+                ),
+          "view2" :
+              (
+                ("ex1.view", "view -bT ex1.fa -o ex1.view2 ex1.sam"),
+                # note that -o ex1.view2 throws exception.
+                ("pysam_ex1.view", (pysam.view, "-bT ex1.fa -oex1.view2 ex1.sam" ) ),
+                ),
+          "sort" :
+              (
+                ( "ex1.sort.bam", "sort ex1.bam ex1.sort" ),
+                ( "pysam_ex1.sort.bam", (pysam.sort, "ex1.bam pysam_ex1.sort" ) ),
+                ),
+          "mpileup" :
+              (
+                ("ex1.pileup", "mpileup ex1.bam > ex1.pileup" ),
+                ("pysam_ex1.mpileup", (pysam.mpileup, "ex1.bam" ) ),
+                ),
+          "depth" :
+              (
+                ("ex1.depth", "depth ex1.bam > ex1.depth" ),
+                ("pysam_ex1.depth", (pysam.depth, "ex1.bam" ) ),
+                ),
+          "faidx" : 
+              ( 
+                ("ex1.fa.fai", "faidx ex1.fa"), 
+                ("pysam_ex1.fa.fai", (pysam.faidx, "ex1.fa") ),
                 ),
           "index":
               (
                 ("ex1.bam.bai", "index ex1.bam" ),
                 ("pysam_ex1.bam.bai", (pysam.index, "pysam_ex1.bam" ) ),
                 ),
-          "pileup1" :
-          (
-                ("ex1.pileup", "pileup -cf ex1.fa ex1.bam > ex1.pileup" ),
-                ("pysam_ex1.pileup", (pysam.pileup, "-c -f ex1.fa ex1.bam" ) )
+          "idxstats" :
+              ( 
+                ("ex1.idxstats", "idxstats ex1.bam > ex1.idxstats" ),
+                ("pysam_ex1.idxstats", (pysam.idxstats, "pysam_ex1.bam" ) ),
                 ),
-          "pileup2" :
-          (
-                ("ex1.glf", "pileup -gf ex1.fa ex1.bam > ex1.glf" ),
-                ("pysam_ex1.glf", (pysam.pileup, "-g -f ex1.fa ex1.bam" ) )
+          "fixmate" :
+              (
+                ("ex1.fixmate", "fixmate ex1.bam ex1.fixmate" ),
+                ("pysam_ex1.fixmate", (pysam.fixmate, "pysam_ex1.bam pysam_ex1.fixmate") ),
                 ),
-          "glfview" :
-        (
-                ("ex1.glfview", "glfview ex1.glf > ex1.glfview"),
-                ("pysam_ex1.glfview", (pysam.glfview, "ex1.glf" ) ),
+          "flagstat" :
+              (
+                ("ex1.flagstat", "flagstat ex1.bam > ex1.flagstat" ),
+                ("pysam_ex1.flagstat", (pysam.flagstat, "pysam_ex1.bam") ),
                 ),
-          "view" :
-        (
-                ("ex1.view", "view ex1.bam > ex1.view"),
-                ("pysam_ex1.view", (pysam.view, "ex1.bam" ) ),
+          "calmd" :
+              (
+                ("ex1.calmd", "calmd ex1.bam ex1.fa > ex1.calmd" ),
+                ("pysam_ex1.calmd", (pysam.calmd, "pysam_ex1.bam ex1.fa") ),
                 ),
-          "view2" :
-        (
-                ("ex1.view", "view -bT ex1.fa -o ex1.view2 ex1.sam"),
-                # note that -o ex1.view2 throws exception.
-                ("pysam_ex1.view", (pysam.view, "-bT ex1.fa -oex1.view2 ex1.sam" ) ),
+          "merge" :
+              (
+                ("ex1.merge", "merge -f ex1.merge ex1.bam ex1.bam" ),
+                # -f option does not work - following command will cause the subsequent
+                # command to fail
+                ("pysam_ex1.merge", (pysam.merge, "pysam_ex1.merge pysam_ex1.bam pysam_ex1.bam") ),
+                ),
+          "rmdup" :
+              (
+                ("ex1.rmdup", "rmdup ex1.bam ex1.rmdup" ),
+                ("pysam_ex1.rmdup", (pysam.rmdup, "pysam_ex1.bam pysam_ex1.rmdup" )),
+                ),
+          "reheader" :
+              (
+                ( "ex1.reheader", "reheader ex1.bam ex1.bam > ex1.reheader"),
+                ( "pysam_ex1.reheader", (pysam.reheader, "ex1.bam ex1.bam" ) ),
+                ),
+          "cat":
+              (
+                ( "ex1.cat", "cat ex1.bam ex1.bam > ex1.cat"),
+                ( "pysam_ex1.cat", (pysam.cat, "ex1.bam ex1.bam" ) ),
+                ),
+          "targetcut":
+              (
+                ("ex1.targetcut", "targetcut ex1.bam > ex1.targetcut" ),
+                ("pysam_ex1.targetcut", (pysam.targetcut, "pysam_ex1.bam") ),
+                ),
+          "phase":
+              (
+                ("ex1.phase", "phase ex1.bam > ex1.phase" ),
+                ("pysam_ex1.phase", (pysam.phase, "pysam_ex1.bam") ),
+                ),
+          "import" :
+              (
+                ("ex1.bam", "import ex1.fa.fai ex1.sam.gz ex1.bam" ),
+                ("pysam_ex1.bam", (pysam.samimport, "ex1.fa.fai ex1.sam.gz pysam_ex1.bam") ),
+                ),
+          "bam2fq":
+              (
+                ("ex1.bam2fq", "bam2fq ex1.bam > ex1.bam2fq" ),
+                ("pysam_ex1.bam2fq", (pysam.bam2fq, "pysam_ex1.bam") ),
                 ),
         }
 
     # some tests depend on others. The order specifies in which order
     # the samtools commands are executed.
-    mOrder = ('faidx', 'import', 'index', 
-              'pileup1', 'pileup2', 
+    # The first three (faidx, import, index) need to be in that order, 
+    # the rest is arbitrary.
+    order = ('faidx', 'import', 'index', 
+              # 'pileup1', 'pileup2', deprecated
               # 'glfview', deprecated
-              'view', 'view2' )
+              'view', 'view2',
+              'sort',
+              'mpileup',
+              'depth',
+              'idxstats',
+              'fixmate',
+              'flagstat',
+              # 'calmd',
+              'merge',
+              'rmdup',
+              'reheader',
+              'cat',
+              'targetcut',
+              'phase',
+              'bam2fq',
+              )
 
     def setUp( self ):
         '''setup tests. 
@@ -124,25 +213,63 @@ class BinaryTest(unittest.TestCase):
         files.
         '''
         if BinaryTest.first_time:
-            # copy the source 
-            shutil.copy( "ex1.fa", "pysam_ex1.fa" )
 
-            for label in self.mOrder:
-                command = self.mCommands[label]
+            # remove previous files
+            if os.path.exists( WORKDIR ):
+                shutil.rmtree( WORKDIR )
+                pass
+
+            # copy the source files to WORKDIR
+            os.makedirs( WORKDIR )
+
+            shutil.copy( "ex1.fa", os.path.join( WORKDIR, "pysam_ex1.fa" ) )
+            shutil.copy( "ex1.fa", os.path.join( WORKDIR, "ex1.fa" ) )
+            shutil.copy( "ex1.sam.gz", os.path.join( WORKDIR, "ex1.sam.gz" ) )
+            shutil.copy( "ex1.sam", os.path.join( WORKDIR, "ex1.sam" ) )
+
+            # cd to workdir
+            savedir = os.getcwd()
+            os.chdir( WORKDIR )
+            
+            for label in self.order:
+                command = self.commands[label]
+                # build samtools command and target and run
                 samtools_target, samtools_command = command[0]
-                pysam_target, pysam_command = command[1]
                 runSamtools( " ".join( (SAMTOOLS, samtools_command )))
+
+                # get pysam command and run
+                try:
+                    pysam_target, pysam_command = command[1]
+                except ValueError as msg:
+                    raise ValueError( "error while setting up %s=%s: %s" %\
+                                          (label, command, msg) )
+
                 pysam_method, pysam_options = pysam_command
-                output = pysam_method( *pysam_options.split(" "), raw=True)
-                if ">" in samtools_command:
-                    outfile = open( pysam_target, "wb" )
-                    for line in output: outfile.write( line )
-                    outfile.close()
+                try:
+                    output = pysam_method( *pysam_options.split(" "), raw=True)
+                except pysam.SamtoolsError as msg:
+                    raise pysam.SamtoolsError( "error while executing %s: options=%s: msg=%s" %\
+                                                   (label, pysam_options, msg) )
+
+                
 
+                if ">" in samtools_command:
+                    with open( pysam_target, "wb" ) as outfile:
+                        if type(output) == list:
+                            if IS_PYTHON3:
+                                for line in output: 
+                                    outfile.write( line.encode('ascii') )
+                            else:
+                                for line in output: outfile.write( line )
+                        else:
+                            outfile.write(output)
+
+            os.chdir( savedir )
             BinaryTest.first_time = False
 
         samtools_version = getSamtoolsVersion()
 
+        
         def _r( s ):
             # patch - remove any of the alpha/beta suffixes, i.e., 0.1.12a -> 0.1.12
             if s.count('-') > 0: s = s[0:s.find('-')]
@@ -155,7 +282,9 @@ class BinaryTest(unittest.TestCase):
 
     def checkCommand( self, command ):
         if command:
-            samtools_target, pysam_target = self.mCommands[command][0][0], self.mCommands[command][1][0]
+            samtools_target, pysam_target = self.commands[command][0][0], self.commands[command][1][0]
+            samtools_target = os.path.join( WORKDIR, samtools_target )
+            pysam_target = os.path.join( WORKDIR, pysam_target )
             self.assertTrue( checkBinaryEqual( samtools_target, pysam_target ), 
                              "%s failed: files %s and %s are not the same" % (command, samtools_target, pysam_target) )
             
@@ -164,12 +293,51 @@ class BinaryTest(unittest.TestCase):
 
     def testIndex( self ):
         self.checkCommand( "index" )
+
+    def testSort( self ):
+        self.checkCommand( "sort" )
+
+    def testMpileup( self ):
+        self.checkCommand( "mpileup" )
+
+    def testDepth( self ):
+        self.checkCommand( "depth" )
+
+    def testIdxstats( self ):
+        self.checkCommand( "idxstats" )
+
+    def testFixmate( self ):
+        self.checkCommand( "fixmate" )
+
+    def testFlagstat( self ):
+        self.checkCommand( "flagstat" )
         
-    def testPileup1( self ):
-        self.checkCommand( "pileup1" )
+    def testMerge( self ):
+        self.checkCommand( "merge" )
+
+    def testRmdup( self ):
+        self.checkCommand( "rmdup" )
+
+    def testReheader( self ):
+        self.checkCommand( "reheader" )
+
+    def testCat( self ):
+        self.checkCommand( "cat" )
+
+    def testTargetcut( self ):
+        self.checkCommand( "targetcut" )
+
+    def testPhase( self ):
+        self.checkCommand( "phase" )
+
+    def testBam2fq( self ):
+        self.checkCommand( "bam2fq" )
+
+    # def testPileup1( self ):
+    #     self.checkCommand( "pileup1" )
     
-    def testPileup2( self ):
-        self.checkCommand( "pileup2" )
+    def testPileup2( self ):
+        self.checkCommand( "pileup2" )
 
     # deprecated
     # def testGLFView( self ):
@@ -182,27 +350,26 @@ class BinaryTest(unittest.TestCase):
         self.assertRaises( IOError, pysam.index, "exdoesntexist.bam" )
 
     def __del__(self):
-        return
-        for label, command in self.mCommands.iteritems():
-            samtools_target, samtools_command = command[0]
-            pysam_target, pysam_command = command[1]
-            if os.path.exists( samtools_target): os.remove( samtools_target )
-            if os.path.exists( pysam_target): os.remove( pysam_target )
-        if os.path.exists( "pysam_ex1.fa" ): os.remove( "pysam_ex1.fa" )
+        if os.path.exists( WORKDIR ):
+            pass
+        # shutil.rmtree( WORKDIR )
 
 class IOTest(unittest.TestCase):
     '''check if reading samfile and writing a samfile are consistent.'''
 
-    def checkEcho( self, input_filename, reference_filename, 
+    def checkEcho( self, input_filename, 
+                   reference_filename, 
                    output_filename, 
-                   input_mode, output_mode, use_template = True):
+                   input_mode, output_mode, use_template = True ):
         '''iterate through *input_filename* writing to *output_filename* and
         comparing the output to *reference_filename*. 
         
         The files are opened according to the *input_mode* and *output_mode*.
 
         If *use_template* is set, the header is copied from infile using the
-        template mechanism, otherwise target names and lengths are passed explicitely. 
+        template mechanism, otherwise target names and lengths are passed 
+        explicitely. 
+
         '''
 
         infile = pysam.Samfile( input_filename, input_mode )
@@ -211,9 +378,11 @@ class IOTest(unittest.TestCase):
         else:
             outfile = pysam.Samfile( output_filename, output_mode, 
                                      referencenames = infile.references,
-                                     referencelengths = infile.lengths )
-
+                                     referencelengths = infile.lengths,
+                                     add_sq_text = False )
+            
         iter = infile.fetch()
+
         for x in iter: outfile.write( x )
         infile.close()
         outfile.close()
@@ -221,6 +390,7 @@ class IOTest(unittest.TestCase):
         self.assertTrue( checkBinaryEqual( reference_filename, output_filename), 
                          "files %s and %s are not the same" % (reference_filename, output_filename) )
 
+
     def testReadWriteBam( self ):
         
         input_filename = "ex1.bam"
@@ -257,6 +427,59 @@ class IOTest(unittest.TestCase):
         self.checkEcho( input_filename, reference_filename, output_filename,
                         "r", "w" )
 
+    def testReadSamWithoutTargetNames( self ):
+        '''see issue 104.'''
+        input_filename = "example_unmapped_reads_no_sq.sam"
+
+        # raise exception in default mode
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r" )
+
+        # raise exception if no SQ files
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r",
+                           check_header = True)
+
+        infile = pysam.Samfile( input_filename, check_header = False, check_sq = False )
+        result = list(infile.fetch())
+
+    def testReadBamWithoutTargetNames( self ):
+        '''see issue 104.'''
+        input_filename = "example_unmapped_reads_no_sq.bam"
+
+        # raise exception in default mode
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r" )
+
+        # raise exception if no SQ files
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r",
+                           check_header = True)
+
+
+        infile = pysam.Samfile( input_filename, check_header = False, check_sq = False )
+        result = list(infile.fetch( until_eof = True))
+
+    def testReadSamWithoutHeader( self ):
+        input_filename = "ex1.sam"
+        output_filename = "pysam_ex1.sam"
+        reference_filename = "ex1.sam"
+
+        # reading from a samfile without header is not implemented.
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r" )
+
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r",
+                           check_header = False )
+
+    def testReadUnformattedFile( self ):
+        '''test reading from a file that is not bam/sam formatted'''
+        input_filename = "example.vcf40"
+
+        # bam - file raise error
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "rb" )
+
+        # sam - file error, but can't fetch
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r" )
+        
+        self.assertRaises( ValueError, pysam.Samfile, input_filename, "r", 
+                           check_header = False)
+
     def testFetchFromClosedFile( self ):
 
         samfile = pysam.Samfile( "ex1.bam", "rb" )
@@ -283,14 +506,22 @@ class IOTest(unittest.TestCase):
         self.assertEqual( 0, samfile.write(None) )
 
     def testAutoDetection( self ):
-        samfile = pysam.Samfile( "ex3.bam" )
-        samfile.fetch()
+        '''test if autodetection works.'''
+
+        samfile = pysam.Samfile( "ex3.sam" )
+        self.assertRaises( ValueError, samfile.fetch, 'chr1' )
         samfile.close()
 
-        samfile = pysam.Samfile( "ex3.sam")
-        samfile.fetch()
+        samfile = pysam.Samfile( "ex3.bam" )
+        samfile.fetch('chr1')
         samfile.close()
 
+    def testReadingFromSamFileWithoutHeader( self ):
+        '''read from samfile without header.
+        '''
+        samfile = pysam.Samfile( "ex7.sam", check_header = False, check_sq = False )
+        self.assertRaises( NotImplementedError, samfile.__iter__ )
+
     def testReadingFromFileWithoutIndex( self ):
         '''read from bam file without index.'''
 
@@ -299,6 +530,87 @@ class IOTest(unittest.TestCase):
         self.assertRaises( ValueError, samfile.fetch )
         self.assertEqual( len(list( samfile.fetch(until_eof = True) )), 3270 )
 
+    def testReadingUniversalFileMode( self ):
+        '''read from samfile without header.
+        '''
+
+        input_filename = "ex2.sam"
+        output_filename = "pysam_ex2.sam"
+        reference_filename = "ex1.sam"
+
+        self.checkEcho( input_filename, reference_filename, output_filename,
+                        "rU", "w" )
+
+class TestFloatTagBug( unittest.TestCase ):
+    '''see issue 71'''
+
+    def testFloatTagBug( self ): 
+        '''a float tag before another exposed a parsing bug in bam_aux_get - expected to fail
+
+        This test is expected to fail until samtools is fixed.
+        '''
+        samfile = pysam.Samfile("tag_bug.bam")
+        read = next(samfile.fetch(until_eof=True))
+        self.assertTrue( ('XC',1) in read.tags )
+        self.assertEqual(read.opt('XC'), 1)
+
+class TestLargeFieldBug( unittest.TestCase ):
+    '''see issue 100'''
+
+    def testLargeFileBug( self ): 
+        '''when creating a read with a large entry in the tag field
+        causes an errror:
+            NotImplementedError: tags field too large
+        '''
+        samfile = pysam.Samfile("issue100.bam")
+        read = next(samfile.fetch(until_eof=True))
+        new_read = pysam.AlignedRead()
+        new_read.tags = read.tags
+        self.assertEqual( new_read.tags, read.tags )
+
+class TestTagParsing( unittest.TestCase ):
+    '''tests checking the accuracy of tag setting and retrieval.'''
+
+    def makeRead( self ):
+        a = pysam.AlignedRead()
+        a.qname = "read_12345"
+        a.tid = 0
+        a.seq="ACGT" * 3
+        a.flag = 0
+        a.rname = 0
+        a.pos = 1
+        a.mapq = 20
+        a.cigar = ( (0,10), (2,1), (0,25) )
+        a.mrnm = 0
+        a.mpos=200
+        a.isize = 0
+        a.qual ="1234" * 3
+        # todo: create tags
+        return a
+
+    def testNegativeIntegers( self ):
+        x = -2
+        aligned_read = self.makeRead()
+        aligned_read.tags = [("XD", int(x) ) ]
+        # print (aligned_read.tags)
+
+    def testNegativeIntegers2( self ):
+        x = -2
+        r = self.makeRead()
+        r.tags = [("XD", int(x) ) ]
+        outfile = pysam.Samfile( "test.bam",
+                                 "wb",
+                                 referencenames = ("chr1",),
+                                 referencelengths = (1000,) )
+        outfile.write (r )
+        outfile.close()
+
+    def testCigarString( self ):
+        r = self.makeRead()
+        self.assertEqual( r.cigarstring, "M10D1M25" )
+        r.cigarstring = "M20D10M20"
+        self.assertEqual( r.cigar, [(0,20), (2,10), (0,20)])
+
 class TestIteratorRow(unittest.TestCase):
 
     def setUp(self):
@@ -310,15 +622,18 @@ class TestIteratorRow(unittest.TestCase):
         sa = list(pysam.view( "ex1.bam", rnge, raw = True) )
         self.assertEqual( len(ps), len(sa), "unequal number of results for range %s: %i != %i" % (rnge, len(ps), len(sa) ))
         # check if the same reads are returned and in the same order
-        for line, pair in enumerate( zip( ps, sa ) ):
-            a,b = pair
+        for line, (a, b) in enumerate( list(zip( ps, sa )) ):
             d = b.split("\t")
             self.assertEqual( a.qname, d[0], "line %i: read id mismatch: %s != %s" % (line, a.rname, d[0]) )
             self.assertEqual( a.pos, int(d[3])-1, "line %i: read position mismatch: %s != %s, \n%s\n%s\n" % \
                                   (line, a.pos, int(d[3])-1,
                                    str(a), str(d) ) )
-            self.assertEqual( a.qual, d[10], "line %i: quality mismatch: %s != %s, \n%s\n%s\n" % \
-                                  (line, a.qual, d[10],
+            if sys.version_info[0] < 3:
+                qual = d[10]
+            else:
+                qual = d[10].encode('ascii')
+            self.assertEqual( a.qual, qual, "line %i: quality mismatch: %s != %s, \n%s\n%s\n" % \
+                                  (line, a.qual, qual,
                                    str(a), str(d) ) )
 
     def testIteratePerContig(self):
@@ -335,6 +650,7 @@ class TestIteratorRow(unittest.TestCase):
     def tearDown(self):
         self.samfile.close()
 
+
 class TestIteratorRowAll(unittest.TestCase):
 
     def setUp(self):
@@ -346,7 +662,7 @@ class TestIteratorRowAll(unittest.TestCase):
         sa = list(pysam.view( "ex1.bam", raw = True) )
         self.assertEqual( len(ps), len(sa), "unequal number of results: %i != %i" % (len(ps), len(sa) ))
         # check if the same reads are returned
-        for line, pair in enumerate( zip( ps, sa ) ):
+        for line, pair in enumerate( list(zip( ps, sa )) ):
             data = pair[1].split("\t")
             self.assertEqual( pair[0].qname, data[0], "read id mismatch in line %i: %s != %s" % (line, pair[0].rname, data[0]) )
 
@@ -366,10 +682,13 @@ class TestIteratorColumn(unittest.TestCase):
     def setUp(self):
         self.samfile=pysam.Samfile( "ex4.bam","rb" )
 
-    def checkRange( self, rnge ):
+    def checkRange( self, contig, start = None, end = None, truncate = False ):
         '''compare results from iterator with those from samtools.'''
         # check if the same reads are returned and in the same order
-        for column in self.samfile.pileup(region=rnge):
+        for column in self.samfile.pileup(contig, start, end, truncate = truncate):
+            if truncate:
+                self.assertGreaterEqual( column.pos, start )
+                self.assertLess( column.pos, end )
             thiscov = len(column.pileups)
             refcov = self.mCoverages[self.samfile.getrname(column.tid)][column.pos]
             self.assertEqual( thiscov, refcov, "wrong coverage at pos %s:%i %i should be %i" % (self.samfile.getrname(column.tid), column.pos, thiscov, refcov))
@@ -387,11 +706,11 @@ class TestIteratorColumn(unittest.TestCase):
         '''check random access per range'''
         for contig, length in zip(self.samfile.references, self.samfile.lengths):
             for start in range( 1, length, 90):
-                self.checkRange( "%s:%i-%i" % (contig, start, start + 90) ) # this includes empty ranges
+                self.checkRange( contig, start, start + 90 ) # this includes empty ranges
 
     def testInverse( self ):
         '''test the inverse, is point-wise pileup accurate.'''
-        for contig, refseq in self.mCoverages.items():
+        for contig, refseq in list(self.mCoverages.items()):
             refcolumns = sum(refseq)
             for pos, refcov in enumerate( refseq ):
                 columns = list(self.samfile.pileup( contig, pos, pos+1) )
@@ -403,10 +722,17 @@ class TestIteratorColumn(unittest.TestCase):
                     self.assertEqual( len(columns), refcolumns, "pileup incomplete at position %i: got %i, expected %i " %\
                                           (pos, len(columns), refcolumns))
 
-                    
-    
+    def testIterateTruncate( self ):
+        '''check random access per range'''
+        for contig, length in zip(self.samfile.references, self.samfile.lengths):
+            for start in range( 1, length, 90):
+                self.checkRange( contig, start, start + 90, truncate = True ) # this includes empty ranges
+                
+        
+        
     def tearDown(self):
         self.samfile.close()
+
     
 class TestAlignedReadFromBam(unittest.TestCase):
 
@@ -441,34 +767,40 @@ class TestAlignedReadFromBam(unittest.TestCase):
     def testARmrnm(self):
         self.assertEqual( self.reads[0].mrnm, 0, "mate reference sequence name mismatch in read 1: %s != %s" % (self.reads[0].mrnm, 0) )
         self.assertEqual( self.reads[1].mrnm, 1, "mate reference sequence name mismatch in read 2: %s != %s" % (self.reads[1].mrnm, 1) )
+        self.assertEqual( self.reads[0].rnext, 0, "mate reference sequence name mismatch in read 1: %s != %s" % (self.reads[0].rnext, 0) )
+        self.assertEqual( self.reads[1].rnext, 1, "mate reference sequence name mismatch in read 2: %s != %s" % (self.reads[1].rnext, 1) )
 
     def testARmpos(self):
         self.assertEqual( self.reads[0].mpos, 200-1, "mate mapping position mismatch in read 1: %s != %s" % (self.reads[0].mpos, 200-1) )
         self.assertEqual( self.reads[1].mpos, 500-1, "mate mapping position mismatch in read 2: %s != %s" % (self.reads[1].mpos, 500-1) )
+        self.assertEqual( self.reads[0].pnext, 200-1, "mate mapping position mismatch in read 1: %s != %s" % (self.reads[0].pnext, 200-1) )
+        self.assertEqual( self.reads[1].pnext, 500-1, "mate mapping position mismatch in read 2: %s != %s" % (self.reads[1].pnext, 500-1) )
 
     def testARisize(self):
         self.assertEqual( self.reads[0].isize, 167, "insert size mismatch in read 1: %s != %s" % (self.reads[0].isize, 167) )
         self.assertEqual( self.reads[1].isize, 412, "insert size mismatch in read 2: %s != %s" % (self.reads[1].isize, 412) )
+        self.assertEqual( self.reads[0].tlen, 167, "insert size mismatch in read 1: %s != %s" % (self.reads[0].tlen, 167) )
+        self.assertEqual( self.reads[1].tlen, 412, "insert size mismatch in read 2: %s != %s" % (self.reads[1].tlen, 412) )
 
     def testARseq(self):
-        self.assertEqual( self.reads[0].seq, "AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG", "sequence mismatch in read 1: %s != %s" % (self.reads[0].seq, "AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG") )
-        self.assertEqual( self.reads[1].seq, "ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA", "sequence size mismatch in read 2: %s != %s" % (self.reads[1].seq, "ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA") )
-        self.assertEqual( self.reads[3].seq, "AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG", "sequence mismatch in read 4: %s != %s" % (self.reads[3].seq, "AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG") )
+        self.assertEqual( self.reads[0].seq, b"AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG", "sequence mismatch in read 1: %s != %s" % (self.reads[0].seq, b"AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG") )
+        self.assertEqual( self.reads[1].seq, b"ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA", "sequence size mismatch in read 2: %s != %s" % (self.reads[1].seq, b"ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA") )
+        self.assertEqual( self.reads[3].seq, b"AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG", "sequence mismatch in read 4: %s != %s" % (self.reads[3].seq, b"AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG") )
 
     def testARqual(self):
-        self.assertEqual( self.reads[0].qual, "<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<", "quality string mismatch in read 1: %s != %s" % (self.reads[0].qual, "<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<") )
-        self.assertEqual( self.reads[1].qual, "<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<", "quality string mismatch in read 2: %s != %s" % (self.reads[1].qual, "<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<") )
-        self.assertEqual( self.reads[3].qual, "<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<", "quality string mismatch in read 3: %s != %s" % (self.reads[3].qual, "<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<") )
+        self.assertEqual( self.reads[0].qual, b"<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<", "quality string mismatch in read 1: %s != %s" % (self.reads[0].qual, b"<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<") )
+        self.assertEqual( self.reads[1].qual, b"<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<", "quality string mismatch in read 2: %s != %s" % (self.reads[1].qual, b"<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<") )
+        self.assertEqual( self.reads[3].qual, b"<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<", "quality string mismatch in read 3: %s != %s" % (self.reads[3].qual, b"<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<") )
 
     def testARquery(self):
-        self.assertEqual( self.reads[0].query, "AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG", "query mismatch in read 1: %s != %s" % (self.reads[0].query, "AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG") )
-        self.assertEqual( self.reads[1].query, "ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA", "query size mismatch in read 2: %s != %s" % (self.reads[1].query, "ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA") )
-        self.assertEqual( self.reads[3].query, "TAGCTAGCTACCTATATCTTGGTCTT", "query mismatch in read 4: %s != %s" % (self.reads[3].query, "TAGCTAGCTACCTATATCTTGGTCTT") )
+        self.assertEqual( self.reads[0].query, b"AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG", "query mismatch in read 1: %s != %s" % (self.reads[0].query, b"AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG") )
+        self.assertEqual( self.reads[1].query, b"ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA", "query size mismatch in read 2: %s != %s" % (self.reads[1].query, b"ACCTATATCTTGGCCTTGGCCGATGCGGCCTTGCA") )
+        self.assertEqual( self.reads[3].query, b"TAGCTAGCTACCTATATCTTGGTCTT", "query mismatch in read 4: %s != %s" % (self.reads[3].query, b"TAGCTAGCTACCTATATCTTGGTCTT") )
 
     def testARqqual(self):
-        self.assertEqual( self.reads[0].qqual, "<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<", "qquality string mismatch in read 1: %s != %s" % (self.reads[0].qqual, "<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<") )
-        self.assertEqual( self.reads[1].qqual, "<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<", "qquality string mismatch in read 2: %s != %s" % (self.reads[1].qqual, "<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<") )
-        self.assertEqual( self.reads[3].qqual, "<<<<<<<<<<<<<<<<<:<9/,&,22", "qquality string mismatch in read 3: %s != %s" % (self.reads[3].qqual, "<<<<<<<<<<<<<<<<<:<9/,&,22") )
+        self.assertEqual( self.reads[0].qqual, b"<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<", "qquality string mismatch in read 1: %s != %s" % (self.reads[0].qqual, b"<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<") )
+        self.assertEqual( self.reads[1].qqual, b"<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<", "qquality string mismatch in read 2: %s != %s" % (self.reads[1].qqual, b"<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<") )
+        self.assertEqual( self.reads[3].qqual, b"<<<<<<<<<<<<<<<<<:<9/,&,22", "qquality string mismatch in read 3: %s != %s" % (self.reads[3].qqual, b"<<<<<<<<<<<<<<<<<:<9/,&,22") )
 
     def testPresentOptionalFields(self):
         self.assertEqual( self.reads[0].opt('NM'), 1, "optional field mismatch in read 1, NM: %s != %s" % (self.reads[0].opt('NM'), 1) )
@@ -529,7 +861,7 @@ class TestHeaderSam(unittest.TestCase):
 
     def compareHeaders( self, a, b ):
         '''compare two headers a and b.'''
-        for ak,av in a.iteritems():
+        for ak,av in a.items():
             self.assertTrue( ak in b, "key '%s' not in '%s' " % (ak,b) )
             self.assertEqual( av, b[ak] )
 
@@ -558,6 +890,7 @@ class TestHeaderBam(TestHeaderSam):
     def setUp(self):
         self.samfile=pysam.Samfile( "ex3.bam","rb" )
 
+
 class TestUnmappedReads(unittest.TestCase):
 
     def testSAM(self):
@@ -596,6 +929,14 @@ class TestPileupObjects(unittest.TestCase):
     def tearDown(self):
         self.samfile.close()
 
+    def testIteratorOutOfScope( self ):
+        '''test if exception is raised if pileup col is accessed after iterator is exhausted.'''
+
+        for pileupcol in self.samfile.pileup():
+            pass
+        
+        self.assertRaises( ValueError, getattr, pileupcol, "pileups" )
+
 class TestContextManager(unittest.TestCase):
 
     def testManager( self ):
@@ -671,10 +1012,16 @@ class TestWrongFormat(unittest.TestCase):
         self.assertRaises( ValueError, pysam.Samfile, 'ex1.sam', 'rb' )
 
     def testOpenBamAsSam( self ):
-        self.assertRaises( ValueError, pysam.Samfile, 'ex1.bam', 'r' )
+        # test fails, needs to be implemented.
+        # sam.fetch() fails on reading, not on opening
+        # self.assertRaises( ValueError, pysam.Samfile, 'ex1.bam', 'r' )
+        pass
 
     def testOpenFastaAsSam( self ):
-        self.assertRaises( ValueError, pysam.Samfile, 'ex1.fa', 'r' )
+        # test fails, needs to be implemented.
+        # sam.fetch() fails on reading, not on opening
+        # self.assertRaises( ValueError, pysam.Samfile, 'ex1.fa', 'r' )
+        pass
 
     def testOpenFastaAsBam( self ):
         self.assertRaises( ValueError, pysam.Samfile, 'ex1.fa', 'rb' )
@@ -682,16 +1029,16 @@ class TestWrongFormat(unittest.TestCase):
 class TestFastaFile(unittest.TestCase):
 
     mSequences = { 'chr1' :
-                       "CACTAGTGGCTCATTGTAAATGTGTGGTTTAACTCGTCCATGGCCCAGCATTAGGGAGCTGTGGACCCTGCAGCCTGGCTGTGGGGGCCGCAGTGGCTGAGGGGTGCAGAGCCGAGTCACGGGGTTGCCAGCACAGGGGCTTAACCTCTGGTGACTGCCAGAGCTGCTGGCAAGCTAGAGTCCCATTTGGAGCCCCTCTAAGCCGTTCTATTTGTAATGAAAACTATATTTATGCTATTCAGTTCTAAATATAGAAATTGAAACAGCTGTGTTTAGTGCCTTTGTTCAACCCCCTTGCAACAACCTTGAGAACCCCAGGGAATTTGTCAATGTCAGGGAAGGAGCATTTTGTCAGTTACCAAATGTGTTTATTACCAGAGGGATGGAGGGAAGAGGGACGCTGAAGAACTTTGATGCCCTCTTCTTCCAAAGATGAAACGCGTAACTGCGCTCTCATTCACTCCAGCTCCCTGTCACCCAATGGACCTGTGATATCTGGATTCTGGGAAATTCTTCATCCTGGACCCTGAGAGATTCTGCAGCCCAGCTCCAGATTGCTTGTGGTCTGACAGGCTGCAACTGTGAGCCATCACAATGAACAACAGGAAGAAAAGGTCTTTCAAAAGGTGATGTGTGTTCTCATCAACCTCATACACACACATGGTTTAGGGGTATAATACCTCTACATGGCTGATTATGAAAACAATGTTCCCCAGATACCATCCCTGTCTTACTTCCAGCTCCCCAGAGGGAAAGCTTTCAACGCTTCTAGCCATTTCTTTTGGCATTTGCCTTCAGACCCTACACGAATGCGTCTCTACCACAGGGGGCTGCGCGGTTTCCCATCATGAAGCACTGAACTTCCACGTCTCATCTAGGGGAACAGGGAGGTGCACTAATGCGCTCCACGCCCAAGCCCTTCTCACAGTTTCTGCCCCCAGCATGGTTGTACTGGGCAATACATGAGATTATTAGGAAATGCTTTACTGTCATAACTATGAAGAGACTATTGCCAGATGAACCACACATTAATACTATGTTTCTTATCTGCACATTACTACCCTGCAATTAATATAATTGTGTCCATGTACACACGCTGTCCTATGTACTTATCATGACTCTATCCCAAATTCCCAATTACGTCCTATCTTCTTCTTAGGGAAGAACAGCTTAGGTATCAATTTGGTGTTCTGTGTAAAGTCTCAGGGAGCCGTCCGTGTCCTCCCATCTGGCCTCGTCCACACTGGTTCTCTTGAAAGCTTGGGCTGTAATGATGCCCCTTGGCCATCACCCAGTCCCTGCCCCATCTCTTGTAATCTCTCTCCTTTTTGCTGCATCCCTGTCTTCCTCTGTCTTGATTTACTTGTTGTTGGTTTTCTGTTTCTTTGTTTGATTTGGTGGAAGACATAATCCCACGCTTCCTATGGAAAGGTTGTTGGGAGATTTTTAATGATTCCTCAATGTTAAAATGTCTATTTTTGTCTTGACACCCAACTAATATTTGTCTGAGCAAAACAGTCTAGATGAGAGAGAACTTCCCTGGAGGTCTGATGGCGTTTCTCCCTCGTCTTCTTA",
+                       b"CACTAGTGGCTCATTGTAAATGTGTGGTTTAACTCGTCCATGGCCCAGCATTAGGGAGCTGTGGACCCTGCAGCCTGGCTGTGGGGGCCGCAGTGGCTGAGGGGTGCAGAGCCGAGTCACGGGGTTGCCAGCACAGGGGCTTAACCTCTGGTGACTGCCAGAGCTGCTGGCAAGCTAGAGTCCCATTTGGAGCCCCTCTAAGCCGTTCTATTTGTAATGAAAACTATATTTATGCTATTCAGTTCTAAATATAGAAATTGAAACAGCTGTGTTTAGTGCCTTTGTTCAACCCCCTTGCAACAACCTTGAGAACCCCAGGGAATTTGTCAATGTCAGGGAAGGAGCATTTTGTCAGTTACCAAATGTGTTTATTACCAGAGGGATGGAGGGAAGAGGGACGCTGAAGAACTTTGATGCCCTCTTCTTCCAAAGATGAAACGCGTAACTGCGCTCTCATTCACTCCAGCTCCCTGTCACCCAATGGACCTGTGATATCTGGATTCTGGGAAATTCTTCATCCTGGACCCTGAGAGATTCTGCAGCCCAGCTCCAGATTGCTTGTGGTCTGACAGGCTGCAACTGTGAGCCATCACAATGAACAACAGGAAGAAAAGGTCTTTCAAAAGGTGATGTGTGTTCTCATCAACCTCATACACACACATGGTTTAGGGGTATAATACCTCTACATGGCTGATTATGAAAACAATGTTCCCCAGATACCATCCCTGTCTTACTTCCAGCTCCCCAGAGGGAAAGCTTTCAACGCTTCTAGCCATTTCTTTTGGCATTTGCCTTCAGACCCTACACGAATGCGTCTCTACCACAGGGGGCTGCGCGGTTTCCCATCATGAAGCACTGAACTTCCACGTCTCATCTAGGGGAACAGGGAGGTGCACTAATGCGCTCCACGCCCAAGCCCTTCTCACAGTTTCTGCCCCCAGCATGGTTGTACTGGGCAATACATGAGATTATTAGGAAATGCTTTACTGTCATAACTATGAAGAGACTATTGCCAGATGAACCACACATTAATACTATGTTTCTTATCTGCACATTACTACCCTGCAATTAATATAATTGTGTCCATGTACACACGCTGTCCTATGTACTTATCATGACTCTATCCCAAATTCCCAATTACGTCCTATCTTCTTCTTAGGGAAGAACAGCTTAGGTATCAATTTGGTGTTCTGTGTAAAGTCTCAGGGAGCCGTCCGTGTCCTCCCATCTGGCCTCGTCCACACTGGTTCTCTTGAAAGCTTGGGCTGTAATGATGCCCCTTGGCCATCACCCAGTCCCTGCCCCATCTCTTGTAATCTCTCTCCTTTTTGCTGCATCCCTGTCTTCCTCTGTCTTGATTTACTTGTTGTTGGTTTTCTGTTTCTTTGTTTGATTTGGTGGAAGACATAATCCCACGCTTCCTATGGAAAGGTTGTTGGGAGATTTTTAATGATTCCTCAATGTTAAAATGTCTATTTTTGTCTTGACACCCAACTAATATTTGTCTGAGCAAAACAGTCTAGATGAGAGAGAACTTCCCTGGAGGTCTGATGGCGTTTCTCCCTCGTCTTCTTA",
                    'chr2' :
-                       "TTCAAATGAACTTCTGTAATTGAAAAATTCATTTAAGAAATTACAAAATATAGTTGAAAGCTCTAACAATAGACTAAACCAAGCAGAAGAAAGAGGTTCAGAACTTGAAGACAAGTCTCTTATGAATTAACCCAGTCAGACAAAAATAAAGAAAAAAATTTTAAAAATGAACAGAGCTTTCAAGAAGTATGAGATTATGTAAAGTAACTGAACCTATGAGTCACAGGTATTCCTGAGGAAAAAGAAAAAGTGAGAAGTTTGGAAAAACTATTTGAGGAAGTAATTGGGGAAAACCTCTTTAGTCTTGCTAGAGATTTAGACATCTAAATGAAAGAGGCTCAAAGAATGCCAGGAAGATACATTGCAAGACAGACTTCATCAAGATATGTAGTCATCAGACTATCTAAAGTCAACATGAAGGAAAAAAATTCTAAAATCAGCAAGAGAAAAGCATACAGTCATCTATAAAGGAAATCCCATCAGAATAACAATGGGCTTCTCAGCAGAAACCTTACAAGCCAGAAGAGATTGGATCTAATTTTTGGACTTCTTAAAGAAAAAAAAACCTGTCAAACACGAATGTTATGCCCTGCTAAACTAAGCATCATAAATGAAGGGGAAATAAAGTCAAGTCTTTCCTGACAAGCAAATGCTAAGATAATTCATCATCACTAAACCAGTCCTATAAGAAATGCTCAAAAGAATTGTAAAAGTCAAAATTAAAGTTCAATACTCACCATCATAAATACACACAAAAGTACAAAACTCACAGGTTTTATAAAACAATTGAGACTACAGAGCAACTAGGTAAAAAATTAACATTACAACAGGAACAAAACCTCATATATCAATATTAACTTTGAATAAAAAGGGATTAAATTCCCCCACTTAAGAGATATAGATTGGCAGAACAGATTTAAAAACATGAACTAACTATATGCTGTTTACAAGAAACTCATTAATAAAGACATGAGTTCAGGTAAAGGGGTGGAAAAAGATGTTCTACGCAAACAGAAACCAAATGAGAGAAGGAGTAGCTATACTTATATCAGATAAAGCACACTTTAAATCAACAACAGTAAAATAAAACAAAGGAGGTCATCATACAATGATAAAAAGATCAATTCAGCAAGAAGATATAACCATCCTACTAAATACATATGCACCTAACACAAGACTACCCAGATTCATAAAACAAATACTACTAGACCTAAGAGGGATGAGAAATTACCTAATTGGTACAATGTACAATATTCTGATGATGGTTACACTAAAAGCCCATACTTTACTGCTACTCAATATATCCATGTAACAAATCTGCGCTTGTACTTCTAAATCTATAAAAAAATTAAAATTTAACAAAAGTAAATAAAACACATAGCTAAAACTAAAAAAGCAAAAACAAAAACTATGCTAAGTATTGGTAAAGATGTGGGGAAAAAAGTAAACTCTCAAATATTGCTAGTGGGAGTATAAATTGTTTTCCACTTTGGAAAACAATTTGGTAATTTCGTTTTTTTTTTTTTCTTTTCTCTTTTTTTTTTTTTTTTTTTTGCATGCCAGAAAAAAATATTTACAGTAACT",
+                       b"TTCAAATGAACTTCTGTAATTGAAAAATTCATTTAAGAAATTACAAAATATAGTTGAAAGCTCTAACAATAGACTAAACCAAGCAGAAGAAAGAGGTTCAGAACTTGAAGACAAGTCTCTTATGAATTAACCCAGTCAGACAAAAATAAAGAAAAAAATTTTAAAAATGAACAGAGCTTTCAAGAAGTATGAGATTATGTAAAGTAACTGAACCTATGAGTCACAGGTATTCCTGAGGAAAAAGAAAAAGTGAGAAGTTTGGAAAAACTATTTGAGGAAGTAATTGGGGAAAACCTCTTTAGTCTTGCTAGAGATTTAGACATCTAAATGAAAGAGGCTCAAAGAATGCCAGGAAGATACATTGCAAGACAGACTTCATCAAGATATGTAGTCATCAGACTATCTAAAGTCAACATGAAGGAAAAAAATTCTAAAATCAGCAAGAGAAAAGCATACAGTCATCTATAAAGGAAATCCCATCAGAATAACAATGGGCTTCTCAGCAGAAACCTTACAAGCCAGAAGAGATTGGATCTAATTTTTGGACTTCTTAAAGAAAAAAAAACCTGTCAAACACGAATGTTATGCCCTGCTAAACTAAGCATCATAAATGAAGGGGAAATAAAGTCAAGTCTTTCCTGACAAGCAAATGCTAAGATAATTCATCATCACTAAACCAGTCCTATAAGAAATGCTCAAAAGAATTGTAAAAGTCAAAATTAAAGTTCAATACTCACCATCATAAATACACACAAAAGTACAAAACTCACAGGTTTTATAAAACAATTGAGACTACAGAGCAACTAGGTAAAAAATTAACATTACAACAGGAACAAAACCTCATATATCAATATTAACTTTGAATAAAAAGGGATTAAATTCCCCCACTTAAGAGATATAGATTGGCAGAACAGATTTAAAAACATGAACTAACTATATGCTGTTTACAAGAAACTCATTAATAAAGACATGAGTTCAGGTAAAGGGGTGGAAAAAGATGTTCTACGCAAACAGAAACCAAATGAGAGAAGGAGTAGCTATACTTATATCAGATAAAGCACACTTTAAATCAACAACAGTAAAATAAAACAAAGGAGGTCATCATACAATGATAAAAAGATCAATTCAGCAAGAAGATATAACCATCCTACTAAATACATATGCACCTAACACAAGACTACCCAGATTCATAAAACAAATACTACTAGACCTAAGAGGGATGAGAAATTACCTAATTGGTACAATGTACAATATTCTGATGATGGTTACACTAAAAGCCCATACTTTACTGCTACTCAATATATCCATGTAACAAATCTGCGCTTGTACTTCTAAATCTATAAAAAAATTAAAATTTAACAAAAGTAAATAAAACACATAGCTAAAACTAAAAAAGCAAAAACAAAAACTATGCTAAGTATTGGTAAAGATGTGGGGAAAAAAGTAAACTCTCAAATATTGCTAGTGGGAGTATAAATTGTTTTCCACTTTGGAAAACAATTTGGTAATTTCGTTTTTTTTTTTTTCTTTTCTCTTTTTTTTTTTTTTTTTTTTGCATGCCAGAAAAAAATATTTACAGTAACT",
                    }
 
     def setUp(self):
         self.file=pysam.Fastafile( "ex1.fa" )
 
     def testFetch(self):
-        for id, seq in self.mSequences.items():
+        for id, seq in list(self.mSequences.items()):
             self.assertEqual( seq, self.file.fetch( id ) )
             for x in range( 0, len(seq), 10):
                 self.assertEqual( seq[x:x+10], self.file.fetch( id, x, x+10) )
@@ -702,15 +1049,15 @@ class TestFastaFile(unittest.TestCase):
 
         
         # unknown sequence returns ""
-        self.assertEqual( "", self.file.fetch("chr12") )
+        self.assertEqual( b"", self.file.fetch("chr12") )
 
     def testOutOfRangeAccess( self ):
         '''test out of range access.'''
         # out of range access returns an empty string
-        for contig, s in self.mSequences.iteritems():
-            self.assertEqual( self.file.fetch( contig, len(s), len(s)+1), "" )
+        for contig, s in self.mSequences.items():
+            self.assertEqual( self.file.fetch( contig, len(s), len(s)+1), b"" )
 
-        self.assertEqual( self.file.fetch( "chr3", 0 , 100), "" ) 
+        self.assertEqual( self.file.fetch( "chr3", 0 , 100), b"" ) 
 
     def testFetchErrors( self ):
         self.assertRaises( ValueError, self.file.fetch )
@@ -753,7 +1100,7 @@ class TestAlignedRead(unittest.TestCase):
         self.assertEqual( a.rname, 0 )
         self.assertEqual( a.mapq, 0 )
         self.assertEqual( a.cigar, None )
-        self.assertEqual( a.tags, None )
+        self.assertEqual( a.tags, [] )
         self.assertEqual( a.mrnm, 0 )
         self.assertEqual( a.mpos, 0 )
         self.assertEqual( a.isize, 0 )
@@ -763,16 +1110,16 @@ class TestAlignedRead(unittest.TestCase):
         
         a = pysam.AlignedRead()
         a.qname = "read_12345"
-        a.seq="ACGT" * 3
+        a.seq="ACGT" * 10
         a.flag = 0
         a.rname = 0
-        a.pos = 33
+        a.pos = 20
         a.mapq = 20
-        a.cigar = ( (0,10), (2,1), (0,25) )
+        a.cigar = ( (0,10), (2,1), (0,9), (1,1), (0,20) )
         a.mrnm = 0
         a.mpos=200
         a.isize=167
-       a.qual="1234" * 3
+        a.qual="1234" * 10
         # todo: create tags
         return a
 
@@ -793,17 +1140,17 @@ class TestAlignedRead(unittest.TestCase):
         # check cigar
         b.cigar = ( (0,10), )
         self.checkFieldEqual( a, b, "cigar" )
-        b.cigar = ( (0,10), (2,1), (0,25), (2,1), (0,25) )
+        b.cigar = ( (0,10), (2,1), (0,10) )
         self.checkFieldEqual( a, b, "cigar" )
-        b.cigar = ( (0,10), (2,1), (0,25) )
+        b.cigar = ( (0,10), (2,1), (0,9), (1,1), (0,20) )
         self.checkFieldEqual( a, b)
 
         # check seq 
         b.seq = "ACGT"
         self.checkFieldEqual( a, b, ("seq", "qual") )
-        b.seq = "ACGT" * 10
-        self.checkFieldEqual( a, b, ("seq", "qual") )
         b.seq = "ACGT" * 3
+        self.checkFieldEqual( a, b, ("seq", "qual") )
+        b.seq = "ACGT" * 10
         self.checkFieldEqual( a, b, ("qual",))
 
         # reset qual
@@ -832,13 +1179,13 @@ class TestAlignedRead(unittest.TestCase):
         a.seq="ACGT" * 200
         a.flag = 0
         a.rname = 0
-        a.pos = 33
+        a.pos = 20
         a.mapq = 20
-        a.cigar = ( (0,10), (2,1), (0,25) )
+        a.cigar = ( (0, 4 * 200), )
         a.mrnm = 0
         a.mpos=200
         a.isize=167
-       a.qual="1234" * 200
+        a.qual="1234" * 200
 
         return a
 
@@ -855,8 +1202,41 @@ class TestAlignedRead(unittest.TestCase):
             after = entry.tags
             self.assertEqual( after, before )
 
+    def testUpdateTlen( self ):
+        '''check if updating tlen works'''
+        a = self.buildRead()
+        oldlen = a.tlen
+        oldlen *= 2
+        a.tlen = oldlen
+        self.assertEqual( a.tlen, oldlen )
+
+    def testPositions( self ):
+        a = self.buildRead()
+        self.assertEqual( a.positions,
+                          [20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 
+                                31, 32, 33, 34, 35, 36, 37, 38, 39, 
+                           40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 
+                           50, 51, 52, 53, 54, 55, 56, 57, 58, 59] )
+
+        self.assertEqual( a.aligned_pairs,
+                          [(0, 20), (1, 21), (2, 22), (3, 23), (4, 24), 
+                           (5, 25), (6, 26), (7, 27), (8, 28), (9, 29), 
+                           (None, 30), 
+                           (10, 31), (11, 32), (12, 33), (13, 34), (14, 35), 
+                           (15, 36), (16, 37), (17, 38), (18, 39), (19, None), 
+                           (20, 40), (21, 41), (22, 42), (23, 43), (24, 44), 
+                           (25, 45), (26, 46), (27, 47), (28, 48), (29, 49), 
+                           (30, 50), (31, 51), (32, 52), (33, 53), (34, 54), 
+                           (35, 55), (36, 56), (37, 57), (38, 58), (39, 59)] )
+
+        self.assertEqual( a.positions, [x[1] for x in a.aligned_pairs if x[0] != None and x[1] != None] )
+        # alen is the length of the aligned read in genome
+        self.assertEqual( a.alen, a.aligned_pairs[-1][0] + 1 )
+        # aend points to one beyond last aligned base in ref
+        self.assertEqual( a.positions[-1], a.aend - 1 )
+
 class TestDeNovoConstruction(unittest.TestCase):
-    '''check BAM/SAM file construction using ex3.sam
+    '''check BAM/SAM file construction using ex6.sam
     
     (note these are +1 coordinates):
     
@@ -890,7 +1270,6 @@ class TestDeNovoConstruction(unittest.TestCase):
 
     def setUp( self ):
 
-        
         a = pysam.AlignedRead()
         a.qname = "read_28833_29006_6945"
         a.seq="AGCTTAGCTAGCTACCTATATCTTGGTCTTGGCCG"
@@ -902,8 +1281,8 @@ class TestDeNovoConstruction(unittest.TestCase):
         a.mrnm = 0
         a.mpos=199
         a.isize=167
-       a.qual="<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<"
-       a.tags = ( ("NM", 1),
+        a.qual="<<<<<<<<<<<<<<<<<<<<<:<9/,&,22;;<<<"
+        a.tags = ( ("NM", 1),
                    ("RG", "L1") )
 
         b = pysam.AlignedRead()
@@ -917,8 +1296,8 @@ class TestDeNovoConstruction(unittest.TestCase):
         b.mrnm = 1
         b.mpos=499
         b.isize=412
-       b.qual="<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<"
-       b.tags = ( ("MF", 18),
+        b.qual="<<<<<;<<<<7;:<<<6;<<<<<<<<<<<<7<<<<"
+        b.tags = ( ("MF", 18),
                    ("RG", "L2") )
 
         self.reads = (a,b)
@@ -931,7 +1310,6 @@ class TestDeNovoConstruction(unittest.TestCase):
 
         for x in self.reads: outfile.write( x )
         outfile.close()
-        
         self.assertTrue( checkBinaryEqual( tmpfilename, self.samfile ),
                          "mismatch when construction SAM file, see %s %s" % (tmpfilename, self.samfile))
         
@@ -969,6 +1347,61 @@ class TestDeNovoConstruction(unittest.TestCase):
         
         os.unlink( tmpfilename )
 
+class TestDeNovoConstructionUserTags(TestDeNovoConstruction):
+    '''test de novo construction with a header that contains lower-case tags.'''
+
+    header = { 'HD': {'VN': '1.0'},
+               'SQ': [{'LN': 1575, 'SN': 'chr1'}, 
+                      {'LN': 1584, 'SN': 'chr2'}],
+               'x1': {'A': 2, 'B': 5 },
+               'x3': {'A': 6, 'B': 5 },
+               'x2': {'A': 4, 'B': 5 } }
+
+    bamfile = "example_user_header.bam"
+    samfile = "example_user_header.sam"
+
+class TestEmptyHeader( unittest.TestCase ):
+    '''see issue 84.'''
+    
+    def testEmptyHeader( self ):
+
+        s = pysam.Samfile('example_empty_header.bam')
+        self.assertEqual( s.header, {'SQ': [{'LN': 1000, 'SN': 'chr1'}]} )
+
+class TestBTagSam( unittest.TestCase ):
+    '''see issue 81.'''
+    
+    compare = [ [100, 1, 91, 0, 7, 101, 0, 201, 96, 204, 0, 0, 87, 109, 0, 7, 97, 112, 1, 12, 78, 197, 0, 7, 100, 95, 101, 202, 0, 6, 0, 1, 186, 0, 84, 0, 244, 0, 0, 324, 0, 107, 195, 101, 113, 0, 102, 0, 104, 3, 0, 101, 1, 0, 212, 6, 0, 0, 1, 0, 74, 1, 11, 0, 196, 2, 197, 103, 0, 108, 98, 2, 7, 0, 1, 2, 194, 0, 180, 0, 108, 0, 203, 104, 16, 5, 205, 0, 0, 0, 1, 1, 100, 98, 0, 0, 204, 6, 0, 79, 0, 0, 101, 7, 109, 90, 265, 1, 27, 10, 109, 102, 9, 0, 292, 0, 110, 0, 0, 102, 112, 0, 0, 84, 100, 103, 2, 81, 126, 0, 2, 90, 0, 15, 96, 15, 1, 0, 2, 0, 107, 92, 0, 0, 101, 3, 98, 15, 102, 13, 116, 116, 90, 93, 198, 0, 0, 0, 199, 92, 26, 495, 100, 5, 0, 100, 5, 209, 0, 92, 107, 90, 0, 0, 0, 0, 109, 194, 7, 94, 200, 0, 40, 197, 0, 11, 0, 0, 112, 110, 6, 4, 200, 28, 0, 196, 0, 203, 1, 129, 0, 0, 1, 0, 94, 0, 1, 0, 107, 5, 201, 3, 3, 100, 0, 121, 0, 7, 0, 1, 105, 306, 3, 86, 8, 183, 0, 12, 163, 17, 83, 22, 0, 0, 1, 8, 109, 103, 0, 0, 295, 0, 200, 16, 172, 3, 16, 182, 3, 11, 0, 0, 223, 111, 103, 0, 5, 225, 0, 95],
+                [-100,200,-300,-400],
+                [-100,12],
+                [12,15],
+                [-1.0,5.0,2.5] ]
+            
+    filename = 'example_btag.sam'
+
+    def testRead( self ):
+
+        s = pysam.Samfile(self.filename)
+        for x, read in enumerate(s):
+            if x == 0:
+                self.assertEqual( read.tags, [('RG', 'QW85I'), ('PG', 'tmap'), ('MD', '140'), ('NM', 0), ('AS', 140), ('FZ', [100, 1, 91, 0, 7, 101, 0, 201, 96, 204, 0, 0, 87, 109, 0, 7, 97, 112, 1, 12, 78, 197, 0, 7, 100, 95, 101, 202, 0, 6, 0, 1, 186, 0, 84, 0, 244, 0, 0, 324, 0, 107, 195, 101, 113, 0, 102, 0, 104, 3, 0, 101, 1, 0, 212, 6, 0, 0, 1, 0, 74, 1, 11, 0, 196, 2, 197, 103, 0, 108, 98, 2, 7, 0, 1, 2, 194, 0, 180, 0, 108, 0, 203, 104, 16, 5, 205, 0, 0, 0, 1, 1, 100, 98, 0, 0, 204, 6, 0, 79, 0, 0, 101, 7, 109, 90, 265, 1, 27, 10, 109, 102, 9, 0, 292, 0, 110, 0, 0, 102, 112, 0, 0, 84, 100, 103, 2, 81, 126, 0, 2, 90, 0, 15, 96, 15, 1, 0, 2, 0, 107, 92, 0, 0, 101, 3, 98, 15, 102, 13, 116, 116, 90, 93, 198, 0, 0, 0, 199, 92, 26, 495, 100, 5, 0, 100, 5, 209, 0, 92, 107, 90, 0, 0, 0, 0, 109, 194, 7, 94, 200, 0, 40, 197, 0, 11, 0, 0, 112, 110, 6, 4, 200, 28, 0, 196, 0, 203, 1, 129, 0, 0, 1, 0, 94, 0, 1, 0, 107, 5, 201, 3, 3, 100, 0, 121, 0, 7, 0, 1, 105, 306, 3, 86, 8, 183, 0, 12, 163, 17, 83, 22, 0, 0, 1, 8, 109, 103, 0, 0, 295, 0, 200, 16, 172, 3, 16, 182, 3, 11, 0, 0, 223, 111, 103, 0, 5, 225, 0, 95]), ('XA', 'map2-1'), ('XS', 53), ('XT', 38), ('XF', 1), ('XE', 0)] 
+                                  )
+                         
+            fz = dict(read.tags)["FZ"]
+            self.assertEqual( fz, self.compare[x] )
+            self.assertEqual( read.opt("FZ"), self.compare[x])
+
+    def testWrite( self ):
+        
+        s = pysam.Samfile(self.filename)
+        for read in s:
+            before = read.tags
+            read.tags = read.tags
+            after = read.tags
+            self.assertEqual( after, before )
+
+class TestBTagBam( TestBTagSam ):
+    filename = 'example_btag.bam'
 
 class TestDoubleFetch(unittest.TestCase):
     '''check if two iterators on the same bamfile are independent.'''
@@ -1061,22 +1494,22 @@ class TestLargeOptValues( unittest.TestCase ):
         
         i = samfile.fetch()
         for exp in self.ints:
-            rr = i.next()
+            rr = next(i)
             obs = rr.opt("ZP")
             self.assertEqual( exp, obs, "expected %s, got %s\n%s" % (str(exp), str(obs), str(rr)))
 
         for exp in [ -x for x in self.ints ]:
-            rr = i.next()
+            rr = next(i)
             obs = rr.opt("ZP")
             self.assertEqual( exp, obs, "expected %s, got %s\n%s" % (str(exp), str(obs), str(rr)))
 
         for exp in self.floats:
-            rr = i.next()
+            rr = next(i)
             obs = rr.opt("ZP")
             self.assertEqual( exp, obs, "expected %s, got %s\n%s" % (str(exp), str(obs), str(rr)))
 
         for exp in [ -x for x in self.floats ]:
-            rr = i.next()
+            rr = next(i)
             obs = rr.opt("ZP")
             self.assertEqual( exp, obs, "expected %s, got %s\n%s" % (str(exp), str(obs), str(rr)))
 
@@ -1088,107 +1521,107 @@ class TestLargeOptValues( unittest.TestCase ):
         samfile = pysam.Samfile("ex10.bam", "rb")
         self.check( samfile )
 
-class TestSNPCalls( unittest.TestCase ):
-    '''test pysam SNP calling ability.'''
-
-    def checkEqual( self, a, b ):
-        for x in ("reference_base", 
-                  "pos",
-                  "genotype",
-                  "consensus_quality",
-                  "snp_quality",
-                  "mapping_quality",
-                  "coverage" ):
-            self.assertEqual( getattr(a, x), getattr(b,x), "%s mismatch: %s != %s\n%s\n%s" % 
-                              (x, getattr(a, x), getattr(b,x), str(a), str(b)))
-
-    def testAllPositionsViaIterator( self ):
-        samfile = pysam.Samfile( "ex1.bam", "rb")  
-        fastafile = pysam.Fastafile( "ex1.fa" )
-        try: 
-            refs = [ x for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ) if x.reference_base != "*"]
-        except pysam.SamtoolsError:
-            pass
-
-        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
-        calls = list(pysam.IteratorSNPCalls(i))
-        for x,y in zip( refs, calls ):
-            self.checkEqual( x, y )
-
-    def testPerPositionViaIterator( self ):
-        # test pileup for each position. This is a slow operation
-        # so this test is disabled 
-        return
-        samfile = pysam.Samfile( "ex1.bam", "rb")  
-        fastafile = pysam.Fastafile( "ex1.fa" )
-        for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ):
-            if x.reference_base == "*": continue
-            i = samfile.pileup( x.chromosome, x.pos, x.pos+1,
-                                fastafile = fastafile,
-                                stepper = "samtools" )
-            z = [ zz for zz in pysam.IteratorSamtools(i) if zz.pos == x.pos ]
-            self.assertEqual( len(z), 1 )
-            self.checkEqual( x, z[0] )
-
-    def testPerPositionViaCaller( self ):
-        # test pileup for each position. This is a fast operation
-        samfile = pysam.Samfile( "ex1.bam", "rb")  
-        fastafile = pysam.Fastafile( "ex1.fa" )
-        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
-        caller = pysam.SNPCaller( i )
-
-        for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ):
-            if x.reference_base == "*": continue
-            call = caller.call( x.chromosome, x.pos )
-            self.checkEqual( x, call )
-
-class TestIndelCalls( unittest.TestCase ):
-    '''test pysam indel calling.'''
-
-    def checkEqual( self, a, b ):
-
-        for x in ("pos",
-                  "genotype",
-                  "consensus_quality",
-                  "snp_quality",
-                  "mapping_quality",
-                  "coverage",
-                  "first_allele",
-                  "second_allele",
-                  "reads_first",
-                  "reads_second",
-                  "reads_diff"):
-            if b.genotype == "*/*" and x == "second_allele":
-                # ignore test for second allele (positions chr2:439 and chr2:1512)
-                continue
-            self.assertEqual( getattr(a, x), getattr(b,x), "%s mismatch: %s != %s\n%s\n%s" % 
-                              (x, getattr(a, x), getattr(b,x), str(a), str(b)))
-
-    def testAllPositionsViaIterator( self ):
-
-        samfile = pysam.Samfile( "ex1.bam", "rb")  
-        fastafile = pysam.Fastafile( "ex1.fa" )
-        try: 
-            refs = [ x for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ) if x.reference_base == "*"]
-        except pysam.SamtoolsError:
-            pass
-
-        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
-        calls = [ x for x in list(pysam.IteratorIndelCalls(i)) if x != None ]
-        for x,y in zip( refs, calls ):
-            self.checkEqual( x, y )
-
-    def testPerPositionViaCaller( self ):
-        # test pileup for each position. This is a fast operation
-        samfile = pysam.Samfile( "ex1.bam", "rb")  
-        fastafile = pysam.Fastafile( "ex1.fa" )
-        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
-        caller = pysam.IndelCaller( i )
-
-        for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ):
-            if x.reference_base != "*": continue
-            call = caller.call( x.chromosome, x.pos )
-            self.checkEqual( x, call )
+class TestSNPCalls( unittest.TestCase ):
+    '''test pysam SNP calling ability.'''
+
+    def checkEqual( self, a, b ):
+        for x in ("reference_base", 
+                  "pos",
+                  "genotype",
+                  "consensus_quality",
+                  "snp_quality",
+                  "mapping_quality",
+                  "coverage" ):
+            self.assertEqual( getattr(a, x), getattr(b,x), "%s mismatch: %s != %s\n%s\n%s" % 
+                              (x, getattr(a, x), getattr(b,x), str(a), str(b)))
+
+    def testAllPositionsViaIterator( self ):
+        samfile = pysam.Samfile( "ex1.bam", "rb")  
+        fastafile = pysam.Fastafile( "ex1.fa" )
+        try: 
+            refs = [ x for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ) if x.reference_base != "*"]
+        except pysam.SamtoolsError:
+            pass
+
+        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
+        calls = list(pysam.IteratorSNPCalls(i))
+        for x,y in zip( refs, calls ):
+            self.checkEqual( x, y )
+
+    def testPerPositionViaIterator( self ):
+        # test pileup for each position. This is a slow operation
+        # so this test is disabled 
+        return
+        samfile = pysam.Samfile( "ex1.bam", "rb")  
+        fastafile = pysam.Fastafile( "ex1.fa" )
+        for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ):
+            if x.reference_base == "*": continue
+            i = samfile.pileup( x.chromosome, x.pos, x.pos+1,
+                                fastafile = fastafile,
+                                stepper = "samtools" )
+            z = [ zz for zz in pysam.IteratorSamtools(i) if zz.pos == x.pos ]
+            self.assertEqual( len(z), 1 )
+            self.checkEqual( x, z[0] )
+
+    def testPerPositionViaCaller( self ):
+        # test pileup for each position. This is a fast operation
+        samfile = pysam.Samfile( "ex1.bam", "rb")  
+        fastafile = pysam.Fastafile( "ex1.fa" )
+        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
+        caller = pysam.SNPCaller( i )
+
+        for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ):
+            if x.reference_base == "*": continue
+            call = caller.call( x.chromosome, x.pos )
+            self.checkEqual( x, call )
+
+class TestIndelCalls( unittest.TestCase ):
+    '''test pysam indel calling.'''
+
+    def checkEqual( self, a, b ):
+
+        for x in ("pos",
+                  "genotype",
+                  "consensus_quality",
+                  "snp_quality",
+                  "mapping_quality",
+                  "coverage",
+                  "first_allele",
+                  "second_allele",
+                  "reads_first",
+                  "reads_second",
+                  "reads_diff"):
+            if b.genotype == "*/*" and x == "second_allele":
+                # ignore test for second allele (positions chr2:439 and chr2:1512)
+                continue
+            self.assertEqual( getattr(a, x), getattr(b,x), "%s mismatch: %s != %s\n%s\n%s" % 
+                              (x, getattr(a, x), getattr(b,x), str(a), str(b)))
+
+    def testAllPositionsViaIterator( self ):
+
+        samfile = pysam.Samfile( "ex1.bam", "rb")  
+        fastafile = pysam.Fastafile( "ex1.fa" )
+        try: 
+            refs = [ x for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ) if x.reference_base == "*"]
+        except pysam.SamtoolsError:
+            pass
+
+        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
+        calls = [ x for x in list(pysam.IteratorIndelCalls(i)) if x != None ]
+        for x,y in zip( refs, calls ):
+            self.checkEqual( x, y )
+
+    def testPerPositionViaCaller( self ):
+        # test pileup for each position. This is a fast operation
+        samfile = pysam.Samfile( "ex1.bam", "rb")  
+        fastafile = pysam.Fastafile( "ex1.fa" )
+        i = samfile.pileup( stepper = "samtools", fastafile = fastafile )
+        caller = pysam.IndelCaller( i )
+
+        for x in pysam.pileup( "-c", "-f", "ex1.fa", "ex1.bam" ):
+            if x.reference_base != "*": continue
+            call = caller.call( x.chromosome, x.pos )
+            self.checkEqual( x, call )
 
 class TestLogging( unittest.TestCase ):
     '''test around bug issue 42,
@@ -1208,7 +1641,7 @@ class TestLogging( unittest.TestCase ):
 
         bam  = pysam.Samfile(bamfile, 'rb')
         cols = bam.pileup()
-        self.assert_( True )
+        self.assertTrue( True )
 
     def testFail1( self ):
         self.check( "ex9_fail.bam", False )
@@ -1234,7 +1667,7 @@ class TestSamfileUtilityFunctions( unittest.TestCase ):
         samfile = pysam.Samfile( "ex1.bam", "rb" )
 
         for contig in ("chr1", "chr2" ):
-            for start in xrange( 0, 2000, 100 ):
+            for start in range( 0, 2000, 100 ):
                 end = start + 1
                 self.assertEqual( len( list( samfile.fetch( contig, start, end ) ) ),
                                   samfile.count( contig, start, end ) )
@@ -1250,7 +1683,11 @@ class TestSamfileUtilityFunctions( unittest.TestCase ):
     def testMate( self ):
         '''test mate access.'''
 
-        readnames = [ x.split("\t")[0] for x in open( "ex1.sam", "rb" ).readlines() ]
+        with open( "ex1.sam", "rb" ) as inf:
+            readnames = [ x.split(b"\t")[0] for x in inf.readlines() ]
+        if sys.version_info[0] >= 3:
+            readnames = [ name.decode('ascii') for name in readnames ]
+            
         counts = collections.defaultdict( int )
         for x in readnames: counts[x] += 1
 
@@ -1271,6 +1708,13 @@ class TestSamfileUtilityFunctions( unittest.TestCase ):
                     self.assertEqual( read.pos, mate.mpos )
                     self.assertEqual( read.mpos, mate.pos )
 
+    def testIndexStats( self ):
+        '''test if total number of mapped/unmapped reads is correct.'''
+
+        samfile = pysam.Samfile( "ex1.bam", "rb" )
+        self.assertEqual( samfile.mapped, 3235 )
+        self.assertEqual( samfile.unmapped, 35 )
+
 class TestSamtoolsProxy( unittest.TestCase ):
     '''tests for sanity checking access to samtools functions.'''
 
@@ -1295,7 +1739,7 @@ class TestSamfileIndex( unittest.TestCase):
 
         for read in samfile: reads[read.qname] += 1
             
-        for qname, counts in reads.iteritems():
+        for qname, counts in reads.items():
             found = list(index.find( qname ))
             self.assertEqual( len(found), counts )
             for x in found: self.assertEqual( x.qname, qname )
@@ -1303,7 +1747,8 @@ class TestSamfileIndex( unittest.TestCase):
 
 if __name__ == "__main__":
     # build data files
-    print "building data files"
+    print ("building data files")
     subprocess.call( "make", shell=True)
-    print "starting tests"
+    print ("starting tests")
     unittest.main()
+    print ("completed tests")