commit 971278a9627a3b3de6c6116c0fb4ebb563f41617
author kjellander@webrtc.org <kjellander@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> Fri Mar 08 10:20:53 2013 +0000
committer kjellander@webrtc.org <kjellander@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> Fri Mar 08 10:20:53 2013 +0000
tree 5ca43a9345dec5a6563b52deb833939d71d3e420
parent 3137a21068e2531ba42285916e60100300628c92

Splitting out video_coding_test executable again.

This CL undoes the merge of the developer test tool and the gtest tests
that was merged in https://code.google.com/p/webrtc/source/detail?r=3176

Doing that, we get a pure gtest executable of
video_coding_integrationtests which can run properly on the bots.

BUG=none
TEST=Trybots passing + local execution on Linux with:
out/Debug/video_coding_integrationtests --gtest_print_time (to ensure it will be possible to run with runtest.py)

Review URL: https://webrtc-codereview.appspot.com/1171007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3638 4adac7df-926f-26a2-2b94-8c16560cd09d

webrtc/modules/video_coding/main/source/video_coding_test.gypi

diff --git a/webrtc/modules/video_coding/main/source/video_coding_test.gypi b/webrtc/modules/video_coding/main/source/video_coding_test.gypi
index b453c57..c00d56c 100644
--- a/webrtc/modules/video_coding/main/source/video_coding_test.gypi
+++ b/webrtc/modules/video_coding/main/source/video_coding_test.gypi
@@ -7,12 +7,12 @@
 # be found in the AUTHORS file in the root of the source tree.
 
 {
-  'targets': [{
-      'target_name': 'video_coding_integrationtests',
+  'targets': [
+    {
+      'target_name': 'video_coding_test',
       'type': 'executable',
       'dependencies': [
          'rtp_rtcp',
-         'video_codecs_test_framework',
          'video_processing',
          'webrtc_video_coding',
          'webrtc_utility',
@@ -63,10 +63,23 @@
         '../test/video_rtp_play_mt.cc',
         '../test/video_rtp_play.cc',
         '../test/video_source.cc',
-        '../../codecs/test/videoprocessor_integrationtest.cc',
       ], # sources
     },
     {
+      'target_name': 'video_coding_integrationtests',
+      'type': 'executable',
+      'dependencies': [
+         'video_codecs_test_framework',
+         'webrtc_video_coding',
+         '<(DEPTH)/testing/gtest.gyp:gtest',
+         '<(webrtc_root)/test/test.gyp:test_support_main',
+         '<(webrtc_root)/test/metrics.gyp:metrics',
+      ],
+      'sources': [
+        '../../codecs/test/videoprocessor_integrationtest.cc',
+      ],
+    },
+    {
       'target_name': 'video_coding_unittests',
       'type': 'executable',
       'dependencies': [

webrtc/modules/video_coding/main/test/tester_main.cc

diff --git a/webrtc/modules/video_coding/main/test/tester_main.cc b/webrtc/modules/video_coding/main/test/tester_main.cc
index fc15ac7..6188cba 100644
--- a/webrtc/modules/video_coding/main/test/tester_main.cc
+++ b/webrtc/modules/video_coding/main/test/tester_main.cc
@@ -9,7 +9,6 @@
  */
 
 #include "google/gflags.h"
-#include "gtest/gtest.h"
 
 #include "receiver_tests.h"
 #include "normal_test.h"
@@ -19,7 +18,6 @@
 #include "media_opt_test.h"
 #include "quality_modes_test.h"
 #include "test_util.h"
-#include "webrtc/test/test_suite.h"
 #include "webrtc/test/testsupport/fileutils.h"
 
 #include <stdlib.h>
@@ -46,8 +44,6 @@
 DEFINE_string(fv_output_filename, webrtc::test::OutputPath() +
               "features.txt", "FV output file.");
 DEFINE_int32(test_number, 0, "Test number.");
-DEFINE_bool(run_gtest_tests, true, "Run gtest tests too (after legacy tests has"
-            " executed).");
 
 using namespace webrtc;
 
@@ -99,7 +95,7 @@
     return -1;
   }
 
-  printf("Running legacy video coding tests...\n");
+  printf("Running video coding tests...\n");
   int ret = 0;
   switch (args.testNum) {
     case 0:
@@ -146,14 +142,5 @@
       ret = -1;
       break;
   }
-  if (ret != 0) {
-    printf("Legacy Tests failed!\n");
-  } else {
-    if (FLAGS_run_gtest_tests) {
-      printf("Running gtest integration tests...\n");
-      webrtc::test::TestSuite test_suite(argc, argv);
-      ret = test_suite.Run();
-    }
-  }
   return ret;
 }