mr.zeroandClaude Opus 5
19d142ef64
Két dolog volt beépítve az engine-be, ami nem az övé. A **képtár** eddig `WarpEngine::Image` volt, pedig a modell teljesen általános: a teletypegames-ben a tagok arcképét is ez hordozza, nem csak a katalógus borítóit. Az `Image` modell, a feltöltött fájlok, az `/api/image/:id` végpont és az admin oldal ezért átkerült a hosztba, az engine pedig adapteren szól hozzá (`WarpEngine::Images`): `url_for` adja a katalógus JSON `imageUrl`-jét, `select_options` a software-form képválasztóját, `build_from_upload` a "tölts fel új képet" ágat. Az alapértelmezés az `Image` osztály, tehát a default útvonal bitre a régi. A `SoftwareImage` (a katalógus-kapcsolat) maradt az engine-ben, és **az `images` tábla nem mozdult**: az engine csak abbahagyta a létrehozását, a generátor írja meg hoszt-kódként. A **CI** eddig végig Woodpecker volt: kliens, aláírás-ellenőrzés, pipeline-receptek, repo-szinkron, secret-kiosztás. Mindez egy adapter mögé került (`WarpEngine.ci`), a Woodpecker-implementáció pedig az engine-ben maradt `WarpEngine::CI::Woodpecker` néven — kliens, adapter, httpsig-ellenőrző és a platformonkénti pipeline-receptek, mert a YAML-dialektus a szolgáltatóé. Az engine saját kódja már nem nevez szolgáltatót: `CI::Repo` és `CI::Run` értékeket kap, `CI::ConnectionError`/`ApiError`/`NotConfigured` hibákat dob, a `Pipeline` pedig `remote_repo_id`-t ad a történelmi `woodpecker_repo_id` kolumna fölött (a tábla itt sem mozdult). `c.ci_adapter = :none` azt jelenti, hogy ez a hoszt nem buildel: az `/api/ci/*` 503, a `/build/config` elutasít, az admin akciók elbújnak. Mindkét seam a hoszt initializerében van kimondva, nem alapértelmezésre hagyva — a hoszt megnevezi, mi a képtára és mi a CI-ja. Törés a 0.6-hoz képest: `image_container_path`, `image_owners`, `ci_platforms`, `ci_update_server`, `ci_extension_public_key(_url)`, `woodpecker_url`, `woodpecker_api_token`, `woodpecker_repo_owner` és a `WarpEngine.woodpecker_configured?` megszűnt; a helyük `c.image_class_name` / `c.image_adapter` és `c.ci_adapter`. Az `/api/ci/*` `latest_run`/`trigger` válasza a normalizált `CI::Run` alakot adja (number, status, branch, message, createdAt, url), a `pipelines` lista pedig `repo_id`-t is közöl a megtartott `woodpecker_repo_id` mellett. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
52 lines
1.8 KiB
Ruby
52 lines
1.8 KiB
Ruby
require "rails_helper"
|
|
|
|
RSpec.describe "Engine migrations" do
|
|
ROUND_VERSION = /\A\d{8}0{4}\d{2}\z/
|
|
|
|
GRANDFATHERED = %w[
|
|
20260805000001
|
|
20260805000003
|
|
20260806000001
|
|
20260806000002
|
|
].freeze
|
|
|
|
let(:versions) do
|
|
Dir.glob(WarpEngine::Engine.root.join("db/migrate/*.rb"))
|
|
.map { |path| File.basename(path)[/\A\d+/] }
|
|
end
|
|
|
|
it "has a migration to check at all" do
|
|
expect(versions).not_to be_empty
|
|
end
|
|
|
|
it "numbers every migration uniquely" do
|
|
expect(versions).to eq(versions.uniq)
|
|
end
|
|
|
|
it "uses a real timestamp rather than a round hand-written number" do
|
|
round = versions.grep(ROUND_VERSION) - GRANDFATHERED
|
|
|
|
expect(round).to be_empty,
|
|
"these would collide with a host that numbers its migrations the " \
|
|
"same way on the same day: #{round.join(', ')}. Use a second-resolution " \
|
|
"timestamp — `date -u +%Y%m%d%H%M%S` — not a hand-picked round number."
|
|
end
|
|
|
|
it "does not create a table the install generator also creates" do
|
|
templates = Dir.glob(
|
|
WarpEngine::Engine.root.join("lib/generators/warp_engine/install/templates/create_*.rb")
|
|
)
|
|
engine_tables = versions.flat_map do |version|
|
|
file = Dir.glob(WarpEngine::Engine.root.join("db/migrate/#{version}_*.rb")).first
|
|
File.read(file).scan(/create_table :(\w+)/).flatten
|
|
end
|
|
template_tables = templates.flat_map { |t| File.read(t).scan(/create_table :(\w+)/).flatten }
|
|
duplicated = (engine_tables & template_tables) - [ "application_tokens" ]
|
|
|
|
expect(duplicated).to be_empty,
|
|
"the install template and an engine migration both create " \
|
|
"#{duplicated.join(', ')} — a host would run CREATE TABLE twice. " \
|
|
"A table added after the initial schema belongs in a migration only."
|
|
end
|
|
end
|