Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

breg tests: stale re-freeze command, unused migration helpers, and a stale source view copy

未关闭 适合新手
#2,034 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
76/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
postgresql, rust
领域
databases, testing

调研方向

阅读 crates/registry-breg/tests/frozen_rhai_package.rs 中的 re-freeze 命令、postgres_migration target 的警告,以及 crates/registry-breg/tests/postgres_statistics_http.rs 中重新创建的源视图。在文档中的命令里添加 runtime 功能,删除或使用列出的未使用 helper,并使视图选项与引擎生成的选项一致。使用给定的 cargo check 命令确认,并验证 fixture 命令会写入 fixture。

由索引模型根据 Issue 内容生成。

描述

area:breg rust

Three small leftovers in the registry-breg test tree, noticed while working on #1564. None changes behaviour.

  1. The re-freeze command for the frozen Rhai package fixture runs no test. The header of crates/registry-breg/tests/frozen_rhai_package.rs documents cargo test --locked -p registry-breg --test frozen_rhai_package -- --ignored write_frozen_fixture. The file is gated on #![cfg(feature = "runtime")] and runtime is not a default feature, so the command exits green with 0 tests and writes nothing. It needs --features runtime.

  2. postgres_migration builds with 10 unused-code warnings under --features postgres-test,tooling, on main as of 2026-10-13:

    • unused imports successor_plan_is_empty_for_predecessor and successor_plan_is_empty
    • unused functions load_rewritten_person_package, copy_package_tree, person_registration_acceptance_root, person_registration_project, person_registration_assets, compile_person_registration, prepare_and_load_person_registration
    • source_bytes and drop_null_members from tests/support/source_bytes.rs, unused in this binary only

    They look like helpers of tests that were removed. No gate reports them because the clippy job does not build the postgres-test targets.

  3. Once #2030 merges, one test keeps a stale copy of the source view options. crates/registry-breg/tests/postgres_statistics_http.rs recreates a source view WITH (security_invoker=true, security_barrier=true) to inject a delay. The engine no longer emits security_barrier there. The test still passes; the copy should match the engine.

Done when

  • The documented re-freeze command writes the fixture.
  • cargo check --locked -p registry-breg --features postgres-test,tooling --test postgres_migration is free of warnings.
  • The statistics test recreates the view with the options the engine generates.
主要语言
Rust
星标
2
派生
0
平均合并
11 小时 3 分钟
30 天内合并 PR
246

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

registrystack/registry-stack 的其他 Issue

查看 registrystack/registry-stack 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。