Skip to content

Commit 43fabe4

Browse files
authored
Add insufficient observations error code (#2072)
1 parent 0640480 commit 43fabe4

2 files changed

Lines changed: 11 additions & 6 deletions

File tree

pkg/capabilities/errors/error_codes.go

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,9 @@ const (
130130

131131
// LimitExceeded indicates that a CRE limit breach has occurred.
132132
LimitExceeded ErrorCode = 101
133+
134+
// InsufficientObservations indicates that there are not enough observations to enable the operation to complete.
135+
InsufficientObservations ErrorCode = 102
133136
)
134137

135138
// String returns the string representation of the ErrorCode.
@@ -157,8 +160,9 @@ var errorCodeToString = map[ErrorCode]string{
157160
Unavailable: "Unavailable",
158161
DataLoss: "DataLoss",
159162
Unauthenticated: "Unauthenticated",
160-
ConsensusFailed: "ConsensusFailed",
161-
LimitExceeded: "LimitExceeded",
163+
ConsensusFailed: "ConsensusFailed",
164+
LimitExceeded: "LimitExceeded",
165+
InsufficientObservations: "InsufficientObservations",
162166
}
163167

164168
var stringToErrorCode = map[string]ErrorCode{
@@ -178,8 +182,9 @@ var stringToErrorCode = map[string]ErrorCode{
178182
"Unavailable": Unavailable,
179183
"DataLoss": DataLoss,
180184
"Unauthenticated": Unauthenticated,
181-
"ConsensusFailed": ConsensusFailed,
182-
"LimitExceeded": LimitExceeded,
185+
"ConsensusFailed": ConsensusFailed,
186+
"LimitExceeded": LimitExceeded,
187+
"InsufficientObservations": InsufficientObservations,
183188
}
184189

185190
func FromErrorCodeString(str string) ErrorCode {

pkg/capabilities/errors/error_serialization_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ func TestErrorSerializationAndDeserialization(t *testing.T) {
1313
// Assuming you have types: Visibility, Origin, and a custom Error type with serialization logic.
1414
visibilities := []caperrors.Visibility{caperrors.VisibilityPublic, caperrors.VisibilityPrivate}
1515
origins := []caperrors.Origin{caperrors.OriginUser, caperrors.OriginSystem}
16-
errorCodes := []caperrors.ErrorCode{caperrors.Unknown, caperrors.ConsensusFailed, caperrors.InvalidArgument}
16+
errorCodes := []caperrors.ErrorCode{caperrors.Unknown, caperrors.ConsensusFailed, caperrors.InsufficientObservations, caperrors.InvalidArgument}
1717

1818
for _, v := range visibilities {
1919
for _, o := range origins {
@@ -33,7 +33,7 @@ func TestRemoteErrorSerializationAndDeserialization(t *testing.T) {
3333
// Assuming you have types: Visibility, Origin, and a custom Error type with serialization logic.
3434
visibilities := []caperrors.Visibility{caperrors.VisibilityPublic, caperrors.VisibilityPrivate}
3535
origins := []caperrors.Origin{caperrors.OriginUser, caperrors.OriginSystem}
36-
errorCodes := []caperrors.ErrorCode{caperrors.Unknown, caperrors.ConsensusFailed, caperrors.InvalidArgument}
36+
errorCodes := []caperrors.ErrorCode{caperrors.Unknown, caperrors.ConsensusFailed, caperrors.InsufficientObservations, caperrors.InvalidArgument}
3737

3838
for _, v := range visibilities {
3939
for _, o := range origins {

0 commit comments

Comments
 (0)