Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix contig test case for new contig header parsing #111

Merged
merged 1 commit into from

2 participants

@martijnvermaat
Collaborator

Pull request #105 introduced improved parsing of Contig header lines. This change updates the unit tests accordingly.

@jamescasbon jamescasbon merged commit 39391ee into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 4 deletions.
  1. +1 −4 vcf/test/test_vcf.py
View
5 vcf/test/test_vcf.py
@@ -48,10 +48,7 @@ def test_vcf_4_1(self):
self.assertEqual(reader.metadata['fileformat'], 'VCFv4.1')
# contigs were added in vcf4.1
- # probably need to add a reader.contigs attribute
- assert 'contig' in reader.metadata
- assert 'ID' in reader.metadata['contig'][0]
- assert reader.metadata['contig'][0]['ID'] == '20'
+ self.assertEqual(reader.contigs['20'].length, 62435964)
# test we can walk the file at least
for r in reader:
Something went wrong with that request. Please try again.