Open AndreasSko opened 3 years ago
While working on https://github.com/AndreasSko/go-jwlm/pull/52 I noticed that for Notes and TagMaps the merging tests don't notice if https://github.com/AndreasSko/go-jwlm/blob/a938374f89b8ffebcb17705f812354e4528ca658/cmd/merge.go#L71 and https://github.com/AndreasSko/go-jwlm/blob/a938374f89b8ffebcb17705f812354e4528ca658/cmd/merge.go#L72 are commented out. There is a similar issue in: https://github.com/AndreasSko/go-jwlm/blob/a938374f89b8ffebcb17705f812354e4528ca658/gomobile/Merge.go#L17. The tests should be updated to fail if LocationsIDs are not properly updated.
While working on https://github.com/AndreasSko/go-jwlm/pull/52 I noticed that for Notes and TagMaps the merging tests don't notice if https://github.com/AndreasSko/go-jwlm/blob/a938374f89b8ffebcb17705f812354e4528ca658/cmd/merge.go#L71 and https://github.com/AndreasSko/go-jwlm/blob/a938374f89b8ffebcb17705f812354e4528ca658/cmd/merge.go#L72 are commented out. There is a similar issue in: https://github.com/AndreasSko/go-jwlm/blob/a938374f89b8ffebcb17705f812354e4528ca658/gomobile/Merge.go#L17. The tests should be updated to fail if LocationsIDs are not properly updated.