get font collection tests

lib/experimental/fonts/font-library/class-wp-font-library.php

@@ -74,7 +74,7 @@ public static function register_font_collection( $id, $config ) {
  }
 
  /**
- * Gets the font collections available.
+ * Gets all the font collections available.
  *
  * @since 6.4.0
  *
@@ -84,6 +84,21 @@ public static function get_font_collections() {
  return self::$collections;
  }
 
+ /**
+ * Gets a font collection.
+ *
+ * @since 6.4.0
+ *
+ * @param string $id Font collection id.
+ * @return array List of font collections.
+ */
+ public static function get_font_collection( $id ) {
+ if ( array_key_exists( $id, self::$collections ) ) {
+ return self::$collections[ $id ];
+ }
+ return new WP_Error( 'font_collection_not_found', 'Font collection not found.' );
+ }
+
  /**
  * Gets the upload directory for fonts.
  *

lib/experimental/fonts/font-library/class-wp-rest-font-library-controller.php

@@ -102,16 +102,17 @@
  * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
  */
  public function get_font_collection( $request ) {
  $id = $request->get_param( 'id' );
- if ( ! array_key_exists( $id, WP_Font_Library::get_font_collections() ) ) {
+ $collection = WP_Font_Library::get_font_collection( $id );
+
+ if ( is_wp_error( $collection ) ) {
  return new WP_Error(
  'font_collection_not_found',
  __( 'Font collection not found.', 'gutenberg' ),
  array( 'status' => 404 )
  );
  }
-
- $collection = WP_Font_Library::get_font_collections()[ $id ];
+
  return new WP_REST_Response( $collection->get_data() );
  }
 
@@ -120,7 +121,7 @@
  *
  * @since 6.4.0
  *
  * @param WP_REST_Request $request Full details about the request.
  * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
  */
  public function get_font_collections() {

phpunit/fonts/font-library/wpFontLibrary/getFontCollection-test.php

@@ -0,0 +1,35 @@
+<?php
+/**
+ * Test WP_Font_Library::get_font_collections().
+ *
+ * @package WordPress
+ * @subpackage Font Library
+ *
+ * @group fonts
+ * @group font-library
+ *
+ * @covers WP_Font_Library::get_font_collection
+ */
+class Tests_Fonts_WpFontLibrary_GetFontCollection extends WP_UnitTestCase {
+
+ public static function set_up_before_class() {
+ $my_font_collection_config = array(
+ 'name' => 'My Font Collection',
+ 'description' => 'Demo about how to a font collection to your WordPress Font Library.',
+ 'data_json_file' => path_join( __DIR__, 'my-font-collection-data.json' ),
+ );
+
+ apply_filters( 'register_font_collection', 'my-font-collection', $my_font_collection_config );
+ }
+
+ public function test_should_get_font_collection() {
+ $font_collection = WP_Font_Library::get_font_collection( 'my-font-collection' );
+ $this->assertInstanceOf( 'WP_Font_Collection', $font_collection );
+ }
+
+ public function test_should_get_no_font_collection_if_the_id_is_not_registered() {
+ $font_collection = WP_Font_Library::get_font_collection( 'not-registered-font-collection' );
+ $this->assertWPError( $font_collection );
+ }
+
+}

phpunit/fonts/font-library/wpFontLibrary/getFontCollections-test.php

@@ -0,0 +1,44 @@
+<?php
+/**
+ * Test WP_Font_Library::get_font_collections().
+ *
+ * @package WordPress
+ * @subpackage Font Library
+ *
+ * @group fonts
+ * @group font-library
+ *
+ * @covers WP_Font_Library::get_font_collections
+ */
+class Tests_Fonts_WpFontLibrary_GetFontCollections extends WP_UnitTestCase {
+
+ public static function set_up_before_class() {
+ $my_font_collection_config = array(
+ 'name' => 'My Font Collection',
+ 'description' => 'Demo about how to a font collection to your WordPress Font Library.',
+ 'data_json_file' => path_join( __DIR__, 'my-font-collection-data.json' ),
+ );
+
+ apply_filters( 'register_font_collection', 'my-font-collection', $my_font_collection_config );
+
+ $another_font_collection_config = array(
+ 'name' => 'Another Font Collection',
+ 'description' => 'Demo about how to a font collection to your WordPress Font Library.',
+ 'data_json_file' => path_join( __DIR__, 'another-font-collection-data.json' ),
+ );
+
+ apply_filters( 'register_font_collection', 'another-font-collection', $another_font_collection_config );
+ }
+
+ public function test_should_get_font_collections() {
+ $font_collections = WP_Font_Library::get_font_collections();
+ $this->assertNotEmpty( $font_collections, 'Sould return an array of font collections.' );
+ $this->assertCount( 2, $font_collections, 'Should return an array with one font collection.' );
+
+ $this->assertArrayHasKey( 'my-font-collection', $font_collections, 'The array should have the key of the registered font collection id.' );
+ $this->assertInstanceOf( 'WP_Font_Collection', $font_collections['my-font-collection'], 'The value of the array $font_collections[id] should be an instance of WP_Font_Collection class.' );
+ $this->assertArrayHasKey( 'another-font-collection', $font_collections, 'The array should have the key of the registered font collection id.' );
+ $this->assertInstanceOf( 'WP_Font_Collection', $font_collections['another-font-collection'], 'The value of the array $font_collections[id] should be an instance of WP_Font_Collection class.' );
+ }
+
+}