Skip to content

Conversation

@patapenka-alexey
Copy link
Contributor

Closes TNTP-4190

@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-4190-namer-2 branch from c205b03 to f7106b0 Compare December 11, 2025 12:19
@patapenka-alexey patapenka-alexey marked this pull request as ready for review December 11, 2025 12:19
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-4190-namer-2 branch from f7106b0 to 436e3b8 Compare December 11, 2025 12:35
@coveralls
Copy link

coveralls commented Dec 11, 2025

Pull Request Test Coverage Report for Build 20173968955

Details

  • 186 of 194 (95.88%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+4.6%) to 24.857%

Changes Missing Coverage Covered Lines Changed/Added Lines %
verification/rsapss.go 2 3 66.67%
namer/namer.go 163 170 95.88%
Totals Coverage Status
Change from base Build 19701817310: 4.6%
Covered Lines: 782
Relevant Lines: 3146

💛 - Coveralls

@bigbes bigbes force-pushed the patapenka-alexey/tntp-4190-namer-2 branch from d610d1e to 440c01c Compare December 12, 2025 14:13
Closes TNTP-4190
@bigbes bigbes force-pushed the patapenka-alexey/tntp-4190-namer-2 branch from 440c01c to 9496be8 Compare December 12, 2025 16:59
@bigbes bigbes requested a review from oleg-jukovec December 12, 2025 17:00
Comment on lines +31 to +32
// NewDefaultKey returns new Key object.
func NewDefaultKey(name string, keytype KeyType, property string, raw string) DefaultKey {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tests for the type are missed.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe use a shortest name verify for the package?

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.

5 participants