More coverage

Author: kunitoki

modules/yup_gui/buttons/yup_TextButton.cpp

@@ -48,8 +48,7 @@ void TextButton::setButtonText (StringRef newButtonText)
     {
         buttonText = newButtonText;
 
-        resized();
-        repaint();
+        updateTextLayout();
     }
 }
 
@@ -64,6 +63,13 @@ void TextButton::paintButton (Graphics& g)
 //==============================================================================
 
 void TextButton::resized()
+{
+    updateTextLayout();
+}
+
+//==============================================================================
+
+void TextButton::updateTextLayout()
 {
     auto textBounds = getTextBounds();
     auto font = ApplicationTheme::getGlobalTheme()->getDefaultFont();
@@ -78,6 +84,8 @@ void TextButton::resized()
 
     if (buttonText.isNotEmpty())
         modifier.appendText (buttonText, font.withHeight (getHeight() * 0.35f));
+
+    repaint();
 }
 
 //==============================================================================

modules/yup_gui/buttons/yup_TextButton.h

@@ -59,6 +59,8 @@ class YUP_API TextButton : public Button
     StyledText& getStyledText() const noexcept { return const_cast<StyledText&> (styledText); }
 
 private:
+    void updateTextLayout();
+
     String buttonText;
     StyledText styledText;
 };

tests/yup_audio_gui.cpp

@@ -24,4 +24,5 @@
 #include "yup_audio_gui/yup_AudioThumbnail.cpp"
 #include "yup_audio_gui/yup_AudioViewComponent.cpp"
 #include "yup_audio_gui/yup_CartesianPlane.cpp"
+#include "yup_audio_gui/yup_KMeterComponent.cpp"
 #include "yup_audio_gui/yup_SpectrumAnalyzerComponent.cpp"

tests/yup_audio_gui/KMeterComponent.cpp …ts/yup_audio_gui/yup_KMeterComponent.cpptests/yup_audio_gui/KMeterComponent.cpp renamed to tests/yup_audio_gui/yup_KMeterComponent.cpp tests/yup_audio_gui/KMeterComponent.cpp renamed to tests/yup_audio_gui/yup_KMeterComponent.cpp

@@ -25,6 +25,11 @@
 
 using namespace yup;
 
+namespace yup
+{
+extern std::unique_ptr<yup::GraphicsContext> yup_constructHeadlessGraphicsContext (yup::GraphicsContext::Options);
+} // namespace yup
+
 //==============================================================================
 TEST (KMeterComponentTests, DefaultsAreConfigured)
 {
@@ -57,3 +62,18 @@ TEST (KMeterComponentTests, SettersUpdateState)
     EXPECT_DOUBLE_EQ (1.5, meterState.getPeakHoldTime());
     EXPECT_EQ (KMeterState::OverCounterMode::total, meter.getOverCounterMode());
 }
+
+TEST (KMeterComponentTests, PaintWithThemeDoesNotCrash)
+{
+    KMeterState meterState (48000.0, 2);
+    KMeterComponent meter (meterState, 0);
+    meter.setBounds (0.0f, 0.0f, 60.0f, 240.0f);
+
+    auto context = yup_constructHeadlessGraphicsContext ({});
+    auto renderer = context->makeRenderer (120, 240);
+    Graphics g (*context, *renderer);
+
+    meter.paint (g);
+
+    EXPECT_TRUE (true);
+}

tests/yup_gui.cpp

@@ -32,4 +32,5 @@
 #include "yup_gui/yup_SwitchButton.cpp"
 #include "yup_gui/yup_TextButton.cpp"
 #include "yup_gui/yup_TextEditor.cpp"
+#include "yup_gui/yup_ThemeVersion1.cpp"
 #include "yup_gui/yup_ToggleButton.cpp"

tests/yup_gui/yup_ThemeVersion1.cpp

@@ -0,0 +1,92 @@
+/*
+  ==============================================================================
+
+   This file is part of the YUP library.
+   Copyright (c) 2026 - kunitoki@gmail.com
+
+   YUP is an open source library subject to open-source licensing.
+
+   The code included in this file is provided under the terms of the ISC license
+   http://www.isc.org/downloads/software-support-policy/isc-license. Permission
+   to use, copy, modify, and/or distribute this software for any purpose with or
+   without fee is hereby granted provided that the above copyright notice and
+   this permission notice appear in all copies.
+
+   YUP IS PROVIDED "AS IS" WITHOUT ANY WARRANTY, AND ALL WARRANTIES, WHETHER
+   EXPRESSED OR IMPLIED, INCLUDING MERCHANTABILITY AND FITNESS FOR PURPOSE, ARE
+   DISCLAIMED.
+
+  ==============================================================================
+*/
+
+#include <yup_audio_gui/yup_audio_gui.h>
+
+#include <gtest/gtest.h>
+
+using namespace yup;
+
+namespace yup
+{
+extern std::unique_ptr<yup::GraphicsContext> yup_constructHeadlessGraphicsContext (yup::GraphicsContext::Options);
+} // namespace yup
+
+TEST (ThemeVersion1Tests, PaintsCoreComponents)
+{
+    auto context = yup_constructHeadlessGraphicsContext ({});
+    auto renderer = context->makeRenderer (800, 600);
+    Graphics g (*context, *renderer);
+
+    Slider slider (Slider::LinearHorizontal);
+    slider.setRange (0.0, 1.0);
+    slider.setValue (0.25);
+    slider.setBounds (20.0f, 20.0f, 240.0f, 40.0f);
+    slider.paint (g);
+
+    TextButton textButton;
+    textButton.setBounds (20.0f, 80.0f, 140.0f, 40.0f);
+    textButton.paint (g);
+
+    ToggleButton toggleButton;
+    toggleButton.setToggleState (true, NotificationType::dontSendNotification);
+    toggleButton.setBounds (20.0f, 140.0f, 160.0f, 40.0f);
+    toggleButton.paint (g);
+
+    SwitchButton switchButton;
+    switchButton.setToggleState (true, NotificationType::dontSendNotification);
+    switchButton.setBounds (20.0f, 200.0f, 120.0f, 40.0f);
+    switchButton.paint (g);
+
+    TextEditor textEditor;
+    textEditor.setText ("Theme", NotificationType::dontSendNotification);
+    textEditor.setBounds (20.0f, 260.0f, 240.0f, 40.0f);
+    textEditor.paint (g);
+
+    ComboBox comboBox;
+    comboBox.addItem ("Item 1", 1);
+    comboBox.setSelectedId (1, NotificationType::dontSendNotification);
+    comboBox.setBounds (20.0f, 320.0f, 200.0f, 40.0f);
+    comboBox.paint (g);
+
+    Label label;
+    label.setText ("Label", NotificationType::dontSendNotification);
+    label.setBounds (20.0f, 380.0f, 200.0f, 30.0f);
+    label.paint (g);
+
+    ScrollBar scrollBar (ScrollBar::Orientation::horizontal);
+    scrollBar.setRangeLimits (0.0, 100.0);
+    scrollBar.setCurrentRange (20.0, 40.0);
+    scrollBar.setBounds (20.0f, 430.0f, 240.0f, 18.0f);
+    scrollBar.paint (g);
+
+    ProgressBar progressBar;
+    progressBar.setProgress (0.5, NotificationType::dontSendNotification);
+    progressBar.setBounds (20.0f, 470.0f, 240.0f, 20.0f);
+    progressBar.paint (g);
+
+    KMeterState meterState (48000.0, 2);
+    KMeterComponent meter (meterState, 0);
+    meter.setBounds (300.0f, 20.0f, 60.0f, 240.0f);
+    meter.paint (g);
+
+    EXPECT_TRUE (true);
+}