Skip to content

sqlite: allow passing conflict resolution handler function #56352

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nodejs-github-bot merged 12 commits into from
Dec 29, 2024
+298 −42
Merged

sqlite: allow passing conflict resolution handler function #56352

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
3635cd9
sqlite: pass conflict type to conflict resolution handler
louwers Dec 24, 2024
e6001b3
sqlite: improve error message when applying changeset fails
louwers Dec 24, 2024
a06a068
sqlite: handle exception thrown in conflict handler
louwers Dec 24, 2024
5d2fc4e
sqlite: avoid coercion when returning invalid value from conflict han…
louwers Dec 28, 2024
a45e4c0
sqlite: remove redundant header
louwers Dec 28, 2024
c4a03b5
doc: sqlite documentation improvements
louwers Dec 28, 2024
c674327
sqlite: improve error handling
louwers Dec 28, 2024
7ccdfd1
sqlite: add test returning Promise from conflict handler
louwers Dec 28, 2024
df86bb1
sqlite: fix error code key and add test
louwers Dec 28, 2024
eb1936b
sqlite: formatting
louwers Dec 28, 2024
9b4255c
sqlite: fix error message test
louwers Dec 28, 2024
5e39d22
doc: sort references
louwers Dec 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
sqlite: fix error code key and add test
  • Loading branch information
@louwers
louwers committed Dec 28, 2024
commit df86bb171438777b6be171adaa9ea4e5a83fe82c
2 changes: 1 addition & 1 deletion src/node_sqlite.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ inline MaybeLocal<Object> CreateSQLiteError(Isolate* isolate,
->ToObject(isolate->GetCurrentContext())
.ToLocal(&e) ||
e->Set(isolate->GetCurrentContext(),
env->error_string(),
env->code_string(),
env->err_sqlite_error_string())
.IsNothing()) {
return MaybeLocal<Object>();
Expand Down
5 changes: 3 additions & 2 deletions test/parallel/test-sqlite-session.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,7 @@ suite('conflict resolution', () => {
() => -1,
() => ({}),
() => null,
async () => constants.SQLITE_CHANGESET_ABORT
async () => constants.SQLITE_CHANGESET_ABORT,
];

for (const invalidHandler of invalidHandlers) {
Expand All @@ -340,7 +340,8 @@ suite('conflict resolution', () => {
}, {
name: 'Error',
message: 'bad parameter or other API misuse',
errcode: 21
errcode: 21,
code: 'ERR_SQLITE_ERROR'
}, `Did not throw expected exception when returning '${invalidHandler}' from conflict handler`);
}
});
Expand Down