check for expect value in unit tests

Signed-off-by: Stephanie <[email protected]>

Author: yangcao77

pkg/devfile/parser/data/v2/commands_test.go

@@ -2,6 +2,7 @@ package v2
 
 import (
 	"fmt"
+	"github.com/kylelemons/godebug/pretty"
 	"reflect"
 	"testing"
 
@@ -318,7 +319,13 @@ func TestDevfile200_AddCommands(t *testing.T) {
 				t.Errorf("TestDevfile200_AddCommands() error = %v, wantErr %v", err, tt.wantErr)
 			} else if tt.wantErr != nil {
 				assert.Regexp(t, *tt.wantErr, err.Error(), "Error message should match")
+			} else {
+				wantCommands := append(tt.currentCommands, tt.newCommands...)
+				if !reflect.DeepEqual(d.Commands, wantCommands) {
+					t.Errorf("TestDevfile200_AddCommands() wanted: %v, got: %v, difference at %v", wantCommands, d.Commands, pretty.Compare(wantCommands, d.Commands))
+				}
 			}
+
 		})
 	}
 }

pkg/devfile/parser/data/v2/components_test.go

@@ -2,6 +2,7 @@ package v2
 
 import (
 	"fmt"
+	"github.com/kylelemons/godebug/pretty"
 	"reflect"
 	"testing"
 
@@ -104,6 +105,11 @@ func TestDevfile200_AddComponent(t *testing.T) {
 				t.Errorf("TestDevfile200_AddComponents() error = %v, wantErr %v", err, tt.wantErr)
 			} else if tt.wantErr != nil {
 				assert.Regexp(t, *tt.wantErr, err.Error(), "Error message should match")
+			} else {
+				wantComponents := append(tt.currentComponents, tt.newComponents...)
+				if !reflect.DeepEqual(d.Components, wantComponents) {
+					t.Errorf("TestDevfile200_AddComponents() wanted: %v, got: %v, difference at %v", wantComponents, d.Components, pretty.Compare(wantComponents, d.Components))
+				}
 			}
 		})
 	}

pkg/devfile/parser/data/v2/events_test.go

@@ -2,6 +2,7 @@ package v2
 
 import (
 	"fmt"
+	"github.com/kylelemons/godebug/pretty"
 	"github.com/stretchr/testify/assert"
 	"reflect"
 	"testing"
@@ -16,6 +17,7 @@ func TestDevfile200_AddEvents(t *testing.T) {
 		name          string
 		currentEvents *v1.Events
 		newEvents     v1.Events
+		wantEvents    v1.Events
 		wantErr       *string
 	}{
 		{
@@ -30,6 +32,12 @@ func TestDevfile200_AddEvents(t *testing.T) {
 					PostStart: []string{"postStart1"},
 				},
 			},
+			wantEvents: v1.Events{
+				DevWorkspaceEvents: v1.DevWorkspaceEvents{
+					PreStart:  []string{"preStart1"},
+					PostStart: []string{"postStart1"},
+				},
+			},
 			wantErr: nil,
 		},
 		{
@@ -40,6 +48,11 @@ func TestDevfile200_AddEvents(t *testing.T) {
 					PostStart: []string{"postStart1"},
 				},
 			},
+			wantEvents: v1.Events{
+				DevWorkspaceEvents: v1.DevWorkspaceEvents{
+					PostStart: []string{"postStart1"},
+				},
+			},
 			wantErr: nil,
 		},
 		{
@@ -78,6 +91,10 @@ func TestDevfile200_AddEvents(t *testing.T) {
 				t.Errorf("TestDevfile200_AddEvents() error = %v, wantErr %v", err, tt.wantErr)
 			} else if tt.wantErr != nil {
 				assert.Regexp(t, *tt.wantErr, err.Error(), "Error message should match")
+			} else {
+				if !reflect.DeepEqual(*d.Events, tt.wantEvents) {
+					t.Errorf("TestDevfile200_AddComponents() wanted: %v, got: %v, difference at %v", tt.wantEvents, *d.Events, pretty.Compare(tt.wantEvents, *d.Events))
+				}
 			}
 
 		})