Changes between Version 4 and Version 5 of FATE/AddingATest

Timestamp:

Apr 12, 2014, 8:57:36 PM (12 years ago)

Author:

Timothy Gu

Comment:

--

FATE/AddingATest

@@ -28 +28 @@
 }}}
 
-This is one of the simplest tests for FFmpeg. Look at the first quoted line. It adds `fate-dts` to a weird variable containing a call to `DEMDEC`. The `DEMDEC` function returns "yes" if the required `DEM`uxer and `DEC`oder, in this case the `mpegts` demuxer and `dca` decoder, are enabled. Then, if you look at the end of the file:
+This is one of the simplest tests for FFmpeg. Look at the first quoted line. It adds `fate-dts` to a weird variable containing a call to `DEMDEC`. The `DEMDEC` function returns "yes" if the required `DEM`uxer and `DEC`oder, in this case the `mpegts` demuxer and `dca` decoder, are enabled. So `fate-dts`, the target name of the test, gets added into `FATE_SAMPLES_AUDIO-yes` variable.
+
+Then, if you look at the end of the file:
 
 {{{
@@ -38 +40 @@
 The enabled tests are added into `FATE_SAMPLES_AUDIO` and subsequently `FATE_SAMPLES_FFMPEG` variable, which then of course gets called when doing `make fate`.
 
+Using this kind of system guarantees that if a person builds FFmpeg with certain parts disabled, the FATE test will skip the disabled parts.
+
 Keep in mind that different tests may have different variable prefixes. Not all tests can be categorized into `FATE_SAMPLES_FFMPEG` variable.
 
-If only one component for the test you want to add is required, you can replace `$(call DEMDEC, MPEGTS, DCA)` to `$(CONFIG_*)`.
+If only one component for the test you want to add is required, you can replace `$(call DEMDEC, MPEGTS, DCA)` with `$(CONFIG_*)`.
 
 == Filter Tests ==