CERBERUS/LibreChat
1
0
Fork 0
mirror of https://github.com/danny-avila/LibreChat.git synced 2026-10-21 17:44:11 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

🪞 chore: Fix isPreviewOnlyArtifact test description parameter order

Browse source 
Follow-up review nit on PR #12934. Jest's `it.each` substitutes `%s`
positionally, and the table rows were `[type, expected]` while the
description template read `'returns %s for type %s'` — outputting
"returns application/vnd.librechat.docx-preview for type true"
instead of the intended "type ... returns true".

Reorder the template to match the column order. Test runner output
now reads naturally: "type application/vnd.librechat.docx-preview
returns true". Pure cosmetic — runtime behavior unchanged.
This commit is contained in:
Danny Avila 2026-05-04 10:57:26 +09:00
parent afa3c54990
commit b21989905e
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
client/src/utils/__tests__/artifacts.test.ts
View file

@ -724,7 +724,7 @@ describe('isPreviewOnlyArtifact', () => {
[TOOL_ARTIFACT_TYPES.MERMAID, false],
[TOOL_ARTIFACT_TYPES.CODE, false],
[TOOL_ARTIFACT_TYPES.PLAIN_TEXT, false],
])('returns %s for type %s', (type, expected) => {
])('type %s returns %s', (type, expected) => {
expect(isPreviewOnlyArtifact(type)).toBe(expected);
});