tbianchini/shopit-back
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'refactor/inventory_stock' into homo

Browse Source
This commit is contained in:
ncoronel
2026-10-01 10:00:41 -03:00
parent c7fc867905 fb5221854c
commit e2cf53eacc
36 changed files with 841 additions and 179 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/Feature/FiestaFutbolInfantil/AccommodationControllerTest.php
View File

@@ -97,6 +97,8 @@ class AccommodationControllerTest extends TestCase
$updated = $this->variantPayload('Casa Rodante Premium', 'Con electricidad', 15, 50000);
$updated['id'] = $variantId;
unset($updated['stock']);
$updated['stock_difference'] = -5;
$this->postJson('/api/v1/adminapp/tenant/accommodations', [
'variants' => [

8
tests/Feature/FiestaFutbolInfantil/EntryControllerTest.php
View File

@@ -194,7 +194,7 @@ class EntryControllerTest extends TestCase
'title' => 'Abono actualizado',
'description' => 'Ahora incluye ambas fechas',
'event_date_ids' => [$firstDate->id, $secondDate->id],
'stock' => 20,
'stock_difference' => 10,
'price' => 250,
],
[
@@ -268,7 +268,7 @@ class EntryControllerTest extends TestCase
'id' => $entryId,
'title' => 'Abono editado',
'event_date_ids' => [$date->id, $otherDate->id],
'stock' => 30,
'stock_difference' => 10,
'price' => 200,
]],
])->assertOk()
@@ -329,7 +329,7 @@ class EntryControllerTest extends TestCase
'title' => 'Entrada inválida',
'description' => null,
'event_date_ids' => [$foreignDate->id],
'stock' => 10,
'stock_difference' => 0,
'price' => 100,
]],
])
@@ -371,7 +371,7 @@ class EntryControllerTest extends TestCase
'title' => 'Entrada inválida',
'description' => null,
'event_date_ids' => [$eventDate->id],
'stock' => 10,
'stock_difference' => 0,
'price' => 100,
]],
])

6
tests/Feature/FiestaFutbolInfantil/FoodControllerTest.php
View File

@@ -124,6 +124,8 @@ class FoodControllerTest extends TestCase
$updated = $this->variantPayload($secondDate->id, 'Cena', 'Vianda', 80, 8500);
$updated['id'] = $variantId;
unset($updated['stock']);
$updated['stock_difference'] = -20;
$updated['description'] = 'Cena para llevar';
$this->postJson('/api/v1/adminapp/tenant/foods', [
@@ -240,7 +242,7 @@ class FoodControllerTest extends TestCase
Inventory::query()->whereKey($replacementInventoryId)->delete();
$updated = $this->patchJson('/api/v1/adminapp/tenant/foods/history-stock', [
'variants' => [['id' => $historicalVariantId, 'stock' => 45]],
'variants' => [['id' => $historicalVariantId, 'stock_difference' => -55]],
])
->assertOk()
->assertJsonPath('data.history.0.variants.0.id', $historicalVariantId)
@@ -258,7 +260,7 @@ class FoodControllerTest extends TestCase
]);
$this->patchJson('/api/v1/adminapp/tenant/foods/history-stock', [
'variants' => [['id' => $activeVariantId, 'stock' => 20]],
'variants' => [['id' => $activeVariantId, 'stock_difference' => -60]],
])
->assertUnprocessable()
->assertJsonValidationErrors(['variants.0.id']);

4
tests/Feature/FiestaFutbolInfantil/MerchandiseControllerTest.php
View File

@@ -112,6 +112,8 @@ class MerchandiseControllerTest extends TestCase
$updatedVariant = $this->variantPayload('Blanco', 'M', 80, 12500);
$updatedVariant['id'] = $variantId;
unset($updatedVariant['stock']);
$updatedVariant['stock_difference'] = -20;
$updatedItem = $this->itemPayload('Camiseta oficial', 2, [
$updatedVariant,
$this->variantPayload('Verde', 'L', 60, 15000),
@@ -234,6 +236,8 @@ class MerchandiseControllerTest extends TestCase
$secondItemVariantId = $created->json('data.1.variants.0.id');
$foreignVariant = $this->variantPayload('Azul Marino', 'XL', 20, 30000);
$foreignVariant['id'] = $secondItemVariantId;
unset($foreignVariant['stock']);
$foreignVariant['stock_difference'] = 0;
$firstItem = $this->itemPayload('Camiseta', 3, [$foreignVariant]);
$firstItem['id'] = $firstItemId;

139
tests/Feature/Migrations/InventoryStockMovementMigrationTest.php Normal file
View File

@@ -0,0 +1,139 @@
<?php
namespace Tests\Feature\Migrations;
use App\Domains\Commerce\Catalog\Enums\InventoryMovementOperation;
use App\Domains\Commerce\Catalog\Models\Inventory;
use App\Domains\Commerce\Catalog\Models\InventoryMovement;
use App\Domains\Core\Auth\Models\User;
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Schema;
use Tests\TestCase;
class InventoryStockMovementMigrationTest extends TestCase
{
protected function setUp(): void
{
parent::setUp();
Schema::create('users', function (Blueprint $table): void {
$table->id();
$table->softDeletes();
});
Schema::create('inventories', function (Blueprint $table): void {
$table->id();
$table->unsignedBigInteger('sold_units')->default(0);
$table->unsignedInteger('reserved_stock')->default(0);
$table->unsignedInteger('real_stock')->default(0);
$table->unsignedBigInteger('refunded_units')->default(0);
$table->unsignedInteger('entry_reserved_stock')->default(0);
});
}
protected function tearDown(): void
{
Schema::disableForeignKeyConstraints();
Schema::dropIfExists('inventory_movements');
Schema::dropIfExists('inventories');
Schema::dropIfExists('users');
Schema::enableForeignKeyConstraints();
parent::tearDown();
}
public function test_migration_and_rollback_preserve_the_available_balance(): void
{
DB::table('inventories')->insert([
'id' => 17,
'real_stock' => 10,
'sold_units' => 3,
'refunded_units' => 1,
'reserved_stock' => 2,
'entry_reserved_stock' => 1,
]);
$stockMigration = $this->stockMigration();
$movementMigration = $this->movementMigration();
$stockMigration->up();
$movementMigration->up();
$inventory = Inventory::findOrFail(17);
$this->assertSame(12, $inventory->real_stock);
$this->assertSame(7, $inventory->availableStock());
$this->assertTrue(Schema::hasTable('inventory_movements'));
$inventory->adjustAvailableStock(2);
$this->assertSame(9, $inventory->availableStock());
$movementMigration->down();
$stockMigration->down();
$this->assertFalse(Schema::hasColumn('inventories', 'available_stock'));
$this->assertFalse(Schema::hasTable('inventory_movements'));
$this->assertSame(12, Inventory::findOrFail(17)->real_stock);
$this->assertSame(9, Inventory::findOrFail(17)->availableStock());
}
public function test_movements_capture_counters_actor_and_idempotency(): void
{
$this->stockMigration()->up();
$this->movementMigration()->up();
DB::table('users')->insert(['id' => 5]);
$user = User::query()->findOrFail(5);
$inventory = Inventory::query()->create(['real_stock' => 10]);
$inventory->recordInitialization($user);
$inventory->adjustAvailableStock(3, $user, '955d2afb-1e36-46df-a9db-d9ee31ed5eb4');
$inventory->adjustAvailableStock(3, $user, '955d2afb-1e36-46df-a9db-d9ee31ed5eb4');
$inventory->reserve(2, true);
$inventory->buy(2, true);
$inventory->refundStock(1, $user);
$this->assertSame(12, $inventory->availableStock());
$this->assertCount(5, $inventory->movements()->get());
$adjustment = InventoryMovement::query()
->where('operation', InventoryMovementOperation::StockIncreased->value)
->firstOrFail();
$this->assertSame(3, $adjustment->available_stock_delta);
$this->assertSame(10, $adjustment->available_stock_before);
$this->assertSame(13, $adjustment->available_stock_after);
$this->assertSame(['real_stock' => 3], $adjustment->counter_deltas);
$this->assertSame(5, $adjustment->responsible_user_id);
$purchase = InventoryMovement::query()
->where('operation', InventoryMovementOperation::PurchaseCommitted->value)
->firstOrFail();
$this->assertSame(0, $purchase->available_stock_delta);
$this->assertSame(['reserved_stock' => -2, 'sold_units' => 2], $purchase->counter_deltas);
}
public function test_migration_rejects_an_inventory_with_negative_available_stock(): void
{
DB::table('inventories')->insert([
'id' => 23,
'real_stock' => 1,
'reserved_stock' => 2,
]);
$this->expectException(\RuntimeException::class);
$this->expectExceptionMessage(
'stock disponible actual es negativo. Inventarios detectados (máximo 20): '
.'id=23 [real=1, vendidas=0, reintegradas=0, reservadas=2, reservas_entradas=0, resultado=-1]'
);
$this->stockMigration()->up();
}
private function stockMigration(): Migration
{
return require database_path('migrations/2026_09_25_000000_generate_available_inventory_stock.php');
}
private function movementMigration(): Migration
{
return require database_path('migrations/2026_09_25_010000_create_inventory_movements_table.php');
}
}