Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

No CI: 152 smoke tests, rigs and a fuzz harness exist but nothing runs them

Đang mở
  • Có gì không ổn?
#414 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
78/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, php
Lĩnh vực
ci-cd, testing

Hướng nghiên cứu

Start with homeboy.json and its scripts.lint and scripts.test recipes, then inspect the repository for the absent .github/workflows directory. Add a workflow that invokes those recipes for pull requests and pushes to main, and verify it reports status checks while a failing smoke test prevents a green result. Use tests/smoke-*.php as the suite being exercised.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Summary

This repository has no CI. There is no .github/workflows directory and no configuration for any other CI system. The only automation running against it is Dependabot's dependency graph updates.

Pull requests therefore report no checks reported, and nothing verifies a change on merge or catches a regression afterwards.

What already exists and is not being run

The repository is well equipped for verification. None of it is wired to anything:

Asset Count / detail
Smoke tests 152 tests/smoke-*.php
Declared recipes homeboy.json → scripts.lint, scripts.test
Rigs rigs/mdi-native, rigs/mdi-primary, rigs/mdi-sqlite
Fuzz harness fuzz/
Benchmarks bench/native-sqlite-differential.workload.json

homeboy.json already declares exactly what a pipeline would need:

"scripts": {
  "lint": ["php -l markdown-database-integration.php && for file in inc/*.php tests/*.php; do php -l \"$file\" || exit 1; done"],
  "test": ["for test_file in tests/smoke-*.php; do php \"$test_file\" || exit 1; done"]
}

So the gap is wiring, not authoring. The full suite runs in seconds locally.

Why this matters more than usual here

MDI is not a peripheral plugin — when MARKDOWN_DB_MODE is primary it is the database. WP_Markdown_Native_WPDB replaces $wpdb and every option, post and table read on the site goes through it.

A silent read-side regression in this component does not surface as a failing feature. It surfaces as data that appears not to exist. #412 is a worked example: an indeterminate option read was reported as "this option is absent", which caused WordPress to regenerate auth_key/auth_salt and permanently destroy every credential encrypted under the previous salt. That happened twice in four days, and was not noticed for three days, because the only symptom was an unrelated remote HTTP 401.

That class of defect is precisely what a test suite catches and a human review does not.

The fix for it (#413) merged — or will merge — with zero automated verification. Its evidence is a local run and a manual mutation check recorded in the PR body. That is not a durable guarantee, and it will not stop the same read path regressing later.

Proposed

Add a workflow that runs the recipes this repository already declares:

  • scripts.lint and scripts.test from homeboy.json on pull requests and on pushes to main.
  • Required status checks on main so a red suite blocks merge.

Worth considering once the basics are green:

  • Run the mdi-native / mdi-primary / mdi-sqlite rigs, since backend parity is the property most likely to drift.
  • Schedule the differential fuzz harness rather than running it per-PR, to keep PR latency low while still exercising it.

152 smoke tests running on every PR is a large improvement over nothing, and needs no new test authoring.

Acceptance

  • A pull request runs lint and the smoke suite and reports a status check.
  • A failing smoke test blocks merge on main.
  • The workflow invokes the recipes declared in homeboy.json rather than duplicating them, so the local command and the CI command cannot drift.

AI assistance disclosure: Claude Sonnet 4.5 via Claude Code noticed this while opening #413 and finding no checks reported, inventoried the existing unused test assets, and drafted this issue at Chris Huber's request.

Ngôn ngữ chính
PHP
Star
5
Fork
1
Merge trung bình
2 giờ 37 phút
Pull request đã merge (30 ngày)
80

Chuẩn bị môi trường

Chúng tôi chưa kiểm tra các tệp thiết lập môi trường của dự án này. Hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Automattic/markdown-database-integration

Tất cả issue của Automattic/markdown-database-integration

Issue tương tự

Thêm issue về PHP

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.