fix: wrap nested property names

[epechuzal]

We have entities with relations using property names in kebab case, like

export class ParentEntity {
  @PrimaryKey()
  id: number;

  @ApiProperty()
  @OneToOne({ entity: () => SomeOtherEntity, nullable: true, mappedBy: 'parent' })
  'kebab-case-property': SomeOtherEntity;
}

When I use an entity factory the hydrator generates code like

if (entity.parent && !entity.parent.kebab-case-property) {
        entity.parent.kebab-case-property = entity;
      }

Which leads to an error because the above code is not valid JS.

This would be fine if only entity.parent.kebab-case-property was formatted like entity.parent['kebab-case-property'].

I noticed we have the this.wrap() method which does this manipulation for us for other properties, but we are not using it in this particular part of the code.

This PR will just add that in so that all properties are "fixed".

[B4nan]

The change is apparently wrong as it breaks existing tests. You should at least adjust the existing tests so they fail without your change, or introduce new test for it in the same manner.

[epechuzal]

The change is apparently wrong as it breaks existing tests. You should at least adjust the existing tests so they fail without your change, or introduce new test for it in the same manner.

The PR here is basically the quick change I made to my local project so that I could use the entity factory, which I was hoping that it would be a low impact fix.

To be honest I don't fully understand exactly how the hydrator works so it is very possible that I broke core functionality with this change - I will take another look but I am not very confident I will figure it out :)