Skip to content

Conversation

@rpedela
Copy link
Contributor

@rpedela rpedela commented Jan 10, 2015

Several error fields were added in PG 9.3:

  • schema
  • table
  • column
  • data type
  • constraint

This PR exposes them in the returned Error object.

@brianc
Copy link
Owner

brianc commented Jan 12, 2015

@rpedela thank you so much! Quick question...is there any way you could write a test for this? I know it's postgres version dependent...you could use this function to only run the test if the database version supports it. Here's an example

@rpedela
Copy link
Contributor Author

rpedela commented Jan 12, 2015

Tests added. I am not sure how to trigger the "data type" error field, but the others are tested. I am happy to add another test for the "data type" error field if someone knows how to trigger it.

@brianc
Copy link
Owner

brianc commented Jan 13, 2015

@rpedela awesome! 👏 💃

Will push a new minor version with this change later today. 👍

brianc added a commit that referenced this pull request Jan 13, 2015
#701 Expose error fields added in PG 9.3.
@brianc brianc merged commit ba84f30 into brianc:master Jan 13, 2015
@rpedela
Copy link
Contributor Author

rpedela commented Jan 14, 2015

Thanks! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants