Skip to content

Add feedback button with unit and UI tests - #111

Open
evatechnosoft wants to merge 3 commits into
lvgl:masterfrom
evatechnosoft:master
Open

Add feedback button with unit and UI tests#111
evatechnosoft wants to merge 3 commits into
lvgl:masterfrom
evatechnosoft:master

Conversation

@evatechnosoft

@evatechnosoft evatechnosoft commented Feb 28, 2026

Copy link
Copy Markdown

Summary

  • add a Give Feedback button to the simulator UI and FreeRTOS path
  • add shared feedback message logic for deterministic behavior
  • add unit test for message formatting
  • add LVGL UI integration test for click behavior
  • update CMake test wiring so logic test always runs and UI test runs when SDL2 is available

Validation

  • feedback_logic_test passes
  • feedback_ui_test passes

Summary by cubic

Add a “Give Feedback” button to the LVGL UI (simulator and FreeRTOS) with deterministic message formatting and click behavior. Update the build to run logic tests always and UI tests when SDL2 is available.

  • New Features

    • Added feedback_add_button: creates a status label and “Give Feedback” button; clicks update the label with “Thanks for your feedback! (n)”.
    • Integrated in simulator (lv_demo_widgets) and FreeRTOS hello world screen.
  • Tests

    • feedback_logic_test validates message formatting.
    • feedback_ui_test simulates button clicks and asserts label updates; built and run only if SDL2 is found.
    • CMake: enable_testing, always build/run logic test; make SDL2 optional and skip simulator target when missing.

Written for commit c2a0a9d. Summary will update on new commits.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 12 files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants