Add a test for including pluginval as a dependency

Author: sudara

.github/workflows/build.yaml

@@ -167,6 +167,33 @@ jobs:
           name: ${{ env.ZIP_FILE_NAME }}
           path: ${{ env.APP_DIR }}/${{ env.ZIP_FILE_NAME }}
 
+  # Test pluginval as a dependency in another JUCE project
+  as_dependency:
+    name: Pluginval as Dependency
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, macos-15, windows-latest]
+        juce: ["8.0.3", "8.0.11"]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install dependencies (Linux)
+        if: runner.os == 'Linux'
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y libasound2-dev libfreetype6-dev libx11-dev libxcomposite-dev libxcursor-dev libxinerama-dev libxrandr-dev mesa-common-dev libwebkit2gtk-4.1-dev
+
+      - name: Configure
+        run: cmake -B build -S tests/pluginval_as_dependency -DJUCE_VERSION=${{ matrix.juce }}
+
+      - name: Build
+        run: cmake --build build --target pluginval --parallel 4
+
+      - name: Verify
+        run: cmake --build build --target verify_pluginval
+
   # Create release for tagged refs
   deploy:
     if: contains(github.ref, 'tags/v')

tests/pluginval_as_dependency/CMakeLists.txt

@@ -0,0 +1,31 @@
+cmake_minimum_required(VERSION 3.22)
+project(PluginvalAsDependency VERSION 1.0.0)
+
+set(JUCE_VERSION "8.0.3" CACHE STRING "JUCE version to test against")
+
+include(${CMAKE_CURRENT_LIST_DIR}/../../cmake/CPM.cmake)
+
+CPMAddPackage("gh:juce-framework/JUCE#${JUCE_VERSION}")
+
+# Minimal plugin to test pluginval as a dependency
+# Plugin name must match CMAKE_PROJECT_NAME for pluginval's CLI target to work
+# Must be defined BEFORE adding pluginval
+juce_add_plugin(${CMAKE_PROJECT_NAME}
+    PLUGIN_MANUFACTURER_CODE Test
+    PLUGIN_CODE Test
+    FORMATS VST3 AU
+    PRODUCT_NAME "TestPlugin")
+
+target_sources(${CMAKE_PROJECT_NAME} PRIVATE TestPlugin.cpp)
+target_link_libraries(${CMAKE_PROJECT_NAME} PRIVATE juce::juce_audio_processors)
+
+CPMAddPackage(
+    NAME pluginval
+    SOURCE_DIR ${CMAKE_CURRENT_LIST_DIR}/../..
+)
+
+# Verify pluginval runs and reports the expected JUCE version
+add_custom_target(verify_pluginval
+    COMMAND $<TARGET_FILE:pluginval> --help | grep "JUCE v${JUCE_VERSION}"
+    DEPENDS pluginval
+    COMMENT "Verify pluginval built with JUCE ${JUCE_VERSION}")

tests/pluginval_as_dependency/TestPlugin.cpp

@@ -0,0 +1,39 @@
+// Minimal JUCE plugin to test pluginval builds as a dependency
+#include <juce_audio_processors/juce_audio_processors.h>
+
+class TestProcessor : public juce::AudioProcessor
+{
+public:
+    TestProcessor()
+        : AudioProcessor (BusesProperties()
+            .withInput ("Input", juce::AudioChannelSet::stereo(), true)
+            .withOutput ("Output", juce::AudioChannelSet::stereo(), true))
+    {
+    }
+
+    const juce::String getName() const override { return "TestPlugin"; }
+    bool acceptsMidi() const override { return false; }
+    bool producesMidi() const override { return false; }
+    double getTailLengthSeconds() const override { return 0.0; }
+
+    int getNumPrograms() override { return 1; }
+    int getCurrentProgram() override { return 0; }
+    void setCurrentProgram (int) override {}
+    const juce::String getProgramName (int) override { return {}; }
+    void changeProgramName (int, const juce::String&) override {}
+
+    void prepareToPlay (double, int) override {}
+    void releaseResources() override {}
+    void processBlock (juce::AudioBuffer<float>&, juce::MidiBuffer&) override {}
+
+    bool hasEditor() const override { return false; }
+    juce::AudioProcessorEditor* createEditor() override { return nullptr; }
+
+    void getStateInformation (juce::MemoryBlock&) override {}
+    void setStateInformation (const void*, int) override {}
+};
+
+juce::AudioProcessor* JUCE_CALLTYPE createPluginFilter()
+{
+    return new TestProcessor();
+}