Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix status bar update when switching to other DB #9073

Merged
merged 4 commits into from
Feb 8, 2023

Conversation

dmaslenko
Copy link
Contributor

Fixes #9071 bug.

Testing strategy

The same as described in the bug.

Type of change

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

@dmaslenko
Copy link
Contributor Author

I will add GUI test later.

@codecov-commenter
Copy link

codecov-commenter commented Feb 6, 2023

Codecov Report

Base: 64.83% // Head: 64.85% // Increases project coverage by +0.02% 🎉

Coverage data is based on head (ec6b4f8) compared to base (f703736).
Patch coverage: 100.00% of modified lines in pull request are covered.

❗ Current head ec6b4f8 differs from pull request most recent head bf976b0. Consider uploading reports for the commit bf976b0 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9073      +/-   ##
===========================================
+ Coverage    64.83%   64.85%   +0.02%     
===========================================
  Files          342      342              
  Lines        44392    44393       +1     
===========================================
+ Hits         28781    28789       +8     
+ Misses       15611    15604       -7     
Impacted Files Coverage Δ
src/gui/MainWindow.cpp 71.59% <100.00%> (+0.02%) ⬆️
src/core/FileWatcher.cpp 85.54% <0.00%> (-1.20%) ⬇️
src/core/Entry.cpp 82.42% <0.00%> (-0.30%) ⬇️
src/fdosecrets/dbus/DBusMgr.cpp 53.67% <0.00%> (+1.47%) ⬆️
...rc/fdosecrets/widgets/SettingsWidgetFdoSecrets.cpp 59.09% <0.00%> (+3.03%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[What]
  1) Defined the object name for the StatusBarLabel.
  2) Implemented a separate method to find the StatusBarLabel, check the
     visibility and compare the actual text with given expected one.
  3) Added the validation of the StatusBarLabel in some tests where the
     entries are adding or deleting.

[Why]
  This is a part of the StatusBarLabel bug fixing.
  The StatusBarLabel was not verified in GUI tests.
[What]
  Added verification of the StatusBarLabel after new DB creation and on
  DB tab switching.

[Why]
  This is a part of the StatusBarLabel bug fixing.
@droidmonkey droidmonkey added this to the v2.7.5 milestone Feb 8, 2023
@droidmonkey droidmonkey merged commit 5bd8715 into keepassxreboot:develop Feb 8, 2023
@dmaslenko dmaslenko deleted the fix/status_bar_update branch February 8, 2023 06:49
pull bot pushed a commit to tigerwill90/keepassxc that referenced this pull request Feb 8, 2023
* Gui tests: add validation of StatusBarLabel in some tests
droidmonkey pushed a commit that referenced this pull request Feb 18, 2023
* Gui tests: add validation of StatusBarLabel in some tests
@droidmonkey droidmonkey added the pr: backported Pull request backported to previous release label Feb 18, 2023
@droidmonkey
Copy link
Member

The gui tests from this commit are causing problems when I merged it over to the 2.7.x branch. I am going to correct them in a follow up PR.

droidmonkey added a commit that referenced this pull request Feb 19, 2023
* Gui tests: add validation of StatusBarLabel in some tests

Correct testing status bar text
droidmonkey added a commit that referenced this pull request Feb 19, 2023
* Gui tests: add validation of StatusBarLabel in some tests
@dmaslenko
Copy link
Contributor Author

Hm, wondering what kind of GUI test problem?

@droidmonkey
Copy link
Member

The checked string is localization dependent with quantity replacements.

Perlover added a commit to Perlover/keepassxc that referenced this pull request May 18, 2023
Release 2.7.5

- Add menu option to allow screenshots [keepassxreboot#8841]
- Add support for Botan 3 [keepassxreboot#9388]
- Increase max TOTP step to 24 hours [keepassxreboot#9149]
- Improve HTML export layout [keepassxreboot#8987]
- Turn search reset off by default [keepassxreboot#9153]
- Use QClipboard::clear() instead of setting blank text [keepassxreboot#9148]
- Hide group column header choice when not in search [keepassxreboot#9171]
- Improve look of KeePassXC logo and icons [keepassxreboot#9355]
- Add keyboard shortcuts for app and database settings [keepassxreboot#9007]
- Hide rename button from attachments preview panel [keepassxreboot#8842]
- Linux: Set SingleMainWindow in .desktop file [keepassxreboot#7430]

- Fix crash when search clears while creating new entry [keepassxreboot#9230]
- Fix crash when using Windows Hello in a Remote Desktop session [keepassxreboot#9006]
- Fix crash in Group Edit after enabling Browser Integration [keepassxreboot#8778]
- Fix canceling quick unlock when it is unavailable [keepassxreboot#9034]
- Set password input field font correctly [keepassxreboot#8732]
- Greatly improve performance when rendering entry view [keepassxreboot#9398]
- Fix various accessibility issues [keepassxreboot#9138]
- Fix arrows size when expand/collapse a group [keepassxreboot#9096]
- Select the clone instead of the original after cloning an entry [keepassxreboot#9070]
- Fix bugs with preview widget [keepassxreboot#9170]
- Fix status bar update when switching to other DB [keepassxreboot#9073]
- Fix database settings spin box bug [keepassxreboot#9101]
- Fix Ctrl+Tab shortcut to cycle databases in unlock dialog [keepassxreboot#8839]
- Fix TOTP QR code maintaining square ratio [keepassxreboot#9027]
- Fix Auto-Type configuration page on custom sequence selection [keepassxreboot#8752]
- Fix unexpected behavior of `--lock` when KeePassXC is not running [keepassxreboot#8889]
- Make open folder icon exempt from "Apply group icon to entry" [keepassxreboot#9205]
- Allow setting default file open directory with env var [keepassxreboot#9192]
- SSH Agent: Fix support for AES-256/GCM openssh keys [keepassxreboot#8968]
- Browser: Fix Native Messaging script path with BSD OS's [keepassxreboot#8835]
- MacOS: Fix text selection for Auto-Type clear field [keepassxreboot#9066]
- MacOS: Don't rely on AppleInterfaceStyle for theme switching [keepassxreboot#8615]
- Windows: Remove registry detection of desktop shortcut [keepassxreboot#9380]
@phoerious phoerious added pr: bugfix Pull request that fixes a bug and removed bug labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: backported Pull request backported to previous release pr: bugfix Pull request that fixes a bug user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Number of entries not updated when switching between databases
4 participants