pyTest: fix seg fault in the readers tests

Clement Champetier · Clement Champetier · commit ca157d83988a · 2017-01-30T19:25:23.000+01:00
* Force to call the reader destructor before the inputFile destructor
(which cannot happen in C++).
* This is a new problem since the decoder has a link to the stream,
which is needed to access the codec to close it. So the file, which
handles the streams, should not be destroyed before the decoders.
diff --git a/test/pyTest/testAudioReader.py b/test/pyTest/testAudioReader.py
@@ -55,15 +55,18 @@ def testAudioReaderChannelsExtraction():
 
     assert_equals( sizeOfFrameWithAllChannels / nbChannels, sizeOfFrameWithOneChannels )
 
+    # Force to call the readers destructor before the inputFile destructor (which cannot happen in C++)
+    readerOfAllChannels = None
+    readerOfOneChannel = None
+
 
 def testAudioReaderWithGenerator():
     """
     Read an audio stream with the AudioReader.
     When there is no more data to decode, switch to a generator and process some frames.
     """
     inputFileName = os.environ['AVTRANSCODER_TEST_AUDIO_WAVE_FILE']
-    inputFile = av.InputFile(inputFileName)
-    reader = av.AudioReader(inputFile)
+    reader = av.AudioReader(inputFileName)
 
     # read all frames and check their size
     while True:
diff --git a/test/pyTest/testVideoReader.py b/test/pyTest/testVideoReader.py
@@ -25,7 +25,6 @@ def testVideoReaderCreateNewInputFile():
         assert_equals( frame.getDataSize(), reader.getOutputWidth() * reader.getOutputHeight() * bytesPerPixel )
 
     # check if there is no next frame
-    frame = reader.readNextFrame()
     assert_equals( reader.readNextFrame(), None )
 
 
@@ -47,6 +46,9 @@ def testVideoReaderReferenceInputFile():
     # check if there is no next frame
     assert_equals( reader.readNextFrame(), None )
 
+    # Force to call the reader destructor before the inputFile destructor (which cannot happen in C++)
+    reader = None
+
 
 def testVideoReaderWithGenerator():
     """
