Merge pull request opencv#10989 from tomoaki0705:fixOldGstreamerTest

alalek · alalek · commit 1393fd9e8e9d · 2018-03-06T16:26:08.000Z
diff --git a/modules/videoio/test/test_video_io.cpp b/modules/videoio/test/test_video_io.cpp
@@ -112,6 +112,13 @@ class Videoio_Test_Base
 
         {
             SCOPED_TRACE("consecutive read");
+            if (apiPref == CAP_GSTREAMER)
+            {
+                // This workaround is for GStreamer 1.3.1.1 and older.
+                // Old Gstreamer has a bug which handles the total duration 1 frame shorter
+                // Old Gstreamer are used in Ubuntu 14.04, so the following code could be removed after it's EOL
+                n_frames--;
+            }
             for (int k = 0; k < n_frames; ++k)
             {
                 checkFrameRead(k, cap);

Original file line number	Diff line number	Diff line change
`@@ -112,6 +112,13 @@ class Videoio_Test_Base`
`112`	`112`
`113`	`113`	`{`
`114`	`114`	`SCOPED_TRACE("consecutive read");`
	`115`	`+ if (apiPref == CAP_GSTREAMER)`
	`116`	`+ {`
	`117`	`+ // This workaround is for GStreamer 1.3.1.1 and older.`
	`118`	`+ // Old Gstreamer has a bug which handles the total duration 1 frame shorter`
	`119`	`+ // Old Gstreamer are used in Ubuntu 14.04, so the following code could be removed after it's EOL`
	`120`	`+ n_frames--;`
	`121`	`+ }`
`115`	`122`	`for (int k = 0; k < n_frames; ++k)`
`116`	`123`	`{`
`117`	`124`	`checkFrameRead(k, cap);`