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

Issue 10917042: Revert 154453 - Remove the translate pyauto test to chrome tests and all the supporting automation … (Closed)

Created:
8 years, 3 months ago by dmazzoni
Modified:
8 years, 3 months ago
Reviewers:
jam
CC:
chromium-reviews, Nirnimesh, kkania, anantha, robertshield, dyu1, dennis_jeffrey
Visibility:
Public.

Description

Revert 154453 - Remove the translate pyauto test to chrome tests and all the supporting automation hooks. I've added new tests for the non-duplicate cases. Most of the Chrome tests use a test harness to avoid network requests to the Translate server, which is inline with our testing phiolosophy (i.e. new bots don't even have network access). There is a browser_test in BrowserTest.PageLanguageDetection. testTranslate is a duplicate of TranslateManagerTest.NormalTranslate testNoTranslate is a duplicate of TranslateManagerTest.MultipleOnPageContents and TranslateManagerTest.DenyTranslateInPageNavigation testNeverTranslateLanguage and testAlwaysTranslateLanguage are duplicates of TranslateManagerTest.BeforeTranslateExtraButtons testNeverTranslateSite is a duplicate of TranslateManagerTest.NeverTranslateSitePref testRevert is a duplicate of TranslateManagerTest.ServerReportsUnsupportedLanguage testBarNotVisibleOnSSLErrorPage seems unnecessary, i.e. we don't check that infobar comes up for all of our internal pages testBarNotVisibleOnEnglishPage is a duplicate of TranslateManagerTest.AutoTranslateOnNavigate testTranslateDiffURLs, like in other pyauto test suites, seems unnecessary (i.e. we don't retest every feature across all schemes) testNotranslateMetaTag is a duplicate of ChromeRenderViewTest.TranslatablePage testToggleTranslateOption is a duplicate of TranslateManagerTest.AlwaysTranslateLanguagePref testGoBackAndForwardToTranslatePage: the interesting bits are tested by ChromeRenderViewTest.BackToTranslatablePage testForCrashedTab seems unnecessary, we don't verify that all infobars are gone when a page crashes testTranslatePrefs is a duplicate of of TranslateManagerTest.TranslateEnabledPref testAlwaysTranslateLanguageButton and testNeverTranslateLanguageButton are duplicates of TranslateManagerTest.BeforeTranslateExtraButtons testChangeTargetLanguageAlwaysTranslate: I added a reload step to TranslateManagerTest.NormalTranslate to cover this testHistoryNotTranslated and testDownloadsNotTranslated: I added a simple unittest for them testSeveralLanguages is redundant with many of the above tests testMultipleTabsAndWindows is not very useful testAlwaysTranslateInIncognito is a duplicate of a subset of TranslateManagerTest.AlwaysTranslateLanguagePref testNoTranslateInfobarAfterNeverTranslate is a duplicate of TranslateManagerTest.NeverTranslateLanguagePref testGoBackAndForwardToTranslatePage is a duplicate of ChromeRenderViewTest.BackToTranslatablePage testSessionRestore: I added a browser test for (BrowserTest.SessionRestore) BUG=143637, 95140, 129802, 12648, 12639 Review URL: https://chromiumcodereview.appspot.com/10922002 TBR=jam@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=154464

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+645 lines, -100 lines) Patch
M chrome/browser/automation/automation_provider_observers.h View 2 chunks +44 lines, -0 lines 0 comments Download
M chrome/browser/automation/automation_provider_observers.cc View 2 chunks +101 lines, -0 lines 0 comments Download
M chrome/browser/automation/testing_automation_provider.h View 2 chunks +18 lines, -0 lines 0 comments Download
M chrome/browser/automation/testing_automation_provider.cc View 7 chunks +233 lines, -0 lines 0 comments Download
M chrome/browser/translate/translate_manager_browsertest.cc View 8 chunks +43 lines, -101 lines 0 comments Download
M chrome/test/functional/PYAUTO_TESTS View 6 chunks +19 lines, -0 lines 0 comments Download
A + chrome/test/functional/translate.py View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/test/pyautolib/pyauto.py View 2 chunks +188 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
dmazzoni
8 years, 3 months ago (2012-08-31 18:05:23 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698