Skip to content

Remove unused old test mocks - #8510

Merged
nbolton merged 1 commit into
masterfrom
rmUnusedMocks
Apr 21, 2025
Merged

Remove unused old test mocks#8510
nbolton merged 1 commit into
masterfrom
rmUnusedMocks

Conversation

@sithlord48

@sithlord48 sithlord48 commented Apr 19, 2025

Copy link
Copy Markdown
Member

I Found some unused mock objects when working on the QtTests. This PR removes them.

  • Remove unused src/test/mock/server/MockServer.h
  • Remove unused src/test/shared/gui/TestQtFullApp.h

@sithlord48 sithlord48 mentioned this pull request Apr 19, 2025
44 tasks
@sithlord48
sithlord48 requested a review from nbolton April 20, 2025 03:12
@sithlord48 sithlord48 added the 🗒️ task Doesn't directly add new functionality or fix bugs in the program label Apr 20, 2025
@sithlord48 sithlord48 changed the title chore: remove unused old test mocks Remove unused old test mocks Apr 20, 2025
@nbolton
nbolton enabled auto-merge (rebase) April 21, 2025 21:01
@nbolton
nbolton merged commit f5f3bc0 into master Apr 21, 2025
@nbolton
nbolton deleted the rmUnusedMocks branch April 21, 2025 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗒️ task Doesn't directly add new functionality or fix bugs in the program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants