Page MenuHomePhabricator
Differential D8557

[web] rework storage engine tests to use CommQueryExecutor
ClosedPublic
Actions

Authored by kamil on Jul 19 2023, 6:58 AM.
Tags
None
Referenced Files
F2157228: D8557.id29130.diff
Mon, Jul 1, 1:23 AM
F2157227: D8557.id29005.diff
Mon, Jul 1, 1:23 AM
F2157226: D8557.id28832.diff
Mon, Jul 1, 1:23 AM
F2157168: D8557.diff
Mon, Jul 1, 1:19 AM
Unknown Object (File)
Sat, Jun 29, 7:16 PM
Unknown Object (File)
Thu, Jun 27, 3:46 AM
Unknown Object (File)
Wed, Jun 26, 3:27 PM
Unknown Object (File)
Wed, Jun 26, 9:29 AM
View All 74 Files
Subscribers
ashoat

Details

Reviewers
michal
tomek
Commits
rCOMMba914f510cfe: [web] rework storage engine tests to use CommQueryExecutor
Summary

Updates test to new module

Depends on D8556

Test Plan

Run tests

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Jul 19 2023, 6:58 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptJul 19 2023, 6:58 AM
kamil added a child revision: D8558: [web] rework database module tests to use CommQueryExecutor.Jul 19 2023, 7:00 AM
Harbormaster completed remote builds in B21036: Diff 28832.Jul 19 2023, 7:35 AM
kamil published this revision for review.Jul 19 2023, 9:07 AM
kamil mentioned this in D8472: [native] add entity and methods for redux-persist storage.Jul 20 2023, 7:07 AM
michal accepted this revision.Jul 21 2023, 6:23 AM
michal added inline comments.
web/database/queries/storage-engine-queries.test.js
42 ↗(On Diff #28832)

Shouldn't we clear this item after the test ends?

This revision is now accepted and ready to land.Jul 21 2023, 6:23 AM
kamil mentioned this in rCOMM6d72d08d6592: [native] add entity and methods for redux-persist storage.Jul 21 2023, 7:26 AM
kamil removed a child revision: D8558: [web] rework database module tests to use CommQueryExecutor.Jul 25 2023, 2:05 AM
kamil updated this revision to Diff 29005.Jul 25 2023, 2:09 AM

rebase

kamil added a child revision: D8560: [web] switch code from `sql.js` API to `.wasm` database module.Jul 25 2023, 2:14 AM
kamil added inline comments.Jul 25 2023, 2:22 AM
web/database/queries/storage-engine-queries.test.js
42 ↗(On Diff #28832)

https://phab.comm.dev/D8555?id=28830#inline-55238

Harbormaster completed remote builds in B21172: Diff 29005.Jul 25 2023, 3:05 AM
Closed by commit rCOMMba914f510cfe: [web] rework storage engine tests to use CommQueryExecutor (authored by kamil). · Explain WhyJul 27 2023, 5:40 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMba914f510cfe: [web] rework storage engine tests to use CommQueryExecutor.

Revision Contents
Changeset List

PathSize
web/
database/
queries/
54 lines

Diff 29130

View Options

web/database/queries/storage-engine-queries.test.js

Loading...