Add Configuration Builder (and tests) (#146)

* Add EditorConfigurationBuilder

* Add EditorConfigurationBuilderTests

* Make Test Suite run on macOS

* docs: Document all editor configuration options

* test: Expand builder tests

* test: Fix EditorManifestTests for macOS runs

Apply iOS conditionals used in other, existing code.

* ci: Revert Swift test script device configuration removal

This was changed while making the tests pass when running on macOS, but
it is unclear why the original CI script was changed. Presumably we
should run the tests with a fixed device on the CI.

debc1c7#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52L62

* test: Remove builder cookie assertions

This functionality was removed in the trunk branch.

---------

Co-authored-by: David Calhoun <[email protected]>

Author: jkmassel

Makefile

@@ -47,6 +47,9 @@ lint-js: npm-dependencies
 test-js: npm-dependencies
 	npm run test -- run
 
+lint-swift:
+	swift package plugin swiftlint
+
 local-android-library: build
 	echo "--- :android: Building Library"
 	./android/gradlew -p ./android :gutenberg:publishToMavenLocal -exclude-task prepareToPublishToS3

ios/Demo-iOS/Sources/ContentView.swift

@@ -66,20 +66,20 @@ struct ContentView: View {
 private extension EditorConfiguration {
 
     static var template: Self {
-        var configuration = EditorConfiguration.default
-
-        #warning("1. Update the property values below")
+        #warning("1. Update the siteURL and authHeader values below")
         #warning("2. Install the Jetpack plugin to the site")
-        configuration.siteURL = "https://modify-me.com"
-        configuration.authHeader = "Insert the Authorization header value here"
+        let siteUrl: String = "https://modify-me.com"
+        let authHeader: String = "Insert the Authorization header value here"
+        let siteApiRoot: String = "\(siteUrl)/wp-json/"
 
-        // DO NOT CHANGE the properties below
-        configuration.siteApiRoot = "\(configuration.siteURL)/wp-json/"
-        configuration.editorAssetsEndpoint = URL(string: configuration.siteApiRoot)!.appendingPathComponent("wpcom/v2/editor-assets")
-        // The `plugins: true` is necessary for the editor to use 'remote.html'
-        configuration.plugins = true
+        let configuration = EditorConfigurationBuilder()
+            .setSiteUrl(siteUrl)
+            .setAuthHeader(authHeader)
+            .setSiteApiRoot(siteApiRoot)
+            .setEditorAssetsEndpoint(URL(string: siteApiRoot)!.appendingPathComponent("wpcom/v2/editor-assets"))
+            .setShouldUsePlugins(true)
 
-        return configuration
+        return configuration.build()
     }
 
 }

ios/Sources/GutenbergKit/Sources/Cache/EditorAssetsLibrary.swift

@@ -245,7 +245,11 @@ struct EditorAssetsMainifest: Codable {
         for script in try document.select("script[src]") {
             if let src = try? script.attr("src") {
                 let link = Self.resolveAssetLink(src, defaultScheme: defaultScheme)
+                #if canImport(UIKit)
                 let newLink = CachedAssetSchemeHandler.cachedURL(forWebLink: link) ?? link
+                #else
+                let newLink = link
+                #endif
                 try script.attr("src", newLink)
             }
         }
@@ -268,7 +272,11 @@ struct EditorAssetsMainifest: Codable {
         for stylesheet in try document.select(#"link[rel="stylesheet"][href]"#) {
             if let href = try? stylesheet.attr("href") {
                 let link = Self.resolveAssetLink(href, defaultScheme: defaultScheme)
+                #if canImport(UIKit)
                 let newLink = CachedAssetSchemeHandler.cachedURL(forWebLink: link) ?? link
+                #else
+                let newLink = link
+                #endif
                 try stylesheet.attr("href", newLink)
             }
         }

ios/Sources/GutenbergKit/Sources/EditorBlockPicker.swift

@@ -1,5 +1,6 @@
 import SwiftUI
 
+#if canImport(UIKit)
 // TODO: Add search
 // TODO: Group these properly
 struct EditorBlockPicker: View {
@@ -217,3 +218,4 @@ struct EditorBlockPickerSection: Identifiable {
     let name: String
     let blockTypes: [EditorBlockType]
 }
+#endif