GDS: add Method CheckRevocationStatus to Client & Server

romanett commented 5 months ago

Proposed changes

Implement the Method CheckRevocationStatus from : https://reference.opcfoundation.org/GDS/v105/docs/7.9.11

Related Issues

2469

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] Enhancement (non-breaking change which adds functionality)
[ ] Test enhancement (non-breaking change to increase test coverage)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected, requires version increase of Nuget packages)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] I have read the CONTRIBUTING doc.
[x] I have signed the CLA.
[x] I ran tests locally with my changes, all passed.
[x] I fixed all failing tests in the CI pipelines.
[x] I fixed all introduced issues with CodeQL and LGTM.
[x] I have added tests that prove my fix is effective or that my feature works and increased code coverage.
[x] I have added necessary documentation (if appropriate).
[x] Any dependent changes have been merged and published in downstream modules.

Further comments

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 80.88235% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 53.96%. Comparing base (22096f4) to head (bed8ee1). Report is 1 commits behind head on master.

Files	Patch %	Lines
...mon/RoleBasedUserManagement/AuthorizationHelper.cs	71.79%	4 Missing and 7 partials :warning:
...a.Gds.Client.Common/GlobalDiscoveryServerClient.cs	83.33%	1 Missing and 1 partial :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #2499 +/- ## ========================================== - Coverage 53.97% 53.96% -0.01% ========================================== Files 335 335 Lines 64526 64558 +32 Branches 13267 13273 +6 ========================================== + Hits 34826 34841 +15 - Misses 25939 25951 +12 - Partials 3761 3766 +5 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

romanett commented 4 months ago

@mregen Now works as expected, ready to review.

OPCFoundation / UA-.NETStandard