Fix unfrozen open position bug

src/lib/trade-executor/state/position-info.test.ts

@@ -81,3 +81,28 @@ describe('closed position with stats entries', () => {
 		expect(spy).toHaveBeenCalledOnce();
 	});
 });
+
+describe('position with frozen_at value', () => {
+	const frozenPosition = { ...position, frozen_at: new Date() };
+	const frozenPositionInfo = createTradingPositionInfo(frozenPosition);
+
+	test('should be frozen', () => {
+		expect(frozenPositionInfo.frozen).toBeTruthy();
+	});
+
+	test('should not be stillOpen', () => {
+		expect(frozenPositionInfo.stillOpen).toBeFalsy();
+	});
+
+	describe('and unfrozen_at value', () => {
+		const unfrozenPositionInfo = createTradingPositionInfo({ ...frozenPosition, unfrozen_at: new Date() });
+
+		test('should no longer be frozen', () => {
+			expect(unfrozenPositionInfo.frozen).toBeFalsy();
+		});
+
+		test('should be stillOpen', () => {
+			expect(unfrozenPositionInfo.stillOpen).toBeTruthy();
+		});
+	});
+});

src/lib/trade-executor/state/position-info.ts

@@ -106,7 +106,7 @@ const tradingPositionInfoPrototype = {
 	},
 
 	get stillOpen() {
-		return this.closed_at == null && this.frozen_at == null;
+		return !this.closed && !this.frozen;
 	},
 
 	get frozen() {

src/routes/strategies/[strategy]/[status=positionStatus]-positions/PositionTable.svelte

@@ -25,7 +25,7 @@
 	$: positionsStore.set(positions);
 
 	const statusColumns = {
-		open: ['description', 'flags', 'profitability', 'value', 'opened_at', 'cta'],
+		open: ['description', 'flags', 'profitability', 'current_value', 'opened_at', 'cta'],
 		closed: ['description', 'flags', 'profitability', 'value_at_open', 'closed_at', 'cta'],
 		frozen: ['description', 'flags', 'frozen_on', 'frozen_value', 'frozen_at', 'cta']
 	};
@@ -76,7 +76,8 @@
 		}),
 		table.column({
 			header: 'Value',
-			accessor: 'value',
+			id: 'current_value',
+			accessor: 'currentValue',
 			cell: ({ value }) => formatDollar(value)
 		}),
 		table.column({
@@ -147,7 +148,7 @@
 			padding-block: 0;
 		}
 
-		:is(.profitability, .value, .value_at_open, .frozen_value, .opened_at, .closed_at, .frozen_at) {
+		:is(.profitability, .current_value, .value_at_open, .frozen_value, .opened_at, .closed_at, .frozen_at) {
 			text-align: right;
 		}