test: support codecov

Signed-off-by: pingkai <[email protected]>

.travis.yml

@@ -26,11 +26,16 @@ before_script:
   - cd ../
 
 script:
-  - cd cmdline/
   - mkdir build
   - cd build
-  - cmake ../
-  - make cicadaPlayer -j8
+  - cmake -DCODE_COVERAGE=ON -DTRAVIS=ON -DCMAKE_BUILD_TYPE=Debug ../
+  - cmake --build . --config Debug -- -j8
+  - cd cmdline/mediaPlayer.out/framework.out/tests/
+  - ctest -j8 --output-on-failure
+
+after_success:
+  - bash <(curl -s https://codecov.io/bash) -x "llvm-cov gcov" || echo "Codecov did not collect coverage reports"
+
 
 addons:
   coverity_scan:

README.md

@@ -3,6 +3,7 @@
 [![HitCount](http://hits.dwyl.io/aliyun/CicadaPlayer.svg)](http://hits.dwyl.io/aliyun/CicadaPlayer)
 [![Build Status](https://api.travis-ci.org/aliyun/CicadaPlayer.svg?branch=develop)](https://travis-ci.org/aliyun/CicadaPlayer)
 [![coverity Status](https://scan.coverity.com/projects/20151/badge.svg?flat=1)](https://scan.coverity.com/projects/aliyun-cicadaplayer)
+[![codecov](https://codecov.io/gh/aliyun/CicadaPlayer/branch/develop/graph/badge.svg)](https://codecov.io/gh/aliyun/CicadaPlayer/branch/develop)
 [![LICENSE](https://img.shields.io/badge/license-MIT-orange.svg)](LICENSE)
 
 <h1 align="center">

cmdline/CMakeLists.txt

@@ -23,6 +23,10 @@ elseif (${CMAKE_SYSTEM_NAME} MATCHES "Windows")
     set(TARGET_PLATFORM windows)
 endif ()
 
+# Code Coverage Configuration
+option(CODE_COVERAGE "Enable coverage reporting" OFF)
+
+include(../framework/code_coverage.cmake)
 add_subdirectory(../mediaPlayer mediaPlayer.out)
 
 include(../framework/${TARGET_PLATFORM}.cmake)
@@ -40,7 +44,7 @@ target_link_directories(cicadaPlayer PRIVATE
         ../external/install/openssl/${CMAKE_SYSTEM_NAME}/x86_64/lib
         ${COMMON_LIB_DIR})
 
-target_link_libraries(cicadaPlayer
+target_link_libraries(cicadaPlayer PRIVATE
         media_player
         demuxer
         data_source
@@ -63,21 +67,22 @@ target_link_libraries(cicadaPlayer
         ${FRAMEWORK_LIBS})
 
 if (${CMAKE_SYSTEM_NAME} MATCHES "Windows")
-    target_link_libraries(cicadaPlayer
+    target_link_libraries(cicadaPlayer PUBLIC
             bcrypt
             )
 else ()
-    target_link_libraries(cicadaPlayer
+    target_link_libraries(cicadaPlayer PUBLIC
             z
             dl
             )
 endif ()
 
 if (${CMAKE_SYSTEM_NAME} MATCHES "Darwin")
-    target_link_libraries(cicadaPlayer
+    target_link_libraries(cicadaPlayer PUBLIC
             bz2
             )
 endif ()
+target_link_libraries(cicadaPlayer PUBLIC coverage_config)
 
 if (USEASAN)
     set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=address -fno-omit-frame-pointer -fsanitize-address-use-after-scope")

framework/cacheModule/CMakeLists.txt

@@ -33,3 +33,7 @@ add_library(cacheModule STATIC
         cache/CachePath.h
         cache/CacheRet.h
         )
+
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(cacheModule PUBLIC coverage_config)
+endif ()

framework/code_coverage.cmake

@@ -0,0 +1,19 @@
+
+add_library(coverage_config INTERFACE)
+
+if (CODE_COVERAGE AND CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
+    # Add required flags (GCC & LLVM/Clang)
+    target_compile_options(coverage_config INTERFACE
+            -O0        # no optimization
+            -g         # generate debug info
+            --coverage # sets all required flags
+            )
+    if (CMAKE_VERSION VERSION_GREATER_EQUAL 3.13)
+        target_link_options(coverage_config INTERFACE --coverage)
+    else ()
+        target_link_libraries(coverage_config INTERFACE --coverage)
+    endif ()
+    message("CODE_COVERAGE Enabled")
+endif ()
+
+set(HAVE_COVERAGE_CONFIG ON)

framework/codec/CMakeLists.txt

@@ -67,3 +67,7 @@ link_libraries(${LINNK_LIBRARYS})
 
 add_library(videodec ${TARGET_LIBRARY_TYPE} ${SOURCE_FILES})
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-deprecated-declarations")
+
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(videodec PUBLIC coverage_config)
+endif ()

framework/data_source/CMakeLists.txt

@@ -75,3 +75,6 @@ endif ()
 link_libraries(${LINNK_LIBRARYS})
 
 add_library(data_source ${TARGET_LIBRARY_TYPE} ${SOURCE_FILES})
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(data_source PUBLIC coverage_config)
+endif ()

framework/demuxer/CMakeLists.txt

@@ -90,4 +90,7 @@ endif ()
 
 link_libraries(${LINNK_LIBRARYS})
 
-add_library(demuxer ${TARGET_LIBRARY_TYPE} ${SOURCE_FILES})
+add_library(demuxer ${TARGET_LIBRARY_TYPE} ${SOURCE_FILES})
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(demuxer PUBLIC coverage_config)
+endif ()

framework/filter/CMakeLists.txt

@@ -5,4 +5,8 @@ set(SOURCE_FILES IAudioFilter.cpp
         ffmpegAudioFilter.cpp filterFactory.cpp filterFactory.h)
 include_directories(
         ${COMMON_INC_DIR})
-add_library(framework_filter ${TARGET_LIBRARY_TYPE} ${SOURCE_FILES})
+add_library(framework_filter ${TARGET_LIBRARY_TYPE} ${SOURCE_FILES})
+
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(framework_filter PUBLIC coverage_config)
+endif ()

framework/muxer/CMakeLists.txt

@@ -40,4 +40,7 @@ endif ()
 
 link_libraries(${LINNK_LIBRARYS})
 
-add_library(muxer STATIC ${SOURCE_FILES})
+add_library(muxer STATIC ${SOURCE_FILES})
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(muxer PUBLIC coverage_config)
+endif ()

framework/render/CMakeLists.txt

@@ -145,3 +145,8 @@ if (ENABLE_GLRENDER)
             )
 
 endif ()
+
+
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(render PUBLIC coverage_config)
+endif ()

framework/tests/CMakeLists.txt

@@ -1,9 +1,15 @@
 project(frameworkUintTest)
 cmake_minimum_required(VERSION 3.15)
 
+if (!HAVE_COVERAGE_CONFIG)
+    include(../code_coverage.cmake)
+endif ()
+
 include(GoogleTest.cmake)
 add_subdirectory(dataSource)
-add_subdirectory(render)
+if (!TRAVIS)
+    add_subdirectory(render)
+endif ()
 add_subdirectory(demuxer)
 add_subdirectory(decoder)
 
@@ -13,12 +19,12 @@ add_test(
         NAME dataSourceTest
         COMMAND $<TARGET_FILE:dataSourceTest>
 )
-
-add_test(
-        NAME renderTest
-        COMMAND $<TARGET_FILE:renderTest>
-)
-
+if (!TRAVIS)
+    add_test(
+            NAME renderTest
+            COMMAND $<TARGET_FILE:renderTest>
+    )
+endif ()
 add_test(
         NAME demuxerUnitTest
         COMMAND $<TARGET_FILE:demuxerUnitTest>

framework/tests/dataSource/CMakeLists.txt

@@ -24,8 +24,7 @@ target_include_directories(
         ../../../plugin
         )
 
-target_link_libraries(
-        dataSourceTest
+target_link_libraries(dataSourceTest PRIVATE
         demuxer
         videodec
         data_source
@@ -43,16 +42,19 @@ target_link_libraries(
         gtest_main)
 if (APPLE)
         target_link_libraries(
-                dataSourceTest
+                dataSourceTest PUBLIC
                 iconv
                 bz2
                 ${FRAMEWORK_LIBS}
         )
-else()
+else ()
         target_link_libraries(
-                dataSourceTest
+                dataSourceTest PUBLIC
                 dl
                 pthread
         )
 
+endif ()
+if (HAVE_COVERAGE_CONFIG)
+        target_link_libraries(dataSourceTest PUBLIC coverage_config)
 endif ()

framework/tests/decoder/CMakeLists.txt

@@ -25,7 +25,7 @@ target_include_directories(
 )
 
 target_link_libraries(
-        decoderUnitTest
+        decoderUnitTest PRIVATE
         demuxer
         videodec
         data_source
@@ -44,16 +44,19 @@ target_link_libraries(
 
 if (APPLE)
     target_link_libraries(
-            decoderUnitTest
+            decoderUnitTest PUBLIC
             iconv
             bz2
             ${FRAMEWORK_LIBS}
     )
-else()
+else ()
     target_link_libraries(
-            decoderUnitTest
+            decoderUnitTest PUBLIC
             dl
             pthread
     )
 
+endif ()
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(decoderUnitTest PUBLIC coverage_config)
 endif ()

framework/tests/decoder/decoderUnitTest.cpp

@@ -76,12 +76,14 @@ int main(int argc, char **argv)
     ::testing::InitGoogleTest(&argc, argv);
     return RUN_ALL_TESTS();
 }
-
+#ifdef  __APPLE__
 TEST(hardCodec, H264)
 {
+
     std::string url = "http://player.alicdn.com/video/aliyunmedia.mp4";
     test_codec(url, AF_CODEC_ID_H264, DECFLAG_HW);
 }
+#endif
 
 TEST(softCodec, H264)
 {

framework/tests/demuxer/CMakeLists.txt

@@ -24,7 +24,7 @@ target_include_directories(
 )
 
 target_link_libraries(
-        demuxerUnitTest
+        demuxerUnitTest PRIVATE
         demuxer
         videodec
         data_source
@@ -43,16 +43,19 @@ target_link_libraries(
 
 if (APPLE)
     target_link_libraries(
-            demuxerUnitTest
+            demuxerUnitTest PUBLIC
             iconv
             bz2
             ${FRAMEWORK_LIBS}
     )
-else()
+else ()
     target_link_libraries(
-            demuxerUnitTest
+            demuxerUnitTest PUBLIC
             dl
             pthread
     )
 
+endif ()
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(demuxerUnitTest PUBLIC coverage_config)
 endif ()

framework/tests/render/CMakeLists.txt

@@ -23,8 +23,7 @@ target_include_directories(
         PRIVATE
 )
 
-target_link_libraries(
-        renderTest
+target_link_libraries(renderTest PRIVATE
         demuxer
         videodec
         data_source
@@ -46,17 +45,20 @@ target_link_libraries(
 
 if (APPLE)
     target_link_libraries(
-            renderTest
+            renderTest PUBLIC
             iconv
             bz2
             ${FRAMEWORK_LIBS}
     )
-else()
+else ()
     target_link_libraries(
-            renderTest
+            renderTest PUBLIC
             dl
             pthread
     )
 
 endif ()
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(renderTest PUBLIC coverage_config)
+endif ()
 

mediaPlayer/CMakeLists.txt

@@ -136,4 +136,7 @@ endif ()
 
 add_library(media_player STATIC ${SOURCE_FILES})
 target_include_directories(media_player PUBLIC ../framework)
+if (HAVE_COVERAGE_CONFIG)
+    target_link_libraries(media_player PUBLIC coverage_config)
+endif ()