Add explicit working folder; rename project from CadSearchGui to PrepStep
Working folder (phase 1): CSV/raw/step/step_finali all used to live split across two different implicit locations (CSV in the "repo root" found by walking up for CLAUDE.md, raw/step next to the input distinta file) with neither shown to the user. That repo-root lookup also silently breaks once the app runs as a standalone published exe on a machine with no repo/ CLAUDE.md at all. Replaced both with one explicit "cartella di lavoro", auto-suggested (last used, remembered the same way as the DB path) and changeable from phase 1, with no dependency on any repo marker file. Rename: CadSearchGui was the working name from before the app had a product identity; renamed folders, csproj files, namespace and the AppData settings folder to PrepStep throughout, matching the exe name already in use. Product-facing name stays "Cevolani StepPrep". Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
davideandClaude Sonnet 5 committed
1 parent
6726dd2216
commit
13798ad47c
24 files changed
+211
-129
No files matched your search
@@ -0,0 +1,151 @@
|
||||
using PrepStep;
|
||||
|
||||
namespace PrepStep.Tests;
|
||||
|
||||
public class AssemblyComponentScannerTests
|
||||
{
|
||||
private static string ScriviAsmFinto(string testo)
|
||||
{
|
||||
string percorso = Path.GetTempFileName();
|
||||
File.WriteAllBytes(percorso, System.Text.Encoding.Latin1.GetBytes(testo));
|
||||
return percorso;
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Scarta_codici_con_una_sola_occorrenza()
|
||||
{
|
||||
// "11112222" compare 2 volte con un vero placement (riferimento vero), "99998888"
|
||||
// una sola volta senza placement (rumore).
|
||||
string percorso = ScriviAsmFinto(
|
||||
"blah Componente ID 1 (11112222.PRT) blah 99998888 blah 11112222 fine");
|
||||
try
|
||||
{
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso);
|
||||
Assert.Contains("11112222", codici);
|
||||
Assert.DoesNotContain("99998888", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Scarta_codice_ripetuto_senza_un_vero_placement()
|
||||
{
|
||||
// Riferimento al nome di un altro assieme (es. il padre che lo contiene): ripetuto
|
||||
// più volte ma mai dentro un placement "Componente ID N (...)" - va scartato.
|
||||
string percorso = ScriviAsmFinto("padre 33334444 assieme 33334444_D200 33334444");
|
||||
try
|
||||
{
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso);
|
||||
Assert.DoesNotContain("33334444", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Riconosce_sotto_assiemi_con_suffisso_stile_D100()
|
||||
{
|
||||
// Visto su 9244228573.asm: i sotto-assiemi hanno nomi come <codice>_D100/_D200/_D300.
|
||||
string percorso = ScriviAsmFinto("x 9244228573_D100 y 9244228573_D100 z 9244228573_D200 9244228573_D200");
|
||||
try
|
||||
{
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso);
|
||||
Assert.Contains("9244228573_D100", codici);
|
||||
Assert.Contains("9244228573_D200", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Non_estrae_nulla_da_una_sequenza_di_cifre_troppo_lunga()
|
||||
{
|
||||
// Una sequenza di 15 cifre di fila non contiene nessun confine di parola valido
|
||||
// per un blocco di 8-12 cifre: non deve produrre falsi positivi troncati.
|
||||
string percorso = ScriviAsmFinto("123456789012345 123456789012345");
|
||||
try
|
||||
{
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso);
|
||||
Assert.Empty(codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Scarta_id_di_viste_salvate_ModelView()
|
||||
{
|
||||
// Visto su 9044207061.asm: "ModelView#1073741828" compare più volte ma è un ID
|
||||
// interno di vista salvata, non un codice componente.
|
||||
string percorso = ScriviAsmFinto("x ModelView#1073741828 y ModelView#1073741828 z");
|
||||
try
|
||||
{
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso);
|
||||
Assert.DoesNotContain("1073741828", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Estrae_il_codice_generico_da_istanze_di_family_table_UNI()
|
||||
{
|
||||
// Visto su 9244228576.asm: le viti normalizzate compaiono come istanza di family
|
||||
// table "R12--_-_U6592<UNI6592>", ma il file reale indicizzato è "uni6592.prt".
|
||||
string percorso = ScriviAsmFinto(
|
||||
"Componente ID 1 (R12--_-_U6592<UNI6592>.PRT) x Componente ID 2 (R12--_-_U6592<UNI6592>.PRT)");
|
||||
try
|
||||
{
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso, codiciNoti: new[] { "UNI6592" });
|
||||
Assert.Contains("UNI6592", codici);
|
||||
Assert.DoesNotContain("R12--_-_U6592<UNI6592>", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void File_inesistente_lancia_eccezione_IO() =>
|
||||
Assert.Throws<DirectoryNotFoundException>(() =>
|
||||
AssemblyComponentScanner.EstraiCodiciReferenziati(@"C:\non\esiste\pippo.asm.1"));
|
||||
|
||||
[Fact]
|
||||
public void Riconosce_parte_commerciale_con_nome_alfabetico_se_nel_db()
|
||||
{
|
||||
// Visto su un caso reale: le parti commerciali scaricate come STEP (es. anelli,
|
||||
// boccole) hanno nomi Creo alfabetici, non un codice numerico Pro-E - si
|
||||
// riconoscono solo se il nome esiste davvero come codice indicizzato.
|
||||
string percorso = ScriviAsmFinto(
|
||||
"blah Componente ID 11 (ANELLO_ESTERNO-1-SOLID1.PRT) blah RUMORE_CASUALE blah");
|
||||
try
|
||||
{
|
||||
var codiciNoti = new[] { "ANELLO_ESTERNO-1-SOLID1" };
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso, codiciNoti);
|
||||
Assert.Contains("ANELLO_ESTERNO-1-SOLID1", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Scarta_token_alfabetico_nel_db_ma_senza_placement()
|
||||
{
|
||||
// Un token che coincide per puro caso con un codice DB corto/generico (es. una
|
||||
// keyword interna di Creo) non basta: senza un vero placement resta rumore.
|
||||
string percorso = ScriviAsmFinto("start_part blah blah start_part");
|
||||
try
|
||||
{
|
||||
var codiciNoti = new[] { "start_part" };
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso, codiciNoti);
|
||||
Assert.DoesNotContain("start_part", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Ignora_token_alfabetici_non_presenti_nel_db()
|
||||
{
|
||||
string percorso = ScriviAsmFinto(
|
||||
"blah Componente ID 1 (QUALCOSA_DI_INESISTENTE-1.PRT) blah");
|
||||
try
|
||||
{
|
||||
var codici = AssemblyComponentScanner.EstraiCodiciReferenziati(percorso, codiciNoti: new[] { "ALTRO_CODICE" });
|
||||
Assert.DoesNotContain("QUALCOSA_DI_INESISTENTE-1", codici);
|
||||
}
|
||||
finally { File.Delete(percorso); }
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,124 @@
|
||||
using PrepStep;
|
||||
using Microsoft.Data.Sqlite;
|
||||
|
||||
namespace PrepStep.Tests;
|
||||
|
||||
/// <summary>Ricrea lo schema reale del DB (stessa costante SQL usata da Indexer.ScanAsync) e
|
||||
/// verifica, con dati inseriti a mano, i due comportamenti che in passato hanno causato bug
|
||||
/// silenziosi: il confronto case-insensitive su codice/tipo_file e la vista
|
||||
/// files_ultima_versione che tiene solo l'ultima versione per cartella_radice+codice+tipo.</summary>
|
||||
public class CaseInsensitiveLookupIntegrationTests : IDisposable
|
||||
{
|
||||
private readonly string _dbPath = Path.Combine(Path.GetTempPath(), $"cadsearch_test_{Guid.NewGuid():N}.db");
|
||||
private readonly SqliteConnection _connection;
|
||||
|
||||
public CaseInsensitiveLookupIntegrationTests()
|
||||
{
|
||||
_connection = new SqliteConnection($"Data Source={_dbPath}");
|
||||
_connection.Open();
|
||||
using var schemaCmd = _connection.CreateCommand();
|
||||
schemaCmd.CommandText = Indexer.Schema;
|
||||
schemaCmd.ExecuteNonQuery();
|
||||
}
|
||||
|
||||
public void Dispose()
|
||||
{
|
||||
_connection.Dispose();
|
||||
SqliteConnection.ClearPool(_connection);
|
||||
File.Delete(_dbPath);
|
||||
}
|
||||
|
||||
private void InserisciFile(string percorso, string cartella, string codice, string tipo, int versione, string dataModifica)
|
||||
{
|
||||
using var cmd = _connection.CreateCommand();
|
||||
cmd.CommandText = """
|
||||
INSERT INTO files (percorso_completo, percorso_relativo, nome_file, cartella_radice,
|
||||
famiglia, codice, tipo_file, versione, dimensione_byte, data_modifica, scansionato_il)
|
||||
VALUES (@p, @p, @p, @cartella, NULL, @codice, @tipo, @versione, 1, @modifica, @modifica)
|
||||
""";
|
||||
cmd.Parameters.AddWithValue("@p", percorso);
|
||||
cmd.Parameters.AddWithValue("@cartella", cartella);
|
||||
cmd.Parameters.AddWithValue("@codice", codice);
|
||||
cmd.Parameters.AddWithValue("@tipo", tipo);
|
||||
cmd.Parameters.AddWithValue("@versione", versione);
|
||||
cmd.Parameters.AddWithValue("@modifica", dataModifica);
|
||||
cmd.ExecuteNonQuery();
|
||||
}
|
||||
|
||||
/// <summary>Stessa query (LOWER su entrambi i lati) usata da Form1 per risolvere un codice
|
||||
/// referenziato in un .asm: la scansione euristica del binario può restituire maiuscole
|
||||
/// diverse da quelle del file reale su disco (es. "_D100" estratto vs "_d100" su X:\).</summary>
|
||||
private List<(string Tipo, string Percorso)> CercaCasoInsensitive(string codiceCercato)
|
||||
{
|
||||
using var cmd = _connection.CreateCommand();
|
||||
cmd.CommandText = "SELECT tipo_file, percorso_completo FROM files_ultima_versione "
|
||||
+ "WHERE LOWER(codice) = LOWER(@codice) AND LOWER(tipo_file) IN ('prt', 'asm') "
|
||||
+ "ORDER BY tipo_file, data_modifica DESC";
|
||||
cmd.Parameters.AddWithValue("@codice", codiceCercato);
|
||||
using var reader = cmd.ExecuteReader();
|
||||
var risultati = new List<(string, string)>();
|
||||
while (reader.Read())
|
||||
risultati.Add((reader.GetString(0), reader.GetString(1)));
|
||||
return risultati;
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Trova_il_file_anche_se_la_maiuscola_del_codice_non_corrisponde()
|
||||
{
|
||||
InserisciFile(@"X:\__Corpi\442\M_B\9244228573_d100.asm.16", "__Corpi", "9244228573_d100", "asm", 16, "2018-08-22T14:51:19Z");
|
||||
|
||||
var risultati = CercaCasoInsensitive("9244228573_D100");
|
||||
|
||||
Assert.Single(risultati);
|
||||
Assert.Equal("asm", risultati[0].Tipo);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Trova_il_file_anche_se_il_tipo_e_maiuscolo()
|
||||
{
|
||||
InserisciFile(@"X:\Commerciali\Elettrici\1504240807.PRT.1", "Commerciali", "1504240807", "PRT", 1, "2020-01-01T00:00:00Z");
|
||||
|
||||
var risultati = CercaCasoInsensitive("1504240807");
|
||||
|
||||
Assert.Single(risultati);
|
||||
Assert.Equal("PRT", risultati[0].Tipo);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Tra_due_cartelle_radice_diverse_tiene_solo_la_versione_piu_recente()
|
||||
{
|
||||
// Stesso codice+tipo duplicato su due cartelle radice diverse (capitato con temp_paris
|
||||
// vs __Corpi): la vista files_ultima_versione deve restituire il più recente per
|
||||
// cartella, ma la query applicativa a valle deve poi tenerne solo uno in assoluto
|
||||
// (comportamento verificato qui a livello di dati grezzi restituiti dalla vista).
|
||||
InserisciFile(@"X:\temp_paris\gabbia45_modelli_copia\9244228573.asm.1", "temp_paris", "9244228573", "asm", 1, "2014-06-25T07:38:13Z");
|
||||
InserisciFile(@"X:\__Corpi\442\M_B\9244228573.asm.41", "__Corpi", "9244228573", "asm", 41, "2019-05-08T13:27:23Z");
|
||||
|
||||
var risultati = CercaCasoInsensitive("9244228573");
|
||||
|
||||
// La vista non deduplica tra cartelle radice diverse: la query applicativa ordina per
|
||||
// data_modifica DESC e tiene solo la prima per tipo, cioè quella più recente.
|
||||
Assert.Equal(2, risultati.Count);
|
||||
Assert.Equal(@"X:\__Corpi\442\M_B\9244228573.asm.41", risultati[0].Percorso);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void La_vista_ultima_versione_scarta_le_versioni_precedenti_nella_stessa_cartella()
|
||||
{
|
||||
InserisciFile(@"X:\__Corpi\442\M_B\9244228573.asm.40", "__Corpi", "9244228573", "asm", 40, "2018-08-22T14:51:27Z");
|
||||
InserisciFile(@"X:\__Corpi\442\M_B\9244228573.asm.41", "__Corpi", "9244228573", "asm", 41, "2019-05-08T13:27:23Z");
|
||||
|
||||
var risultati = CercaCasoInsensitive("9244228573");
|
||||
|
||||
Assert.Single(risultati);
|
||||
Assert.Contains(".asm.41", risultati[0].Percorso);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Nessun_risultato_per_un_codice_non_indicizzato()
|
||||
{
|
||||
InserisciFile(@"X:\__Corpi\442\M_B\9244228573.asm.41", "__Corpi", "9244228573", "asm", 41, "2019-05-08T13:27:23Z");
|
||||
|
||||
Assert.Empty(CercaCasoInsensitive("0000000000"));
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,69 @@
|
||||
using PrepStep;
|
||||
|
||||
namespace PrepStep.Tests;
|
||||
|
||||
public class DistintaBaseImporterTests
|
||||
{
|
||||
private const string SpreadsheetMlMinimo = """
|
||||
<?xml version="1.0"?>
|
||||
<Workbook xmlns="urn:schemas-microsoft-com:office:spreadsheet">
|
||||
<Worksheet ss:Name="Distinta Base" xmlns:ss="urn:schemas-microsoft-com:office:spreadsheet">
|
||||
<Table>
|
||||
<Row><Cell ss:Index="2"><Data>Componente</Data></Cell><Cell><Data>Descrizione</Data></Cell></Row>
|
||||
<Row><Cell ss:Index="2"><Data>1234567890</Data></Cell><Cell><Data>Vite</Data></Cell></Row>
|
||||
</Table>
|
||||
</Worksheet>
|
||||
</Workbook>
|
||||
""";
|
||||
|
||||
private static string ScriviFileDistinta(string nomeFile, string contenuto)
|
||||
{
|
||||
string percorso = Path.Combine(Path.GetTempPath(), $"cadsearch_test_{Guid.NewGuid():N}", nomeFile);
|
||||
Directory.CreateDirectory(Path.GetDirectoryName(percorso)!);
|
||||
File.WriteAllText(percorso, contenuto);
|
||||
return percorso;
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Aggiunge_assieme_generale_dedotto_dal_nome_file_DBP()
|
||||
{
|
||||
string percorso = ScriviFileDistinta("DBP_9244228573_20260916144238.xls", SpreadsheetMlMinimo);
|
||||
try
|
||||
{
|
||||
var (_, righe) = DistintaBaseImporter.ConvertiInCsv(percorso);
|
||||
|
||||
Assert.Contains(righe, r => r.Componente == "9244228573" && r.Descrizione == "Assieme generale");
|
||||
Assert.Contains(righe, r => r.Componente == "1234567890");
|
||||
}
|
||||
finally { Directory.Delete(Path.GetDirectoryName(percorso)!, recursive: true); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Non_duplica_lassieme_generale_se_gia_presente_tra_i_componenti()
|
||||
{
|
||||
// Caso limite: il codice del nome file coincide con un componente già elencato nel foglio.
|
||||
string contenutoConGeneraleGiaPresente = SpreadsheetMlMinimo.Replace("1234567890", "9244228573");
|
||||
string percorso = ScriviFileDistinta("DBP_9244228573_20260916144238.xls", contenutoConGeneraleGiaPresente);
|
||||
try
|
||||
{
|
||||
var (_, righe) = DistintaBaseImporter.ConvertiInCsv(percorso);
|
||||
|
||||
Assert.Single(righe, r => r.Componente == "9244228573");
|
||||
}
|
||||
finally { Directory.Delete(Path.GetDirectoryName(percorso)!, recursive: true); }
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Nessun_assieme_generale_aggiunto_se_il_nome_file_non_segue_il_pattern_DBP()
|
||||
{
|
||||
string percorso = ScriviFileDistinta("estrazione_qualsiasi.xls", SpreadsheetMlMinimo);
|
||||
try
|
||||
{
|
||||
var (_, righe) = DistintaBaseImporter.ConvertiInCsv(percorso);
|
||||
|
||||
Assert.DoesNotContain(righe, r => r.Descrizione == "Assieme generale");
|
||||
Assert.Single(righe);
|
||||
}
|
||||
finally { Directory.Delete(Path.GetDirectoryName(percorso)!, recursive: true); }
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,31 @@
|
||||
using PrepStep;
|
||||
|
||||
namespace PrepStep.Tests;
|
||||
|
||||
public class IndexerParseFilenameTests
|
||||
{
|
||||
[Theory]
|
||||
[InlineData("9244228573.asm.41", "9244228573", "asm", 41)]
|
||||
[InlineData("9244228573_D100.asm.16", "9244228573_D100", "asm", 16)]
|
||||
[InlineData("1504240807.PRT.1", "1504240807", "PRT", 1)]
|
||||
public void Estrae_codice_tipo_e_versione_dal_nome_file(
|
||||
string nomeFile, string codiceAtteso, string tipoAtteso, int versioneAttesa)
|
||||
{
|
||||
var (codice, tipo, versione) = Indexer.ParseFilename(nomeFile);
|
||||
Assert.Equal(codiceAtteso, codice);
|
||||
Assert.Equal(tipoAtteso, tipo);
|
||||
Assert.Equal(versioneAttesa, versione);
|
||||
}
|
||||
|
||||
[Theory]
|
||||
[InlineData("9244228573.edz")]
|
||||
[InlineData("readme.txt")]
|
||||
[InlineData("senza_estensioni")]
|
||||
public void File_senza_pattern_codice_tipo_versione_finisce_in_altro(string nomeFile)
|
||||
{
|
||||
var (codice, tipo, versione) = Indexer.ParseFilename(nomeFile);
|
||||
Assert.Equal(nomeFile, codice);
|
||||
Assert.Equal("altro", tipo);
|
||||
Assert.Null(versione);
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,26 @@
|
||||
<Project Sdk="Microsoft.NET.Sdk">
|
||||
|
||||
<PropertyGroup>
|
||||
<TargetFramework>net10.0-windows</TargetFramework>
|
||||
<ImplicitUsings>enable</ImplicitUsings>
|
||||
<Nullable>enable</Nullable>
|
||||
<IsPackable>false</IsPackable>
|
||||
</PropertyGroup>
|
||||
|
||||
<ItemGroup>
|
||||
<PackageReference Include="coverlet.collector" Version="6.0.4" />
|
||||
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.14.1" />
|
||||
<PackageReference Include="xunit" Version="2.9.3" />
|
||||
<PackageReference Include="xunit.runner.visualstudio" Version="3.1.4" />
|
||||
<PackageReference Include="Microsoft.Data.Sqlite" Version="10.0.12" />
|
||||
</ItemGroup>
|
||||
|
||||
<ItemGroup>
|
||||
<ProjectReference Include="..\PrepStep\PrepStep.csproj" />
|
||||
</ItemGroup>
|
||||
|
||||
<ItemGroup>
|
||||
<Using Include="Xunit" />
|
||||
</ItemGroup>
|
||||
|
||||
</Project>
|
||||
Reference in new issue
Block a user