Skip to content

chore: delete obsolete e2e tests#4419

Merged
whysosaket merged 1 commit intomainfrom
chore/remove-dead-e2e-tests
Mar 19, 2026
Merged

chore: delete obsolete e2e tests#4419
whysosaket merged 1 commit intomainfrom
chore/remove-dead-e2e-tests

Conversation

@kartik-mem0
Copy link
Copy Markdown
Contributor

Description

Remove 5 dead E2E/integration test files that are always skipped in CI and never run anywhere in the pipeline. These tests are gated behind environment variables (MEM0_RUN_E2E,
LMSTUDIO_INTEGRATION) or local service availability checks (Qdrant, Redis) that are never set in any CI workflow, resulting in 101 permanently skipped tests and 4 skipped test suites on every PR.

Removed files:

  • src/client/tests/memoryClient.e2e.test.ts (61 tests)
  • src/oss/tests/memory.e2e.test.ts (19 tests)
  • src/oss/tests/lmstudio-integration.test.ts (10 tests)
  • src/oss/tests/qdrant-e2e.test.ts (11 tests)
  • src/oss/tests/redis-e2e.test.ts (17 tests)

Before: 28 passed, 4 skipped suites — 486 passed, 101 skipped tests
After: 28 passed, 0 skipped suites — 486 passed, 0 skipped tests

Type of change

  • Refactor (does not change functionality, e.g. code style improvements, linting)

How Has This Been Tested?

  • Unit Test

Ran pnpm run test:ci — all 28 test suites pass, 486 tests pass, 0 skipped.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@kartik-mem0 kartik-mem0 requested a review from whysosaket March 19, 2026 12:11
@mem0ai mem0ai deleted a comment Mar 19, 2026
@whysosaket whysosaket merged commit 66e6f58 into main Mar 19, 2026
6 checks passed
jamebobob pushed a commit to jamebobob/mem0-vigil-recall that referenced this pull request Mar 29, 2026
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.

2 participants