SQLite transactions do not work.

When tx.executeSql(x, [], onSuccess, onError); is run inside a db.transaction(t, error, success), each statement runs as its own transaction.

The expo API does not issue any BEGIN TRANSACTION or COMMIT TRANSACTION statements. https://www.sqlite.org/lang_transaction.html

This is misleading as the documentation does not mention this, and the API specifically mentions transaction.

Here is a reproduction:

https://snack.expo.io/r1A-1yPw4

• A table is created, with a single row inserted.
• Inside the same transaction, a SQL syntax error is purposely thrown.
• What should happen: Transaction is rolled back, no changes applied to DB, transaction error callback called.
• What happens: The new table and row are written to disk, the transaction success callback is called.

Please can you update your documentation to mention that transactions are not supported.

[ghost]

@viktor-podzigun I think you are right, I tested with returning true from the error handler and this rolled back the transaction. Thanks for posting the solution.

Expo includes node-websql here: https://github.com/expo/expo/blob/master/packages/expo-sqlite/src/SQLite.ts#L3

ROLLBACK is defined here: https://github.com/nolanlawson/node-websql/blob/master/lib/websql/WebSQLDatabase.js#L10

I think maybe this should be documented/added to the Expo test cases.

[viktor-podzigun]

@enzomd it works for me, as per spec here to rollback transaction you should either do not specify error handler (null) or return true from it:

tx.executeSql("...", [], null, null)
//or
tx.executeSql("...", [], null, (tx, error) => {
  return true //rollback
})