ON CONFLICT support doesn't take camelCase into account

[artis3n]

The recent addition of ON CONFLICT in #335 uses in a test the format:

post.insert({
 +    content: 'test',
 +    userId: 2
 +  }).onConflict({
 +    columns: ['userId'],
 +    update: ['content']
 +  }),

However, if a SQL definition is constructed as:

sql.define({
  name: 'test',
  columns: [
    { name: 'start_time' }
  ],
  snakeToCamel: true,
});

Then, the ON CONFLICT clause will not respect the snakeToCamel setting, and will require update: ['start_time'] instead of update: ['startTime'].

This should be noted somewhere or modified to support the snakeToCamel property.

[rohitishere1]

Hi, Thanks for reporting, I will share an update soon.

[charsleysa]

@brianc @arikalfus issue is resolved by #342