Fix failing remote tests

keepassxreboot / keepassxc

KeePassXC is a cross-platform community-driven port of the Windows application “Keepass Password Safe”.

https://keepassxc.org/

Other

20.07k stars 1.42k forks source link

Fix failing remote tests #10912

Closed t-h-e closed 2 weeks ago

t-h-e commented 2 weeks ago

Fixes failing remote tests:

QFATAL : TestGui::testRemoteSyncDatabaseRequiresPassword() Received signal 11
         Function time: 658ms Total time: 5010ms
FAIL!  : TestGui::testRemoteSyncDatabaseRequiresPassword() Received a fatal error.
   Loc: [Unknown file(0)]

Button text changed from "OK" to "&OK". With this PR, it should not matter anymore what the button text is.

Screenshots

Testing strategy

Type of change

✅ Bug fix (non-breaking change that fixes an issue)

t-h-e commented 2 weeks ago

Does not seem to fix the Received signal 11 issue, but seemed to fix one of the issue I had in https://github.com/keepassxreboot/keepassxc/pull/10896. As this code change is included in https://github.com/keepassxreboot/keepassxc/pull/10896 anyway, I will close this PR.