[PR #195] [MERGED] test(sandbox): add standard tests for all provider #198

Closed
opened 2026-02-16 06:17:25 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/deepagentsjs/pull/195
Author: @christian-bromann
Created: 2/6/2026
Status: Merged
Merged: 2/6/2026
Merged by: @christian-bromann

Base: mainHead: cb/sandbox-standard-tests


📝 Commits (10+)

📊 Changes

19 files changed (+1233 additions, -1334 deletions)

View changed files

📝 .github/workflows/ci.yml (+24 -2)
internal/standard-tests/package.json (+18 -0)
internal/standard-tests/src/index.ts (+16 -0)
internal/standard-tests/src/sandbox.ts (+600 -0)
internal/standard-tests/tsconfig.json (+11 -0)
📝 libs/providers/daytona/package.json (+1 -0)
📝 libs/providers/daytona/src/sandbox.int.test.ts (+124 -272)
📝 libs/providers/daytona/src/sandbox.ts (+55 -4)
📝 libs/providers/daytona/vitest.config.ts (+1 -0)
📝 libs/providers/deno/package.json (+1 -0)
📝 libs/providers/deno/src/sandbox.int.test.ts (+127 -499)
📝 libs/providers/deno/vitest.config.ts (+1 -0)
📝 libs/providers/modal/package.json (+1 -0)
📝 libs/providers/modal/src/sandbox.int.test.ts (+220 -544)
📝 libs/providers/modal/vitest.config.ts (+1 -0)
📝 libs/providers/node-vfs/package.json (+1 -0)
📝 libs/providers/node-vfs/src/sandbox.int.test.ts (+12 -13)
📝 pnpm-lock.yaml (+18 -0)
📝 pnpm-workspace.yaml (+1 -0)

📄 Description

With standard tests we can make sure we have common functionality for all provider.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/deepagentsjs/pull/195 **Author:** [@christian-bromann](https://github.com/christian-bromann) **Created:** 2/6/2026 **Status:** ✅ Merged **Merged:** 2/6/2026 **Merged by:** [@christian-bromann](https://github.com/christian-bromann) **Base:** `main` ← **Head:** `cb/sandbox-standard-tests` --- ### 📝 Commits (10+) - [`17ac3b9`](https://github.com/langchain-ai/deepagentsjs/commit/17ac3b992a5fbb330f9f3ad67867a874ec7cdf4c) test(sandbox): add standard tests for all provider - [`00ef0fb`](https://github.com/langchain-ai/deepagentsjs/commit/00ef0fbde195ec572b78cfebe8f3dd019166cbc1) cr - [`7775041`](https://github.com/langchain-ai/deepagentsjs/commit/77750412e471cbde47f61ab5b8278a7f540499cd) fix: allow to clean all daytona sandboxes - [`4fe2432`](https://github.com/langchain-ai/deepagentsjs/commit/4fe24320980a33446a1f2b2e1926d63361e5d009) apply CI labels - [`e6f06ee`](https://github.com/langchain-ai/deepagentsjs/commit/e6f06eebb55e22ae59db66547e0b8fc79d9023cc) withRetry - [`ce2a025`](https://github.com/langchain-ai/deepagentsjs/commit/ce2a025e189279b53bf5498374f193528d54691c) bump retry - [`f588425`](https://github.com/langchain-ai/deepagentsjs/commit/f588425303931ca63711abf5ee7544e34fc0c92d) format - [`20d6d5f`](https://github.com/langchain-ai/deepagentsjs/commit/20d6d5f27483a747c57191daedbd114c4997ab1c) cr - [`11d3f9b`](https://github.com/langchain-ai/deepagentsjs/commit/11d3f9be06421e72b05d3405c1282df4601095da) cr - [`485deaa`](https://github.com/langchain-ai/deepagentsjs/commit/485deaa60501c3893696f4fd3f13dacee790e519) tweak CI ### 📊 Changes **19 files changed** (+1233 additions, -1334 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+24 -2) ➕ `internal/standard-tests/package.json` (+18 -0) ➕ `internal/standard-tests/src/index.ts` (+16 -0) ➕ `internal/standard-tests/src/sandbox.ts` (+600 -0) ➕ `internal/standard-tests/tsconfig.json` (+11 -0) 📝 `libs/providers/daytona/package.json` (+1 -0) 📝 `libs/providers/daytona/src/sandbox.int.test.ts` (+124 -272) 📝 `libs/providers/daytona/src/sandbox.ts` (+55 -4) 📝 `libs/providers/daytona/vitest.config.ts` (+1 -0) 📝 `libs/providers/deno/package.json` (+1 -0) 📝 `libs/providers/deno/src/sandbox.int.test.ts` (+127 -499) 📝 `libs/providers/deno/vitest.config.ts` (+1 -0) 📝 `libs/providers/modal/package.json` (+1 -0) 📝 `libs/providers/modal/src/sandbox.int.test.ts` (+220 -544) 📝 `libs/providers/modal/vitest.config.ts` (+1 -0) 📝 `libs/providers/node-vfs/package.json` (+1 -0) 📝 `libs/providers/node-vfs/src/sandbox.int.test.ts` (+12 -13) 📝 `pnpm-lock.yaml` (+18 -0) 📝 `pnpm-workspace.yaml` (+1 -0) </details> ### 📄 Description With standard tests we can make sure we have common functionality for all provider. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 06:17:25 -05:00
yindo closed this issue 2026-02-16 06:17:25 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/deepagentsjs#198