Skip to content

Commit

Permalink
Update bench test
Browse files Browse the repository at this point in the history
  • Loading branch information
JOOHOJANG committed Sep 9, 2024
1 parent 6f383e6 commit 13f528b
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 5 deletions.
6 changes: 5 additions & 1 deletion packages/sdk/test/bench/document.bench.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import { MaxTimeTicket } from '@yorkie-js-sdk/src/document/time/ticket';
import { InitialCheckpoint } from '@yorkie-js-sdk/src/document/change/checkpoint';
import { DocumentStatus } from '@yorkie-js-sdk/src/document/document';
import { describe, bench, assert } from 'vitest';
import { MaxVersionVector } from '../helper/helper';

const benchmarkObject = (size: number) => {
const doc = new Document<{ k1: number }>('test-doc');
Expand Down Expand Up @@ -40,7 +41,10 @@ const benchmarkArrayGC = (size: number) => {
delete root.k1;
});

assert.equal(size + 1, doc.garbageCollect(MaxTimeTicket));
assert.equal(
size + 1,
doc.garbageCollect(MaxVersionVector([doc.getChangeID().getActorID()])),
);
};

describe('Document', () => {
Expand Down
11 changes: 9 additions & 2 deletions packages/sdk/test/bench/text.bench.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import { Document, Text } from '@yorkie-js-sdk/src/yorkie';
import { MaxTimeTicket } from '@yorkie-js-sdk/src/document/time/ticket';

Check failure on line 2 in packages/sdk/test/bench/text.bench.ts

View workflow job for this annotation

GitHub Actions / build (18.x)

'MaxTimeTicket' is defined but never used
import { assert, bench, describe } from 'vitest';
import { MaxVersionVector } from '../helper/helper';

const benchmarkTextEditGC = (size: number) => {
const doc = new Document<{ text: Text }>('test-doc');
Expand All @@ -22,7 +23,10 @@ const benchmarkTextEditGC = (size: number) => {
}, `modify ${size} nodes`);
// 03. GC
assert.equal(size, doc.getGarbageLen());
assert.equal(size, doc.garbageCollect(MaxTimeTicket));
assert.equal(
size,
doc.garbageCollect(MaxVersionVector([doc.getChangeID().getActorID()])),
);
const empty = 0;
assert.equal(empty, doc.getGarbageLen());
};
Expand All @@ -45,7 +49,10 @@ const benchmarkTextSplitGC = (size: number) => {
}, 'Modify one node multiple times');
// 03. GC
assert.equal(size, doc.getGarbageLen());
assert.equal(size, doc.garbageCollect(MaxTimeTicket));
assert.equal(
size,
doc.garbageCollect(MaxVersionVector([doc.getChangeID().getActorID()])),
);
const empty = 0;
assert.equal(empty, doc.getGarbageLen());
};
Expand Down
11 changes: 9 additions & 2 deletions packages/sdk/test/bench/tree.bench.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import { converter, Document, Tree, TreeNode } from '@yorkie-js-sdk/src/yorkie';
import { MaxTimeTicket } from '@yorkie-js-sdk/src/document/time/ticket';

Check failure on line 2 in packages/sdk/test/bench/tree.bench.ts

View workflow job for this annotation

GitHub Actions / build (18.x)

'MaxTimeTicket' is defined but never used
import { describe, bench, assert } from 'vitest';
import { MaxVersionVector } from '../helper/helper';

const benchmarkTreeEdit = (size: number) => {
const doc = new Document<{ tree: Tree }>('test-doc');
Expand Down Expand Up @@ -56,7 +57,10 @@ const benchmarkTreeSplitGC = (size: number) => {
}, `modify ${size} nodes`);
// 03. GC
assert.equal(size, doc.getGarbageLen());
assert.equal(size, doc.garbageCollect(MaxTimeTicket));
assert.equal(
size,
doc.garbageCollect(MaxVersionVector([doc.getChangeID().getActorID()])),
);
const empty = 0;
assert.equal(empty, doc.getGarbageLen());
};
Expand All @@ -83,7 +87,10 @@ const benchmarkTreeEditGC = (size: number) => {
}, `modify ${size} nodes`);
// 03. GC
assert.equal(size, doc.getGarbageLen());
assert.equal(size, doc.garbageCollect(MaxTimeTicket));
assert.equal(
size,
doc.garbageCollect(MaxVersionVector([doc.getChangeID().getActorID()])),
);
const empty = 0;
assert.equal(empty, doc.getGarbageLen());
};
Expand Down

0 comments on commit 13f528b

Please sign in to comment.