Encoder test updates

- Rename AvcEncoderTest as AvcEncTest
- Enable AvcEncTest in cmake builds
- Rename AvcEncoderTestEnvironment.h as TestArgs.h

Change-Id: Iceba9c33e060bcb1138f03833da67c0ff6e84b2b

9 files changed, 136 insertions, 70 deletions

diff --git a/.github/workflows/cmake.yml b/.github/workflows/cmake.yml
index b3e7e40..bda845e 100644
--- a/.github/workflows/cmake.yml
+++ b/.github/workflows/cmake.yml
@@ -17,7 +17,7 @@ jobs:
     - uses: actions/checkout@v3
 
     - name: Configure CMake
-      run: cmake -B ${{github.workspace}}/out -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}} -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DENABLE_MVC=1 -DENABLE_SVC=1
+      run: cmake -B ${{github.workspace}}/out -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}} -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DENABLE_MVC=1 -DENABLE_SVC=1 -DENABLE_TESTS=1
 
     - name: Build
       run: cmake --build ${{github.workspace}}/out --config ${{env.BUILD_TYPE}}
diff --git a/.gitignore b/.gitignore
new file mode 100644
index 0000000..912eacc
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1 @@
+third_party
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 6b95a00..d0c37d3 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -6,6 +6,7 @@ set(AVC_ROOT "${CMAKE_CURRENT_SOURCE_DIR}")
 set(AVC_CONFIG_DIR "${CMAKE_CURRENT_BINARY_DIR}")
 option(ENABLE_MVC "Enables svcenc and svcdec builds" OFF)
 option(ENABLE_SVC "Enables svcenc and svcdec builds" OFF)
+option(ENABLE_TESTS "Enables gtest based unit tests" OFF)
 
 if("${AVC_ROOT}" STREQUAL "${AVC_CONFIG_DIR}")
   message(
@@ -67,3 +68,7 @@ if (${ENABLE_SVC})
   include("${AVC_ROOT}/fuzzer/svc_enc_fuzzer.cmake")
   include("${AVC_ROOT}/fuzzer/svc_dec_fuzzer.cmake")
 endif()
+
+if (${ENABLE_TESTS})
+    include("${AVC_ROOT}/tests/AvcEncTest.cmake")
+endif()
diff --git a/tests/Android.bp b/tests/Android.bp
index 14ba11d..656d3b7 100644
--- a/tests/Android.bp
+++ b/tests/Android.bp
@@ -24,10 +24,10 @@ package {
 }
 
 cc_test {
-    name: "AvcEncoderTest",
+    name: "AvcEncTest",
     gtest: true,
 
-    srcs : [ "AvcEncoderTest.cpp" ],
+    srcs : [ "AvcEncTest.cpp" ],
 
     shared_libs: [
         "libutils",
diff --git a/tests/AndroidTest.xml b/tests/AndroidTest.xml
index 6ad3f01..36d240d 100644
--- a/tests/AndroidTest.xml
+++ b/tests/AndroidTest.xml
@@ -14,18 +14,18 @@
      limitations under the License.
 -->
 <configuration description="Test module config for AVC encoder unit tests">
-    <option name="test-suite-tag" value="AvcEncoderTest" />
+    <option name="test-suite-tag" value="AvcEncTest" />
     <target_preparer class="com.android.tradefed.targetprep.PushFilePreparer">
         <option name="cleanup" value="true" />
-        <option name="push" value="AvcEncoderTest->/data/local/tmp/AvcEncoderTest/" />
+        <option name="push" value="AvcEncTest->/data/local/tmp/AvcEncTest/" />
         <option name="push-file"
             key="https://storage.googleapis.com/android_media/external/libavc/tests/AvcEncoder.zip?unzip=true"
-            value="/data/local/tmp/AvcEncoderTestRes/" />
+            value="/data/local/tmp/AvcEncTestRes/" />
     </target_preparer>
 
     <test class="com.android.tradefed.testtype.GTest" >
         <option name="native-test-device-path" value="/data/local/tmp" />
-        <option name="module-name" value="AvcEncoderTest" />
-        <option name="native-test-flag" value="-P /data/local/tmp/AvcEncoderTestRes/" />
+        <option name="module-name" value="AvcEncTest" />
+        <option name="native-test-flag" value="-P /data/local/tmp/AvcEncTestRes/" />
     </test>
 </configuration>
diff --git a/tests/AvcEncTest.cmake b/tests/AvcEncTest.cmake
new file mode 100644
index 0000000..8b1c76c
--- /dev/null
+++ b/tests/AvcEncTest.cmake
@@ -0,0 +1,21 @@
+include(ExternalProject)
+ExternalProject_Add(googletest
+    GIT_REPOSITORY https://android.googlesource.com/platform/external/googletest
+    GIT_TAG main
+    PREFIX ${AVC_ROOT}/third_party/build/googletest
+    SOURCE_DIR ${AVC_ROOT}/third_party/googletest
+    TMP_DIR ${AVC_ROOT}/third_party/build/googletest/tmp
+    INSTALL_COMMAND ""
+)
+
+list(
+  APPEND
+  AVCENCTEST_SRCS
+  "${AVC_ROOT}/tests/AvcEncTest.cpp")
+
+libavc_add_executable(AvcEncTest libavcenc
+    SOURCES ${AVCENCTEST_SRCS}
+    INCLUDES "${AVC_ROOT}/third_party/googletest/googletest/include")
+target_link_libraries(AvcEncTest
+    ${AVC_ROOT}/third_party/build/googletest/src/googletest-build/lib/libgtest.a
+    ${AVC_ROOT}/third_party/build/googletest/src/googletest-build/lib/libgtest_main.a)
diff --git a/tests/AvcEncoderTest.cpp b/tests/AvcEncTest.cpp
index 20c85e8..83ea700 100644
--- a/tests/AvcEncoderTest.cpp
+++ b/tests/AvcEncTest.cpp
@@ -14,30 +14,26 @@
  * limitations under the License.
  */
 
-#include <utils/Log.h>
-
 #include "ih264_defs.h"
 #include "ih264_typedefs.h"
 #include "ih264e.h"
 #include "ih264e_error.h"
 
-#include "AvcEncoderTestEnvironment.h"
+#include "TestArgs.h"
 
 #define MAX_FRAME_HEIGHT 1080
 #define MAX_FRAME_WIDTH 1920
 #define MAX_OUTPUT_BUFFER_SIZE (MAX_FRAME_HEIGHT * MAX_FRAME_WIDTH)
 
-#define ENCODED_FILE "/data/local/tmp/AvcOutput"
-
 #define ive_api_function ih264e_api_function
 
 constexpr int16_t kCompressionRatio = 1;
 constexpr size_t kMinQP = 4;
 constexpr uint32_t kHeaderLength = 0x800;
 
-static AvcEncoderTestEnvironment* gEnv = nullptr;
+static TestArgs* gArgs = nullptr;
 
-class AvcEncoderTest
+class AvcEncTest
     : public ::testing::TestWithParam<tuple<string, int32_t, int32_t, float, int32_t>> {
   private:
     void setRawBuf(iv_raw_buf_t* psInpRawBuf, const uint8_t* data);
@@ -110,10 +106,10 @@ class AvcEncoderTest
     IV_PROFILE_T mProfile = IV_PROFILE_BASE;
 
   public:
-    AvcEncoderTest()
+    AvcEncTest()
         : mInputBuffer(nullptr), mOutputBuffer(nullptr), mFpInput(nullptr), mFpOutput(nullptr) {}
 
-    ~AvcEncoderTest() {
+    ~AvcEncTest() {
         iv_mem_rec_t* ps_mem_rec = mMemRecords;
         for (size_t i = 0; i < mNumMemRecords; ++i) {
             if (ps_mem_rec) {
@@ -131,11 +127,12 @@ class AvcEncoderTest
         tuple<string /* fileName */, int32_t /* frameWidth */, int32_t /* frameHeight */,
               float /* frameRate */, int32_t /* bitRate */>
                 params = GetParam();
-        mFileName = gEnv->getRes() + get<0>(params);
+        mFileName = gArgs->getRes() + get<0>(params);
         mFrameWidth = get<1>(params);
         mFrameHeight = get<2>(params);
         mFrameRate = get<3>(params);
         mBitRate = get<4>(params);
+        mOutFileName = gArgs->getRes() + "out.bin";
 
         ASSERT_LE(mFrameWidth, 1080) << "Frame Width <= 1080";
 
@@ -153,8 +150,8 @@ class AvcEncoderTest
         mFpInput = fopen(mFileName.c_str(), "rb");
         ASSERT_NE(mFpInput, nullptr) << "Failed to open the input file: " << mFileName;
 
-        mFpOutput = fopen(ENCODED_FILE, "wb");
-        ASSERT_NE(mFpOutput, nullptr) << "Failed to open the output file:" << ENCODED_FILE;
+        mFpOutput = fopen(mOutFileName.c_str(), "wb");
+        ASSERT_NE(mFpOutput, nullptr) << "Failed to open the output file:" << mOutFileName;
 
         /* Getting Number of MemRecords */
         iv_num_mem_rec_ip_t sNumMemRecIp = {};
@@ -316,6 +313,7 @@ class AvcEncoderTest
     int32_t mBitRate = 256000;
     int64_t mOutputBufferSize = MAX_OUTPUT_BUFFER_SIZE;
     string mFileName;
+    string mOutFileName;
     uint8_t* mInputBuffer = nullptr;
     uint8_t* mOutputBuffer = nullptr;
     FILE* mFpInput = nullptr;
@@ -323,7 +321,7 @@ class AvcEncoderTest
     IV_STATUS_T status;
 };
 
-void AvcEncoderTest::setDimensions() {
+void AvcEncTest::setDimensions() {
     ive_ctl_set_dimensions_ip_t sDimensionsIp = {};
     ive_ctl_set_dimensions_op_t sDimensionsOp = {};
 
@@ -344,7 +342,7 @@ void AvcEncoderTest::setDimensions() {
     return;
 }
 
-void AvcEncoderTest::setNumCores() {
+void AvcEncTest::setNumCores() {
     ive_ctl_set_num_cores_ip_t sNumCoresIp = {};
     ive_ctl_set_num_cores_op_t sNumCoresOp = {};
 
@@ -364,7 +362,7 @@ void AvcEncoderTest::setNumCores() {
     return;
 }
 
-void AvcEncoderTest::setDefault() {
+void AvcEncTest::setDefault() {
     ive_ctl_setdefault_ip_t sDefaultIp = {};
     ive_ctl_setdefault_op_t sDefaultOp = {};
 
@@ -383,7 +381,7 @@ void AvcEncoderTest::setDefault() {
     return;
 }
 
-void AvcEncoderTest::getBufInfo() {
+void AvcEncTest::getBufInfo() {
     ih264e_ctl_getbufinfo_ip_t sGetBufInfoIp = {};
     ih264e_ctl_getbufinfo_op_t sGetBufInfoOp = {};
 
@@ -402,7 +400,7 @@ void AvcEncoderTest::getBufInfo() {
     return;
 }
 
-void AvcEncoderTest::setFrameRate() {
+void AvcEncTest::setFrameRate() {
     ive_ctl_set_frame_rate_ip_t sFrameRateIp = {};
     ive_ctl_set_frame_rate_op_t sFrameRateOp = {};
 
@@ -423,7 +421,7 @@ void AvcEncoderTest::setFrameRate() {
     return;
 }
 
-void AvcEncoderTest::setIpeParams() {
+void AvcEncTest::setIpeParams() {
     ive_ctl_set_ipe_params_ip_t sIpeParamsIp = {};
     ive_ctl_set_ipe_params_op_t sIpeParamsOp = {};
 
@@ -445,7 +443,7 @@ void AvcEncoderTest::setIpeParams() {
     return;
 }
 
-void AvcEncoderTest::setBitRate() {
+void AvcEncTest::setBitRate() {
     ive_ctl_set_bitrate_ip_t sBitrateIp = {};
     ive_ctl_set_bitrate_op_t sBitrateOp = {};
 
@@ -465,7 +463,7 @@ void AvcEncoderTest::setBitRate() {
     return;
 }
 
-void AvcEncoderTest::setFrameType(IV_PICTURE_CODING_TYPE_T eFrameType) {
+void AvcEncTest::setFrameType(IV_PICTURE_CODING_TYPE_T eFrameType) {
     ive_ctl_set_frame_type_ip_t sFrameTypeIp = {};
     ive_ctl_set_frame_type_op_t sFrameTypeOp = {};
 
@@ -484,7 +482,7 @@ void AvcEncoderTest::setFrameType(IV_PICTURE_CODING_TYPE_T eFrameType) {
     return;
 }
 
-void AvcEncoderTest::setQp() {
+void AvcEncTest::setQp() {
     ive_ctl_set_qp_ip_t s_QpIp = {};
     ive_ctl_set_qp_op_t s_QpOp = {};
 
@@ -515,7 +513,7 @@ void AvcEncoderTest::setQp() {
     return;
 }
 
-void AvcEncoderTest::setEncMode(IVE_ENC_MODE_T eEncMode) {
+void AvcEncTest::setEncMode(IVE_ENC_MODE_T eEncMode) {
     ive_ctl_set_enc_mode_ip_t sEncModeIp = {};
     ive_ctl_set_enc_mode_op_t sEncModeOp = {};
 
@@ -535,7 +533,7 @@ void AvcEncoderTest::setEncMode(IVE_ENC_MODE_T eEncMode) {
     return;
 }
 
-void AvcEncoderTest::setVbvParams() {
+void AvcEncTest::setVbvParams() {
     ive_ctl_set_vbv_params_ip_t sVbvIp = {};
     ive_ctl_set_vbv_params_op_t sVbvOp = {};
 
@@ -556,7 +554,7 @@ void AvcEncoderTest::setVbvParams() {
     return;
 }
 
-void AvcEncoderTest::setAirParams() {
+void AvcEncTest::setAirParams() {
     ive_ctl_set_air_params_ip_t sAirIp = {};
     ive_ctl_set_air_params_op_t sAirOp = {};
 
@@ -577,7 +575,7 @@ void AvcEncoderTest::setAirParams() {
     return;
 }
 
-void AvcEncoderTest::setMeParams() {
+void AvcEncTest::setMeParams() {
     ive_ctl_set_me_params_ip_t sMeParamsIp = {};
     ive_ctl_set_me_params_op_t sMeParamsOp = {};
 
@@ -604,7 +602,7 @@ void AvcEncoderTest::setMeParams() {
     return;
 }
 
-void AvcEncoderTest::setGopParams() {
+void AvcEncTest::setGopParams() {
     ive_ctl_set_gop_params_ip_t sGopParamsIp = {};
     ive_ctl_set_gop_params_op_t sGopParamsOp = {};
 
@@ -626,7 +624,7 @@ void AvcEncoderTest::setGopParams() {
     return;
 }
 
-void AvcEncoderTest::setProfileParams() {
+void AvcEncTest::setProfileParams() {
     ive_ctl_set_profile_params_ip_t sProfileParamsIp = {};
     ive_ctl_set_profile_params_op_t sProfileParamsOp = {};
 
@@ -651,7 +649,7 @@ void AvcEncoderTest::setProfileParams() {
     return;
 }
 
-void AvcEncoderTest::setDeblockParams() {
+void AvcEncTest::setDeblockParams() {
     ive_ctl_set_deblock_params_ip_t sDeblockParamsIp = {};
     ive_ctl_set_deblock_params_op_t sDeblockParamsOp = {};
 
@@ -672,7 +670,7 @@ void AvcEncoderTest::setDeblockParams() {
     return;
 }
 
-void AvcEncoderTest::setVuiParams() {
+void AvcEncTest::setVuiParams() {
     ih264e_vui_ip_t sVuiParamsIp = {};
     ih264e_vui_op_t sVuiParamsOp = {};
 
@@ -694,7 +692,7 @@ void AvcEncoderTest::setVuiParams() {
     return;
 }
 
-void AvcEncoderTest::setSeiMdcvParams() {
+void AvcEncTest::setSeiMdcvParams() {
     ih264e_ctl_set_sei_mdcv_params_ip_t sSeiMdcvParamsIp = {};
     ih264e_ctl_set_sei_mdcv_params_op_t sSeiMdcvParamsOp = {};
 
@@ -723,7 +721,7 @@ void AvcEncoderTest::setSeiMdcvParams() {
     return;
 }
 
-void AvcEncoderTest::setSeiCllParams() {
+void AvcEncTest::setSeiCllParams() {
     ih264e_ctl_set_sei_cll_params_ip_t sSeiCllParamsIp = {};
     ih264e_ctl_set_sei_cll_params_op_t sSeiCllParamsOp = {};
 
@@ -747,7 +745,7 @@ void AvcEncoderTest::setSeiCllParams() {
     return;
 }
 
-void AvcEncoderTest::setSeiAveParams() {
+void AvcEncTest::setSeiAveParams() {
     ih264e_ctl_set_sei_ave_params_ip_t sSeiAveParamsIp = {};
     ih264e_ctl_set_sei_ave_params_op_t sSeiAveParamsOp = {};
 
@@ -772,7 +770,7 @@ void AvcEncoderTest::setSeiAveParams() {
     return;
 }
 
-void AvcEncoderTest::setSeiCcvParams() {
+void AvcEncTest::setSeiCcvParams() {
     ih264e_ctl_set_sei_ccv_params_ip_t sSeiCcvParamsIp = {};
     ih264e_ctl_set_sei_ccv_params_op_t sSeiCcvParamsOp = {};
 
@@ -808,7 +806,7 @@ void AvcEncoderTest::setSeiCcvParams() {
     return;
 }
 
-void AvcEncoderTest::logVersion() {
+void AvcEncTest::logVersion() {
     ive_ctl_getversioninfo_ip_t sCtlIp = {};
     ive_ctl_getversioninfo_op_t sCtlOp = {};
     UWORD8 au1Buf[512];
@@ -827,7 +825,7 @@ void AvcEncoderTest::logVersion() {
     return;
 }
 
-void AvcEncoderTest::encodeFrames(int64_t numFramesToEncode) {
+void AvcEncTest::encodeFrames(int64_t numFramesToEncode) {
     ih264e_video_encode_ip_t ih264e_video_encode_ip = {};
     ih264e_video_encode_op_t ih264e_video_encode_op = {};
 
@@ -897,7 +895,7 @@ void AvcEncoderTest::encodeFrames(int64_t numFramesToEncode) {
 
         int32_t numOutputBytes = fwrite((UWORD8*)sEncodeOp->s_out_buf.pv_buf, sizeof(UWORD8),
                                         sEncodeOp->s_out_buf.u4_bytes, mFpOutput);
-        ASSERT_NE(numOutputBytes, 0) << "Failed to write the output!" << ENCODED_FILE;
+        ASSERT_NE(numOutputBytes, 0) << "Failed to write the output!" << mOutFileName;
 
         numFramesToEncode--;
         numFrame++;
@@ -914,11 +912,11 @@ void AvcEncoderTest::encodeFrames(int64_t numFramesToEncode) {
     if (sEncodeOp->output_present) {
         int32_t numOutputBytes = fwrite((UWORD8*)sEncodeOp->s_out_buf.pv_buf, sizeof(UWORD8),
                                         sEncodeOp->s_out_buf.u4_bytes, mFpOutput);
-        ASSERT_NE(numOutputBytes, 0) << "Failed to write the output!" << ENCODED_FILE;
+        ASSERT_NE(numOutputBytes, 0) << "Failed to write the output!" << mOutFileName;
     }
 }
 
-void AvcEncoderTest::setRawBuf(iv_raw_buf_t* psInpRawBuf, const uint8_t* data) {
+void AvcEncTest::setRawBuf(iv_raw_buf_t* psInpRawBuf, const uint8_t* data) {
     switch (mIvVideoColorFormat) {
         case IV_YUV_420SP_UV:
             [[fallthrough]];
@@ -982,7 +980,7 @@ void AvcEncoderTest::setRawBuf(iv_raw_buf_t* psInpRawBuf, const uint8_t* data) {
     return;
 }
 
-int64_t AvcEncoderTest::getTotalFrames() {
+int64_t AvcEncTest::getTotalFrames() {
     struct stat buf;
     stat(mFileName.c_str(), &buf);
     size_t fileSize = buf.st_size;
@@ -990,23 +988,22 @@ int64_t AvcEncoderTest::getTotalFrames() {
     return totalFrames;
 }
 
-TEST_P(AvcEncoderTest, EncodeTest) {
+TEST_P(AvcEncTest, EncodeTest) {
     ASSERT_NO_FATAL_FAILURE(encodeFrames(mTotalFrames)) << "Failed to Encode: " << mFileName;
 }
 
-INSTANTIATE_TEST_SUITE_P(EncodeTest, AvcEncoderTest,
+INSTANTIATE_TEST_SUITE_P(EncodeTest, AvcEncTest,
                          ::testing::Values(make_tuple("bbb_352x288_420p_30fps_32frames.yuv", 352,
                                                       288, 30, 2048),
                                            make_tuple("football_qvga.yuv", 320, 240, 30, 1024)));
 
 int32_t main(int argc, char** argv) {
-    gEnv = new AvcEncoderTestEnvironment();
-    ::testing::AddGlobalTestEnvironment(gEnv);
+    gArgs = new TestArgs();
+    ::testing::AddGlobalTestEnvironment(gArgs);
     ::testing::InitGoogleTest(&argc, argv);
-    uint8_t status = gEnv->initFromOptions(argc, argv);
+    uint8_t status = gArgs->initFromOptions(argc, argv);
     if (status == 0) {
         status = RUN_ALL_TESTS();
-        ALOGI("Encoder Test Result = %d\n", status);
     }
     return status;
 }
diff --git a/tests/README.md b/tests/README.md
index 78c6328..43aa2db 100644
--- a/tests/README.md
+++ b/tests/README.md
@@ -1,38 +1,80 @@
-## Media Testing ##
----
+# AvcEncTest
+The AvcEncoder Test Suite validates the Avc encoder.
 
-#### AvcEncoder
-The AvcEncoder Test Suite validates the Avc encoder library available in external/libavc.
-Run the following steps to build the test suite:
+## Linux x86/x64
+
+###  Requirements
+- cmake (3.9.1 or above)
+- make
+- clang (12.0 or above)
+
+### Steps to build
+Clone libavc repository
+```
+$ git clone https://android.googlesource.com/platform/external/libavc
+```
+Create a directory inside libavc and change directory
+```
+ $ cd libavc
+ $ mkdir build
+ $ cd build
+```
+
+Build with -DENABLE_TESTS=1.
+```
+ $ cmake .. -DENABLE_TESTS=1 -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ \
+   -DCMAKE_BUILD_TYPE=Debug
+ $ make
+```
+
+Optionally, enable sanitizers by passing -DSANITIZE
 ```
-m AvcEncoderTest
+ $ cmake .. -DENABLE_TESTS=1 -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ \
+   -DCMAKE_BUILD_TYPE=Debug -DSANITIZE=fuzzer-no-link,address,\
+   signed-integer-overflow,unsigned-integer-overflow
+ $ make
 ```
 
-The 32-bit binaries will be created in the following path : ${OUT}/data/nativetest/
-The 64-bit binaries will be created in the following path : ${OUT}/data/nativetest64/
+The media files for the tests are present [at](https://storage.googleapis.com/android_media/external/libavc/tests/AvcEncoder.zip).
+Download and extract these the current folder.
+
+usage: AvcEncTest -P \<path_to_the local folder\>
+
+```
+$./AvcEncTest -P ./
+```
+
+## Android
+
+Run the following steps to build the test suite:
+```
+m AvcEncTest
+```
 
 To test 64-bit binary push binaries from nativetest64.
 ```
-adb push ${OUT}/data/nativetest64/AvcEncoderTest/AvcEncoderTest /data/local/tmp/
+adb push ${OUT}/data/nativetest64/AvcEncTest/AvcEncTest /data/local/tmp/
 ```
 
 To test 32-bit binary push binaries from nativetest.
 ```
-adb push ${OUT}/data/nativetest/AvcEncoderTest/AvcEncoderTest /data/local/tmp/
+adb push ${OUT}/data/nativetest/AvcEncTest/AvcEncTest /data/local/tmp/
 ```
 
-The resource file for the tests is taken from [here](https://storage.googleapis.com/android_media/frameworks/av/media/libstagefright/codecs/m4v_h263/enc/test/Mpeg4H263Encoder.zip ) Download, unzip and push these files into device for testing.
+The resource file for the tests is taken from [here](https://storage.googleapis.com/android_media/external/libavc/tests/AvcEncoder.zip)
+
+Download, unzip and push these files into device for testing.
 
 ```
 adb push AvcEncoder/. /data/local/tmp/
 ```
 
-usage: AvcEncoderTest -P \<path_to_folder\>
+usage: AvcEncTest -P \<path_to_folder\>
 ```
-adb shell /data/local/tmp/AvcEncoderTest -P /data/local/tmp/
+adb shell /data/local/tmp/AvcEncTest -P /data/local/tmp/
 ```
 Alternatively, the test can also be run using atest command.
 
 ```
-atest AvcEncoderTest -- --enable-module-dynamic-download=true
+atest AvcEncTest -- --enable-module-dynamic-download=true
 ```
diff --git a/tests/AvcEncoderTestEnvironment.h b/tests/TestArgs.h
index c3cb434..684925c 100644
--- a/tests/AvcEncoderTestEnvironment.h
+++ b/tests/TestArgs.h
@@ -22,9 +22,9 @@
 
 using namespace std;
 
-class AvcEncoderTestEnvironment : public::testing::Environment {
+class TestArgs : public::testing::Environment {
   public:
-    AvcEncoderTestEnvironment() : res("/data/local/tmp/AvcEncoderTest/") {}
+    TestArgs() : res("/data/local/tmp/AvcEncTestRes/") {}
 
     // Parses the command line arguments
     int initFromOptions(int argc, char **argv);
@@ -37,7 +37,7 @@ class AvcEncoderTestEnvironment : public::testing::Environment {
     string res;
 };
 
-int AvcEncoderTestEnvironment::initFromOptions(int argc, char **argv) {
+int TestArgs::initFromOptions(int argc, char **argv) {
     static struct option options[] = {{"path", required_argument, 0, 'P'}, {0, 0, 0, 0}};
 
     while (true) {