Skip to content

#1631 Add unit tests for key validation#1679

Merged
tomholub merged 1 commit intomasterfrom
bugfix/issue-1631-ekm-test-fail
Jun 7, 2022
Merged

#1631 Add unit tests for key validation#1679
tomholub merged 1 commit intomasterfrom
bugfix/issue-1631-ekm-test-fail

Conversation

@sosnovsky
Copy link
Collaborator

@sosnovsky sosnovsky commented Jun 7, 2022

This PR replaces UI test for invalid EKM key with unit test.

close #1631


Tests (delete all except exactly one):

  • Tests added

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@sosnovsky sosnovsky requested review from ioanmo226 and tomholub June 7, 2022 09:10
@tomholub tomholub enabled auto-merge (squash) June 7, 2022 09:19
@tomholub tomholub merged commit bf4785f into master Jun 7, 2022
@tomholub tomholub deleted the bugfix/issue-1631-ekm-test-fail branch June 7, 2022 09:20
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.

CheckRefreshKeyFromEkmErrorHandling test periodically fails on Semaphore

2 participants