Never return "null" in getFederationBackend(s)

see #69 (comment)

Author: christophfriedrich

openeo.d.ts

@@ -410,19 +410,19 @@ declare module OpenEO {
          */
         listFederation(): Array<FederationBackend>;
         /**
-         * Given just the string ID of a backend within the federation, returns that backend's full details as a FederationBackend object
+         * Given just the string ID of a backend within the federation, returns that backend's full details as a FederationBackend object.
          * 
          * @param {string} backendId - The ID of a backend within the federation
-         * @returns {FederationBackend} The full details of the backend, or `null` if no backend with the given ID exists
+         * @returns {FederationBackend} The full details of the backend
          */
-        getFederationBackend(backendId: string): FederationBackend | null;
+        getFederationBackend(backendId: string): FederationBackend;
         /**
-         * Given a list of string IDs of backends within the federation, returns those backends' full details as FederationBackend objects
+         * Given a list of string IDs of backends within the federation, returns those backends' full details as FederationBackend objects.
          * 
          * @param {Array<string>} backendIds - The IDs of backends within the federation
-         * @returns {Array<FederationBackend>} An array in the same order as the input, containing for each position either the full details of the backend, or `null` if no backend with the given ID exists
+         * @returns {Array<FederationBackend>} An array in the same order as the input, containing for each position the full details of the backend
          */
-        getFederationBackends(backendIds: Array<string>): Array<FederationBackend | null>;
+        getFederationBackends(backendIds: Array<string>): Array<FederationBackend>;
         /**
          * Lists all supported features.
          *

src/capabilities.js

@@ -210,21 +210,22 @@ class Capabilities {
 	}
 
 	/**
-	 * Given just the string ID of a backend within the federation, returns that backend's full details as a FederationBackend object
+	 * Given just the string ID of a backend within the federation, returns that backend's full details as a FederationBackend object.
 	 * 
 	 * @param {string} backendId - The ID of a backend within the federation
-	 * @returns {FederationBackend | null} The full details of the backend, or `null` if no backend with the given ID exists
+	 * @returns {FederationBackend} The full details of the backend
 	 */
 	getFederationBackend(backendId) {
-		// Add `id` property to make it a proper FederationBackend object, but check for null case beforehand
-		return this.data.federation[backendId] ? { id: backendId, ...this.data.federation[backendId] } : null;
+		// Add `id` property to make it a proper FederationBackend object
+		// If backendId doesn't exist in this.data.federation, will contain just the `id` field (intended behaviour)
+		return { id: backendId, ...this.data.federation[backendId] }
 	}
 
 	/**
-	 * Given a list of string IDs of backends within the federation, returns those backends' full details as FederationBackend objects
+	 * Given a list of string IDs of backends within the federation, returns those backends' full details as FederationBackend objects.
 	 * 
 	 * @param {Array<string>} backendIds - The IDs of backends within the federation
-	 * @returns {Array<FederationBackend | null>} An array in the same order as the input, containing for each position either the full details of the backend, or `null` if no backend with the given ID exists
+	 * @returns {Array<FederationBackend>} An array in the same order as the input, containing for each position the full details of the backend
 	 */
 	getFederationBackends(backendIds) {
 		// Let 'single case' function do the work, but pass `this` so that `this.data.federation` can be accessed in the callback context