Skip to content

Browser SDK OPFS test suite is fully skipped #3970

Description

@omerbek

Summary

sdks/js/browser-sdk/test/Opfs.test.ts currently wraps the entire OPFS suite in describe.skip("Opfs", ...), so none of the tests for Opfs run.

The skipped coverage includes:

  • listFiles() / fileCount()
  • poolCapacity()
  • fileExists()
  • deleteFile()
  • exportDb() / importDb()
  • clearAll()

Impact

Opfs is exported by the browser SDK and manages the SDK's OPFS-backed SQLite files, but its main behavior is not currently covered by CI. Regressions in import/export/delete/cleanup behavior can slip through unnoticed.

Expected behavior

The OPFS tests should either run in an environment that supports OPFS, or be conditionally skipped with an explicit feature/environment guard instead of skipping the whole suite unconditionally.

Possible fix

Move the suite to a browser-capable test environment, or replace the unconditional describe.skip with a feature-detected describe.skipIf(...)/equivalent so the tests run wherever OPFS is available.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions