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
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: avoid coercion when returning invalid value from conflict han… 
…dler
  • Loading branch information
@louwers
louwers committed Dec 28, 2024
commit 5d2fc4e38fe37732c535e687cc4534aa88a83ad9
2 changes: 2 additions & 0 deletions src/node_sqlite.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -805,6 +805,8 @@ void DatabaseSync::ApplyChangeset(const FunctionCallbackInfo<Value>& args) {
try_catch.ReThrow();
return SQLITE_CHANGESET_ABORT;
}
constexpr auto invalid_value = -1;
if (!result->IsInt32()) return invalid_value;
return result->Int32Value(env->context()).FromJust();
Comment thread
louwers marked this conversation as resolved.
};
Comment thread
louwers marked this conversation as resolved.
}
Expand Down
27 changes: 16 additions & 11 deletions test/parallel/test-sqlite-session.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -324,17 +324,22 @@ suite('conflict resolution', () => {
});

test('conflict resolution handler returns invalid value', (t) => {
const { database2, changeset } = prepareConflict();
t.assert.throws(() => {
database2.applyChangeset(changeset, {
onConflict: () => {
return -1;
}
});
}, {
name: 'Error',
message: 'bad parameter or other API misuse'
});
const invalidValues = [-1, {}, null];

for (const invalidValue of invalidValues) {
const { database2, changeset } = prepareConflict();
t.assert.throws(() => {
database2.applyChangeset(changeset, {
onConflict: () => {
return invalidValue;
}
});
}, {
name: 'Error',
message: 'bad parameter or other API misuse',
errcode: 21
}, `Did not throw expected exception when returning '${invalidValue}' from conflict handler`);
}
});

test('conflict resolution handler throws', (t) => {
Expand Down