refactor: ensure type safety of Cffu; if return type is Cffu, the…

… result is not a minimal stage 🦺 🧬

this type safety strategy is consistent with `CompletableFuture`:
if return type is `CompletableFuture`, the result is not a minimal stage

cffu-core/src/main/java/io/foldright/cffu/Cffu.java

@@ -46,7 +46,7 @@ private <U> Cffu<U> reset0(CompletableFuture<U> cf) {
     }
 
     @Contract(pure = true)
-    private <U> Cffu<U> resetToMin(CompletableFuture<U> cf) {
+    private <U> CompletionStage<U> resetToMin(CompletableFuture<U> cf) {
         return new Cffu<>(fac, true, cf);
     }
 

cffu-core/src/main/java/io/foldright/cffu/CffuFactory.java

@@ -73,7 +73,7 @@ private <T> Cffu<T> create(CompletableFuture<T> cf) {
     }
 
     @Contract(pure = true)
-    private <T> Cffu<T> createMin(CompletableFuture<T> cf) {
+    private <T> CompletionStage<T> createMin(CompletableFuture<T> cf) {
         return new Cffu<>(this, true, cf);
     }
 
@@ -245,32 +245,20 @@ public <T> Cffu<T> newIncompleteCffu() {
     }
 
     /**
-     * Wrap an existed {@link CompletableFuture} / {@link CompletionStage} / {@link Cffu} to {@link Cffu}.
-     * For {@link CompletableFuture} class instances,
-     * {@link Cffu#cffuUnwrap()} is the inverse operation to this method.
-     * <p>
-     * <strong>NOTE</strong>, keep input stage unchanged if possible when wrap:<br>
-     * <ol>
-     * <li>if input stage is a {@link Cffu}, re-wrapped with the config of
-     *     this {@link CffuFactory} by {@link Cffu#resetCffuFactory(CffuFactory)}.
-     * <li>if input stage is a CompletableFuture, wrap it by setting it as the underlying cf of returned cffu.
-     * <li>otherwise use input {@code stage.toCompletableFuture} as the underlying cf of returned cffu.
-     * </ol>
+     * Returns a Cffu maintaining the same completion properties as this stage and this {@code CffuFactory} config.
+     * If this stage is already a Cffu and have the same {@code CffuFactory}, this method may return this stage itself.
      *
      * @throws NullPointerException if the given stage is null
      * @see #toCffuArray(CompletionStage[])
      * @see CompletionStage#toCompletableFuture()
-     * @see Cffu#cffuUnwrap()
      * @see Cffu#resetCffuFactory(CffuFactory)
      */
     @Contract(pure = true)
     public <T> Cffu<T> toCffu(CompletionStage<T> stage) {
         requireNonNull(stage, "stage is null");
-        if (stage instanceof CompletableFuture) {
-            final CompletableFuture<T> cf = (CompletableFuture<T>) stage;
-            return CompletableFutureUtils.isMinStageCf(cf) ? createMin(cf) : create(cf);
-        } else if (stage instanceof Cffu) {
-            return ((Cffu<T>) stage).resetCffuFactory(this);
+        if (stage instanceof Cffu) {
+            Cffu<T> f = ((Cffu<T>) stage);
+            if (f.cffuFactory() == this && !f.isMinimalStage()) return f;
         }
         return create(stage.toCompletableFuture());
     }

cffu-core/src/test/java/io/foldright/cffu/CffuFactoryTest.java

@@ -143,12 +143,12 @@ void test_toCffu__for_factoryMethods_of_Java9() {
         shouldNotBeMinimalStage(cf1);
 
         Cffu<Integer> cf2 = cffuFactory.toCffu(CompletableFuture.completedStage(n));
-        assertTrue(cf2.isMinimalStage());
-        shouldBeMinimalStage(cf2);
+        assertFalse(cf2.isMinimalStage());
+        shouldNotBeMinimalStage(cf2);
 
         Cffu<Object> cf3 = cffuFactory.toCffu(CompletableFuture.failedStage(rte));
-        assertTrue(cf3.isMinimalStage());
-        shouldBeMinimalStage(cf3);
+        assertFalse(cf3.isMinimalStage());
+        shouldNotBeMinimalStage(cf3);
     }
 
     @Test
@@ -157,7 +157,7 @@ void test_toCffuArray() throws Exception {
         Cffu<Integer>[] cffus = cffuFactory.toCffuArray(CompletableFuture.completedStage(n), completedFuture(n));
         assertEquals(n, cffus[1].get());
 
-        shouldBeMinimalStage(cffus[0]);
+        shouldNotBeMinimalStage(cffus[0]);
         shouldNotBeMinimalStage(cffus[1]);
     }
 

cffu-core/src/test/java/io/foldright/cffu/CffuTest.java

@@ -338,7 +338,8 @@ void test_cffuUnwrap_9_completedStage() {
         CompletionStage<Integer> stage = CompletableFuture.completedStage(n);
         Cffu<Integer> cffu = cffuFactory.toCffu(stage);
 
-        assertSame(stage, cffu.cffuUnwrap());
+        assertNotSame(stage, cffu.cffuUnwrap());
+        assertEquals(n, cffu.join());
     }
 
     @Test