Paths

Table of Contentst

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

	F3375354: D9056.id30680.diff
	Tue, Nov 26, 7:47 PM

	F3374951: D9056.diff
	Tue, Nov 26, 5:31 PM

	Unknown Object (File)
	Fri, Nov 8, 8:53 AM

	Unknown Object (File)
	Fri, Nov 8, 5:44 AM

	Unknown Object (File)
	Fri, Nov 8, 4:21 AM

	Unknown Object (File)
	Tue, Nov 5, 7:33 PM

	Unknown Object (File)
	Sun, Nov 3, 11:35 PM

	Unknown Object (File)
	Oct 27 2024, 2:04 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

Branch

patryk/chat-mentioning

Lint

No Lint Coverage

Unit

No Test Coverage

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

55 lines

Diff 30680

lib/shared/markdown.test.js

Loading...