Merge pull request #39 from 30x/ddl-fix
fix: do not report DDL changes on empty changelist
diff --git a/apigee_sync_test.go b/apigee_sync_test.go
index 68db99f..a3ba0f5 100644
--- a/apigee_sync_test.go
+++ b/apigee_sync_test.go
@@ -231,7 +231,7 @@
)).To(BeFalse())
//test b nil
- Expect(changesHaveNewTables(map[string]bool{"a": true, "b": true}, nil)).To(BeTrue())
+ Expect(changesHaveNewTables(map[string]bool{"a": true, "b": true}, nil)).To(BeFalse())
//test a nil
Expect(changesHaveNewTables(nil,
diff --git a/changes.go b/changes.go
index 0b77cba..3a480e4 100644
--- a/changes.go
+++ b/changes.go
@@ -278,7 +278,8 @@
func changesHaveNewTables(a map[string]bool, changes []common.Change) bool {
//nil maps should not be passed in. Making the distinction between nil map and empty map
- if a == nil || changes == nil {
+ if a == nil {
+ log.Warn("Nil map passed to function changesHaveNewTables, may be bug")
return true
}