Skip to content

test: Make test class names uniq#60

Merged
tomasz-blasz merged 1 commit intomainfrom
topic/uniq-test-names
Mar 12, 2026
Merged

test: Make test class names uniq#60
tomasz-blasz merged 1 commit intomainfrom
topic/uniq-test-names

Conversation

@tomasz-blasz
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 12, 2026 09:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates GoogleTest fixture class names to be unique across unit and component test binaries, avoiding naming collisions (e.g., in unity/jumbo builds or when linking tests together).

Changes:

  • Rename unit test fixtures from *Test to *UTest (and update corresponding TEST_F usages).
  • Rename component test fixtures from *Test to *CTest (and update corresponding TEST_F usages).
  • Apply the renaming consistently across multiple feature test suites (TextToSpeech, Metrics, Network, etc.).

Reviewed changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/unit/textToSpeechTest.cpp Renames fixture to TextToSpeechUTest and updates all TEST_F references.
test/unit/statsTest.cpp Renames fixture to StatsUTest and updates TEST_F references.
test/unit/presentationTest.cpp Renames fixture to PresentationUTest and updates TEST_F references.
test/unit/networkTest.cpp Renames fixture to NetworkUTest and updates TEST_F references.
test/unit/metricsTest.cpp Renames fixture to MetricsUTest and updates TEST_F references.
test/unit/lifecycleTest.cpp Renames fixture to LifecycleUTest and updates TEST_F references.
test/unit/displayTest.cpp Renames fixture to DisplayUTest and updates TEST_F references.
test/unit/discoveryTest.cpp Renames fixture to DiscoveryUTest and updates TEST_F references.
test/unit/deviceTest.cpp Renames fixture to DeviceUTest and updates TEST_F references.
test/unit/advertisingTest.cpp Renames fixture to AdvertisingUTest and updates TEST_F references.
test/unit/accessibilityTest.cpp Renames fixture to AccessibilityUTest and updates TEST_F references.
test/component/textToSpeechTest.cpp Renames fixture to TextToSpeechCTest and updates all TEST_F references.
test/component/statsTest.cpp Renames fixture to StatsCTest and updates TEST_F references.
test/component/presentationTest.cpp Renames fixture to PresentationCTest and updates TEST_F references.
test/component/networkTest.cpp Renames fixture to NetworkCTest and updates TEST_F references.
test/component/metricsTest.cpp Renames fixture to MetricsCTest and updates TEST_F references.
test/component/localizationTest.cpp Renames fixture to LocalizationCTest and updates TEST_F references.
test/component/lifecycleTest.cpp Renames fixture to LifecycleCTest and updates TEST_F references.
test/component/displayTest.cpp Renames fixture to DisplayCTest and updates TEST_F references.
test/component/discoveryTest.cpp Renames fixture to DiscoveryCTest and updates TEST_F references.
test/component/deviceTest.cpp Renames fixture to DeviceCTest and updates TEST_F references.
test/component/advertisingTest.cpp Renames fixture to AdvertisingCTest and updates TEST_F references.
test/component/accessibilityTest.cpp Renames fixture to AccessibilityCTest and updates TEST_F references.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tomasz-blasz tomasz-blasz merged commit 0600a3c into main Mar 12, 2026
16 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 12, 2026
@tomasz-blasz tomasz-blasz deleted the topic/uniq-test-names branch March 12, 2026 09:40
@tomasz-blasz
Copy link
Contributor Author

🎉 This PR is included in version 0.5.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants