From 8f2838184ee8c08d7c9f78b9f5126816b655c539 Mon Sep 17 00:00:00 2001
From: Carlos Filoteo <c.filoteo12@gmail.com>
Date: Tue, 26 Jul 2022 12:59:08 -0600
Subject: [PATCH] Self-review

---
 src/single-spa-react.js | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/src/single-spa-react.js b/src/single-spa-react.js
index 8343ecc..632960f 100644
--- a/src/single-spa-react.js
+++ b/src/single-spa-react.js
@@ -221,29 +221,26 @@ function getRenderType(opts) {
 }
 
 function reactDomRender({ reactDom, renderType, elementToRender, domElement }) {
-  const renderRoot = reactDom[renderType];
-  if (typeof renderRoot !== "function")
-    throw new Error( // TODO: add tests for this block
-      `renderRoot returned from ${renderType} is not a function.`
-    );
+  const renderFn = reactDom[renderType];
+  if (typeof renderFn !== "function")
+    throw new Error(`renderType "${renderType}" did not return a function.`);
 
   switch (renderType) {
     case "createRoot":
     case "unstable_createRoot":
     case "createBlockingRoot":
     case "unstable_createBlockingRoot": {
-      const root = renderRoot(domElement);
+      const root = renderFn(domElement);
       root.render(elementToRender);
       return root;
     }
     case "hydrateRoot": {
-      const root = renderRoot(domElement, elementToRender);
+      const root = renderFn(domElement, elementToRender);
       return root;
     }
     case "hydrate":
     default: {
-      renderRoot(elementToRender, domElement);
-      //  TODO: add tests for this block
+      renderFn(elementToRender, domElement);
       // The renderRoot function should return a react root, but ReactDOM.hydrate() and ReactDOM.render()
       // do not return a react root. So instead, we return null which indicates that there is no react root
       // that can be used for updates or unmounting