Fix/27203/enable context path completion for chart (#677)

* fix: contextpath completion enabled for chart * fix: change set
SAP · Nov 22, 2023 · c8768ac · c8768ac
1 parent 4b4acce
commit c8768ac
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/.changeset/giant-jobs-yawn.md b/.changeset/giant-jobs-yawn.md
@@ -0,0 +1,7 @@
+---
+"@ui5-language-assistant/fe": patch
+"vscode-ui5-language-assistant": patch
+"@ui5-language-assistant/vscode-ui5-language-assistant-bas-ext": patch
+---
+
+contextPath attribute value completion enabled for Chart macros element
diff --git a/packages/fe/src/services/completion/providers/context-path.ts b/packages/fe/src/services/completion/providers/context-path.ts
@@ -47,14 +47,9 @@ export function contextPathSuggestions({
     context.ui5Model
   );
 
-  // provider is blocked and is used in tests only
-  // reserved for the future to be reused in binding expressions
-  if (!(context as unknown as { forTest: boolean }).forTest) {
-    return [];
-  }
-
   if (
     ui5Property?.library === SAP_FE_MACROS &&
+    ui5Property.parent?.name === "Chart" &&
     ui5Property.name === "contextPath"
   ) {
     const mainServicePath = context.manifestDetails.mainServicePath;