fix: use custom decorator on column have a property descriptor

[caolvchong]

we add custom decorator(@CustomField) for a column to change it to be an accessor. for example:

function CustomField(target: any, key: string): void {
  Object.defineProperty(target, key, {
    configurable: true,
    enumerable: true,
    get() {
      const value = this.getDataValue(key);
      return `prefix/${value}`;
    },
    set(value: string): void {
      const _value = value?.replace(/^prefix/, '');
      this.setDataValue(key, _value);
    }
  });
}

@Table
export class ModelA extends Model {
  @Column
  name: string;

  @Column({
    type: DataType.VIRTUAL
  })
  @CustomField
  bio: string;
}

@Table
export class ModelB extends Model {
  @PrimaryKey
  @AutoIncrement
  @Column
  id: number;

  @Column({
    type: DataType.VIRTUAL
  })
  @CustomField
  bio: string;
}

the @Column decorator will lose propertyDescriptor(because class field decorator does not hava descriptor)

[jagged3dge]

Apologies for the errant review.

Tagging @RobinBuschmann for review

[theoludwig]

LGTM!
Thanks for your contribution! 👍

[codecov]

Codecov Report

Merging #1070 (11487e9) into master (f16baa4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1070   +/-   ##
=======================================
  Coverage   95.11%   95.11%           
=======================================
  Files         117      117           
  Lines        1125     1126    +1     
  Branches      128      129    +1     
=======================================
+ Hits         1070     1071    +1     
  Misses         23       23           
  Partials       32       32           

Impacted Files	Coverage Δ
src/model/column/column.ts	96.00% <100.00%> (+0.16%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f16baa4...11487e9. Read the comment docs.