From 964976b5c546684dcb4ae2f269eeec7b1f61817a Mon Sep 17 00:00:00 2001
From: kishore4242 <kishoresachin5214715@gmail.com>
Date: Sat, 21 Mar 2026 16:49:37 +0530
Subject: [PATCH] Adding unit test cases for PShapeOBJ addMaterials() and
 rgbaValue()

---
 core/test/processing/core/PShapeOBJTest.java | 96 ++++++++++++++++++++
 1 file changed, 96 insertions(+)
 create mode 100644 core/test/processing/core/PShapeOBJTest.java

diff --git a/core/test/processing/core/PShapeOBJTest.java b/core/test/processing/core/PShapeOBJTest.java
new file mode 100644
index 000000000..065af52d6
--- /dev/null
+++ b/core/test/processing/core/PShapeOBJTest.java
@@ -0,0 +1,96 @@
+package processing.core;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+import java.util.ArrayList;
+import java.util.HashMap;
+
+/*
+* Unit tests for PShapeOBJ class
+* */
+public class PShapeOBJTest {
+
+    /*
+    * Test rgbaValue method with white color
+    * */
+    @Test
+    public void testRgbaValueWhiteColor() {
+        PVector whiteColor = new PVector(1.0f, 1.0f, 1.0f);
+        int rgba = PShapeOBJ.rgbaValue(whiteColor);
+
+        // White: 0xFF
+        Assert.assertEquals(0xFFFFFFFF, rgba);
+    }
+
+    /*
+    * Test addMaterial method
+    * */
+    @Test
+    public void testAddMaterials(){
+        ArrayList<PShapeOBJ.OBJMaterial> materialsList = new ArrayList<>();
+        HashMap<String, Integer> materialHash = new HashMap<>();
+
+        PShapeOBJ.OBJMaterial material = PShapeOBJ.addMaterial("testMaterial",materialsList, materialHash);
+
+        Assert.assertEquals("testMaterial",material.name);
+        Assert.assertEquals(1,materialHash.size());
+        Assert.assertEquals(1, materialsList.size());
+    }
+
+    /*
+    * Test addMaterial method with multiple value
+    * */
+    @Test
+    public void testAddMaterialsMultiple(){
+        ArrayList<PShapeOBJ.OBJMaterial> materialsList = new ArrayList<>();
+        HashMap<String, Integer> materialHash = new HashMap<>();
+
+        PShapeOBJ.OBJMaterial material1 = PShapeOBJ.addMaterial("testMaterial1",materialsList, materialHash);
+        PShapeOBJ.OBJMaterial material2 = PShapeOBJ.addMaterial("testMaterial2",materialsList, materialHash);
+        PShapeOBJ.OBJMaterial material3 = PShapeOBJ.addMaterial("testMaterial3",materialsList, materialHash);
+
+        // Checking the size
+        Assert.assertEquals(3,materialsList.size());
+        Assert.assertEquals(3,materialHash.size());
+
+        // Checking the material name
+        Assert.assertEquals("testMaterial1",material1.name);
+        Assert.assertEquals("testMaterial2",material2.name);
+        Assert.assertEquals("testMaterial3",material3.name);
+    }
+
+    /*
+    * Test addMaterial with null value
+    * */
+    @Test
+    public void testAddMaterialWithNullValue(){
+        ArrayList<PShapeOBJ.OBJMaterial> materialsList = new ArrayList<>();
+        HashMap<String, Integer> materialHash = new HashMap<>();
+
+        PShapeOBJ.OBJMaterial material1 = PShapeOBJ.addMaterial(null,materialsList, materialHash);
+        PShapeOBJ.OBJMaterial material2 = PShapeOBJ.addMaterial(null,materialsList, materialHash);
+
+        Assert.assertNull(materialsList.get(0).name);
+        // Checking null value
+        Assert.assertNotNull(materialHash.get(null));
+        Assert.assertEquals(1, materialHash.size());
+    }
+
+    /*
+    * Test addMaterial with duplicate value
+    * */
+    @Test
+    public void testAddMaterialDuplicateNameOverwritesHash(){
+        ArrayList<PShapeOBJ.OBJMaterial> materialsList = new ArrayList<>();
+        HashMap<String, Integer> materialHash = new HashMap<>();
+
+        PShapeOBJ.addMaterial("material", materialsList, materialHash);
+        PShapeOBJ.addMaterial("material", materialsList, materialHash);
+
+        Assert.assertEquals(2, materialsList.size());
+        Assert.assertEquals(1, materialHash.size());
+
+        Assert.assertEquals(1, materialHash.get("material").intValue());
+    }
+}