generic-library/webrtc
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Try to fix MixingTest on the Win bots.

Browse Source 
- Relax the constraints on recording duration.
- Remove unneeded file deletes. (These files will be properly
  overwritten anyway).

TBR=henrike@webrtc.org
BUG=issue534
TEST=voe_auto_test

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

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2295 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
andrew@webrtc.org 2012-05-24 17:26:05 +00:00
parent 42e78ac087
commit 51b4f3e6a8
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/voice_engine/main/test/auto_test/standard/mixing_test.cc
View File

@ -80,10 +80,6 @@ class MixingTest : public AfterInitializationFixture {
StopRemoteStreams(remote_streams); StopRemoteStreams(remote_streams);
VerifyMixedOutput(max_output_value, min_output_value); VerifyMixedOutput(max_output_value, min_output_value);
// Cleanup the files in case another test uses different lengths.
ASSERT_EQ(0, remove(input_filename_.c_str()));
ASSERT_EQ(0, remove(output_filename_.c_str()));
} }
private: private:
@ -116,8 +112,8 @@ class MixingTest : public AfterInitializationFixture {
EXPECT_GE(output_value, min_output_value); EXPECT_GE(output_value, min_output_value);
} }
// Ensure the recording length is close to the duration of the test. // Ensure the recording length is close to the duration of the test.
ASSERT_GE((samples_read * 1000.0f) / kSampleRateHz, ASSERT_GE((samples_read * 1000.0) / kSampleRateHz,
kTestDurationMs - kSkipOutputMs); 0.9 * (kTestDurationMs - kSkipOutputMs));
// Ensure we read the entire file. // Ensure we read the entire file.
ASSERT_NE(0, feof(output_file)); ASSERT_NE(0, feof(output_file));
ASSERT_EQ(0, fclose(output_file)); ASSERT_EQ(0, fclose(output_file));