Merge pull request #10589 from cjcullen/testfix

Fix errors test
This commit is contained in:
Zach Loafman 2015-06-30 17:27:20 -07:00
commit ceaddc0c44
4 changed files with 4 additions and 4 deletions

View File

@ -149,7 +149,7 @@ func TestNewInvalid(t *testing.T) {
}
for i, testCase := range testCases {
vErr, expected := testCase.Err, testCase.Details
expected.Causes[0].Message = vErr.Error()
expected.Causes[0].Message = vErr.ErrorBody()
err := NewInvalid("kind", "name", fielderrors.ValidationErrorList{vErr})
status := err.(*StatusError).ErrStatus
if status.Code != 422 || status.Reason != api.StatusReasonInvalid {

View File

@ -316,7 +316,7 @@ func validateEvent(actualEvent *api.Event, expectedEvent *api.Event, t *testing.
if actualEvent.FirstTimestamp.IsZero() || actualEvent.LastTimestamp.IsZero() {
t.Errorf("timestamp wasn't set: %#v", *actualEvent)
}
if actualEvent.FirstTimestamp.Equal(actualEvent.LastTimestamp.Time) {
if actualEvent.FirstTimestamp.Equal(actualEvent.LastTimestamp) {
if expectCompression {
t.Errorf("FirstTimestamp (%q) and LastTimestamp (%q) must be equal to indicate only one occurance of the event, but were different. Actual Event: %#v", actualEvent.FirstTimestamp, actualEvent.LastTimestamp, *actualEvent)
}

View File

@ -173,7 +173,7 @@ func compareEventWithHistoryEntry(expected *api.Event, actual *history, t *testi
t.Fatalf("There should be one existing instance of this event in the hash table.")
}
if !actual.FirstTimestamp.Equal(expected.FirstTimestamp.Time) {
if !actual.FirstTimestamp.Equal(expected.FirstTimestamp) {
t.Fatalf("Unexpected FirstTimestamp. Expected: <%v> Actual: <%v>", expected.FirstTimestamp, actual.FirstTimestamp)
}

View File

@ -1136,7 +1136,7 @@ func TestOverlappingRCs(t *testing.T) {
controllers = append(controllers, controllerSpec)
}
shuffledControllers := shuffle(controllers)
for j, _ := range shuffledControllers {
for j := range shuffledControllers {
manager.controllerStore.Store.Add(shuffledControllers[j])
}
// Add a pod and make sure only the oldest rc is synced