Merge pull request #592 from kayac/fix/add-test-for-591

Add tests for #591

diff_test.go

@@ -212,3 +212,46 @@ func TestServiceDefinitionDiffer(t *testing.T) {
  t.Error("failed to SortTaskDefinitionForDiff", diff)
  }
 }
+
+var testServiceDefinitionNoDesiredCount = &ecspresso.Service{
+ Service: types.Service{
+ LaunchType: types.LaunchTypeFargate,
+ },
+ DesiredCount: nil,
+}
+
+var testServiceDefinitionHasDesiredCount = &ecspresso.Service{
+ Service: types.Service{
+ LaunchType: types.LaunchTypeFargate,
+ },
+ DesiredCount: ptr(int32(2)),
+}
+
+func TestDiffServices(t *testing.T) {
+ t.Run("when local.DesiredCount is nil, ignore diff of DesiredCount", func (t *testing.T) {
+ diff, err := ecspresso.DiffServices(
+ testServiceDefinitionNoDesiredCount,
+ testServiceDefinitionHasDesiredCount,
+ "arn", "file", true,
+ )
+ if err != nil {
+ t.Error(err)
+ }
+ if diff != "" {
+ t.Errorf("unexpected diff: %s", diff)
+ }
+ })
+ t.Run("when local.DesiredCount is not nil, detect diff of DesiredCount.", func(t *testing.T) {
+ diff, err := ecspresso.DiffServices(
+ testServiceDefinitionHasDesiredCount,
+ testServiceDefinitionNoDesiredCount,
+ "arn", "file", true,
+ )
+ if err != nil {
+ t.Error(err)
+ }
+ if diff == "" {
+ t.Errorf("unexpected diff: %s", diff)
+ }
+ })
+}

export_test.go

@@ -24,6 +24,7 @@ var (
  InitVerifyState = initVerifyState
  VerifyResource = verifyResource
  Map2str = map2str
+ DiffServices = diffServices
 )
 
 type ModifyAutoScalingParams = modifyAutoScalingParams