dmolinari/SIG-CM2.0
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

ADM-009: Tablas Fiscales (IVA + IIBB) — append-only versioned ref data #22

Merged
dmolinari merged 36 commits from feature/ADM-009 into main 2026-04-18 11:45:13 +00:00
Conversation 0 Commits 36 Files Changed 129 +114 -66
2 changed files with 114 additions and 66 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 8c08a706f0 - Show all commits

131
tests/SIGCM2.Api.Tests/Admin/V014MigrationTests.cs
View File

@@ -93,6 +93,8 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
} }
// ── REQ-SEED-001 ────────────────────────────────────────────────────────── // ── REQ-SEED-001 ──────────────────────────────────────────────────────────
// NOTE: Filters use Codigo IN (...) + PredecesorId IS NULL + VigenciaDesde='2020-01-01'
// to isolate ONLY the 4 canonical seed rows, ignoring rows inserted by repo integration tests.
[Fact] [Fact]
public async Task TipoDeIva_Seed_HasExactly4Rows() public async Task TipoDeIva_Seed_HasExactly4Rows()
@@ -100,8 +102,12 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
await using var conn = new SqlConnection(ConnectionString); await using var conn = new SqlConnection(ConnectionString);
await conn.OpenAsync(); await conn.OpenAsync();
var count = await conn.ExecuteScalarAsync<int>( var count = await conn.ExecuteScalarAsync<int>("""
"SELECT COUNT(1) FROM dbo.TipoDeIva"); SELECT COUNT(1) FROM dbo.TipoDeIva
WHERE Codigo IN ('EXENTO', 'NO_GRAVADO', 'IVA_105', 'IVA_21')
AND PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
""");
count.Should().Be(4, "El seed de V014 debe generar exactamente 4 TipoDeIva canónicos"); count.Should().Be(4, "El seed de V014 debe generar exactamente 4 TipoDeIva canónicos");
} }
@@ -112,8 +118,13 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
await using var conn = new SqlConnection(ConnectionString); await using var conn = new SqlConnection(ConnectionString);
await conn.OpenAsync(); await conn.OpenAsync();
var codigos = (await conn.QueryAsync<string>( var codigos = (await conn.QueryAsync<string>("""
"SELECT Codigo FROM dbo.TipoDeIva ORDER BY Codigo")).ToList(); SELECT Codigo FROM dbo.TipoDeIva
WHERE Codigo IN ('EXENTO', 'NO_GRAVADO', 'IVA_105', 'IVA_21')
AND PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
ORDER BY Codigo
""")).ToList();
codigos.Should().BeEquivalentTo( codigos.Should().BeEquivalentTo(
new[] { "EXENTO", "IVA_105", "IVA_21", "NO_GRAVADO" }, new[] { "EXENTO", "IVA_105", "IVA_21", "NO_GRAVADO" },
@@ -126,8 +137,13 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
await using var conn = new SqlConnection(ConnectionString); await using var conn = new SqlConnection(ConnectionString);
await conn.OpenAsync(); await conn.OpenAsync();
var rows = (await conn.QueryAsync<(string Codigo, decimal Porcentaje)>( var rows = (await conn.QueryAsync<(string Codigo, decimal Porcentaje)>("""
"SELECT Codigo, Porcentaje FROM dbo.TipoDeIva ORDER BY Codigo")).ToList(); SELECT Codigo, Porcentaje FROM dbo.TipoDeIva
WHERE Codigo IN ('EXENTO', 'NO_GRAVADO', 'IVA_105', 'IVA_21')
AND PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
ORDER BY Codigo
""")).ToList();
rows.Should().ContainSingle(r => r.Codigo == "EXENTO" && r.Porcentaje == 0m); rows.Should().ContainSingle(r => r.Codigo == "EXENTO" && r.Porcentaje == 0m);
rows.Should().ContainSingle(r => r.Codigo == "NO_GRAVADO" && r.Porcentaje == 0m); rows.Should().ContainSingle(r => r.Codigo == "NO_GRAVADO" && r.Porcentaje == 0m);
@@ -143,16 +159,20 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
var invalidRows = await conn.ExecuteScalarAsync<int>(""" var invalidRows = await conn.ExecuteScalarAsync<int>("""
SELECT COUNT(1) FROM dbo.TipoDeIva SELECT COUNT(1) FROM dbo.TipoDeIva
WHERE Activo = 0 WHERE Codigo IN ('EXENTO', 'NO_GRAVADO', 'IVA_105', 'IVA_21')
OR PredecesorId IS NOT NULL AND VigenciaDesde = CAST('2020-01-01' AS DATE)
OR VigenciaHasta IS NOT NULL AND PredecesorId IS NULL
AND (Activo = 0 OR VigenciaHasta IS NOT NULL)
"""); """);
invalidRows.Should().Be(0, invalidRows.Should().Be(0,
"Todas las filas seed deben tener Activo=1, PredecesorId=NULL, VigenciaHasta=NULL"); "Las 4 filas seed canónicas deben tener Activo=1, PredecesorId=NULL, VigenciaHasta=NULL");
} }
// ── REQ-SEED-002 ────────────────────────────────────────────────────────── // ── REQ-SEED-002 ──────────────────────────────────────────────────────────
// NOTE: Filters use Alicuota=0 + PredecesorId IS NULL + VigenciaDesde='2020-01-01'
// to isolate ONLY the 24 canonical seed rows, ignoring rows inserted by repo integration tests.
// Seed provinces are stored as PascalCase matching enum ProvinciaArgentina.ToString() (T700 cleanup).
[Fact] [Fact]
public async Task IngresosBrutos_Seed_HasExactly24Rows() public async Task IngresosBrutos_Seed_HasExactly24Rows()
@@ -160,14 +180,18 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
await using var conn = new SqlConnection(ConnectionString); await using var conn = new SqlConnection(ConnectionString);
await conn.OpenAsync(); await conn.OpenAsync();
var count = await conn.ExecuteScalarAsync<int>(
"SELECT COUNT(1) FROM dbo.IngresosBrutos");
// Design canónico: 23 provincias INDEC + CABA = 24 jurisdicciones. // Design canónico: 23 provincias INDEC + CABA = 24 jurisdicciones.
// La lista del design incluye CABA como elemento propio junto a BUENOS_AIRES (provincia). // La lista del design incluye CABA (CiudadAutonomaDeBuenosAires) como elemento propio.
// REQ-SEED-002 especifica "25" pero la lista canónica del design tiene 24 entradas únicas. // REQ-SEED-002 especifica "25" pero la lista canónica del design tiene 24 entradas únicas.
// DISCOVERY: posible discrepancia spec vs. design — anotado en apply-progress. // DISCOVERY: posible discrepancia spec vs. design — anotado en apply-progress.
// Implementamos lo que la lista del design establece explícitamente: 24 filas. // Implementamos lo que la lista del design establece explícitamente: 24 filas.
var count = await conn.ExecuteScalarAsync<int>("""
SELECT COUNT(1) FROM dbo.IngresosBrutos
WHERE Alicuota = 0
AND PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
""");
count.Should().Be(24, "El seed de V014 debe generar 24 IngresosBrutos (23 provincias INDEC + CABA)"); count.Should().Be(24, "El seed de V014 debe generar 24 IngresosBrutos (23 provincias INDEC + CABA)");
} }
@@ -177,23 +201,31 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
await using var conn = new SqlConnection(ConnectionString); await using var conn = new SqlConnection(ConnectionString);
await conn.OpenAsync(); await conn.OpenAsync();
var provincias = (await conn.QueryAsync<string>( var provincias = (await conn.QueryAsync<string>("""
"SELECT Provincia FROM dbo.IngresosBrutos ORDER BY Provincia")).ToList(); SELECT Provincia FROM dbo.IngresosBrutos
WHERE Alicuota = 0
AND PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
ORDER BY Provincia
""")).ToList();
// Lista canónica del design ADM-009: 23 provincias argentinas INDEC + CABA = 24 // Lista canónica del design ADM-009: 23 provincias argentinas INDEC + CABA = 24
// Stored as PascalCase matching ProvinciaArgentina enum values (T700 cleanup).
var expectedCanonical = new[] var expectedCanonical = new[]
{ {
"BUENOS_AIRES", "CABA", "CATAMARCA", "CHACO", "CHUBUT", "BuenosAires", "CiudadAutonomaDeBuenosAires", "Catamarca", "Chaco", "Chubut",
"CORDOBA", "CORRIENTES", "ENTRE_RIOS", "FORMOSA", "JUJUY", "Cordoba", "Corrientes", "EntreRios", "Formosa", "Jujuy",
"LA_PAMPA", "LA_RIOJA", "MENDOZA", "MISIONES", "NEUQUEN", "LaPampa", "LaRioja", "Mendoza", "Misiones", "Neuquen",
"RIO_NEGRO", "SALTA", "SAN_JUAN", "SAN_LUIS", "SANTA_CRUZ", "RioNegro", "Salta", "SanJuan", "SanLuis", "SantaCruz",
"SANTA_FE", "SANTIAGO_DEL_ESTERO", "TIERRA_DEL_FUEGO", "TUCUMAN" "SantaFe", "SantiagoDelEstero", "TierraDelFuego", "Tucuman"
}; };
provincias.Should().Contain("CABA", "CABA debe estar entre las provincias"); provincias.Should().Contain("CiudadAutonomaDeBuenosAires",
provincias.Should().Contain("BUENOS_AIRES", "Buenos Aires (provincia) debe estar como BUENOS_AIRES"); "CABA debe estar almacenada como CiudadAutonomaDeBuenosAires (PascalCase enum)");
provincias.Should().Contain("BuenosAires",
"Buenos Aires (provincia) debe estar como BuenosAires (PascalCase enum)");
foreach (var prov in expectedCanonical) foreach (var prov in expectedCanonical)
provincias.Should().Contain(prov, $"Provincia {prov} debe estar en el seed"); provincias.Should().Contain(prov, $"Provincia {prov} debe estar en el seed (PascalCase)");
} }
[Fact] [Fact]
@@ -202,10 +234,15 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
await using var conn = new SqlConnection(ConnectionString); await using var conn = new SqlConnection(ConnectionString);
await conn.OpenAsync(); await conn.OpenAsync();
var nonZero = await conn.ExecuteScalarAsync<int>( // Verify all 24 seed rows (VigenciaDesde='2020-01-01', PredecesorId IS NULL) have Alicuota=0.
"SELECT COUNT(1) FROM dbo.IngresosBrutos WHERE Alicuota <> 0"); var nonZero = await conn.ExecuteScalarAsync<int>("""
SELECT COUNT(1) FROM dbo.IngresosBrutos
WHERE PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
AND Alicuota <> 0
""");
nonZero.Should().Be(0, "Todas las filas seed de IngresosBrutos deben tener Alicuota=0 (placeholder)"); nonZero.Should().Be(0, "Las 24 filas seed de IngresosBrutos deben tener Alicuota=0 (placeholder)");
} }
[Fact] [Fact]
@@ -216,13 +253,14 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
var invalidRows = await conn.ExecuteScalarAsync<int>(""" var invalidRows = await conn.ExecuteScalarAsync<int>("""
SELECT COUNT(1) FROM dbo.IngresosBrutos SELECT COUNT(1) FROM dbo.IngresosBrutos
WHERE Activo = 0 WHERE VigenciaDesde = CAST('2020-01-01' AS DATE)
OR PredecesorId IS NOT NULL AND PredecesorId IS NULL
OR VigenciaHasta IS NOT NULL AND Alicuota = 0
AND (Activo = 0 OR VigenciaHasta IS NOT NULL)
"""); """);
invalidRows.Should().Be(0, invalidRows.Should().Be(0,
"Todas las filas seed deben tener Activo=1, PredecesorId=NULL, VigenciaHasta=NULL"); "Las 24 filas seed deben tener Activo=1, PredecesorId=NULL, VigenciaHasta=NULL");
} }
// ── REQ-FISCAL-AUTH-002 ─────────────────────────────────────────────────── // ── REQ-FISCAL-AUTH-002 ───────────────────────────────────────────────────
@@ -283,8 +321,13 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
VALUES (s.Codigo, s.Descripcion, s.Porcentaje, s.AplicaIVA, 1, s.VigenciaDesde, NULL, NULL); VALUES (s.Codigo, s.Descripcion, s.Porcentaje, s.AplicaIVA, 1, s.VigenciaDesde, NULL, NULL);
"""); """);
var count = await conn.ExecuteScalarAsync<int>( // Count only the 4 canonical seed rows — not test-inserted rows.
"SELECT COUNT(1) FROM dbo.TipoDeIva"); var count = await conn.ExecuteScalarAsync<int>("""
SELECT COUNT(1) FROM dbo.TipoDeIva
WHERE Codigo IN ('EXENTO', 'NO_GRAVADO', 'IVA_105', 'IVA_21')
AND PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
""");
count.Should().Be(4, "Re-ejecutar el seed MERGE no debe duplicar filas en TipoDeIva"); count.Should().Be(4, "Re-ejecutar el seed MERGE no debe duplicar filas en TipoDeIva");
} }
@@ -295,16 +338,15 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
await using var conn = new SqlConnection(ConnectionString); await using var conn = new SqlConnection(ConnectionString);
await conn.OpenAsync(); await conn.OpenAsync();
// Re-aplicar el MERGE de provincias (simula re-ejecución de V014) // Re-aplicar el MERGE de provincias (simula re-ejecución de V014) — ahora con PascalCase.
// Las 25 provincias canónicas: 24 INDEC + CABA (CABA es la #25)
await conn.ExecuteAsync(""" await conn.ExecuteAsync("""
MERGE dbo.IngresosBrutos AS t MERGE dbo.IngresosBrutos AS t
USING (VALUES USING (VALUES
('BUENOS_AIRES'),('CABA'),('CATAMARCA'),('CHACO'),('CHUBUT'), ('BuenosAires'),('CiudadAutonomaDeBuenosAires'),('Catamarca'),('Chaco'),('Chubut'),
('CORDOBA'),('CORRIENTES'),('ENTRE_RIOS'),('FORMOSA'),('JUJUY'), ('Cordoba'),('Corrientes'),('EntreRios'),('Formosa'),('Jujuy'),
('LA_PAMPA'),('LA_RIOJA'),('MENDOZA'),('MISIONES'),('NEUQUEN'), ('LaPampa'),('LaRioja'),('Mendoza'),('Misiones'),('Neuquen'),
('RIO_NEGRO'),('SALTA'),('SAN_JUAN'),('SAN_LUIS'),('SANTA_CRUZ'), ('RioNegro'),('Salta'),('SanJuan'),('SanLuis'),('SantaCruz'),
('SANTA_FE'),('SANTIAGO_DEL_ESTERO'),('TIERRA_DEL_FUEGO'),('TUCUMAN') ('SantaFe'),('SantiagoDelEstero'),('TierraDelFuego'),('Tucuman')
) AS s (Provincia) ) AS s (Provincia)
ON t.Provincia = s.Provincia AND t.PredecesorId IS NULL ON t.Provincia = s.Provincia AND t.PredecesorId IS NULL
WHEN NOT MATCHED BY TARGET THEN WHEN NOT MATCHED BY TARGET THEN
@@ -312,8 +354,13 @@ public sealed class V014MigrationTests : IClassFixture<SIGCM2.TestSupport.TestWe
VALUES (s.Provincia, N'Ingresos Brutos ' + s.Provincia, CAST(0 AS DECIMAL(5,2)), 1, '2020-01-01', NULL, NULL); VALUES (s.Provincia, N'Ingresos Brutos ' + s.Provincia, CAST(0 AS DECIMAL(5,2)), 1, '2020-01-01', NULL, NULL);
"""); """);
var count = await conn.ExecuteScalarAsync<int>( // Count only the 24 canonical seed rows — not test-inserted rows.
"SELECT COUNT(1) FROM dbo.IngresosBrutos"); var count = await conn.ExecuteScalarAsync<int>("""
SELECT COUNT(1) FROM dbo.IngresosBrutos
WHERE Alicuota = 0
AND PredecesorId IS NULL
AND VigenciaDesde = CAST('2020-01-01' AS DATE)
""");
count.Should().Be(24, "Re-ejecutar el seed MERGE no debe duplicar filas en IngresosBrutos"); count.Should().Be(24, "Re-ejecutar el seed MERGE no debe duplicar filas en IngresosBrutos");
} }

49
tests/SIGCM2.TestSupport/SqlTestFixture.cs
View File

@@ -722,34 +722,35 @@ public sealed class SqlTestFixture : IAsyncLifetime
// ── 4. Seed IngresosBrutos ──────────────────────────────────────────── // ── 4. Seed IngresosBrutos ────────────────────────────────────────────
// 24 filas: 23 provincias INDEC + CABA. Alicuota=0 placeholder. // 24 filas: 23 provincias INDEC + CABA. Alicuota=0 placeholder.
// T700 cleanup: values in PascalCase matching ProvinciaArgentina enum.ToString().
const string seedIIBB = """ const string seedIIBB = """
SET QUOTED_IDENTIFIER ON; SET QUOTED_IDENTIFIER ON;
MERGE dbo.IngresosBrutos AS t MERGE dbo.IngresosBrutos AS t
USING (VALUES USING (VALUES
('BUENOS_AIRES', N'Ingresos Brutos - Buenos Aires'), ('BuenosAires', N'Ingresos Brutos - Buenos Aires'),
('CABA', N'Ingresos Brutos - Ciudad Autonoma de Buenos Aires'), ('CiudadAutonomaDeBuenosAires', N'Ingresos Brutos - Ciudad Autonoma de Buenos Aires'),
('CATAMARCA', N'Ingresos Brutos - Catamarca'), ('Catamarca', N'Ingresos Brutos - Catamarca'),
('CHACO', N'Ingresos Brutos - Chaco'), ('Chaco', N'Ingresos Brutos - Chaco'),
('CHUBUT', N'Ingresos Brutos - Chubut'), ('Chubut', N'Ingresos Brutos - Chubut'),
('CORDOBA', N'Ingresos Brutos - Cordoba'), ('Cordoba', N'Ingresos Brutos - Cordoba'),
('CORRIENTES', N'Ingresos Brutos - Corrientes'), ('Corrientes', N'Ingresos Brutos - Corrientes'),
('ENTRE_RIOS', N'Ingresos Brutos - Entre Rios'), ('EntreRios', N'Ingresos Brutos - Entre Rios'),
('FORMOSA', N'Ingresos Brutos - Formosa'), ('Formosa', N'Ingresos Brutos - Formosa'),
('JUJUY', N'Ingresos Brutos - Jujuy'), ('Jujuy', N'Ingresos Brutos - Jujuy'),
('LA_PAMPA', N'Ingresos Brutos - La Pampa'), ('LaPampa', N'Ingresos Brutos - La Pampa'),
('LA_RIOJA', N'Ingresos Brutos - La Rioja'), ('LaRioja', N'Ingresos Brutos - La Rioja'),
('MENDOZA', N'Ingresos Brutos - Mendoza'), ('Mendoza', N'Ingresos Brutos - Mendoza'),
('MISIONES', N'Ingresos Brutos - Misiones'), ('Misiones', N'Ingresos Brutos - Misiones'),
('NEUQUEN', N'Ingresos Brutos - Neuquen'), ('Neuquen', N'Ingresos Brutos - Neuquen'),
('RIO_NEGRO', N'Ingresos Brutos - Rio Negro'), ('RioNegro', N'Ingresos Brutos - Rio Negro'),
('SALTA', N'Ingresos Brutos - Salta'), ('Salta', N'Ingresos Brutos - Salta'),
('SAN_JUAN', N'Ingresos Brutos - San Juan'), ('SanJuan', N'Ingresos Brutos - San Juan'),
('SAN_LUIS', N'Ingresos Brutos - San Luis'), ('SanLuis', N'Ingresos Brutos - San Luis'),
('SANTA_CRUZ', N'Ingresos Brutos - Santa Cruz'), ('SantaCruz', N'Ingresos Brutos - Santa Cruz'),
('SANTA_FE', N'Ingresos Brutos - Santa Fe'), ('SantaFe', N'Ingresos Brutos - Santa Fe'),
('SANTIAGO_DEL_ESTERO', N'Ingresos Brutos - Santiago del Estero'), ('SantiagoDelEstero', N'Ingresos Brutos - Santiago del Estero'),
('TIERRA_DEL_FUEGO', N'Ingresos Brutos - Tierra del Fuego'), ('TierraDelFuego', N'Ingresos Brutos - Tierra del Fuego'),
('TUCUMAN', N'Ingresos Brutos - Tucuman') ('Tucuman', N'Ingresos Brutos - Tucuman')
) AS s (Provincia, Descripcion) ) AS s (Provincia, Descripcion)
ON t.Provincia = s.Provincia AND t.PredecesorId IS NULL ON t.Provincia = s.Provincia AND t.PredecesorId IS NULL
WHEN NOT MATCHED BY TARGET THEN WHEN NOT MATCHED BY TARGET THEN