swap errstr and errmsg

Author: islandryu

src/node_sqlite.cc

@@ -197,8 +197,8 @@ inline MaybeLocal<Object> CreateSQLiteError(Isolate* isolate,
   Local<String> js_errstr;
   Local<String> js_errmsg;
   Local<Object> e;
-  if (!String::NewFromUtf8(isolate, errstr).ToLocal(&js_errmsg) ||
-      !String::NewFromUtf8(isolate, errmsg).ToLocal(&js_errstr) ||
+  if (!String::NewFromUtf8(isolate, errstr).ToLocal(&js_errstr) ||
+      !String::NewFromUtf8(isolate, errmsg).ToLocal(&js_errmsg) ||
       !CreateSQLiteError(isolate, message).ToLocal(&e) ||
       e->Set(isolate->GetCurrentContext(),
              Environment::GetCurrent(isolate)->errcode_string(),

test/parallel/test-sqlite-template-tag.js

@@ -103,13 +103,13 @@ test('sql.db returns the associated DatabaseSync instance', () => {
 
 test('failed prepares throw', () => {
   assert.throws(() => {
-    sql.all`SELECT * FROM does_not_exist`; // eslint-disable-line no-unused-expressions
+    sql.all`SELECT * FROM does_not_exist`;
   }, {
     name: 'Error',
     message: 'Failed to prepare statement',
     code: 'ERR_SQLITE_ERROR',
     errcode: 1,
-    errstr: 'no such table: does_not_exist',
-    errmsg: 'SQL logic error'
+    errstr: 'SQL logic error',
+    errmsg: 'no such table: does_not_exist'
   });
 });