From 207c979f7198f9a4bd872deb5b5485442a81fce9 Mon Sep 17 00:00:00 2001
From: ncoronel <ncoronel@quo.ar>
Date: Tue, 18 Aug 2026 08:51:33 -0300
Subject: [PATCH] feat(product-ticket-selector): simplify row selection
 handling and improve test coverage for variant selection

---
 .../product-ticket-selector.component.html                  | 2 +-
 .../product-ticket-selector.component.spec.ts               | 6 ++++++
 .../product-ticket-selector.component.ts                    | 6 +-----
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.html b/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.html
index 56915eb..f8d68f0 100644
--- a/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.html
+++ b/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.html
@@ -27,7 +27,7 @@
               <select
                 class="form-select variant-selector__select"
                 [attr.aria-label]="selector.label"
-                [disabled]="rowDisabled(row) || !selector.enabled"
+                [disabled]="rowDisabled(row)"
                 [value]="selectedOptionKey(row, selector.key)"
                 (change)="onSelectionKeyChange(row.id, selector, $any($event.target).value)"
               >
diff --git a/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.spec.ts b/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.spec.ts
index df2f903..64b24c5 100644
--- a/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.spec.ts
+++ b/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.spec.ts
@@ -133,9 +133,15 @@ describe('ProductTicketSelectorComponent', () => {
     fixture.componentRef.setInput('title', 'Entrada');
     fixture.detectChanges();
     await fixture.whenStable();
+    fixture.detectChanges();
 
     expect(catalogService.getVariantOptions).toHaveBeenCalledTimes(1);
     expect(catalogService.withoutLoading).toHaveBeenCalled();
+    expect(
+      Array.from<HTMLSelectElement>(fixture.nativeElement.querySelectorAll('select')).every(
+        (select) => !select.disabled,
+      ),
+    ).toBe(true);
 
     fixture.componentInstance['onSelectionChange'](1, 'sector', {
       value: 'a',
diff --git a/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.ts b/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.ts
index 219e9d1..31058e8 100644
--- a/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.ts
+++ b/src/app/shared/components/product-ticket-selector/product-ticket-selector.component.ts
@@ -206,11 +206,7 @@ export class ProductTicketSelectorComponent {
     const row = this.findRow(rowId);
     if (!row || this.isBusy(row)) return;
 
-    const selectorIndex = row.selectors.findIndex((selector) => selector.key === selectorKey);
-    const retainedKeys = new Set(row.selectors.slice(0, selectorIndex + 1).map(({ key }) => key));
-    const selectedValues = Object.fromEntries(
-      Object.entries(row.selectedValues).filter(([key]) => retainedKeys.has(key)),
-    );
+    const selectedValues = { ...row.selectedValues };
 
     if (value === null) delete selectedValues[selectorKey];
     else selectedValues[selectorKey] = this.normalizeValue(value);