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

Added additional unit tests to SQLiteKeyInfoManager. #511

Conversation

MattDavis00
Copy link
Member

These tests ensure that keys are segmented correctly by the defined namespace.

Separated by:
- Application Name
- Authenticator
- Key Name

Not separated by:
- Provider Name
- Provider UUID

Closes #510

Signed-off-by: Matt Davis matt.davis@arm.com

@MattDavis00 MattDavis00 force-pushed the feature-sqlite-kim-testing branch 2 times, most recently from 665f060 to 6648907 Compare September 1, 2021 09:07
@MattDavis00 MattDavis00 requested a review from a team as a code owner September 1, 2021 09:42
Signed-off-by: Matt Davis <matt.davis@arm.com>
These tests ensure that key are segmented correctly by the defined namespace.
Separated by:
- Application Name
- Authenticator
- Key Name
Not separated by:
- Provider Name
- Provider UUID

Signed-off-by: Matt Davis <matt.davis@arm.com>
Copy link
Member

@ionut-arm ionut-arm left a comment

Choose a reason for hiding this comment

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

Looks GOOD! 🧪

@MattDavis00 MattDavis00 merged commit 85f560a into parallaxsecond:feature-sqlite-kim Sep 1, 2021
@MattDavis00 MattDavis00 deleted the feature-sqlite-kim-testing branch September 2, 2021 14:33
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