Skip to content

Conversation

@altaskur
Copy link
Owner

@altaskur altaskur commented Feb 6, 2026

Description

Add coverage and configuration updates for the automatic updates flow.

Goal

Ensure coverage of new update paths (including getAppVersion) and use the updated Electron rebuild.

Key Changes

  • Replaced electron-rebuild with @electron/rebuild in postinstall and devDependencies; bumped version to 1.0.0-alpha.4.
  • Added spec for the Settings > Updates page covering toasts, dialogs, and the releases link.
  • Added UpdateService spec covering getAppVersion (success, error, missing version, API not available).

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 💥 Breaking change
  • 📝 Documentation update
  • 🎨 Style/UI changes
  • ♻️ Refactoring
  • ⚡ Performance improvement
  • ✅ Test updates
  • 🔧 Build/configuration changes

Impact Assessment

Database Impact

  • No database changes
  • New migration(s) included
  • Existing data migration required

Backup Impact

  • No impact on backups
  • Backup format changed
  • Restore compatibility maintained

Testing

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual testing
  • Tested with SonarQube analysis

(Not run in this iteration; pending.)

Test Steps

  1. npm test
  2. npm run test:electron
  3. npm run sonar:check

Test Configuration

  • Node version: (not provided)
  • npm version: (not provided)
  • Platform tested: (not provided)

UI Changes

No UI changes.

Checklist

  • My code follows the project's coding standards
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have run npm run lint and fixed any issues
  • I have run npm test and all tests pass
  • I have run npm run test:electron and all tests pass
  • I have run npm run sonar:check and the analysis passes
  • Any dependent changes have been merged and published

Breaking Changes

  • This PR contains no breaking changes

Related Issues

Closes #
Related to #

Additional Context

  • Added coverage for getAppVersion and new tests for the Settings > Updates page.

Reviewer Notes

  • Please verify the new tests pass in your environment (Karma and Vitest/Electron).
  • No database changes
  • New migration(s) included
  • Existing data migration required

Backup Impact

  • No impact on backups
  • Backup format changed
  • Restore compatibility maintained

Testing

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual testing
  • Tested with SonarQube analysis

Test Steps

Test Configuration

  • Node version:
  • npm version:
  • Platform tested:

UI Changes

Before

After

Checklist

  • My code follows the project's coding standards
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have run npm run lint and fixed any issues
  • I have run npm test and all tests pass
  • I have run npm run test:electron and all tests pass
  • I have run npm run sonar:check and the analysis passes
  • Any dependent changes have been merged and published

Breaking Changes

  • This PR contains breaking changes

Related Issues

Closes #
Related to #

Additional Context

Reviewer Notes

- Added "hono" version 4.11.8 and "lodash" version 4.17.21 to overrides.
- Moved "@electron/rebuild" to the devDependencies section.
- Updated "prisma" from version 6.19.2 to 7.1.0.
@altaskur altaskur merged commit 6f989ff into develop Feb 6, 2026
1 check passed
@altaskur altaskur deleted the update/bump-alfa-version branch February 6, 2026 21:37
@altaskur altaskur restored the update/bump-alfa-version branch February 6, 2026 22:53
@altaskur altaskur deleted the update/bump-alfa-version branch February 6, 2026 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant