reenable office test suit

nfdi4plants · Nov 7, 2024 · 6f2e4dd · 6f2e4dd
1 parent a81a746
commit 6f2e4dd
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 16 deletions.
diff --git a/build/Build.fs b/build/Build.fs
@@ -347,13 +347,15 @@ Target.create "RunDB" (fun _ ->
 module Tests =
 
     let buildSharedTests () = run dotnet [ "build" ] sharedTestsPath
+    /// This disables microsoft data collection for using `office-addin-mock`
+    let disableUserData() = run npx [ "office-addin-usage-data"; "off" ] __SOURCE_DIRECTORY__
 
     let Watch() =
         [
             "server", dotnet [ "watch"; "run" ] serverTestsPath
             // This below will start web ui for tests, but cannot execute due to office-addin-mock
-            "client", dotnet [ "fable"; "watch"; "-o"; "output"; "-s"; "--run"; "npx"; "vite" ] clientTestsPath
-            // "client", dotnet [ "fable"; "watch"; "-o"; "output"; "-s"; "--run"; "npx"; "mocha"; $"{clientTestsPath}/output/Client.Tests.js" ] clientTestsPath
+            //"client", dotnet [ "fable"; "watch"; "-o"; "output"; "-s"; "--run"; "npx"; "vite" ] clientTestsPath
+            "client", dotnet [ "fable"; "watch"; "-o"; "output"; "-s"; "--run"; "npx"; "mocha"; $"{clientTestsPath}/output/Client.Tests.js" ] clientTestsPath
         ]
         |> runParallel
 
@@ -387,6 +389,7 @@ let main args =
         | _ -> Run(false); 0
     | "test" :: a ->
         Tests.buildSharedTests()
+        Tests.disableUserData()
         match a with
         | "watch" :: _ -> Tests.Watch(); 0
         | _ -> Tests.Run(); 0

diff --git a/src/Client/MainComponents/Cells.fs b/src/Client/MainComponents/Cells.fs
@@ -36,7 +36,6 @@ module private CellComponents =
             prop.children [Html.i [prop.classes ["fa-sharp"; "fa-solid"; "fa-angles-up"; if isExtended then "fa-rotate-270" else "fa-rotate-90"]; prop.style [style.fontSize(length.em 1)]]]
         ]
 
-
 module private CellAux =
 
     let headerTSRSetter (columnIndex: int, s: string, header: CompositeHeader, dispatch) =

diff --git a/tests/Client/Client.Tests.fs b/tests/Client/Client.Tests.fs
@@ -16,7 +16,7 @@ let all =
 #if FABLE_COMPILER // This preprocessor directive makes editor happy
             Shared.Tests.shared
 #endif
-            // OfficeAddin.Tests.Main
+            OfficeAddin.Tests.Main
             client
         ]
 

diff --git a/tests/Client/OfficeAddin.Tests.fs b/tests/Client/OfficeAddin.Tests.fs
@@ -8,16 +8,14 @@ open ExcelJS.Fable.Excel
 // ⚠️ These tests do only work on node.js as the office-addin-mock library sents user data to microsoft !!! ⚠️
 // Check out https://github.com/OfficeDev/Office-Addin-Scripts/issues/905
 
-// let private TestsBasic = testList "Basic tests" [
-//     testCaseAsync "develop mock" <| async {
-//         let testContext: RequestContext = importDefault "./OfficeMockObjects/ExampleObject.js"
-//         let! result = OfficeInterop.ExcelUtil.getSelectedRangeAdress testContext |> Async.AwaitPromise
-//         Expect.equal result "C2:G3" "Verify correct setup"
-//     }
-// ]
+let private TestsBasic = testList "Basic tests" [
+    testCaseAsync "develop mock" <| async {
+        let testContext: RequestContext = importDefault "./OfficeMockObjects/ExampleObject.js"
+        let! result = OfficeInterop.ExcelUtil.getSelectedRangeAdress testContext |> Async.AwaitPromise
+        Expect.equal result "C2:G3" "Verify correct setup"
+    }
+]
 
-// let Main = testList "OfficeAddin" [
-//     TestsBasic
-// ]
-
-let x = 0
+let Main = testList "OfficeAddin" [
+    TestsBasic
+]