Add macOS-x64 testing

This PR adds PR tests on macOS-x64
microsoft · Jul 20, 2021 · d66868a · d66868a
1 parent 7aa7bda
commit d66868a
Show file tree

Hide file tree

Showing 6 changed files with 67 additions and 8 deletions.
diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -116,6 +116,7 @@ jobs:
         which gdb
         echo 0 | sudo tee /proc/sys/kernel/yama/ptrace_scope
         # TODO: Try to make core dumps work
+        # Related CoreDump Tests: https://github.com/microsoft/MIEngine/issues/1170
         # echo 1 | sudo tee /proc/sys/kernel/core_uses_pid
         # ulimit -S -c unlimited
         # sudo sysctl -w kernel.core_pattern=${{ github.workspace }}/core.%e
@@ -150,4 +151,24 @@ jobs:
       run: dotnet build ${{ github.workspace }}/src/MIDebugEngine-Unix.sln
 
     - run: |
-        ${{ github.workspace }}/PublishOpenDebugAD7.sh -c Debug -o ${{ github.workspace }}/bin/DebugAdapterProtocolTests/Debug/extension/debugAdapters
+        ${{ github.workspace }}/PublishOpenDebugAD7.sh -c Debug -o ${{ github.workspace }}/bin/DebugAdapterProtocolTests/Debug/extension/debugAdapters
+        cp ${{ github.workspace }}/bin/DebugAdapterProtocolTests/Debug/CppTests/TestConfigurations/config_lldb.xml ${{ github.workspace }}/bin/DebugAdapterProtocolTests/Debug/CppTests/config.xml
+
+    - name: Download LLDB-MI
+      run: |
+        # Download the latest version of lldb-ui that vscode-cpptools uses.
+        curl https://download.visualstudio.microsoft.com/download/pr/173e6ced-0717-401d-87fc-169ca3424c72/f1228fd847c140b7f9839612f497bb7a/lldb-mi-10.0.0.zip --output ${{ github.workspace }}/lldb-mi-10.0.0.zip
+        unzip ${{ github.workspace }}/lldb-mi-10.0.0.zip
+
+        # Ensure we can run it or we will get permission denied.
+        sudo chmod 755 /Users/runner/work/MIEngine/MIEngine/debugAdapters/lldb-mi/bin/lldb-mi
+
+    - run: |
+        dotnet test ${{ github.workspace }}/bin/DebugAdapterProtocolTests/Debug/CppTests/CppTests.dll
+
+    - name: 'Upload Test Results'
+      uses: actions/upload-artifact@v2
+      if: ${{ always() }}
+      with:
+        name: osx_x64_results
+        path: ${{ github.workspace }}/bin/DebugAdapterProtocolTests/Debug/CppTests/results.trx
diff --git a/test/CppTests/TestConfigurations/config_lldb.xml b/test/CppTests/TestConfigurations/config_lldb.xml
@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<TestMachineConfiguration>
+  <TestConfigurations>
+    <TestConfiguration
+      Compiler="Clang"
+      DebuggeeArchitecture="x64"
+      Debugger="Lldb" />
+  </TestConfigurations>
+  <Compilers>
+    <Compiler
+      Name="Clang"
+      Type="ClangPlusPlus"
+      Path="/usr/bin/clang++" />
+  </Compilers>
+  <Debuggers>
+    <!-- By not specifying the Path, LLDB that ships with the extension will be used. -->
+    <Debugger
+      Name="Lldb"
+      Type="Lldb"
+      MIMode="lldb"
+      Path="/Users/runner/work/MIEngine/MIEngine/debugAdapters/lldb-mi/bin/lldb-mi" />
+  </Debuggers>
+</TestMachineConfiguration>
diff --git a/test/CppTests/Tests/BreakpointTests.cs b/test/CppTests/Tests/BreakpointTests.cs
@@ -157,8 +157,10 @@ public void FunctionBreakpointsBasic(ITestSettings settings)
         [Theory]
         [DependsOnTest(nameof(CompileKitchenSinkForBreakpointTests))]
         [RequiresTestSettings]
-        // TODO: Re-enable for Gdb_Gnu
-        [UnsupportedDebugger(SupportedDebugger.Gdb_Gnu | SupportedDebugger.Gdb_Cygwin | SupportedDebugger.Gdb_MinGW, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - gdb_gnu
+        // - lldb
+        [UnsupportedDebugger(SupportedDebugger.Lldb | SupportedDebugger.Gdb_Gnu | SupportedDebugger.Gdb_Cygwin | SupportedDebugger.Gdb_MinGW, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
         public void RunModeBreakpoints(ITestSettings settings)
         {
             this.TestPurpose("Tests setting breakpoints while in run mode");

diff --git a/test/CppTests/Tests/CoreDumpTests.cs b/test/CppTests/Tests/CoreDumpTests.cs
@@ -51,7 +51,8 @@ public CoreDumpTests(ITestOutputHelper outputHelper) : base(outputHelper)
         [Theory]
         [RequiresTestSettings]
         [SupportedPlatform(SupportedPlatform.Linux, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
-        // TODO: Re-enable for Gdb_Gnu
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - gdb_gnu
         [UnsupportedDebugger(SupportedDebugger.Gdb_Gnu, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
         public void CoreDumpBasic(ITestSettings settings)
         {
@@ -68,7 +69,8 @@ public void CoreDumpBasic(ITestSettings settings)
         [Theory]
         [RequiresTestSettings]
         [SupportedPlatform(SupportedPlatform.Linux, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
-        // TODO: Re-enable for Gdb_Gnu
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - gdb_gnu
         [UnsupportedDebugger(SupportedDebugger.Gdb_Gnu, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
         public void CoreDumpBasicMismatchedSourceAndSymbols(ITestSettings settings)
         {
@@ -88,7 +90,8 @@ public void CoreDumpBasicMismatchedSourceAndSymbols(ITestSettings settings)
         [Theory]
         [RequiresTestSettings]
         [SupportedPlatform(SupportedPlatform.Linux, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
-        // TODO: Re-enable for Gdb_Gnu
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - gdb_gnu
         [UnsupportedDebugger(SupportedDebugger.Gdb_Gnu, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
         public void CoreDumpVerifyActions(ITestSettings settings)
         {

diff --git a/test/CppTests/Tests/ExpressionTests.cs b/test/CppTests/Tests/ExpressionTests.cs
@@ -199,6 +199,9 @@ public void DataTipBasic(ITestSettings settings)
 
         [Theory]
         [DependsOnTest(nameof(CompileKitchenSinkForExpressionTests))]
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - lldb
+        [UnsupportedDebugger(SupportedDebugger.Lldb, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
         [RequiresTestSettings]
         public void CallStackBasic(ITestSettings settings)
         {
@@ -328,6 +331,9 @@ public void EvaluateOnFrames(ITestSettings settings)
 
         [Theory]
         [DependsOnTest(nameof(CompileKitchenSinkForExpressionTests))]
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - lldb
+        [UnsupportedDebugger(SupportedDebugger.Lldb, SupportedArchitecture.x64 | SupportedArchitecture.x86)]
         [RequiresTestSettings]
         public void EvaluateInvalidExpression(ITestSettings settings)
         {

diff --git a/test/CppTests/Tests/SharedLibTests.cs b/test/CppTests/Tests/SharedLibTests.cs
@@ -54,7 +54,9 @@ public void CompileSharedLibDebuggee(ITestSettings settings)
 
         [Theory]
         [DependsOnTest(nameof(CompileSharedLibDebuggee))]
-        [UnsupportedDebugger(SupportedDebugger.VsDbg, SupportedArchitecture.x86 | SupportedArchitecture.x64)]
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - lldb
+        [UnsupportedDebugger(SupportedDebugger.VsDbg | SupportedDebugger.Lldb, SupportedArchitecture.x86 | SupportedArchitecture.x64)]
         [RequiresTestSettings]
         public void SharedLibBasic(ITestSettings settings)
         {
@@ -66,7 +68,9 @@ public void SharedLibBasic(ITestSettings settings)
         }
 
         [Theory]
-        [UnsupportedDebugger(SupportedDebugger.VsDbg, SupportedArchitecture.x86 | SupportedArchitecture.x64)]
+        // TODO: https://github.com/microsoft/MIEngine/issues/1170
+        // - lldb
+        [UnsupportedDebugger(SupportedDebugger.VsDbg | SupportedDebugger.Lldb, SupportedArchitecture.x86 | SupportedArchitecture.x64)]
         [RequiresTestSettings]
         public void SharedLibMismatchSourceAndSymbols(ITestSettings settings)
         {