Paths

Table of Contentst

-
lib/shared/
-
shared/
-
markdown.test.js

Differential D9056

[lib] Add unit tests for chat mention regex
ClosedPublic
Actions

Authored by • patryk on Aug 31 2023, 2:06 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Mar 23, 6:08 PM

	Unknown Object (File)
	Wed, Mar 19, 2:21 PM

	Unknown Object (File)
	Mon, Mar 17, 3:22 AM

	Unknown Object (File)
	Mon, Mar 17, 3:21 AM

	Unknown Object (File)
	Thu, Mar 13, 11:34 AM

	Unknown Object (File)
	Wed, Mar 12, 1:03 PM

	Unknown Object (File)
	Mon, Mar 10, 12:21 AM

	Unknown Object (File)
	Tue, Feb 25, 3:32 PM

Subscribers

Details

Reviewers

tomek
inka
rohan

Commits

rCOMM9d4b7ccde32f: [lib] Add unit tests for chat mention regex

Summary

This diff adds chat mention regex unit tests to markdown.test.js.

Depends on D8834.

Test Plan

Run yarn test in lib folder and check if all tests pass.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• patryk created this revision.Aug 31 2023, 2:06 PM

• patryk held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 31 2023, 2:06 PM

• patryk edited the summary of this revision. (Show Details)Aug 31 2023, 2:09 PM

• patryk edited the test plan for this revision. (Show Details)

• patryk added reviewers: tomek, inka, rohan.

• patryk added a parent revision: D8834: [lib] Introduce chat mention match and parse utilities.

• patryk edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B22328: Diff 30680.Aug 31 2023, 2:26 PM

• patryk added a child revision: D9057: [web] Add webMentionTypeaheadRegex tests.Aug 31 2023, 2:55 PM

• patryk published this revision for review.Aug 31 2023, 11:53 PM

Change test messages

Fix nits

Harbormaster completed remote builds in B22336: Diff 30688.Sep 1 2023, 12:16 AM

Harbormaster completed remote builds in B22337: Diff 30689.Sep 1 2023, 12:18 AM

Thanks for doing this!! Tests look great. I think outside of this file the pattern is to typically use it('should...), but I don't think it's a big deal

lib/shared/markdown.test.js
120 ↗	(On Diff #30689)
124 ↗	(On Diff #30689)

This revision is now accepted and ready to land.Sep 1 2023, 7:24 AM

Move match word

Harbormaster completed remote builds in B22705: Diff 31241.Sep 19 2023, 1:46 AM

Closed by commit rCOMM9d4b7ccde32f: [lib] Add unit tests for chat mention regex (authored by • patryk). · Explain WhySep 28 2023, 1:11 AM

This revision was automatically updated to reflect the committed changes.

• patryk added a commit: rCOMM9d4b7ccde32f: [lib] Add unit tests for chat mention regex.

Revision Contents
Changeset List

Path

Size

lib/

shared/

markdown.test.js

56 lines

Diff 31485

lib/shared/markdown.test.js

	Show All 20 Lines
	async function preloadImage(uri: string): Promise<{			async function preloadImage(uri: string): Promise<{
	steps: $ReadOnlyArray<MediaMissionStep>,			steps: $ReadOnlyArray<MediaMissionStep>,
	result: ?Image,			result: ?Image,
	}> {			}> {
	let image, exceptionMessage;			let image, exceptionMessage;
	const start = Date.now();			const start = Date.now();
	const imageURI = fetchableMediaURI(uri);			const imageURI = fetchableMediaURI(uri);
	try {			try {
	image = await new Promise((resolve, reject) => {			image = await new Promise<Image>((resolve, reject) => {
	const img = new Image();			const img = new Image();
	img.src = imageURI;			img.src = imageURI;
	img.onload = () => {			img.onload = () => {
	resolve(img);			resolve(img);
	};			};
	img.onerror = e => {			img.onerror = e => {
	reject(e);			reject(e);
	};			};
	▲ Show 20 Lines • Show All 129 Lines • ▼ Show 20 Lines		if (orientation === 2) {
	context.transform(0, -1, -1, 0, dimensions.width, dimensions.height);			context.transform(0, -1, -1, 0, dimensions.width, dimensions.height);
	} else if (orientation === 8) {			} else if (orientation === 8) {
	context.transform(0, -1, 1, 0, 0, dimensions.height);			context.transform(0, -1, 1, 0, 0, dimensions.height);
	} else {			} else {
	context.transform(1, 0, 0, 1, 0, 0);			context.transform(1, 0, 0, 1, 0, 0);
	}			}

	context.drawImage(image, 0, 0);			context.drawImage(image, 0, 0);
	reorientedBlob = await new Promise(resolve =>			reorientedBlob = await new Promise<Blob>(resolve =>
	canvas.toBlob(blobResult => resolve(blobResult)),			canvas.toBlob(blobResult => resolve(blobResult)),
	);			);
	} catch (e) {			} catch (e) {
	reorientExceptionMessage = getMessageForException(e);			reorientExceptionMessage = getMessageForException(e);
	}			}
	URL.revokeObjectURL(initialURI);			URL.revokeObjectURL(initialURI);
	const uri = reorientedBlob && URL.createObjectURL(reorientedBlob);			const uri = reorientedBlob && URL.createObjectURL(reorientedBlob);
	steps.push({			steps.push({
	▲ Show 20 Lines • Show All 91 Lines • Show Last 20 Lines