Skip to content

Feature/cb2 19647 - Add Vehicle Load Status to Test Result schema in NOP#138

Open
jackanderson02 wants to merge 2 commits into
developfrom
feature/CB2-19647-jack
Open

Feature/cb2 19647 - Add Vehicle Load Status to Test Result schema in NOP#138
jackanderson02 wants to merge 2 commits into
developfrom
feature/CB2-19647-jack

Conversation

@jackanderson02

@jackanderson02 jackanderson02 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description

Cherry picks commits from #137 relevant specific to this ticket for

Introduce a new field in both VTA and VTM to record the mandatory capture of vehicle load status for a HGV or trailer when presented for a full annual test or when conducting an annual retest when the retest has IM numbers 59, 71, 72 and/or 73 or full prohibition ( with or without certificates). This field should capture whether the vehicle/trailer is:

  • Unladen
  • Partially laden
  • Partially laden 50% to 65%
  • Fully laden
  • Load simulated partially laden
  • Load simulated fully laden

Include a summary of the change here.

Added functions and tests for vehicle load status

Related issue: CB2-19647

Before submitting (or marking as "ready for review")

  • Does the pull request title follow the conventional commit specification?
  • Have you performed a self-review of the code
  • Have you have added tests that prove the fix or feature is effective and working
  • Did you make sure to update any documentation relating to this change?

@jackanderson02 jackanderson02 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The integration tests don't pass for me @meys-bjss. I think it's because of the JSON test data having vrm and trailerIds too long for the VARCHAR columns.
Error when running tests: error: Error: Data too long for column 'vrm_trm' at row 1

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants