remove a debug comment and updated unit test for p5.color.js

Author: Piyushrathoree

src/color/p5.Color.js

@@ -331,10 +331,7 @@ class Color {
       colorString = serialize(this._color, {
         format: outputFormat
       });
-    // // DEBUG LOG
-    //   if (format === '#rrggbb') {
-    //      console.log('DEBUG:', format, 'Input:', this._color, 'Got:', colorString);
-    //   }
+      
       if (format === '#rrggbb') {
         colorString = String(colorString);
 

test/unit/color/p5.Color.js

@@ -791,15 +791,58 @@ suite('p5.Color', function() {
     });
   });
 
-  suite.todo('p5.Color.prototype.toString', function() {
-    // var colorStr;
+  suite('p5.Color.prototype.toString', function() {
+    suite('#rrggbb format', function() {
+      test('should expand short hex (#rgb) to full 6-digit format', function() {
+        mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
+        const c = mockP5Prototype.color('#f06');
+        const result = c.toString('#rrggbb');
+        assert.equal(result, '#ff0066');
+      });
+
+      test('should truncate hex with alpha (#rrggbbaa) to 6 digits', function() {
+        mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
+        const c = mockP5Prototype.color(255, 0, 102, 128);
+        const result = c.toString('#rrggbb');
+        assert.equal(result.length, 7);
+        assert.equal(result, '#ff0066');
+      });
+
+      test('should output lowercase hex string', function() {
+        mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
+        const c = mockP5Prototype.color(255, 170, 187);
+        const result = c.toString('#rrggbb');
+        assert.equal(result, result.toLowerCase());
+        assert.equal(result, '#ffaabb');
+      });
+
+      test('should correctly format standard RGB colors', function() {
+        mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
+        const c = mockP5Prototype.color(255, 0, 102);
+        const result = c.toString('#rrggbb');
+        assert.equal(result, '#ff0066');
+      });
+
+      test('should correctly format black color', function() {
+        mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
+        const c = mockP5Prototype.color(0, 0, 0);
+        const result = c.toString('#rrggbb');
+        assert.equal(result, '#000000');
+      });
 
-    // beforeEach(function() {
-    //   mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
-    //   c = mockP5Prototype.color(128, 0, 128, 128);
-    //   colorStr = c.toString();
-    // });
+      test('should correctly format white color', function() {
+        mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
+        const c = mockP5Prototype.color(255, 255, 255);
+        const result = c.toString('#rrggbb');
+        assert.equal(result, '#ffffff');
+      });
 
-    // NOTE: need some tests here
+      test('should handle colors created from 6-digit hex string', function() {
+        mockP5Prototype.colorMode(mockP5Prototype.RGB, 255, 255, 255, 255);
+        const c = mockP5Prototype.color('#9932cc');
+        const result = c.toString('#rrggbb');
+        assert.equal(result, '#9932cc');
+      });
+    });
   });
 });