[PR #719] [MERGED] Fix building of tests #1690

New issue

Closed

opened 2026-05-05 07:59:34 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 07:59:34 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/debauchee/barrier/pull/719
Author: @p12tic
Created: 5/30/2020
Status: ✅ Merged
Merged: 5/30/2020
Merged by: @shymega

Base: master ← Head: tests

📝 Commits (8)

b99ca80 test: Enable building of tests
56f1232 test: Reduce the number of implicit include directories
a6e5e47 test: Remove unresolved merge conflict markers
7868043 test: Fix referenced library name
f23e8a6 test: Update to new APIs
13a0359 tests: Fix crash in XWindowsKeyStateTests
460f5df test: Fix undefined behavior of deleting incomplete type
64265e9 tests: Disable test that depends on external state

📊 Changes

9 files changed (+47 additions, -101 deletions)

View changed files

📝 src/CMakeLists.txt (+1 -0)
📝 src/lib/platform/XWindowsKeyState.h (+1 -1)
📝 src/test/global/TestEventQueue.h (+1 -1)
📝 src/test/integtests/CMakeLists.txt (+27 -15)
📝 src/test/integtests/arch/ArchInternetTests.cpp (+3 -3)
📝 src/test/integtests/platform/XWindowsKeyStateTests.cpp (+11 -24)
📝 src/test/integtests/platform/XWindowsScreenTests.cpp (+2 -3)
📝 src/test/unittests/CMakeLists.txt (+1 -2)
📝 src/test/unittests/barrier/ClipboardChunkTests.cpp (+0 -52)

📄 Description

This PR fixes building of tests. Two new executables appear in build/bin: integtests and unittests. The next step is to hook them to our CI.

This PR depends on #717 to fix build issue due to missing #include.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/debauchee/barrier/pull/719 **Author:** [@p12tic](https://github.com/p12tic) **Created:** 5/30/2020 **Status:** ✅ Merged **Merged:** 5/30/2020 **Merged by:** [@shymega](https://github.com/shymega) **Base:** `master` ← **Head:** `tests` --- ### 📝 Commits (8) - [`b99ca80`](https://github.com/debauchee/barrier/commit/b99ca80eaa15e93b6d0bda5138a5cf016f32cd04) test: Enable building of tests - [`56f1232`](https://github.com/debauchee/barrier/commit/56f1232352fb62592fe069c8a7317e1500885d5b) test: Reduce the number of implicit include directories - [`a6e5e47`](https://github.com/debauchee/barrier/commit/a6e5e47366bdd7b61b839287175636bdccd4e3fa) test: Remove unresolved merge conflict markers - [`7868043`](https://github.com/debauchee/barrier/commit/7868043ac4f6b515788118f89ef4ea10b3489f31) test: Fix referenced library name - [`f23e8a6`](https://github.com/debauchee/barrier/commit/f23e8a6ad92d2a56f2bf7bdcadaa779b56a376c4) test: Update to new APIs - [`13a0359`](https://github.com/debauchee/barrier/commit/13a0359c58f28789c76b633c463b837c0bb7fc92) tests: Fix crash in XWindowsKeyStateTests - [`460f5df`](https://github.com/debauchee/barrier/commit/460f5df8380e35989dbd64376fecf9c14a2ea2b0) test: Fix undefined behavior of deleting incomplete type - [`64265e9`](https://github.com/debauchee/barrier/commit/64265e926f1b3da9ac915bea5cad05e352707874) tests: Disable test that depends on external state ### 📊 Changes **9 files changed** (+47 additions, -101 deletions) <details> <summary>View changed files</summary> 📝 `src/CMakeLists.txt` (+1 -0) 📝 `src/lib/platform/XWindowsKeyState.h` (+1 -1) 📝 `src/test/global/TestEventQueue.h` (+1 -1) 📝 `src/test/integtests/CMakeLists.txt` (+27 -15) 📝 `src/test/integtests/arch/ArchInternetTests.cpp` (+3 -3) 📝 `src/test/integtests/platform/XWindowsKeyStateTests.cpp` (+11 -24) 📝 `src/test/integtests/platform/XWindowsScreenTests.cpp` (+2 -3) 📝 `src/test/unittests/CMakeLists.txt` (+1 -2) 📝 `src/test/unittests/barrier/ClipboardChunkTests.cpp` (+0 -52) </details> ### 📄 Description This PR fixes building of tests. Two new executables appear in `build/bin`: `integtests` and `unittests`. The next step is to hook them to our CI. This PR depends on #717 to fix build issue due to missing #include. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>