balderdashy/sails-memory

Adapter does not honor columnName configuration.

Closed this issue · 1 comments

We ran into an issue in our tests where we would have a model configured like so

Cat: {
  attributes: {
    myId: {
      columnName: 'my_id'
    }
  }
}

The memory adapter would create the models, as you'd expect, with the myId property. Waterline criteria, unfortunately but correctly, mapped the field to my_id. Therefore, any queries like findOne({myId: 1}) would fail in tests since the actual criteria would be my_id: 1.

PR submitted. Thanks!