Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

Nightly e2e_bun_lockb red: offline takeover test predates #1648

Fechada
  • Algo errado?
#1,770 2 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Um pull request relacionado já foi integrado.

  • #1771 de @mikolalysenko — integrado

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
35/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
github-actions, rust
Domínio
testing

Direção de pesquisa

Start at crates/socket-patch-cli/tests/e2e_bun_lockb.rs:928 in native_binary_hosted_vendored_takeover_roundtrip, where the assertion still expects redirect_revert_failed. Run that test with bun 1.0.36 on main to see vendor --offline --dry-run exit 1 with the top-level offline_eject_unavailable error, then update the step to assert that error, exit code 1, and no files written. Check the remaining roundtrip assertions still hold. Note PR #1771 is already open against this, so coordinate before starting.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

agent:claimed agent:triaged priority:p3

[agent] The nightly e2e-extended legs e2e_bun_lockb (bun 1.0.36 and 1.1.45) have been red on main since #1648 merged.

Evidence

  • Run https://github.com/SocketDev/socket-patch/actions/runs/40639505834 (schedule, main b787454, 2026-10-28), jobs 122654524514 and 122654524565.
  • native_binary_hosted_vendored_takeover_roundtrip panics at crates/socket-patch-cli/tests/e2e_bun_lockb.rs:928: expected redirect_revert_failed, but vendor --offline --dry-run now returns the top-level error offline_eject_unavailable ("ejecting 1 hosted package needs 1 patch record(s) from the Socket API, and this run is offline; re-run without --offline").
  • Reproduces locally with bun 1.0.36 on origin/main.

Cause
#1648 (vendor hosted patches directly from lockfiles) made vendor --offline refuse up front whenever a hosted pin is in scope, which its description records as the chosen behaviour. This test still expects the older per-pin redirect_revert_failed event. It only runs on the bun 1.0/1.1 legs, which are nightly-only, so the PR and merge-queue runs never exercised it.

Fix
Update the test's offline-refusal step to the new contract (exit 1, offline_eject_unavailable, nothing written), and check that the rest of the roundtrip still holds.

Generated by Claude Code

Linguagem predominante
Rust
Estrelas
8
Forks
2
Merge médio
19h 31min
PRs com merge (30d)
686

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de SocketDev/socket-patch

Todas as issues de SocketDev/socket-patch

Issues semelhantes

Mais issues de Rust

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.