Die Live-Läufe vom 11.09. mit Schwellwert 2, 3 und 4 haben zwei Schwächen der
Klassifizierung gezeigt, die vor 1a.2 behoben sein müssen — dort wird sie
dauerhaft in die Datenbank geschrieben.
Erstens stand JS Research mit 204 Firmen an zweiter Stelle der Agenturliste:
eine einzelne Firma, deren Name siebzehnmal leicht anders geschrieben war
("JS Research", "S Research UG", ...). Ebenso Dr. Müller / Dr. Mueller /
Dr. D. Müller und die Praxis Prof. Dhom mit Datumszusätzen im Namen. Ein
Tippfehler ist kein zweiter Kunde. Normalisierte Namen werden jetzt zu
Ähnlichkeits-Clustern zusammengefasst: geringe Levenshtein-Distanz oder ein
gemeinsamer Anfang aus drei Wörtern gilt unter EINER Firmen-E-Mail als dieselbe
Einheit. Echte Agenturen mit verschiedenen Kundennamen bleiben davon unberührt.
Zweitens galt Welcon Europe (Ltd. & Co. KG neben GmbH & Co. KG) als Agentur,
weil die Rechtsformen erst nach dem Entfernen der Satzzeichen abgeglichen
wurden — "gmbh & co kg" passte dann nie auf "gmbh co kg". Rechtsformen werden
jetzt als einzelne Wörter vom Ende her gestrippt, bis keines mehr passt; das
deckt alle Kombinationen ab, ohne sie einzeln aufzuführen.
Auf dem Bestand steigen die automatisch zusammenführbaren Namensvarianten
damit von 723 auf 1603 Adressen, die Sammeladressen sinken von 1425 auf 1082 —
was jetzt dort steht, sind durchweg Agenturen oder Mehrkunden-Absender.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
181 lines
7.2 KiB
PHP
181 lines
7.2 KiB
PHP
<?php
|
|
|
|
use App\Models\Company;
|
|
use App\Models\PressRelease;
|
|
use App\Services\Company\CompanyMergeReport;
|
|
use Illuminate\Support\Collection;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* Trockenlauf zur Newsroom-Konsolidierung (Roadmap 1a.1). Schreibt nichts —
|
|
* er entscheidet, was später geschrieben werden darf.
|
|
*/
|
|
function mergeCompany(string $email, string $name, bool $published = false): Company
|
|
{
|
|
$company = Company::factory()->create(['email' => $email, 'name' => $name]);
|
|
|
|
if ($published) {
|
|
PressRelease::factory()->published()->create(['company_id' => $company->id]);
|
|
}
|
|
|
|
return $company;
|
|
}
|
|
|
|
function mergeClasses(int $threshold = 3): Collection
|
|
{
|
|
return collect(app(CompanyMergeReport::class)->build($threshold)['klassen'])->keyBy('klasse');
|
|
}
|
|
|
|
test('identical names on one address are a mergeable duplicate', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('info@muster.de', 'Muster GmbH');
|
|
mergeCompany('INFO@Muster.de', 'Muster GmbH');
|
|
|
|
$classes = mergeClasses();
|
|
|
|
// Groß-/Kleinschreibung der Adresse darf keine zweite Identität erzeugen.
|
|
expect($classes['dublette']['adressen'])->toBe(1)
|
|
->and($classes['dublette']['firmen'])->toBe(2)
|
|
->and($classes['dublette']['faellt_weg'])->toBe(1);
|
|
});
|
|
|
|
test('a differing legal form is the same company', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('info@hoppe-net.de', 'Hoppe Unternehmensberatung');
|
|
mergeCompany('info@hoppe-net.de', 'Hoppe Unternehmensberatung GmbH');
|
|
|
|
// Genau der Fall „gleicher ODER ÄHNLICHER Firmenname" aus dem Regelwerk.
|
|
expect(mergeClasses()['namensvariante']['faellt_weg'])->toBe(1);
|
|
});
|
|
|
|
test('a legal form with ampersand is stripped like any other', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('info@welcon.de', 'Welcon Europe Ltd. & Co. KG');
|
|
mergeCompany('info@welcon.de', 'Welcon Europe GmbH & Co. KG');
|
|
|
|
// Im Live-Lauf vom 11.09. stand Welcon als „Agentur" — weil „& Co. KG"
|
|
// nach dem Entfernen der Satzzeichen auf keine Rechtsform mehr passte.
|
|
expect(mergeClasses()['namensvariante']['faellt_weg'])->toBe(1);
|
|
});
|
|
|
|
test('typos and small additions do not make a company an agency', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('info@js-research.de', 'JS Research');
|
|
mergeCompany('info@js-research.de', 'JS Research UG (haftungsbeschränkt)');
|
|
mergeCompany('info@js-research.de', 'S Research UG (haftungsbeschränkt)');
|
|
mergeCompany('info@js-research.de', 'JS Research');
|
|
|
|
$classes = mergeClasses(3);
|
|
|
|
// JS Research stand mit 204 Firmen an zweiter Stelle der Agenturliste —
|
|
// eine einzelne Firma, deren Name siebzehnmal leicht anders geschrieben
|
|
// wurde. Ein Tippfehler ist kein zweiter Kunde.
|
|
expect($classes['namensvariante']['adressen'])->toBe(1)
|
|
->and($classes['sammeladresse']['adressen'])->toBe(0);
|
|
});
|
|
|
|
test('a shared three-word prefix under one address is the same entity', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('praxis@prof-dhom.de', 'Praxis Prof. Dr. Dhom & Kollegen MVZ GmbH (Juli 2016)');
|
|
mergeCompany('praxis@prof-dhom.de', 'Praxis Prof. Dr. Dhom & Partner');
|
|
mergeCompany('praxis@prof-dhom.de', 'Praxis Prof. Dr. Dhom');
|
|
|
|
expect(mergeClasses(3)['namensvariante']['adressen'])->toBe(1);
|
|
});
|
|
|
|
test('genuinely different clients under one agency address stay distinct', function () {
|
|
/** @var TestCase $this */
|
|
foreach (['Auto Hein', 'Ihr Personalberater', 'Designwerkstatt Coburg'] as $name) {
|
|
mergeCompany('presse@agentur-zb.de', $name);
|
|
}
|
|
|
|
// Die Ähnlichkeitsregel darf echte Agenturen nicht wegnormalisieren.
|
|
expect(mergeClasses(3)['sammeladresse']['adressen'])->toBe(1);
|
|
});
|
|
|
|
test('two genuinely different names stay below the threshold as a proposal', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('sandra@buschgroup.com', 'Busch Group');
|
|
mergeCompany('sandra@buschgroup.com', 'Pfeiffer Vacuum');
|
|
|
|
$classes = mergeClasses(3);
|
|
|
|
// Keine automatische Aktion: Zwei Marken unter einem Pressekontakt sind
|
|
// nicht dasselbe Unternehmen, aber auch keine Agentur.
|
|
expect($classes['vorschlag']['adressen'])->toBe(1)
|
|
->and($classes['vorschlag']['faellt_weg'])->toBe(0)
|
|
->and($classes['sammeladresse']['adressen'])->toBe(0);
|
|
});
|
|
|
|
test('the threshold decides where a collection address begins', function () {
|
|
/** @var TestCase $this */
|
|
foreach (['Alpha', 'Beta', 'Gamma'] as $name) {
|
|
mergeCompany('presse@agentur.de', $name);
|
|
}
|
|
|
|
// E1 ist genau dieser Schwellwert — der Bericht macht ihn verstellbar,
|
|
// damit die Entscheidung an Zahlen und nicht an einem Gefühl hängt.
|
|
expect(mergeClasses(3)['sammeladresse']['adressen'])->toBe(1)
|
|
->and(mergeClasses(4)['sammeladresse']['adressen'])->toBe(0)
|
|
->and(mergeClasses(4)['vorschlag']['adressen'])->toBe(1);
|
|
});
|
|
|
|
test('many names without a single publication are flagged, not merged', function () {
|
|
/** @var TestCase $this */
|
|
foreach (range(1, 30) as $i) {
|
|
mergeCompany('yahoo@yahoo.com', fake()->unique()->company().' '.fake()->unique()->word());
|
|
}
|
|
|
|
$classes = mergeClasses();
|
|
|
|
// Der teuerste denkbare Fehler dieses Blocks: über eine Platzhalteradresse
|
|
// zusammenzuführen und tausende unbeteiligte Firmen zu verschmelzen.
|
|
expect($classes['nie_veroeffentlicht']['adressen'])->toBe(1)
|
|
->and($classes['nie_veroeffentlicht']['firmen'])->toBe(30)
|
|
->and($classes['nie_veroeffentlicht']['faellt_weg'])->toBe(0)
|
|
->and($classes['sammeladresse']['adressen'])->toBe(0);
|
|
});
|
|
|
|
test('a large but working agency is not mistaken for a placeholder', function () {
|
|
/** @var TestCase $this */
|
|
// Echte, verschiedene Kundennamen — „Kunde 1" bis „Kunde 30" lägen
|
|
// einander so nah, dass die Ähnlichkeitsregel sie zu Recht als eine
|
|
// Firma mit Filialnummern läse.
|
|
foreach (range(1, 30) as $i) {
|
|
mergeCompany('pr@agentur.ag', fake()->unique()->company().' '.fake()->unique()->word(), published: true);
|
|
}
|
|
|
|
// Die Namensanzahl allein trennt nicht. Erst die Veröffentlichung tut es.
|
|
$classes = mergeClasses();
|
|
|
|
expect($classes['sammeladresse']['adressen'])->toBe(1)
|
|
->and($classes['nie_veroeffentlicht']['adressen'])->toBe(0);
|
|
});
|
|
|
|
test('a single company on its own address is not part of the report', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('allein@muster.de', 'Allein GmbH');
|
|
|
|
$data = app(CompanyMergeReport::class)->build();
|
|
|
|
expect($data['gesamt']['adressen_mehrfach'])->toBe(0);
|
|
});
|
|
|
|
test('the report writes nothing', function () {
|
|
/** @var TestCase $this */
|
|
mergeCompany('info@muster.de', 'Muster GmbH');
|
|
mergeCompany('info@muster.de', 'Muster GmbH');
|
|
|
|
app(CompanyMergeReport::class)->build();
|
|
|
|
// Für die rückwirkende Zusammenführung im Altbestand trägt das Portal die
|
|
// Verantwortung — deshalb steht der Trockenlauf davor und fasst nichts an.
|
|
expect(Company::query()->count())->toBe(2);
|
|
});
|
|
|
|
test('the command runs and reports the threshold in use', function () {
|
|
/** @var TestCase $this */
|
|
$this->artisan('companies:merge-report', ['--threshold' => 4])
|
|
->expectsOutputToContain('ab 4 abweichenden Namen')
|
|
->assertSuccessful();
|
|
});
|