-
Notifications
You must be signed in to change notification settings - Fork 0
Cameo testing framework
Since Cameo is designed to be built upon content layer, besides the test suites written in content layer, it is important for Cameo to setup its own testing framework besides the existing tests written in content module.
All content tests can be found in src/content/content_tests.gypi file.
- content_unittests
- content_browsertests
- video_decode_accelerator_unittest (For Windows, ChromeOS)
- h264_parser_unittest (For ChromeOS, Linux)
We can make use of these tests for Cameo as possible to make sure nothing is broken once code change in content layer, and we might still need to build content_shell executable binary to run these tests.
In Chromium, several kinds of testing are provided from Chrome, see Chromium Testing and Infrastructure.
For Cameo, to ensure the code quality, at least 2 kinds of testing should be developed along with the process of feature development and bug fixing.
Unit test is intended to test whether a function works as expected. The unit test writing is based on gtest framework. Use TEST_F to add test case into testing framework. See here for more information about gtest.
Browser test is another special unit test which is considered as Cameo integration test. It requires all test cases MUST run in the browser process, which means the browser environment and context must have been setup and bring up a browser window to load a given URL. For example, you can query the title and URL of web contents loaded in the browser window.
Similarly with Chromium does, we also define InProcessBrowserTest derived from content::BrowserTestBase to help set up the browser testing environment and use IN_PROC_BROWSER_TEST to define browser test case. See content/public/test/browser_test.h and content/public/test/browser_test_base.h for IN_RPOC_BROWSER_TEST macro definition and BrowserTestBase class.
To use InProcessBrowserTest do the following:
- Use the macro
IN_PROC_BROWSER_TEST_Fto define your test. - Your test method is invoked on the UI thread. If you need to block until state changes you'll need to run the message loop from your test method.
- If you subclass and override SetUp, be sure and invoke
InProcessBrowserTest::SetUp. (But see alsoSetUpOnMainThread,SetUpInProcessBrowserTestFixtureand other related hook methods for a cleaner alternative).
Cameo's developer is required to add test for any non-trivial code change. A effective and collaborate testing for your code is always a good practice when uploading a patch to trunk. Generally, a reviewer might reject a patch without testing code.
The trybots will run the tests firstly before checking in a patch into trunk. The patch is allowed to be checked in only if all tests are passed (Green).