@iamsergio/qttest-utils 2.1.1 → 2.2.0

package/Changelog

@@ -2,6 +2,12 @@
 
 
 
+## [2.2.0] - 2024-04-25
+
+### 🚀 Features
+
+- Allow to workaround microsoft/vscode-cmake-tools-api/issues/7
+
 ## [2.1.1] - 2024-04-25
 
 ### 🐛 Bug Fixes
@@ -11,6 +17,7 @@
 ### ⚙️ Miscellaneous Tasks
 
 - Improve CONTRIBUTING.md
+- Bump version
 
 ## [2.1.0] - 2024-04-25
 

package/out/cmake.d.ts

@@ -13,8 +13,9 @@ export declare class CMakeTests {
      */
     tests(): Promise<CMakeTest[] | undefined>;
     private ctestJsonToList;
-    targetNameForExecutable(executable: string, codemodel: any): string | undefined;
-    cppFilesForExecutable(executable: string, codemodel: any): string[];
+    targetNameForExecutable(executable: string, codemodel: any, workaround?: boolean): string | undefined;
+    filenamesAreEqual(file1: string, file2: string, workaround?: boolean): boolean;
+    cppFilesForExecutable(executable: string, codemodel: any, workaround?: boolean): string[];
 }
 export declare class CMakeTest {
     command: string[];

package/out/cmake.js

@@ -74,13 +74,7 @@ class CMakeTests {
     }
     /// Returns the cmake target name for the specified executable
     /// codemodel should have a "projects" key at root.
-    targetNameForExecutable(executable, codemodel) {
-        // simplify:
-        if (executable.endsWith(".exe")) {
-            executable = executable.substring(0, executable.length - 4);
-        }
-        // replace backslashes with forward slashes
-        executable = executable.replace(/\\/g, "/");
+    targetNameForExecutable(executable, codemodel, workaround = false) {
         let projects = codemodel["projects"];
         if (!projects)
             return undefined;
@@ -96,9 +90,7 @@ class CMakeTests {
                     if (artifact.endsWith(".exe")) {
                         artifact = artifact.substring(0, artifact.length - 4);
                     }
-                    // replace backslashes with forward slashes
-                    artifact = artifact.replace(/\\/g, "/");
-                    if (artifact == executable) {
+                    if (this.filenamesAreEqual(executable, artifact, workaround)) {
                         let name = target["name"];
                         if (name) {
                             // We found the target name
@@ -110,16 +102,39 @@ class CMakeTests {
         }
         return undefined;
     }
+    /// Returns whether the two filenames are equal
+    /// If workaround is true, then we workaround microsoft/vscode-cmake-tools-api/issues/7 where
+    /// the basename is correct but the path is bogus, and we only compare the basenames
+    filenamesAreEqual(file1, file2, workaround = false) {
+        if (file1.endsWith(".exe"))
+            file1 = file1.substring(0, file1.length - 4);
+        if (file2.endsWith(".exe"))
+            file2 = file2.substring(0, file2.length - 4);
+        file1 = file1.replace(/\\/g, "/");
+        file2 = file2.replace(/\\/g, "/");
+        if (process.platform === "win32") {
+            file1 = file1.toLowerCase();
+            file2 = file2.toLowerCase();
+        }
+        if (file1 == file2)
+            return true;
+        if (!workaround) {
+            // files aren't equal!
+            return false;
+        }
+        const fs = require('fs');
+        if (fs.existsSync(file2)) {
+            // It's a real file, not bogus.
+            return false;
+        }
+        /// Compare only basename, since path is bogus
+        return path_1.default.basename(file1, ".exe") == path_1.default.basename(file2, ".exe");
+    }
     /// Returns the list of .cpp files for the specified executable
     /// codemodel is the CMake codemodel JSON object
     /// codemodel should have a "projects" key at root.
-    cppFilesForExecutable(executable, codemodel) {
-        // simplify:
-        if (executable.endsWith(".exe")) {
-            executable = executable.substring(0, executable.length - 4);
-        }
-        // replace backslashes with forward slashes
-        executable = executable.replace(/\\/g, "/");
+    /// @param workaround If true, worksaround https://github.com/microsoft/vscode-cmake-tools-api/issues/7
+    cppFilesForExecutable(executable, codemodel, workaround = false) {
         let projects = codemodel["projects"];
         if (!projects)
             return [];
@@ -141,7 +156,7 @@ class CMakeTests {
                     }
                     // replace backslashes with forward slashes
                     artifact = artifact.replace(/\\/g, "/");
-                    if (artifact == executable) {
+                    if (this.filenamesAreEqual(executable, artifact, workaround)) {
                         let fileGroups = target["fileGroups"];
                         if (!fileGroups)
                             continue;

package/out/test.js

@@ -143,6 +143,27 @@ function runCodeModelTests(codeModelFile) {
             console.error("Expected test2, got " + targetName);
             process.exit(1);
         }
+        // test workaround for microsoft/vscode-cmake-tools-api/issues/7
+        files = cmake.cppFilesForExecutable("/vscode-qttest/test/qt_test/build-dev/test3", codemodelJson, /*workaround=*/ false);
+        if (files.length !== 0) {
+            console.error("Expected 0 files, got " + files.length);
+            process.exit(1);
+        }
+        files = cmake.cppFilesForExecutable("/vscode-qttest/test/qt_test/build-dev/test3", codemodelJson, /*workaround=*/ true);
+        if (files.length !== 1) {
+            console.error("Expected 0 files, got " + files.length);
+            process.exit(1);
+        }
+        targetName = cmake.targetNameForExecutable("/vscode-qttest/test/qt_test/build-dev/test3", codemodelJson, /*workaround=*/ false);
+        if (targetName) {
+            console.error("Expected null, got " + targetName);
+            process.exit(1);
+        }
+        targetName = cmake.targetNameForExecutable("/vscode-qttest/test/qt_test/build-dev/test3", codemodelJson, /*workaround=*/ true);
+        if (targetName != "test3") {
+            console.error("Expected null, got " + targetName);
+            process.exit(1);
+        }
     });
 }
 runTests("test/qt_test/build-dev/");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iamsergio/qttest-utils",
-  "version": "2.1.1",
+  "version": "2.2.0",
   "description": "API for listing QtTest executables from a build directory and which individual test slots each executable contains. Useful for a Text Explorer VSCode extension.",
   "repository": {
     "type": "git",