WP_Fonts_Library: split tests

WordPress · Aug 16, 2023 · bd91b81 · bd91b81
1 parent b9d87ad
commit bd91b81
Show file tree

Hide file tree

Showing 4 changed files with 52 additions and 55 deletions.
diff --git a/phpunit/fonts-library/class-wp-fonts-library-test.php b/phpunit/fonts-library/class-wp-fonts-library-test.php
diff --git a/phpunit/fonts-library/wpFontFamily/uninstall-test.php b/phpunit/fonts-library/wpFontFamily/uninstall-test.php
@@ -22,10 +22,11 @@ public function test_should_return_error_when_font_not_found() {
 
 		// Test.
 		$actual = $font->uninstall();
-		$this->assertInstanceOf( WP_Error::class, $actual, 'WP_Error instance should have been returned' );
+		$this->assertWPError( $actual, 'WP_Error should have been returned' );
 		$this->assertSame(
 			array( 'font_family_not_found' => array( 'The font family could not be found.' ) ),
-			$actual->errors
+			$actual->errors,
+			'WP_Error should have "fonts_must_have_same_slug" error'
 		);
 	}
 
@@ -42,7 +43,7 @@ public function test_should_return_error_when_not_able_to_uninstall( $failure_to
 
 		// Test.
 		$actual = $font->uninstall();
-		$this->assertInstanceOf( WP_Error::class, $actual, 'WP_Error instance should be returned' );
+		$this->assertWPError( $actual, 'WP_Error should be returned' );
 		$this->assertSame(
 			array( 'font_family_not_deleted' => array( 'The font family could not be deleted.' ) ),
 			$actual->errors,

diff --git a/phpunit/fonts-library/wpFontsLibrary/getFontsDir-test.php b/phpunit/fonts-library/wpFontsLibrary/getFontsDir-test.php
@@ -0,0 +1,18 @@
+<?php
+/**
+ * Test WP_Fonts_Library::get_fonts_dir().
+ *
+ * @package WordPress
+ * @subpackage Fonts Library
+ *
+ * @group fonts
+ * @group fonts-library
+ *
+ * @covers WP_Fonts_Library::get_fonts_dir
+ */
+class Tests_Fonts_WpFontsLibrary_GetFontsDir extends WP_UnitTestCase {
+
+	public function test_get_fonts_dir() {
+		$this->assertStringEndsWith( '/wp-content/uploads/fonts', WP_Fonts_Library::get_fonts_dir() );
+	}
+}
diff --git a/phpunit/fonts-library/wpFontsLibrary/setUploadDir-test.php b/phpunit/fonts-library/wpFontsLibrary/setUploadDir-test.php
@@ -0,0 +1,30 @@
+<?php
+/**
+ * Test WP_Fonts_Library::set_upload_dir().
+ *
+ * @package WordPress
+ * @subpackage Fonts Library
+ *
+ * @group fonts
+ * @group fonts-library
+ *
+ * @covers WP_Fonts_Library::set_upload_dir
+ */
+class Tests_Fonts_WpFontsLibrary_SetUploadDir extends WP_UnitTestCase {
+
+	public function test_should_set_fonts_upload_dir() {
+		$defaults = array(
+			'subdir'  => '/abc',
+			'basedir' => '/var/www/html/wp-content/uploads',
+			'baseurl' => 'http://example.com/wp-content/uploads',
+		);
+		$expected = array(
+			'subdir'  => '/fonts',
+			'basedir' => '/var/www/html/wp-content/uploads',
+			'baseurl' => 'http://example.com/wp-content/uploads',
+			'path'    => '/var/www/html/wp-content/uploads/fonts',
+			'url'     => 'http://example.com/wp-content/uploads/fonts',
+		);
+		$this->assertSame( $expected, WP_Fonts_Library::set_upload_dir( $defaults ) );
+	}
+}