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

CDMS-200 add test for movement not found #23

Merged
merged 19 commits into from
Dec 18, 2024
Merged

CDMS-200 add test for movement not found #23

merged 19 commits into from
Dec 18, 2024

Conversation

craigedmunds
Copy link
Contributor

No description provided.

@craigedmunds
CDMS-200 add test for movement not found
b259cd7 
CDMS-200 improved CachingBlobService implementation
@craigedmunds
CMDS-200 fixes tests
aa45e3f
@craigedmunds craigedmunds force-pushed the feature/cdms-200-decision-analytics branch from 317f8a9 to aa45e3f Compare December 17, 2024 16:48
craigedmunds and others added 16 commits December 17, 2024 19:48
@craigedmunds
CDMS-200 refactors message processing
5c406f7
@craigedmunds
removes new test
9a94992
@craigedmunds
Add decions in test generator and push to blob storage
df948d4
@craigedmunds
Tidies up audit entry timings and source systems
62d1617
@craigedmunds
Fixes up tests
a2c3bce
@craigedmunds
Adds context to decisions
1edd5f9
@craigedmunds
make audit entry version nullable
483ce25
@craigedmunds
Handles checks in clearance request builder
15c7fe7
@craigedmunds
Try to send clearance requests with the DecisionNumber set to the dec…
ef92850 
…ision consumer
Updated TestDataGeneratorHelpers to use the bus to route to the consu…
8500064 
…mers rather than call directly
tidy up the switch statement further
392a198
@craigedmunds
CDMS-200 adds ability to control sequencing of messages to builders
4752aa0
@craigedmunds
Switch Decisions to their own type
af76474
@craigedmunds
Switched back to using consumers directly
52ad762
@craigedmunds
Finishes initial decision metrics
7069677
@craigedmunds
Tidied up code errors
796a517
@craigedmunds craigedmunds marked this pull request as ready for review December 18, 2024 15:16
@github-actions GitHub Actions
Copy link

Code Coverage

Package Line Rate Branch Rate Health
Btms.Business 62% 36% ➖
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 73% 30% ➖
TestDataGenerator 61% 48% ➖
Btms.Azure 43% 75% ❌
Btms.BlobService 31% 19% ❌
Btms.Analytics 86% 71% ✔
Btms.Backend.Data 65% 61% ➖
Btms.Consumers 76% 68% ➖
Btms.Common 82% 25% ✔
Btms.Model 91% 98% ✔
Btms.SyncJob 73% 62% ➖
Btms.SensitiveData 80% 81% ✔
Btms.Types.Ipaffs 94% 56% ✔
Btms.Types.Alvs.Mapping.V1.Tests 0% 0% ❌
Btms.Types.Alvs 84% 67% ✔
Btms.Emf 11% 6% ❌
Btms.Backend 64% 50% ➖
Summary 72% (6287 / 8765) 46% (507 / 1102) ➖

@craigedmunds craigedmunds merged commit 5259079 into main Dec 18, 2024
1 check passed
@craigedmunds craigedmunds deleted the feature/cdms-200-decision-analytics branch December 18, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t11omas t11omas t11omas approved these changes

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
@craigedmunds @t11omas