Merge pull request #58 from Yoast/stories/fix-path-assets-class

fix path and test that has the same bug

Author: kraftner

inc/class-yoast-acf-analysis-assets.php

@@ -13,7 +13,7 @@ class Yoast_ACF_Analysis_Assets {
 	 * Initialize.
 	 */
 	public function init() {
-		$this->plugin_data = get_plugin_data( dirname( AC_SEO_ACF_ANALYSIS_PLUGIN_FILE ) );
+		$this->plugin_data = get_plugin_data( AC_SEO_ACF_ANALYSIS_PLUGIN_FILE );
 
 		add_filter( 'admin_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
 	}

tests/php/unit/AssetsTest.php

@@ -18,8 +18,12 @@ public function testInitHook() {
 		define( 'AC_SEO_ACF_ANALYSIS_PLUGIN_FILE', '/directory/file' );
 		Functions\expect( 'get_plugin_data' )
 			->once()
-			->with( dirname( AC_SEO_ACF_ANALYSIS_PLUGIN_FILE ) )
-			->andReturn( array( 'Version' => '1.0.0' ) );
+			->with( AC_SEO_ACF_ANALYSIS_PLUGIN_FILE )
+			->andReturn(
+				array(
+					'Version' => '2.0.0',
+				)
+			);
 
 		$testee = new \Yoast_ACF_Analysis_Assets();
 		$testee->init();