From b54e4e14d413b9e43c1f021419bf21bade6b3015 Mon Sep 17 00:00:00 2001
From: Ben Deane <ben.deane@intel.com>
Date: Thu, 14 Mar 2024 07:59:54 -0600
Subject: [PATCH] CI updates for python tests

---
 .github/workflows/asciidoctor-ghpages.yml |  2 +-
 .github/workflows/unit_tests.yml          |  9 +++++++--
 test/mmio_bus_pbt.hpp                     | 12 ++++++------
 3 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/asciidoctor-ghpages.yml b/.github/workflows/asciidoctor-ghpages.yml
index f28b77d..0e55494 100644
--- a/.github/workflows/asciidoctor-ghpages.yml
+++ b/.github/workflows/asciidoctor-ghpages.yml
@@ -33,7 +33,7 @@ jobs:
     runs-on: ${{ github.repository_owner == 'intel' && 'intel-' || '' }}ubuntu-22.04
     steps:
       - name: Checkout source
-        uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
+        uses: actions/checkout@9bb56186c3b09b4f86b1c65136769dd318469633 # v4.1.2
       - name: Setup Node.js
         uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2
         with:
diff --git a/.github/workflows/unit_tests.yml b/.github/workflows/unit_tests.yml
index 3cd1deb..13b1b64 100644
--- a/.github/workflows/unit_tests.yml
+++ b/.github/workflows/unit_tests.yml
@@ -111,7 +111,12 @@ jobs:
       - name: Install build tools
         run: |
           ${{ matrix.install }}
-          sudo apt install -y ninja-build
+          sudo apt install -y python3-pip ninja-build
+
+      - name: Install python test tools
+        run: |
+          sudo pip3 install --upgrade pip
+          sudo pip3 install pytest pytest-forked hypothesis cppyy
 
       - name: Configure CMake
         env:
@@ -185,7 +190,7 @@ jobs:
         run: cmake -B ${{github.workspace}}/build -DCMAKE_CXX_STANDARD=${{env.DEFAULT_CXX_STANDARD}}
 
       - name: Build Unit Tests
-        run: cmake --build ${{github.workspace}}/build -t unit_tests
+        run: cmake --build ${{github.workspace}}/build -t cpp_tests
 
   valgrind:
     runs-on: ${{ github.repository_owner == 'intel' && 'intel-' || '' }}ubuntu-22.04
diff --git a/test/mmio_bus_pbt.hpp b/test/mmio_bus_pbt.hpp
index c0fd648..2cdd287 100644
--- a/test/mmio_bus_pbt.hpp
+++ b/test/mmio_bus_pbt.hpp
@@ -6,15 +6,15 @@
 #include <async/then.hpp>
 
 #include <groov/mmio_bus.hpp>
-#include <groov/path.hpp>       
+#include <groov/path.hpp>
 
 namespace test {
 uint64_t volatile reg64{};
 auto addr64 = reinterpret_cast<std::uintptr_t>(&reg64);
-            
-template<typename T, T mask, T id_mask, T id_value>
+
+template <typename T, T mask, T id_mask, T id_value>
 void bus_write(std::uintptr_t addr64, T write_value) {
-    groov::mmio_bus<>::write<mask, id_mask, id_value>(addr64, write_value) | 
-        async::sync_wait();                     
+    groov::mmio_bus<>::write<mask, id_mask, id_value>(addr64, write_value) |
+        async::sync_wait();
 }
-} // namespace test     
\ No newline at end of file
+} // namespace test