Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Side by Side Diff: chrome_frame/test/navigation_test.cc

Issue 5380005: Enable Selenium test for chrome Frame... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 6
7 #include "base/scoped_comptr_win.h" 7 #include "base/scoped_comptr_win.h"
8 #include "base/test/test_file_util.h" 8 #include "base/test/test_file_util.h"
9 #include "base/win/windows_version.h" 9 #include "base/win/windows_version.h"
10 #include "chrome_frame/test/chrome_frame_test_utils.h" 10 #include "chrome_frame/test/chrome_frame_test_utils.h"
(...skipping 959 matching lines...) Expand 10 before | Expand all | Expand 10 after
970 970
971 // Currently we run total 505 tests and 8 steps fail. 971 // Currently we run total 505 tests and 8 steps fail.
972 // TODO(amit): send results as JSON, diagnose and eliminate failures. 972 // TODO(amit): send results as JSON, diagnose and eliminate failures.
973 ASSERT_EQ(num_tests, 505) << "Expected to run: " << 505 << " tests." << 973 ASSERT_EQ(num_tests, 505) << "Expected to run: " << 505 << " tests." <<
974 " Actual number of tests run: " << num_tests; 974 " Actual number of tests run: " << num_tests;
975 ASSERT_LE(failed_tests, 8) << "Expected failures: " << 8 << 975 ASSERT_LE(failed_tests, 8) << "Expected failures: " << 8 <<
976 " Actual failures: " << failed_tests; 976 " Actual failures: " << failed_tests;
977 } 977 }
978 978
979 // Tests refreshing causes a page load. 979 // Tests refreshing causes a page load.
980 TEST_F(FullTabSeleniumTest, DISABLED_Core) { 980 TEST_F(FullTabSeleniumTest, Core) {
981 server_mock_.ExpectAndServeAnyRequests(CFInvocation::HttpHeader()); 981 server_mock_.ExpectAndServeAnyRequests(CFInvocation::HttpHeader());
982 std::wstring url = GetTestUrl(L"core/TestRunner.html"); 982 std::wstring url = GetTestUrl(L"core/TestRunner.html");
983 983
984 // Expectations for TestRunner.html 984 // Expectations for TestRunner.html
985 EXPECT_CALL(ie_mock_, OnFileDownload(_, _)).Times(testing::AnyNumber()); 985 EXPECT_CALL(ie_mock_, OnFileDownload(_, _)).Times(testing::AnyNumber());
986 EXPECT_CALL(ie_mock_, OnBeforeNavigate2(_, 986 EXPECT_CALL(ie_mock_, OnBeforeNavigate2(_,
987 testing::Field(&VARIANT::bstrVal, 987 testing::Field(&VARIANT::bstrVal,
988 testing::StartsWith(url)), _, _, _, _, _)) 988 testing::StartsWith(url)), _, _, _, _, _))
989 .Times(testing::AnyNumber()); 989 .Times(testing::AnyNumber());
990 EXPECT_CALL(ie_mock_, OnNavigateComplete2(_, 990 EXPECT_CALL(ie_mock_, OnNavigateComplete2(_,
(...skipping 22 matching lines...) Expand all
1013 .WillOnce(testing::DoAll(VerifySeleniumCoreTestResults(), 1013 .WillOnce(testing::DoAll(VerifySeleniumCoreTestResults(),
1014 CloseBrowserMock(&ie_mock_))); 1014 CloseBrowserMock(&ie_mock_)));
1015 1015
1016 // Selenium tests take longer to finish, lets give it 2 mins. 1016 // Selenium tests take longer to finish, lets give it 2 mins.
1017 const int kSeleniumTestTimeout = 120; 1017 const int kSeleniumTestTimeout = 120;
1018 LaunchIENavigateAndLoop(url, kSeleniumTestTimeout); 1018 LaunchIENavigateAndLoop(url, kSeleniumTestTimeout);
1019 } 1019 }
1020 1020
1021 1021
1022 } // namespace chrome_frame_test 1022 } // namespace chrome_frame_test
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698