Skip to content

refactor: Split SubcSubcontractingTest into 3 focused codeunits#8225

Draft
ChethanT wants to merge 1 commit into
mainfrom
refactor/Subcontracting/SplitTestCodeunit
Draft

refactor: Split SubcSubcontractingTest into 3 focused codeunits#8225
ChethanT wants to merge 1 commit into
mainfrom
refactor/Subcontracting/SplitTestCodeunit

Conversation

@ChethanT
Copy link
Copy Markdown
Contributor

@ChethanT ChethanT commented May 20, 2026

Summary

Splits the monolithic test codeunit \Subc. Subcontracting Test\ (139989, 42 tests, 3562 lines) into three focused codeunits for better maintainability:

New Codeunits

Codeunit ID Tests Focus
Subc. Transfer Order Test 139990 13 Transfer order creation, location handling, returns, reservations, bin codes
Subc. Purchase Order Test 139991 12 PO creation from routing, posting groups, calculate subcontracts, descriptions
Subc. Subcontracting Test 139989 18 Remaining: setup, factbox, planning, item charge, copy document, dispatching

Motivation

The original codeunit was growing unbounded due to its generic name. This split groups tests by feature area while avoiding over-fragmentation.

Each new codeunit is self-contained with required handlers and helper procedures.

Split the monolithic test codeunit (139989, 42 tests, 3562 lines) into
three focused codeunits:

- Subc. Transfer Order Test (139990): 13 tests covering transfer order
  creation, location handling, returns, reservations, bin codes
- Subc. Purchase Order Test (139991): 12 tests covering PO creation from
  routing, posting groups, calculate subcontracts, descriptions
- Subc. Subcontracting Test (139989): 18 remaining misc tests (setup,
  factbox, planning, item charge, copy document, dispatching)

Each new codeunit is self-contained with required handlers and helpers.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Could not find a linked ADO work item. Please link one by using the pattern 'AB#' followed by the relevant work item number. You may use the 'Fixes' keyword to automatically resolve the work item when the pull request is merged. E.g. 'Fixes AB#1234'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant