bcquality/custom/knowledge/testing/ui-test-codeunit-naming.md
Michael Dieringer dd5637b1db Custom-laget bestaar nu begge CI-checks: 72 validator-fejl -> 0
Normalisering af alle 39 custom knowledge-filer til READ-kontraktens
skema (validate_frontmatter.py + Test-KnowledgeIndex.ps1 begge groenne):

- R01/R02: 28 filer manglede frontmatter eller brugte aeldre skemaer
  (title/category/severity/rule-id m.fl.) - alle har nu praecis de 6
  kraevede noegler; keywords haandskrevet pr. fil da de driver
  worklist-selektionen i INDEX/knowledge-index
- R09: manglende Description-sektion - regel-agtige foersteoverskrifter
  (Core Rule/Rule/Regel/Core Principle) omdoebt, eller sektion indsat
  efter titlen hvor intro-tekst fandtes
- R10: fenced code blocks konverteret til 4-space indrykkede blokke
  i alle filer (indhold uaendret)
- R11: 4 filer over 100 linjer fortaettet redaktionelt uden semantisk
  tab (ai-eval-scores 143->100, git-lifecycle 121->97,
  permission-sets 113->99, test-feature-scenario-tags 105->91)
- R05: AL0197->al0197, add_repo->add-repo; keyword-lister trimmet
  til maks 10

Ingen regler er fjernet eller aendret i betydning - kun form.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-01 23:47:31 +02:00

2.8 KiB

bc-version domain keywords technologies countries application-area
all
testing
test
ui
testpage
naming
suffix
codeunit
page-testing
al
w1
all

Description

Test codeunits that interact with pages via TestPage must carry a _UT suffix (Unit Test — UI layer) in their name. This distinguishes them from codeunits that test business logic directly by calling codeunit/table procedures.

The suffix signals to every reader that the codeunit opens pages, uses TestPage.OpenNew(), reads FactBox parts, or drives field validates through the page's OnValidate triggers — i.e. it exercises the UI layer, not just the logic layer.

Convention:

Layer tested Suffix Example
Business logic (codeunits, tables) (none) FindPriceTesting
Page / UI layer (TestPage) _UT FindPriceTesting_UT

A codeunit may contain only UI tests or only logic tests — never mix both in the same codeunit.

Anti Pattern

// WRONG: UI test codeunit without _UT suffix
codeunit 99007 "Find Price Page Testing"
{
    Subtype = Test;
    // contains TestPage calls — should be named "Find Price Testing_UT"
    ...
}

// WRONG: mixing direct codeunit calls and TestPage calls in the same codeunit
codeunit 99007 "Find Price Testing"
{
    Subtype = Test;

    [Test]
    procedure GetPrice_LogicTest()   // logic test — fine here
    begin
        FindPriceMgt.GetSalesPrice(...);
    end;

    [Test]
    procedure Page_ShowsPrice_UT()   // UI test — belongs in separate _UT codeunit
    var
        FindPricePage: TestPage "Find Price";
    begin
        FindPricePage.OpenNew();
        ...
    end;
}

Best Practice

// CORRECT: separate codeunits per layer

// Logic tests — no _UT suffix
codeunit 99006 "Find Price Testing"
{
    Subtype = Test;
    [Test]
    procedure GetPrice_CustomerPrice_ReturnsUnitPrice()
    begin
        FindPriceMgt.GetSalesPrice(...);
    end;
}

// UI tests — _UT suffix
codeunit 99007 "Find Price Testing_UT"
{
    Subtype = Test;
    [Test]
    procedure Page_EnterCustomerAndItem_FactBoxShowsPrice()
    var
        FindPricePage: TestPage "Find Price";
    begin
        FindPricePage.OpenNew();
        FindPricePage.CustomerNo.SetValue(Customer."No.");
        FindPricePage.ItemNo.SetValue(Item."No.");
        Assert.AreEqual('100,00', FindPricePage.FindPriceInfo.UnitPrice.Value(), '');
    end;
}

Object ID allocation

Allocate adjacent IDs for the two related codeunits (e.g. 99006 logic, 99007 UI) so they sort together in the object list and their relationship is self-evident.