Build/Test tools: In Travis, skip some tests when not on trunk.

johnbillion · johnbillion · commit 6e9b29e13ad8 · 2017-03-08T00:47:48.000Z
This skips time sensitive tests (copyright year and PHP/MySQL version requirements) when tests are run on branches on Travis. Props netweb, jorbin Fixes #39486 Merges [40241] to the 3.9 branch. git-svn-id: https://develop.svn.wordpress.org/branches/3.9@40248 602fd350-edb4-49c9-b593-d223f7449a82
diff --git a/tests/phpunit/includes/testcase.php b/tests/phpunit/includes/testcase.php
@@ -49,6 +49,27 @@ function clean_up_global_scope() {
 		$this->flush_cache();
 	}
 
+	/**
+	 * Allow tests to be skipped on some automated runs
+	 *
+	 * For test runs on Travis for something other than trunk/master 
+	 * we want to skip tests that only need to run for master.
+	 */
+	public function skipOnAutomatedBranches() {
+		// gentenv can be disabled
+		if ( ! function_exists( 'getenv' ) ) {
+			return false;
+		}
+
+		// https://docs.travis-ci.com/user/environment-variables/#Default-Environment-Variables
+		$travis_branch       = getenv( 'TRAVIS_BRANCH' );
+		$travis_pull_request = getenv( 'TRAVIS_PULL_REQUEST' );
+
+		if ( false !== $travis_pull_request && 'master' !== $travis_branch ) {
+			$this->markTestSkipped( 'For automated test runs, this test is only run on trunk/master' );
+		}
+	}
+
 	function flush_cache() {
 		global $wp_object_cache;
 		$wp_object_cache->group_ops = array();
diff --git a/tests/phpunit/tests/basic.php b/tests/phpunit/tests/basic.php
@@ -30,6 +30,9 @@ function test_readme() {
 	}
 
 	function test_license() {
+		// This test is designed to only run on trunk/master
+		$this->skipOnAutomatedBranches();
+
 		$license = file_get_contents( ABSPATH . 'license.txt' );
 		preg_match( '#Copyright (\d+) by the contributors#', $license, $matches );
 		$this_year = date( 'Y' );

Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,9 @@ function test_readme() {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`function test_license() {`
	`33`	`+ // This test is designed to only run on trunk/master`
	`34`	`+ $this->skipOnAutomatedBranches();`
	`35`	`+`
`33`	`36`	`$license = file_get_contents( ABSPATH . 'license.txt' );`
`34`	`37`	`preg_match( '#Copyright (\d+) by the contributors#', $license, $matches );`
`35`	`38`	`$this_year = date( 'Y' );`