Replace duplicate test assertion

ampproject · Aug 30, 2018 · d73ab0d · d73ab0d
1 parent 1870954
commit d73ab0d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/tests/test-class-amp-http.php b/tests/test-class-amp-http.php
@@ -175,22 +175,22 @@ public function test_send_cors_headers() {
 		AMP_HTTP::send_cors_headers();
 		$this->assertEmpty( AMP_HTTP::$headers_sent );
 
-		// Origin header is not allowed.
+		// Try an invalid Origin header.
 		AMP_HTTP::$headers_sent          = array();
 		AMP_HTTP::$purged_amp_query_vars = array();
 		$_SERVER['HTTP_ORIGIN']          = 'https://evil.example.com';
 		AMP_HTTP::send_cors_headers();
 		$this->assertEmpty( AMP_HTTP::$headers_sent );
 
-		// The __amp_source_origin is not allowed.
+		// Try an invalid __amp_source_origin.
 		AMP_HTTP::$headers_sent          = array();
 		AMP_HTTP::$purged_amp_query_vars = array();
 		unset( $_SERVER['HTTP_ORIGIN'] );
 		$_GET['__amp_source_origin'] = 'https://evil.example.com';
 		AMP_HTTP::send_cors_headers();
 		$this->assertEmpty( AMP_HTTP::$headers_sent );
 
-		// The origin header is allowed.
+		// Try an allowed Origin header.
 		AMP_HTTP::$headers_sent          = array();
 		AMP_HTTP::$purged_amp_query_vars = array();
 		$_SERVER['HTTP_ORIGIN']          = home_url();

diff --git a/tests/test-class-amp-theme-support.php b/tests/test-class-amp-theme-support.php
@@ -63,7 +63,7 @@ public function test_init() {
 		$_GET['__amp_source_origin']     = 'foo';
 		AMP_Theme_Support::init();
 		$this->assertFalse( has_action( 'widgets_init', array( self::TESTED_CLASS, 'register_widgets' ) ) );
-		$this->assertFalse( has_action( 'widgets_init', array( self::TESTED_CLASS, 'register_widgets' ) ) );
+		$this->assertFalse( has_action( 'wp', array( self::TESTED_CLASS, 'finish_init' ) ) );
 
 		add_theme_support( 'amp' );
 		AMP_Theme_Support::init();