Paths

Table of Contentst

Differential D8555

[web] rework metadata tests to use CommQueryExecutor
ClosedPublic
Actions

Authored by kamil on Jul 19 2023, 6:55 AM.

Tags

None

Referenced Files

	F3522915: D8555.diff
	Mon, Dec 23, 7:52 AM

	Unknown Object (File)
	Sat, Dec 14, 5:09 AM

	Unknown Object (File)
	Sat, Dec 14, 5:09 AM

	Unknown Object (File)
	Sat, Dec 14, 5:09 AM

	Unknown Object (File)
	Sat, Dec 14, 5:09 AM

	Unknown Object (File)
	Sat, Dec 14, 5:08 AM

	Unknown Object (File)
	Sat, Dec 14, 5:01 AM

	Unknown Object (File)
	Wed, Dec 11, 9:33 PM

Subscribers

Details

Reviewers

michal
tomek

Commits

rCOMMcfb5ea11fd26: [web] rework metadata tests to use CommQueryExecutor

Summary

Updates test to new module

Depends on D8554

Test Plan

Run tests

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Jul 19 2023, 6:55 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJul 19 2023, 6:55 AM

kamil added a child revision: D8556: [web] rework draft tests to use CommQueryExecutor.Jul 19 2023, 6:56 AM

Harbormaster completed remote builds in B21034: Diff 28830.Jul 19 2023, 7:20 AM

kamil published this revision for review.Jul 19 2023, 9:05 AM

kamil added inline comments.

web/database/queries/metadata-queries.test.js
30 ↗	(On Diff #28830)	native getMetadata returns `std::string` always

michal accepted this revision.Jul 21 2023, 6:20 AM

This revision is now accepted and ready to land.Jul 21 2023, 6:20 AM

michal added inline comments.Jul 21 2023, 6:23 AM

web/database/queries/metadata-queries.test.js
42 ↗	(On Diff #28830)	Shouldn't we clear this metadata after the test?

rebase

kamil added inline comments.Jul 25 2023, 2:21 AM

web/database/queries/metadata-queries.test.js
42 ↗	(On Diff #28830)	indeed, I think to make those tests valid we should delete the entire database after each test, I updated the whole stack to reflect this need

kamil mentioned this in D8557: [web] rework storage engine tests to use CommQueryExecutor.Jul 25 2023, 2:22 AM

Harbormaster failed remote builds in B21170: Diff 29003!Jul 25 2023, 2:46 AM

rebase

This revision was landed with ongoing or failed builds.Jul 27 2023, 5:40 AM

Closed by commit rCOMMcfb5ea11fd26: [web] rework metadata tests to use CommQueryExecutor (authored by kamil). · Explain Why

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMcfb5ea11fd26: [web] rework metadata tests to use CommQueryExecutor.

Harbormaster failed remote builds in B21272: Diff 29122!Jul 27 2023, 5:59 AM

Revision Contents
Changeset List

Path

Size

web/

database/

queries/

metadata-queries.test.js

50 lines

Diff 29128

web/database/queries/metadata-queries.test.js

Loading...