Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: sqliteai/sqlite-sync-react-native
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: sqliteai/sqlite-sync-react-native
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: feat/implement-tests
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 20 commits
  • 49 files changed
  • 2 contributors

Commits on Mar 3, 2026

  1. fix: parse JSON response from cloudsync_network_sync

    @damlayildiz
    damlayildiz committed Mar 3, 2026
    Configuration menu
    Copy the full SHA
    f1ceacf View commit details
    Browse the repository at this point in the history

Commits on Mar 6, 2026

  1. chore: add test-suite plan

    @damlayildiz
    damlayildiz committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    38a6fad View commit details
    Browse the repository at this point in the history

  2. chore: add @testing-library/react-native for hook tests 

    Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    b9816db View commit details
    Browse the repository at this point in the history

  3. test: add shared mocks for native modules 

    Mock files for op-sqlite, NetInfo, expo-notifications,
expo-secure-store, expo-task-manager, expo-constants,
and expo-application.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    4091751 View commit details
    Browse the repository at this point in the history

  4. test: add pure function and createDatabase tests (46 tests) 

    - testUtils.tsx with createTestWrapper for hook tests
- calculateAdaptiveSyncInterval (10 tests)
- isSqliteCloudNotification (12 tests)
- logger (8 tests)
- pushNotificationSyncCallbacks (5 tests)
- constants (2 tests)
- createDatabase (9 tests)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    7f57f07 View commit details
    Browse the repository at this point in the history

  5. test: add core logic tests (67 tests) 

    - initializeSyncExtension (11 tests)
- executeSync (14 tests)
- backgroundSyncConfig (10 tests)
- backgroundSyncRegistry (7 tests)
- executeBackgroundSync (13 tests)
- registerPushToken (12 tests)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    0f8638b View commit details
    Browse the repository at this point in the history

  6. test: add pushNotificationSyncTask, optionalDependencies, and context… 

    … hook tests (28 tests)

- pushNotificationSyncTask (8 tests)
- optionalDependencies (10 tests)
- useSqliteDb (2 tests)
- useSyncStatus (2 tests)
- useSqliteSync (2 tests)
- useInternalLogger (2 tests)
- useTriggerSqliteSync (2 tests)
- Fix react-test-renderer version to match react 19.1.0

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    cfe6d90 View commit details
    Browse the repository at this point in the history

  7. test: add hook tests for useSqliteExecute, useSqliteTransaction, useO… 

    …nTableUpdate, useInitialSync

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    99eaf44 View commit details
    Browse the repository at this point in the history

  8. test: add complex hook tests for sync manager, lifecycle, polling, pu… 

    …sh notifications, and reactive queries

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    524f7d7 View commit details
    Browse the repository at this point in the history

  9. test: add useDatabaseInitialization and SQLiteSyncProvider integratio… 

    …n tests

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    f78808d View commit details
    Browse the repository at this point in the history

  10. test: add coverage for permission fallback, auth, and config in SQLit… 

    …eSyncProvider

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    6e7cec3 View commit details
    Browse the repository at this point in the history

  11. chore: update gitignore

    @damlayildiz
    damlayildiz committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    adcd0a3 View commit details
    Browse the repository at this point in the history

  12. fix: skip push token registration when siteId is unavailable 

    When cloudsync_init fails or returns no siteId, skip registerPushToken
entirely instead of registering with undefined siteId. The registration
will retry on next app open since hasRequestedPermissionsRef resets on mount.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
    @damlayildiz @claude
    damlayildiz and claude committed Mar 6, 2026
    Configuration menu
    Copy the full SHA
    387c95d View commit details
    Browse the repository at this point in the history

Commits on Mar 9, 2026

  1. test: add Tier 1 coverage gap tests (262 total, 82.96% branches) 

    Add 10 targeted tests covering uncovered branches:
- useDatabaseInitialization: empty name, empty tables, close errors
- useSqliteExecute/Transaction: non-Error thrown value wrapping
- useSqliteSyncQuery: debounce timer clearing on query change
- usePushNotificationSync: registerPushToken failure handling
- isSqliteCloudNotification: Android dataString wrong URI
- useSyncManager: push-mode error skips interval recalculation
    @damlayildiz
    damlayildiz committed Mar 9, 2026
    Configuration menu
    Copy the full SHA
    29ec55b View commit details
    Browse the repository at this point in the history

  2. test: add Tier 2 coverage gap tests (272 total, 85.47% branches) 

    Add 10 targeted tests covering additional uncovered branches:
- logger: default parameter (debug=false) branch
- useNetworkListener: null isInternetReachable/isConnected coercion
- useAdaptivePollingSync: interval-to-null stops scheduling
- useOnTableUpdate: empty rows returns null row
- useSyncManager: Android with null isInternetReachable
- initializeSyncExtension: empty rows, site_id logging branches
- useSqliteSyncQuery: stale subscription signature skip
    @damlayildiz
    damlayildiz committed Mar 9, 2026
    Configuration menu
    Copy the full SHA
    34f43b5 View commit details
    Browse the repository at this point in the history

  3. test: suppress console.warn leak in backgroundSyncRegistry test

    @damlayildiz
    damlayildiz committed Mar 9, 2026
    Configuration menu
    Copy the full SHA
    7c87fdb View commit details
    Browse the repository at this point in the history

  4. chore: update .claude/plans/test-suite-design.md

    @damlayildiz
    damlayildiz committed Mar 9, 2026
    Configuration menu
    Copy the full SHA
    38491da View commit details
    Browse the repository at this point in the history

  5. fix: correct push token registration auth header and base URL 

    - Change auth header from Bearer connectionString?apikey=key to Bearer apiKey
- Update base URL to cloudsync-staging-testing.fly.dev
- Remove unused connectionString param from registerPushToken
    @damlayildiz
    damlayildiz committed Mar 9, 2026
    Configuration menu
    Copy the full SHA
    39f2073 View commit details
    Browse the repository at this point in the history

  6. fix: resolve TypeScript errors across test files 

    Add jest and node types to tsconfig.json, fix missing required fields
(syncMode, siteId, createTableSql) in test mocks, correct adaptive
config field names, and prefix unused variables with underscore.
    @damlayildiz
    damlayildiz committed Mar 9, 2026
    Configuration menu
    Copy the full SHA
    09e341f View commit details
    Browse the repository at this point in the history

  7. fix: imports

    @damlayildiz
    damlayildiz committed Mar 9, 2026
    Configuration menu
    Copy the full SHA
    6847368 View commit details
    Browse the repository at this point in the history
Loading