| romangg |
| KWin |
The current code used the same outputInterface between all tests, with
data and outdated connections slowly accumulating.
This meant most the code didn't work as it was intended, for
example testExampleConfig had the config applied from the connect in the
previous test. It just happened to pass.
This resets everything between each test.
Can now add a test without going insane
Existing tests still pass
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
There is no reason to use the member variable m_outputConfiguration and the method createConfig anymore. Just use a scoped pointer in the tests you need it. But we can leave it like this as well. Other inline comments are up to you.
| autotests/client/test_wayland_outputmanagement.cpp | ||
|---|---|---|
| 68 | Btw as a private method this test is not performed at all. | |
| 180 | m_registry->create(m_connection); | |
| 184 | m_connection->flush(); | |
| 476 | rm new line | |
| 512 | rm new line | |