testcase and fix for issue #1462

jme3-core/src/main/java/com/jme3/scene/debug/WireFrustum.java

@@ -41,6 +41,12 @@ import java.nio.FloatBuffer;
 
 public class WireFrustum extends Mesh {
 
+    /**
+     * This constructor is for serialization only. Do not use.
+     */
+    protected WireFrustum() {
+    }
+
     public WireFrustum(Vector3f[] points){
         initGeom(this, points);
     }

jme3-core/src/test/java/com/jme3/scene/debug/TestCloneMesh.java

@@ -0,0 +1,77 @@
+/*
+ * Copyright (c) 2021 jMonkeyEngine
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ * * Redistributions of source code must retain the above copyright
+ *   notice, this list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright
+ *   notice, this list of conditions and the following disclaimer in the
+ *   documentation and/or other materials provided with the distribution.
+ *
+ * * Neither the name of 'jMonkeyEngine' nor the names of its contributors
+ *   may be used to endorse or promote products derived from this software
+ *   without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+ * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+package com.jme3.scene.debug;
+
+import com.jme3.asset.AssetManager;
+import com.jme3.asset.DesktopAssetManager;
+import com.jme3.export.binary.BinaryExporter;
+import com.jme3.math.Vector3f;
+import com.jme3.util.clone.Cloner;
+import org.junit.Assert;
+import org.junit.Test;
+
+/**
+ * Test cloning/saving/loading debug meshes of various types.
+ *
+ * @author Stephen Gold [email protected]
+ */
+public class TestCloneMesh {
+    // *************************************************************************
+    // fields
+
+    /**
+     * AssetManager required by the BinaryImporter
+     */
+    final private static AssetManager assetManager = new DesktopAssetManager();
+    // *************************************************************************
+    // new methods exposed
+
+    /**
+     * Test cloning/saving/loading a WireFrustum.
+     */
+    @Test
+    public void testIssue1462() {
+        Vector3f[] vertices = new Vector3f[8];
+        for (int vertexIndex = 0; vertexIndex < 8; vertexIndex++) {
+            vertices[vertexIndex] = new Vector3f();
+        }
+        WireFrustum wireFrustum = new WireFrustum(vertices);
+        WireFrustum deepClone = (WireFrustum) Cloner.deepClone(wireFrustum);
+        Assert.assertNotNull(deepClone);
+        Assert.assertNotEquals(deepClone, wireFrustum);
+
+        WireFrustum saveAndLoad
+                = BinaryExporter.saveAndLoad(assetManager, wireFrustum);
+        Assert.assertNotNull(saveAndLoad);
+        Assert.assertNotEquals(deepClone, saveAndLoad);
+    }
+}