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.

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

Add unit test for subject scheme reader #4296

Merged
merged 7 commits into from Oct 8, 2023

Conversation

jelovirt
Copy link
Member

@jelovirt jelovirt commented Oct 6, 2023

Description

Add unit test for SubjectSchemeReader.

Motivation and Context

Improve test coverage for old code to make maintaining the code easier.

Type of Changes

  • New feature (non-breaking change which adds functionality)

@jelovirt jelovirt added preprocess enhancement Changes to an existing feature internal Internal refactoring of unreleased code or tests (no docs required) labels Oct 6, 2023
@jelovirt jelovirt self-assigned this Oct 6, 2023
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
@jelovirt jelovirt force-pushed the feature/subject-scheme-reader-test branch from 93b322b to 8770d82 Compare October 6, 2023 13:39
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
@jelovirt jelovirt force-pushed the feature/subject-scheme-reader-test branch from 1926573 to 68ebdd1 Compare October 8, 2023 11:54
@jelovirt jelovirt merged commit 716d960 into develop Oct 8, 2023
4 checks passed
@jelovirt jelovirt deleted the feature/subject-scheme-reader-test branch October 8, 2023 11:58
@jelovirt jelovirt added this to the Next milestone Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Changes to an existing feature internal Internal refactoring of unreleased code or tests (no docs required) preprocess
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant