From b21989905ec532880d71023b394180bebc1bbd99 Mon Sep 17 00:00:00 2001
From: Danny Avila <danny@librechat.ai>
Date: Mon, 4 May 2026 10:57:26 +0900
Subject: [PATCH] =?UTF-8?q?=F0=9F=AA=9E=20chore:=20Fix=20isPreviewOnlyArti?=
 =?UTF-8?q?fact=20test=20description=20parameter=20order?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

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.
---
 client/src/utils/__tests__/artifacts.test.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/client/src/utils/__tests__/artifacts.test.ts b/client/src/utils/__tests__/artifacts.test.ts
index b6c55c39a8..607297f70b 100644
--- a/client/src/utils/__tests__/artifacts.test.ts
+++ b/client/src/utils/__tests__/artifacts.test.ts
@@ -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);
   });