[FFmpeg-devel] [PATCH v1 3/3] avutil/tests: add more avstring test case for fate

lance.lmwang at gmail.com lance.lmwang at gmail.com
Mon Jan 13 15:44:51 EET 2020


From: Limin Wang <lance.lmwang at gmail.com>

One test case is different, before the patchset, it's:
"path" + "/" = "path/"
After applied the patchset, it's:
"path" + "/" = "path"

Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
---
 libavutil/tests/avstring.c | 3 +++
 tests/ref/fate/avstring    | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/libavutil/tests/avstring.c b/libavutil/tests/avstring.c
index 887bd25..a94f7a5 100644
--- a/libavutil/tests/avstring.c
+++ b/libavutil/tests/avstring.c
@@ -76,6 +76,9 @@ int main(void)
     TEST_APPEND_PATH_COMPONENT(NULL, NULL, "(null)")
     TEST_APPEND_PATH_COMPONENT("path", NULL, "path");
     TEST_APPEND_PATH_COMPONENT(NULL, "comp", "comp");
+    TEST_APPEND_PATH_COMPONENT("path", "", "path");
+    TEST_APPEND_PATH_COMPONENT("", "comp", "comp");
+    TEST_APPEND_PATH_COMPONENT("path", "/", "path");
     TEST_APPEND_PATH_COMPONENT("path", "comp", "path/comp");
     TEST_APPEND_PATH_COMPONENT("path/", "comp", "path/comp");
     TEST_APPEND_PATH_COMPONENT("path", "/comp", "path/comp");
diff --git a/tests/ref/fate/avstring b/tests/ref/fate/avstring
index 1ca9be5..92ec918 100644
--- a/tests/ref/fate/avstring
+++ b/tests/ref/fate/avstring
@@ -29,6 +29,9 @@ Testing av_append_path_component()
 (null) = (null)
 path = path
 comp = comp
+path = path
+comp = comp
+path/ = path
 path/comp = path/comp
 path/comp = path/comp
 path/comp = path/comp
-- 
2.9.5



More information about the ffmpeg-devel mailing list