MatchRelationship fails on matching relationship properties

[amyprime]

MatchRelationship fails when attempting to match relationship properties.

You can see the problem in the test Neo4jClient.Extension.Test.Cypher.MatchRelationshipWithProperty. That is indeed the string that is generated, but it is not valid Cypher. "agenthomeAddressMatchKey" should be replaced with the property name and value, like the "id: 7" in the MatchEntity tests. An integration test would demonstrate the failure.

I believe the resolution is that in Neo4jClient.Extension.Cypher, MatchRelationshipWorker should make use of the dynamic cutdown like MatchWorker does.

[simonpinn]

Hi @amyprime ,

Yes, I agree. I don't think relationship properties were fully supported for matches, @neutmute do you remember? I'd want to add support for override properties, and support match attribute on the relationship entity.
I'll have a closer look, feel free to raise a PR @amyprime otherwise I'll get to it soon.

cheers

Simon

[simonpinn]

@amyprime / @neutmute just had a quick play something like #26 might be all that's needed, need to add some extra tests...

[amyprime]

Hi @simonpinn, yes, I think your proposed fix should work. Thanks for looking into it!