From a2e8be5e786172550cf2b42d529201e4a86c6b24 Mon Sep 17 00:00:00 2001 From: Richard Connon Date: Thu, 2 Aug 2018 15:49:43 -0700 Subject: [PATCH] Equality operator on app should include syslogurl field (#1153) --- api/models/app.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/api/models/app.go b/api/models/app.go index c994b43b6..7a4d06805 100644 --- a/api/models/app.go +++ b/api/models/app.go @@ -127,6 +127,7 @@ func (a1 *App) Equals(a2 *App) bool { eq = eq && a1.ID == a2.ID eq = eq && a1.Name == a2.Name eq = eq && a1.Config.Equals(a2.Config) + eq = eq && a1.SyslogURL == a2.SyslogURL eq = eq && a1.Annotations.Equals(a2.Annotations) // NOTE: datastore tests are not very fun to write with timestamp checks, // and these are not values the user may set so we kind of don't care. @@ -143,6 +144,7 @@ func (a1 *App) EqualsWithAnnotationSubset(a2 *App) bool { eq = eq && a1.ID == a2.ID eq = eq && a1.Name == a2.Name eq = eq && a1.Config.Equals(a2.Config) + eq = eq && a1.SyslogURL == a2.SyslogURL eq = eq && a1.Annotations.Subset(a2.Annotations) // NOTE: datastore tests are not very fun to write with timestamp checks, // and these are not values the user may set so we kind of don't care.