Fixes panic while having 2 structs having slices being compared

[dolanor]

Hi!

I'm using mongodm with a project using some protobuf, and I stumbled upon this panic:

2016/05/06 17:48:18 runtime error: comparing uncomparable type proto.Profilegoroutine 36 [running]:
runtime/debug.Stack(0x0, 0x0, 0x0)
    /usr/local/go/src/runtime/debug/stack.go:24 +0x80
github.com/micro/go-micro/server.(*rpcServer).accept.func1(0x7f0f670a8240, 0xc820196140)
    /home/dolanor/go/src/github.com/micro/go-micro/server/rpc_server.go:50 +0x53
panic(0xb6aa60, 0xc820486be0)
    /usr/local/go/src/runtime/panic.go:426 +0x4e9
github.com/zebresel-com/mongodm.(*DocumentBase).DefaultValidate(0xc82014c480, 0xc820129a68, 0x0, 0x0, 0x0)
    /home/dolanor/go/src/github.com/zebresel-com/mongodm/document_base.go:185 +0x3c2a
github.com/zebresel-com/mongodm.(*DocumentBase).Validate(0xc82014c480, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
    /home/dolanor/go/src/github.com/zebresel-com/mongodm/document_base.go:83 +0x32
github.com/zebresel-com/mongodm.(*DocumentBase).Save(0xc82014c480, 0x0, 0x0)
    /home/dolanor/go/src/github.com/zebresel-com/mongodm/document_base.go:410 +0x9c
github.com/org/api/user-srv/db.CreateProfile(0xc8200bc300, 0x0, 0x0)

From my investigation, it is the line I corrected that crashed because of comparing 2 slices.
I don't have more time to investigate, but this fixed my problem.

[moehlone]

I will look into this next week :) thank you!

[moehlone]

Could you attach the piece of your code which triggers the panic, so that I have an example to reproduce?

[dolanor]

Hello,

sorry for the delay but I've set up an example where it fails.
It is here : https://github.com/tuxago/test-mongodm/blob/master/test_mongodm.go

The 2 functions called from main are identical except for using imports from here or from my repo having the fixed proposed in this PR. Yours panic (it seems to be caused by slices in a struct) and mine handle it gracefully with the DeepEquals.

PS: I couldn't make "go test" run in your project because of the L function.

[moehlone]

Thank you in advance - I will check this soon! And thank you for the hint with my tests.. Forgot to encapsulate this..

[lacion]

any news on this @moehlone ?

[moehlone]

I finally found some time yesterday to fix the tests and migrated your issue.
Thank you!