From: Kunal Mehta <legoktm@member.fsf.org>
Date: Wed, 30 Jan 2019 05:33:42 +0000 (-0800)
Subject: Use TestingAccessWrapper in FormOptionsInitializationTest
X-Git-Tag: 1.34.0-rc.0~2994
X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=commitdiff_plain;h=8ff88cc4d2128a940e1ae77dc44baf1cf6516b41

Use TestingAccessWrapper in FormOptionsInitializationTest

Avoids the need for an extra wrapper class just to make a property
public.

Change-Id: I08ae2fb24604a2bde352525abdcadf6251045f5b
---

diff --git a/tests/phpunit/includes/FormOptionsInitializationTest.php b/tests/phpunit/includes/FormOptionsInitializationTest.php
index 0c853e0833..2c78618aa1 100644
--- a/tests/phpunit/includes/FormOptionsInitializationTest.php
+++ b/tests/phpunit/includes/FormOptionsInitializationTest.php
@@ -1,23 +1,6 @@
 <?php
-/**
- * This file host two test case classes for the MediaWiki FormOptions class:
- *  - FormOptionsInitializationTest : tests initialization of the class.
- *  - FormOptionsTest : tests methods an on instance
- *
- * The split let us take advantage of setting up a fixture for the methods
- * tests.
- */
 
-/**
- * Dummy class to makes FormOptions::$options public.
- * Used by FormOptionsInitializationTest which need to verify the $options
- * array is correctly set through the FormOptions::add() function.
- */
-class FormOptionsExposed extends FormOptions {
-	public function getOptions() {
-		return $this->options;
-	}
-}
+use Wikimedia\TestingAccessWrapper;
 
 /**
  * Test class for FormOptions initialization
@@ -39,11 +22,11 @@ class FormOptionsInitializationTest extends MediaWikiTestCase {
 	 */
 	protected function setUp() {
 		parent::setUp();
-		$this->object = new FormOptionsExposed();
+		$this->object = TestingAccessWrapper::newFromObject( new FormOptions() );
 	}
 
 	/**
-	 * @covers FormOptionsExposed::add
+	 * @covers FormOptions::add
 	 */
 	public function testAddStringOption() {
 		$this->object->add( 'foo', 'string value' );
@@ -56,12 +39,12 @@ class FormOptionsInitializationTest extends MediaWikiTestCase {
 					'value' => null,
 				]
 			],
-			$this->object->getOptions()
+			$this->object->options
 		);
 	}
 
 	/**
-	 * @covers FormOptionsExposed::add
+	 * @covers FormOptions::add
 	 */
 	public function testAddIntegers() {
 		$this->object->add( 'one', 1 );
@@ -81,7 +64,7 @@ class FormOptionsInitializationTest extends MediaWikiTestCase {
 					'type' => FormOptions::INT,
 				]
 			],
-			$this->object->getOptions()
+			$this->object->options
 		);
 	}
 }