fix: exclude tables if simple product (#55)

Author: dylandepass

src/templates/html/HTMLTemplate.js

@@ -234,7 +234,7 @@ ${attributes.map((attr) => /* html */`\
    * @returns {string}
    */
   renderProductOptions(options) {
-    return /* html */ `\
+    return options.length > 0 ? /* html */ `\
 <div class="product-options">
 ${options.map((opt) => /* html */ `\
   <div>
@@ -246,7 +246,7 @@ ${options.map((opt) => /* html */ `\
     <div>${opt.required === true ? 'required' : ''}</div>
   </div>
 ${HTMLTemplate.indent(this.renderProductItems(opt.items), 2)}`).join('\n')}
-</div>`;
+</div>` : '';
   }
 
   /**
@@ -280,7 +280,7 @@ ${HTMLTemplate.indent(this.renderProductItems(opt.items), 2)}`).join('\n')}
    * @returns {string}
    */
   renderProductVariants() {
-    if (!this.variants) {
+    if (!this.variants || this.variants.length === 0) {
       return '';
     }
 
@@ -306,7 +306,7 @@ ${HTMLTemplate.indent(this.renderVariantImages(v.images), 6)}
    * @returns {string}
    */
   renderProductVariantsAttributes() {
-    if (!this.variants) {
+    if (!this.variants || this.variants.length === 0) {
       return '';
     }
 

test/templates/html/index.test.js

@@ -233,6 +233,34 @@ describe('Render Product HTML', () => {
     });
   });
 
+  it('should not render product options table if there are none', () => {
+    product.options = [];
+    const html = htmlTemplateFromContext(DEFAULT_CONTEXT({ config }), product, variations).render();
+    dom = new JSDOM(html);
+    document = dom.window.document;
+
+    const optionsTable = document.querySelector('.product-options');
+    assert.strictEqual(optionsTable, null, 'Options table should not be rendered');
+  });
+
+  it('should not render product variants table if there are none', () => {
+    variations = [];
+    const html = htmlTemplateFromContext(DEFAULT_CONTEXT({ config }), product, variations).render();
+    dom = new JSDOM(html);
+    document = dom.window.document;
+
+    const variantsTable = document.querySelector('.product-variants');
+    assert.strictEqual(variantsTable, null, 'Variants table should not be rendered');
+  });
+
+  it('should not render variant attributes table if there are none', () => {
+    const html = htmlTemplateFromContext(DEFAULT_CONTEXT({ config }), product, []).render();
+    dom = new JSDOM(html);
+    document = dom.window.document;
+    const variantAttributesTable = document.querySelector('.variant-attributes');
+    assert.strictEqual(variantAttributesTable, null, 'Variant attributes table should not be rendered');
+  });
+
   it('template should allow for missing prices', () => {
     config.confMap = {
       '/us/p/{{urlkey}}/{{sku}}': {},