Repository: scikit-hep/awkward

Issue:
run cpp tests in CI 
### Version of Awkward Array

2.0.6

### Description and code to reproduce

@agoose77 and @jpivarski - I think, we need to have at least one node to run the cpp tests.


Discussion:
Agreed! IIRC the C++ tests are only testing the shared libraries, right? Perhaps they should be moved back into the root Git directory, and then built as a dedicated target. We can discuss today :)

Resolving patch:
diff --git a/dev/copy-cpp-headers.py b/dev/copy-cpp-headers.py
--- a/dev/copy-cpp-headers.py
+++ b/dev/copy-cpp-headers.py
@@ -3,7 +3,7 @@
 import shutil
 
 root_path = pathlib.Path(__file__).absolute().parents[1]
-source_path = root_path / "header-only"
+source_path = root_path / "header-only" / "include"
 dest_paths = (
     root_path / "awkward-cpp" / "header-only",
     root_path / "src" / "awkward" / "_connect" / "header-only",


Test patch:
diff --git a/.github/workflows/tests.yml b/.github/workflows/packaging-test.yml
similarity index 81%
rename from .github/workflows/tests.yml
rename to .github/workflows/packaging-test.yml
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/packaging-test.yml
@@ -1,4 +1,4 @@
-name: Tests
+name: Packaging Tests
 
 on:
   pull_request:
@@ -6,28 +6,13 @@ on:
 
 
 concurrency:
-  group: wheels-${{ github.head_ref }}
+  group: packaging-test-${{ github.head_ref }}
   cancel-in-progress: true
 
 env:
   SOURCE_DATE_EPOCH: "1668811211"
 
 jobs:
-  pylint:
-    name: "PyLint"
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v3
-      with:
-        submodules: true
-
-    - name: Run PyLint
-      run: |
-        echo "::add-matcher::$GITHUB_WORKSPACE/.github/matchers/pylint.json"
-        pipx run nox -s pylint
-
-
   make_sdists:
     name: "Make SDists"
     runs-on: ubuntu-latest
@@ -56,7 +41,7 @@ jobs:
 
 
   build_wheels:
-    name: "Wheel: ${{ matrix.os }}"
+    name: "Make Wheel: ${{ matrix.os }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
diff --git a/.github/workflows/build-test.yml b/.github/workflows/test.yml
similarity index 99%
rename from .github/workflows/build-test.yml
rename to .github/workflows/test.yml
--- a/.github/workflows/build-test.yml
+++ b/.github/workflows/test.yml
@@ -1,4 +1,4 @@
-name: Build Test
+name: Tests
 
 on:
   pull_request:
@@ -19,7 +19,7 @@ on:
   workflow_dispatch:
 
 concurrency:
-  group: 'build-test-${{ github.head_ref || github.run_id }}'
+  group: 'test-${{ github.head_ref || github.run_id }}'
   cancel-in-progress: true
 
 env:
diff --git a/awkward-cpp/tests-cpp/CMakeLists.txt b/header-only/tests/CMakeLists.txt
similarity index 95%
rename from awkward-cpp/tests-cpp/CMakeLists.txt
rename to header-only/tests/CMakeLists.txt
--- a/awkward-cpp/tests-cpp/CMakeLists.txt
+++ b/header-only/tests/CMakeLists.txt
@@ -14,7 +14,7 @@ set(CMAKE_VISIBILITY_INLINES_HIDDEN ON)
 
 # C++ dependencies (header-only): GrowableBuffer
 add_library(awkward-parent INTERFACE)
-target_include_directories(awkward-parent INTERFACE ../header-only)
+target_include_directories(awkward-parent INTERFACE ../include)
 target_compile_features(awkward-parent INTERFACE cxx_std_11)
 
 macro(addtest name filename)
diff --git a/awkward-cpp/tests-cpp/test_1494-layout-builder.cpp b/header-only/tests/test_1494-layout-builder.cpp
similarity index 100%
rename from awkward-cpp/tests-cpp/test_1494-layout-builder.cpp
rename to header-only/tests/test_1494-layout-builder.cpp
diff --git a/awkward-cpp/tests-cpp/test_1542-array-builder.cpp b/header-only/tests/test_1542-array-builder.cpp
similarity index 100%
rename from awkward-cpp/tests-cpp/test_1542-array-builder.cpp
rename to header-only/tests/test_1542-array-builder.cpp
diff --git a/awkward-cpp/tests-cpp/test_1542-growable-buffer.cpp b/header-only/tests/test_1542-growable-buffer.cpp
similarity index 100%
rename from awkward-cpp/tests-cpp/test_1542-growable-buffer.cpp
rename to header-only/tests/test_1542-growable-buffer.cpp
diff --git a/awkward-cpp/tests-cpp/test_1560-builder-options.cpp b/header-only/tests/test_1560-builder-options.cpp
similarity index 100%
rename from awkward-cpp/tests-cpp/test_1560-builder-options.cpp
rename to header-only/tests/test_1560-builder-options.cpp