deno: 22 of 24 orchestrion suites never load the real library
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 74/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- deno, typescript
- Domaine
- testing
Piste de recherche
Commencez dans dev-packages/deno-integration-tests/suites/orchestrion-mysql et orchestrion-postgres, puis comparez les autres suites orchestrion-*. Réutilisez la structure de scenario.mjs et de Deno.Command pour transformer express, redis et une suite d’un fournisseur d’AI en vrais scénarios. C’est terminé lorsque ces suites chargent la vraie bibliothèque et exécutent la transformation spécifique à Deno au lieu de publier manuellement le diagnostics channel.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In dev-packages/deno-integration-tests/suites/orchestrion-*, only orchestrion-mysql and orchestrion-postgres spawn a real deno run scenario. The other 22 publish the diagnostics channel by hand:
const channel = tracingChannel('orchestrion:openai:chat');
channel.start.runStores(ctx, () => undefined);
channel.end.publish(ctx);
That tests the subscriber, which is shared with Node and already covered there. It proves nothing about whether Deno's Module.registerHooks transform actually injects the channel into that package. The Deno-specific risk is exactly the part not covered.
Work item. Promote three or four suites to real scenarios: express (the most-used), redis, and one AI provider. Reuse the scenario.mjs + Deno.Command shape from orchestrion-mysql.
**Prior art **(related history only). No issue. getsentry/sentry-javascript#21827 (merged) created the deno-integration-tests package that these suites live in. #18635 (open) is the nearest umbrella.
- Langage dominant
- TypeScript
- Étoiles
- 8.7k
- Forks
- 1.9k
- Merge moyen
- 1 j 16 h
- PR mergées (30 j)
- 576
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de getsentry/sentry-javascript
-
Browser Bug Next.js Traces Waiting for: Product Owner
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
getsentry/sentry-javascript#24672 · 1 commentaire ·
-
javascript
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
getsentry/sentry-javascript#24200 · 2 commentaires ·
-
javascript Task
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
getsentry/sentry-javascript#24134 · 1 commentaire ·
-
Cloudflare Workers javascript Tests
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
getsentry/sentry-javascript#24051 · 1 commentaire ·
-
Bug Bun javascript
Difficulté 2/5 1-3 heures Accessibilité débutants 92/100
getsentry/sentry-javascript#24045 · 1 commentaire ·
Toutes les issues de getsentry/sentry-javascript
Issues similaires
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Crush Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
ElementsProject/cln-application#167 · 1 commentaire · 1 réaction ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Quantco/pnpm-licenses#17 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100