Merge pull request kubernetes#125712 from benluddy/remove-cbor-test-skips

KEP-4222: Remove temporary mechanism for skipping CBOR tests.

Author: k8s-ci-robot

staging/src/k8s.io/apimachinery/pkg/runtime/serializer/cbor/internal/modes/appendixa_test.go

@@ -69,10 +69,6 @@ func TestAppendixA(t *testing.T) {
 		reject  string   // reason the decoder rejects the example
 		encoded []byte   // re-encoded object (only if different from example encoding)
 		reasons []string // reasons for re-encode difference
-
-		// TODO: The cases with nonempty fixme are known to be not working and fixing them
-		// is an alpha criteria. They're present and skipped for visibility.
-		fixme string
 	}{
 		{
 			example: hex("00"),
@@ -558,10 +554,6 @@ func TestAppendixA(t *testing.T) {
 		},
 	} {
 		t.Run(fmt.Sprintf("%x", tc.example), func(t *testing.T) {
-			if tc.fixme != "" {
-				t.Skip(tc.fixme) // TODO: Remove once all cases are fixed.
-			}
-
 			var decoded interface{}
 			err := modes.Decode.Unmarshal(tc.example, &decoded)
 			if err != nil {

staging/src/k8s.io/apimachinery/pkg/runtime/serializer/cbor/internal/modes/decode_test.go

@@ -51,11 +51,6 @@ func TestDecode(t *testing.T) {
 		into          interface{} // prototype for concrete destination type. if nil, decode into empty interface value.
 		want          interface{}
 		assertOnError func(t *testing.T, e error)
-
-		// TODO: Some failing test cases are included for completeness. The next library
-		// minor version should allow them all to be fixed. In the meantime, this field
-		// explains the behavior reason for a particular failure.
-		fixme string
 	}
 
 	// Test cases are grouped by the kind of the CBOR data item being decoded, as enumerated in
@@ -76,13 +71,6 @@ func TestDecode(t *testing.T) {
 						}
 
 						t.Run(fmt.Sprintf("%s/mode=%s", test.name, modeName), func(t *testing.T) {
-							if test.fixme != "" {
-								// TODO: Remove this along with the
-								// fixme field when the last skipped
-								// test case is passing.
-								t.Skip(test.fixme)
-							}
-
 							var dst reflect.Value
 							if test.into == nil {
 								var i interface{}