diff --git a/docs/investmoz/consolidated-review-matrix-2026-10-03.md b/docs/investmoz/consolidated-review-matrix-2026-10-03.md index c7788223..51b576b1 100644 --- a/docs/investmoz/consolidated-review-matrix-2026-10-03.md +++ b/docs/investmoz/consolidated-review-matrix-2026-10-03.md @@ -1751,3 +1751,10 @@ Caso da Tempo: Agronegócio / Maputo Província. A leitura QA reproduziu zero re Os resultados apresentam a hierarquia territorial disponível; o contexto de corredor é identificado e os níveis não indicados continuam explícitos. Pesquisa/API, vista por corredores e lista sem coordenadas do mapa concordam. O atalho «Ver toda a província» elimina a necessidade de limpar vários níveis manualmente. Não foram criadas publicações, relações a mestres ou permissões. Provas QA: 4 testes/70 verificações, zero falhas, transacções revertidas; tipos, compilação QA e sintaxe PHP passaram. O inventário anterior foi registado antes dos testes. Menus 6e8de43a, base interna, históricos e aceitação anterior preservados; L8 permanece parcial. Relatório: `catalogue-province-coverage-qa-20261028.json`. Instalação pelo processo Git e conferência visual autenticada continuam pendentes. Não declarar auditados todos os outros menus por este delta. Cobertura actualizada nos requisitos existentes V2-07/V2-10; sem nova matriz. + + +28/10 — Âmbito nacional confirmado pela Tempo: antes da extensão dos testes, preservar a correcção QA 1b2ede41 e verificar os seis corredores aprovados nas onze unidades provinciais, hierarquia descendente e reutilização entre sectores/entradas de localização. Não presumir cobertura de corredor onde não está documentada. Nenhuma prova anterior é revogada. + + + +28/10 - National coverage: 6 tests / 617 assertions passed across all 11 provincial units, 6 configured corridors and 2 sectors, including descendant hierarchy, sector separation and both province/location entry points. The runtime correction already used the common national hierarchy; Maputo was the reported reproduction, not a geographic limit. Existing publication decisions and unknown district/post/site information remain preserved. This is not a claim that every unrelated portal menu has been audited. No additional runtime change since 1b2ede41; types/build accepted evidence reused. Installation remains pending. diff --git a/docs/investmoz/unification/catalogue-province-coverage-qa-20261028.json b/docs/investmoz/unification/catalogue-province-coverage-qa-20261028.json index 7252620b..d4150736 100644 --- a/docs/investmoz/unification/catalogue-province-coverage-qa-20261028.json +++ b/docs/investmoz/unification/catalogue-province-coverage-qa-20261028.json @@ -23,8 +23,8 @@ "One action returns from district to whole province; no requirement to select every level" ], "QA": { - "tests": 4, - "assertions": 70, + "tests": 6, + "assertions": 617, "failures": 0, "database": "proactive_investment_20261027_test", "port": 55441, @@ -53,5 +53,15 @@ "corridor_coverage": "Tempo express decision,23/10/2026", "no_new_publication_or_roles": "existing approved scope", "UI_summary_and_province_shortcut": "implementation choice" + }, + "scope": "national;shared investment catalogue and similar location entry points", + "national_verification": { + "provincial_units": 11, + "approved_corridors": 6, + "sectors_for_descendant_fixtures": 2, + "province_and_legacy_location_entry_points": true, + "district_containment": true, + "unrelated_provinces_excluded": true, + "test_data": "synthetic fixtures rolled back;accepted published source reused" } } diff --git a/docs/investmoz/unification/traceability-and-manual.md b/docs/investmoz/unification/traceability-and-manual.md index 586a2cea..4186c284 100644 --- a/docs/investmoz/unification/traceability-and-manual.md +++ b/docs/investmoz/unification/traceability-and-manual.md @@ -7309,3 +7309,10 @@ Caso da Tempo: Agronegócio / Maputo Província. A leitura QA reproduziu zero re Os resultados apresentam a hierarquia territorial disponível; o contexto de corredor é identificado e os níveis não indicados continuam explícitos. Pesquisa/API, vista por corredores e lista sem coordenadas do mapa concordam. O atalho «Ver toda a província» elimina a necessidade de limpar vários níveis manualmente. Não foram criadas publicações, relações a mestres ou permissões. Provas QA: 4 testes/70 verificações, zero falhas, transacções revertidas; tipos, compilação QA e sintaxe PHP passaram. O inventário anterior foi registado antes dos testes. Menus 6e8de43a, base interna, históricos e aceitação anterior preservados; L8 permanece parcial. Relatório: `catalogue-province-coverage-qa-20261028.json`. Instalação pelo processo Git e conferência visual autenticada continuam pendentes. Não declarar auditados todos os outros menus por este delta. Cobertura actualizada nos requisitos existentes V2-07/V2-10; sem nova matriz. + + +28/10 — Âmbito nacional confirmado pela Tempo: antes da extensão dos testes, preservar a correcção QA 1b2ede41 e verificar os seis corredores aprovados nas onze unidades provinciais, hierarquia descendente e reutilização entre sectores/entradas de localização. Não presumir cobertura de corredor onde não está documentada. Nenhuma prova anterior é revogada. + + + +28/10 - National coverage: 6 tests / 617 assertions passed across all 11 provincial units, 6 configured corridors and 2 sectors, including descendant hierarchy, sector separation and both province/location entry points. The runtime correction already used the common national hierarchy; Maputo was the reported reproduction, not a geographic limit. Existing publication decisions and unknown district/post/site information remain preserved. This is not a claim that every unrelated portal menu has been audited. No additional runtime change since 1b2ede41; types/build accepted evidence reused. Installation remains pending. diff --git a/tests/Feature/CatalogueProvinceCoverageTest.php b/tests/Feature/CatalogueProvinceCoverageTest.php index 59ed835d..7bd523c9 100644 --- a/tests/Feature/CatalogueProvinceCoverageTest.php +++ b/tests/Feature/CatalogueProvinceCoverageTest.php @@ -77,4 +77,50 @@ final class CatalogueProvinceCoverageTest extends TestCase $map=app(InvestmentCatalogue::class)->map($filters); $this->assertCount(5,$map['unlocated']);$this->assertSame([],$map['features']); } + + public function test_all_approved_corridors_are_found_in_their_provinces_and_not_in_unrelated_provinces(): void + { + $catalogue=app(InvestmentCatalogue::class); + $provinces=app(InvestmentGeography::class)->publicReferences()->where('location_type','province')->get(); + $this->assertCount(11,$provinces); + foreach(config('investment_corridors')as$entry){ + $records=Opportunity::published()->whereNull('primary_territory_id') + ->where('workflow_context->corridor_agribusiness->source_sha256',CorridorAgribusinessOpportunities::SOURCE_HASH) + ->where('workflow_context->corridor_agribusiness->corridor',$entry['name'])->pluck('id')->all(); + $this->assertNotEmpty($records,$entry['name']); + foreach($provinces as$province){ + $ids=$catalogue->query(['province_id'=>$province->id,'sector'=>'agronegocio'])->pluck('id')->all(); + foreach($records as$id){ + if(in_array($province->source_code,$entry['provinces'],true))$this->assertContains($id,$ids); + else $this->assertNotContains($id,$ids); + } + } + } + } + + public function test_descendant_coverage_is_national_and_shared_by_sectors_and_location_entry_points(): void + { + $geo=app(InvestmentGeography::class);$catalogue=app(InvestmentCatalogue::class); + $provinces=$geo->publicReferences()->where('location_type','province')->get(); + $sectors=DB::table('sectors')->where('is_active',true)->whereNotNull('published_at')->whereNull('deleted_at')->orderBy('id')->limit(2)->get(); + $this->assertCount(2,$sectors); + foreach($provinces as$province){ + $district=$geo->publicReferences()->where('parent_id',$province->id)->where('location_type','district')->first(); + $this->assertNotNull($district,$province->name); + $post=$geo->publicReferences()->where('parent_id',$district->id)->where('location_type','admin_post')->first(); + $leaf=$post?:$district; + foreach($sectors as$sector){ + $record=Opportunity::create(['sector_id'=>$sector->id,'reference'=>'SYNTHETIC-QA-'.Str::uuid(),'slug'=>'synthetic-'.Str::uuid(), + 'title'=>'SYNTHETIC QA national coverage','status'=>'published','published_at'=>now()->subMinute(),'primary_territory_id'=>$leaf->id,'geography_scope'=>'specific']); + foreach([['province_id'=>$province->id],['geo_location_id'=>$province->id],['province_id'=>$province->id,'district_id'=>$district->id]]as$entry){ + $this->assertContains($record->id,$catalogue->query([...$entry,'sector'=>$sector->slug])->pluck('id')->all()); + } + $other=$sectors->firstWhere('id','!=',$sector->id); + $this->assertNotContains($record->id,$catalogue->query(['province_id'=>$province->id,'sector'=>$other->slug])->pluck('id')->all()); + $label=$catalogue->presentLocation($record)->territorial_summary; + $this->assertStringContainsString($province->name,$label);$this->assertStringContainsString($district->name,$label); + if($post)$this->assertStringContainsString($post->name,$label); + } + } + } }