add: proper roundoff

.eslintrc.json

@@ -21,6 +21,18 @@
     "semi": [
       "error",
       "always"
+    ],
+    "no-unused-vars": [
+      "error",
+      {
+        "args": "all",
+        "argsIgnorePattern": "^_",
+        "caughtErrors": "all",
+        "caughtErrorsIgnorePattern": "^_",
+        "destructuredArrayIgnorePattern": "^_",
+        "varsIgnorePattern": "^_",
+        "ignoreRestSiblings": true
+      }
     ]
   }
 }

script.js

@@ -103,16 +103,13 @@ if (typeof document !== 'undefined') {
           amount: parseFloat(row.querySelector('.item-amount').textContent) || 0
         });
       });
-      const subtotal = items.reduce((sum, i) => sum + i.amount, 0);
-      const igstAmount = (subtotal * igstRate) / 100;
-      const total = subtotal + igstAmount + freightCharges;
+      const { subtotal, igstAmount, _total, rOff, finalTotal } = calculateQuotation(items, igstRate, freightCharges);
+
       // convert total to words (Rupees and Paise)
-      const rupeePart = Math.floor(total);
-      const paisePart = Math.round((total - rupeePart) * 100);
+      const rupeePart = Math.floor(finalTotal);
+      const paisePart = Math.round((finalTotal - rupeePart) * 100);
       const rupeeWords = numberToWords(rupeePart);
       const paiseWords = paisePart > 0 ? numberToWords(paisePart) : '';
-      const rOff = 0; // Or get from a form field if you add one
-      const finalTotal = total - rOff;
 
       let html = `
         <div class="quotation-print">
@@ -234,7 +231,24 @@ if (typeof document !== 'undefined') {
   });
 }
 
+function calculateQuotation(items, igstRate, freightCharges) {
+  const subtotal = items.reduce((sum, i) => sum + i.amount, 0);
+  const igstAmount = (subtotal * igstRate) / 100;
+  const total = subtotal + igstAmount + freightCharges;
+  const totalBeforeRoundOff = total;
+  const finalTotal = Math.round(totalBeforeRoundOff);
+  const rOff = totalBeforeRoundOff - finalTotal;
+
+  return {
+    subtotal,
+    igstAmount,
+    total,
+    rOff,
+    finalTotal
+  };
+}
+
 // Export for testing (Node.js)
 if (typeof module !== 'undefined' && module.exports) {
-  module.exports = { numberToWords };
+  module.exports = { numberToWords, calculateQuotation };
 }

style.css

@@ -90,7 +90,10 @@ button {
 }
 
 .company-details {
-  font-size: 14px;
+  margin: 0;
+  font-size: 24px;
+  color: #036;
+  font-family: Arial, Helvetica, sans-serif;
 }
 
 .quotation-title h1 {

tests/test.js

@@ -1,5 +1,5 @@
 const assert = require('assert');
-const { numberToWords } = require('../script.js');
+const { numberToWords, calculateQuotation } = require('../script.js');
 
 // Test cases for Indian number-to-words conversion (Crore/Lakh system)
 const cases = [
@@ -31,3 +31,49 @@ cases.forEach(({ num, expected }) => {
 });
 
 console.log('✅ All numberToWords tests passed');
+
+// --- Tests for calculateQuotation ---
+
+const calculationCases = [
+  {
+    name: 'Basic case with one item',
+    items: [{ amount: 100 }],
+    igstRate: 18,
+    freightCharges: 50,
+    expected: { subtotal: 100, igstAmount: 18, total: 168, finalTotal: 168, rOff: 0 }
+  },
+  {
+    name: 'Multiple items',
+    items: [{ amount: 100 }, { amount: 250.50 }],
+    igstRate: 18,
+    freightCharges: 50,
+    expected: { subtotal: 350.50, igstAmount: 63.09, total: 463.59, finalTotal: 464, rOff: -0.41 }
+  },
+  {
+    name: 'No IGST or freight',
+    items: [{ amount: 500 }],
+    igstRate: 0,
+    freightCharges: 0,
+    expected: { subtotal: 500, igstAmount: 0, total: 500, finalTotal: 500, rOff: 0 }
+  },
+  {
+    name: 'Rounding down',
+    items: [{ amount: 99.20 }],
+    igstRate: 10, // 9.92
+    freightCharges: 0, // total = 109.12
+    expected: { subtotal: 99.20, igstAmount: 9.92, total: 109.12, finalTotal: 109, rOff: 0.12 }
+  }
+];
+
+calculationCases.forEach(({ name, items, igstRate, freightCharges, expected }) => {
+  const actual = calculateQuotation(items, igstRate, freightCharges);
+  // Need to compare floating point numbers with a tolerance
+  Object.keys(expected).forEach(key => {
+    assert(
+      Math.abs(actual[key] - expected[key]) < 0.001,
+      `[${name}] ${key} => ${actual[key]} (expected ${expected[key]})`
+    );
+  });
+});
+
+console.log('✅ All calculateQuotation tests passed');