Skip to content

Commit 79b5e99

Browse files
committed
feat(builder)!: rename updateTableSchema to syncDB
1 parent c76eb56 commit 79b5e99

File tree

3 files changed

+17
-18
lines changed

3 files changed

+17
-18
lines changed

packages/sqlite-builder/README.md

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -65,14 +65,14 @@ const db = new SqliteBuilder<DB>({
6565
onQuery: true,
6666
})
6767

68-
// update tables using syncTable
69-
await db.updateTableSchema(useSchema(baseTables, { logger: false }))
68+
// update table using schema
69+
await db.syncDB(useSchema(baseTables, { logger: false }))
7070

7171
import { useMigrator } from 'kysely-sqlite-builder'
7272
import { FileMigrationProvider } from 'kysely'
7373

7474
// update tables using MigrationProvider and migrate to latest
75-
await db.updateTableSchema(useMigrator(new FileMigrationProvider(/* ... */)))
75+
await db.syncDB(useMigrator(new FileMigrationProvider(/* ... */)))
7676
```
7777

7878
### Usage
@@ -82,21 +82,20 @@ await db.updateTableSchema(useMigrator(new FileMigrationProvider(/* ... */)))
8282
await db.transaction(async (trx) => {
8383
// use transaction
8484
await trx.insertInto('test').values({ gender: false }).execute()
85+
// auto use `trx`
86+
await db.execute(d => d.insertInto('test').values({ gender: true }).execute())
8587
// nest transaction, using savepoint
8688
await db.transaction(async () => {
87-
// auto load transaction with savepoint
88-
await db.execute(
89-
d => d.selectFrom('test').where('gender', '=', true),
90-
'this is error message'
91-
)
89+
// auto use savepoint
90+
await db.execute(d => d.selectFrom('test').where('gender', '=', true))
9291
})
9392
})
9493

9594
// use origin instance
9695
await db.kysely.insertInto('test').values({ gender: false }).execute()
9796

9897
// run raw
99-
await db.raw(sql`PRAGMA user_version = 2`, 'this is error message')
98+
await db.raw(sql`PRAGMA user_version = 2`)
10099

101100
// destroy
102101
await db.destroy()

packages/sqlite-builder/src/builder.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -124,16 +124,16 @@ export class SqliteBuilder<DB extends Record<string, any>> {
124124
* onQuery: true,
125125
* })
126126
*
127-
* // update tables using syncTable
128-
* await db.updateTableSchema(useSchema(baseTables, { logger: false }))
127+
* // update tables using schema
128+
* await db.syncDB(useSchema(baseTables, { logger: false }))
129129
*
130130
* import { useMigrator } from 'kysely-sqlite-builder'
131131
* import { FileMigrationProvider } from 'kysely'
132132
*
133133
* // update tables using MigrationProvider and migrate to latest
134-
* await db.updateTableSchema(useMigrator(new FileMigrationProvider(...)))
134+
* await db.syncDB(useMigrator(new FileMigrationProvider(...)))
135135
*/
136-
public async updateTableSchema(updater: TableUpdater, checkIntegrity?: boolean): Promise<StatusResult> {
136+
public async syncDB(updater: TableUpdater, checkIntegrity?: boolean): Promise<StatusResult> {
137137
try {
138138
if (checkIntegrity && !(await runCheckIntegrity(this.kysely))) {
139139
this.logger?.error('integrity check fail')

test/builder.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,15 +40,15 @@ describe('test sync table', async () => {
4040
let db: SqliteBuilder<any>
4141
beforeEach(async () => {
4242
db = getDatabase()
43-
await db.updateTableSchema(useSchema(baseTables, { log: false }))
43+
await db.syncDB(useSchema(baseTables, { log: false }))
4444
})
4545
it('should create new table', async () => {
4646
const foo = defineTable({
4747
col1: { type: 'increments' },
4848
col2: { type: 'string' },
4949
})
5050

51-
await db.updateTableSchema(useSchema({
51+
await db.syncDB(useSchema({
5252
...baseTables,
5353
foo,
5454
}, { log: false }))
@@ -59,7 +59,7 @@ describe('test sync table', async () => {
5959
expect(_tables[1].name).toBe('test')
6060
})
6161
it('should drop old table', async () => {
62-
await db.updateTableSchema(useSchema({ }, { log: false }))
62+
await db.syncDB(useSchema({ }, { log: false }))
6363

6464
const _tables = await db.kysely.introspection.getTables()
6565
expect(_tables.length).toBe(0)
@@ -78,7 +78,7 @@ describe('test sync table', async () => {
7878
timeTrigger: { create: true, update: true },
7979
},
8080
)
81-
await db.updateTableSchema(useSchema({ test: foo }, { log: false }))
81+
await db.syncDB(useSchema({ test: foo }, { log: false }))
8282
const [_tables] = await db.kysely.introspection.getTables()
8383
expect(_tables
8484
.columns
@@ -100,7 +100,7 @@ describe('test sync table', async () => {
100100
describe('test builder', async () => {
101101
const db = getDatabase()
102102
await getOrSetDBVersion(db.kysely, 2)
103-
await db.updateTableSchema(useSchema(baseTables))
103+
await db.syncDB(useSchema(baseTables))
104104
it('should insert', async () => {
105105
// generate table
106106
console.log(await db.transaction(async () => {

0 commit comments

Comments
 (0)