[#3033] fix: sqlite3 drop/renameColumn() breaks with postProcessResponse #3040
…esponse * When postProcessResponse is configured, and client.processResponse() returns a Promise (e.g. for custom cases such as sqlite3 dropColumn()), then that Promise is not awaited, but handed to postProcessResponse, which might break is (e.g. with Objection's knexSnakeCaseMappers()). * when reinserting data in the modified table, the rows are now being handled with the "mapped" identifiers (instead of the unmapped) * add tests, fix hasColumn * add hasColumn tests for add mysql + snakeCaseMappers
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments.