Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added automated test to check unmatched CHED #37

Merged
merged 15 commits into from
Jan 3, 2025
Merged

Added automated test to check unmatched CHED #37

merged 15 commits into from
Jan 3, 2025

Conversation

limsim
Copy link
Contributor

@limsim limsim commented Jan 2, 2025

No description provided.

Lim Sim and others added 15 commits December 31, 2024 16:07
Added automated test to check unmatched CHED
7f56c08
Updated notification to use the enum rather than a string
a37fcf0
@craigedmunds
CDMS-200 removes ordering of builders in SpecificFilesScenarioGenerat…
9b4ecb1 
…or and orders messages instead. Removes the non-windows regex (#38)
Fixed issue with serializing strings to an enum
6397bc5
@limsim
Merge pull request #39 from DEFRA/feature/cdms-210
504ff2d 
Updated notification to use the enum rather than a string
@craigedmunds
CDMS-200 captures ched types of movements & adds to analytics. One of… (
a80c298 
#36)

* CDMS-200 captures ched types of movements & adds to analytics. One of the tests is failing so needs investigation

* CDMS-200 finalising country filtering

* Adds test generation unit test that instantiates and validates all scenario generators

* Refactors to reuse chedtype & country filtering

* Removing comments

* Refactoring analytics to re-use link status analysis

* Refactored Link status out of analytics to be calculated at write time and stored on movement

* Fixed tests

* Fixes tests
Added automated test to check unmatched CHED
a961536
Fixed issue with serializing strings to an enum
871ae66
Changed AllChedsNoMatch test to use specific files pattern.
3629055
Added automated test to check unmatched CHED
be75f94
Fixed issue with serializing strings to an enum
71b0876
some tidy up.
1f98749
Merge remote-tracking branch 'origin/feature/CDMS-144-add-automated-t…
facb434 
…est' into feature/CDMS-144-add-automated-test

# Conflicts:
#	Btms.Backend.IntegrationTests/DecisionTests/ChedPSimpleTests.cs
#	Btms.Backend.IntegrationTests/MatchingTests/ChedNoMatchTests.cs
Make sure file properties are correct.
0b31fd7
use correct chedp-one-commodity.json file.
48ac665
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 3, 2025

Code Coverage

Package Line Rate Branch Rate Health
Btms.Business 72% 53% ➖
Btms.Emf 11% 6% ❌
Btms.Types.Gvms.Mapping 38% 26% ❌
Btms.Metrics 82% 0% ✔
Btms.Types.Gvms 67% 100% ➖
Btms.Types.Alvs.Mapping 76% 0% ➖
Btms.Types.Ipaffs.Mapping 75% 39% ➖
Btms.Azure 43% 75% ❌
Btms.BlobService 33% 23% ❌
Btms.Backend 64% 47% ➖
Btms.Analytics 74% 61% ➖
Btms.Backend.Data 65% 61% ➖
Btms.Consumers 78% 69% ➖
Btms.Common 82% 50% ✔
Btms.Model 91% 97% ✔
Btms.SyncJob 73% 62% ➖
Btms.SensitiveData 80% 83% ✔
Btms.Types.Ipaffs 95% 61% ✔
Btms.Types.Alvs.Mapping.V1.Tests 0% 0% ❌
Btms.Types.Alvs 84% 50% ✔
TestDataGenerator 73% 66% ➖
Summary 74% (7482 / 10110) 53% (686 / 1292) ➖

@limsim limsim merged commit 4f680ac into main Jan 3, 2025
1 check passed
@limsim limsim deleted the feature/CDMS-144-add-automated-test branch January 3, 2025 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@limsim @craigedmunds