Ludovic Marie
9a3237c5ac
docs/versioning.md: with .github/release-line.txt (MAJOR.MINOR) the train tags game releases vMAJOR.MINOR.PATCH and launcher releases launcher@MAJOR.MINOR.PATCH, the patch after the newest release on the line; the line's first release goes out at once (still not while RELEASES_PAUSED is set). Without the file nothing changes: vYYYY.MM.DD[.N] and launcher-vX.Y.Z. The file is not added here; the switch to 0.2 is a separate one-line pull request for the day Ludovic says go. - tag_key: every 0.2 tag ranks above every dated or launcher-v tag; semver order within, a pre-release before its release. release_channels.py uses the same key and tag test, so promotion and "latest only moves forward" work across the switch. - A line's first release says so in its notes; 0.2.0's explains the numbering and how a server that says it is up to date takes it (horizon-server update --version v0.2.0). - launcher.yml strips either prefix for HORIZON_LAUNCHER_VERSION and passes HORIZON_LAUNCHER_TAG, which the launcher uses to know its own line. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
184 lines
12 KiB
Python
184 lines
12 KiB
Python
import datetime as dt
|
|
from pathlib import Path
|
|
import tempfile
|
|
import unittest
|
|
|
|
from tools import release_train as train
|
|
|
|
NOW = dt.datetime(2026, 10, 15, 12, 0, tzinfo=dt.timezone.utc)
|
|
|
|
|
|
def pr(number, title="Fix a thing", labels=(), paths=("crates/horizon-client/src/game.rs",), issues=(), author="ludovic111",
|
|
merged=dt.timedelta(hours=9)):
|
|
return {"number": number, "title": title, "labels": list(labels), "paths": list(paths), "issues": list(issues),
|
|
"author": author, "url": f"https://github.com/ludovic111/horizon/pull/{number}", "merged_at": NOW - merged}
|
|
|
|
|
|
class ReleaseTrainTests(unittest.TestCase):
|
|
def test_player_facing_paths(self):
|
|
exporters = {"mods/horizon/src/test/java/dev/horizonlab/parity/VanillaMobDataExporter.java"}
|
|
self.assertTrue(train.player_facing("crates/horizon-client/src/game.rs", "game"))
|
|
self.assertTrue(train.player_facing("Cargo.lock", "game"))
|
|
self.assertTrue(train.player_facing("xtask/src/main.rs", "game"))
|
|
self.assertTrue(train.player_facing(next(iter(exporters)), "game", exporters))
|
|
self.assertFalse(train.player_facing("mods/horizon/src/test/java/dev/horizonlab/parity/VanillaFoodOracle.java", "game", exporters))
|
|
self.assertFalse(train.player_facing("crates/horizon-world/examples/probe.rs", "game"))
|
|
self.assertFalse(train.player_facing("crates/horizon-client/README.md", "game"))
|
|
self.assertFalse(train.player_facing("tools/test_food_parity.py", "game"))
|
|
self.assertFalse(train.player_facing("launcher/src/main.rs", "game"))
|
|
self.assertTrue(train.player_facing("launcher/src/main.rs", "launcher"))
|
|
self.assertTrue(train.player_facing("crates/horizon-prepare/src/lib.rs", "launcher"))
|
|
self.assertFalse(train.player_facing("launcher/links.json", "launcher"))
|
|
self.assertFalse(train.player_facing("crates/horizon-client/src/game.rs", "launcher"))
|
|
|
|
def test_the_exporters_are_those_horizon_prepare_names(self):
|
|
found = train.exporters()
|
|
self.assertIn(train.EXPORTERS + "VanillaMobDataExporter.java", found)
|
|
self.assertTrue(all(path.endswith(".java") for path in found))
|
|
|
|
def test_game_tags_count_up_within_a_day(self):
|
|
today = dt.date(2026, 10, 15)
|
|
self.assertEqual(train.next_game_tag(["v2026.10.14", "launcher-v0.1.4"], today), "v2026.10.15")
|
|
self.assertEqual(train.next_game_tag(["v2026.10.15"], today), "v2026.10.15.01")
|
|
self.assertEqual(train.next_game_tag(["v2026.10.15", "v2026.10.15.01", "v2026.10.15.02"], today), "v2026.10.15.03")
|
|
# Unpadded tags from before the change still count, and the next one sorts after them as text too.
|
|
self.assertEqual(train.next_game_tag(["v2026.10.15", "v2026.10.15.9", "v2026.10.15.20"], today), "v2026.10.15.21")
|
|
self.assertEqual(train.next_game_tag(["v2026.10.15", "v2026.10.15.08"], today), "v2026.10.15.09")
|
|
tags = ["v2026.10.15.09", "v2026.10.15.10", "v2026.10.15"]
|
|
self.assertEqual(sorted(tags), sorted(tags, key=train.tag_key))
|
|
|
|
def test_launcher_tags_bump_the_patch_or_the_minor(self):
|
|
tags = ["launcher-v0.1.3", "launcher-v0.1.10", "v2026.10.15", "launcher-v0.1.4"]
|
|
self.assertEqual(train.next_launcher_tag(tags, minor=False), "launcher-v0.1.11")
|
|
self.assertEqual(train.next_launcher_tag(tags, minor=True), "launcher-v0.2.0")
|
|
|
|
def test_0_2_tags_follow_every_tag_from_before(self):
|
|
order = ["launcher-v0.1.9", "launcher-v0.16.0", "v2026.10.19.15", "v2026.10.21", "v0.2.0-beta.1", "v0.2.0",
|
|
"v0.2.1-beta", "v0.2.1-beta.1", "v0.2.1-beta.2", "v0.2.1-beta.11", "v0.2.1-rc.1", "v0.2.1", "v0.2.9",
|
|
"v0.2.10", "v0.3.0-1", "v0.3.0-alpha", "v0.3.0", "v1.0.0"]
|
|
self.assertEqual(sorted(reversed(order), key=train.tag_key), order)
|
|
self.assertGreater(train.tag_key("launcher@0.2.0"), train.tag_key("launcher-v0.16.0"))
|
|
self.assertEqual(train.tag_key("v2026.10.19"), train.tag_key("v2026.10.19.0"))
|
|
for tag in ["v0.2.0", "v0.3.0-rc.1", "v2026.10.21", "v2026.10.19.15"]:
|
|
self.assertTrue(train.is_game_tag(tag), tag)
|
|
for tag in ["launcher@0.2.0", "launcher-v0.2.0", "v0.2", "vnext", "v2026.10.21-x", "v0.2.0-"]:
|
|
self.assertFalse(train.is_game_tag(tag), tag)
|
|
self.assertTrue(train.is_launcher_tag("launcher@0.2.0") and train.is_launcher_tag("launcher-v0.16.0"))
|
|
self.assertFalse(train.is_launcher_tag("v0.2.0"))
|
|
|
|
def test_tags_on_a_release_line(self):
|
|
line = (0, 2)
|
|
before = ["v2026.10.19.15", "v2026.10.21", "launcher-v0.16.0", "launcher-v0.2.0", "launcher-v0.2.1"]
|
|
today = dt.date(2026, 10, 23)
|
|
self.assertEqual(train.next_game_tag(before, today, line), "v0.2.0")
|
|
self.assertEqual(train.next_launcher_tag(before, minor=True, line=line), "launcher@0.2.0")
|
|
self.assertTrue(train.first_on_line(before, "v", line))
|
|
after = before + ["v0.2.0", "v0.2.1", "launcher@0.2.0", "v0.3.0-rc.1"]
|
|
self.assertEqual(train.next_game_tag(after, today, line), "v0.2.2")
|
|
# A feature doesn't move the line; only .github/release-line.txt does.
|
|
self.assertEqual(train.next_launcher_tag(after, minor=True, line=line), "launcher@0.2.1")
|
|
self.assertFalse(train.first_on_line(after, "v", line))
|
|
self.assertTrue(train.first_on_line(after, "v", (0, 3)), "a pre-release isn't the line's first release")
|
|
self.assertEqual(train.next_game_tag(after, today, (0, 3)), "v0.3.0")
|
|
with self.assertRaises(SystemExit):
|
|
train.next_game_tag(after + ["v0.3.0"], today, line)
|
|
# Without a line, the tags from before 0.2, whatever 0.2 tags exist.
|
|
self.assertEqual(train.next_game_tag(before, today), "v2026.10.23")
|
|
self.assertEqual(train.next_launcher_tag(after, minor=False), "launcher-v0.16.1")
|
|
|
|
def test_the_release_line_file(self):
|
|
with tempfile.TemporaryDirectory() as folder:
|
|
path = Path(folder) / "release-line.txt"
|
|
self.assertIsNone(train.release_line(path))
|
|
path.write_text("0.2\n")
|
|
self.assertEqual(train.release_line(path), (0, 2))
|
|
path.write_text("0.2.0\n")
|
|
with self.assertRaises(SystemExit):
|
|
train.release_line(path)
|
|
# Not set on main until the switch to 0.2 (docs/versioning.md).
|
|
self.assertIsNone(train.release_line()) if not train.LINE_FILE.exists() else self.assertIsNotNone(train.release_line())
|
|
|
|
def test_a_new_line_releases_at_once_unless_paused(self):
|
|
self.assertEqual(train.decision("game", [], NOW, new_line="0.2"), (True, "the first release of 0.2"))
|
|
self.assertFalse(train.decision("game", [], NOW, paused=True, new_line="0.2")[0])
|
|
self.assertFalse(train.decision("game", [], NOW)[0])
|
|
|
|
def test_the_first_release_of_0_2_says_what_changed(self):
|
|
title, body = train.notes("game", "v0.2.0", [pr(1)], [], beta=True, previous="v2026.10.21")
|
|
self.assertTrue(title.startswith("Horizon 0.2.0: "), title)
|
|
self.assertTrue(body.startswith("Horizon is numbered from now on: this is Horizon 0.2.0, the release after 2026.10.21."))
|
|
self.assertIn("`horizon-server update --version v0.2.0`", body)
|
|
title, body = train.notes("launcher", "launcher@0.2.0", [], [], previous="launcher-v0.17.0")
|
|
self.assertEqual(title, "Horizon launcher 0.2.0")
|
|
self.assertIn("the release after launcher 0.17.0", body)
|
|
self.assertIn("Rebuilt from the latest main.", body)
|
|
self.assertTrue(train.notes("game", "v0.3.0", [pr(1)], [], previous="v0.2.14")[1].startswith("The first release of Horizon 0.3."))
|
|
for version, previous in [("v0.2.1", "v0.2.0"), ("v0.3.0-rc.1", "v0.2.9"), ("v2026.10.23", "v2026.10.21")]:
|
|
body = train.notes("game", version, [pr(1)], [], previous=previous)[1]
|
|
self.assertTrue(body.startswith("### Fixed"), (version, body))
|
|
|
|
def test_decision(self):
|
|
minute = dt.timedelta(minutes=1)
|
|
decide = lambda prs, **kw: train.decision("game", prs, NOW, **kw)[0]
|
|
self.assertTrue(decide([pr(1, merged=16 * minute)]), "settled for the debounce")
|
|
self.assertFalse(decide([pr(1, merged=5 * minute)]), "merged moments ago")
|
|
self.assertFalse(decide([pr(1, merged=40 * minute), pr(2, merged=5 * minute)]), "a burst still merging")
|
|
self.assertTrue(decide([pr(1, merged=61 * minute), pr(2, merged=2 * minute)]), "merges that never pause still ship")
|
|
self.assertTrue(decide([pr(1, labels=["release-now"], merged=minute)]))
|
|
self.assertFalse(decide([pr(1, paths=[])]), "nothing players run")
|
|
self.assertFalse(decide([pr(1, labels=["skip-release"])]))
|
|
self.assertFalse(decide([pr(1, labels=["release-now", "skip-release"])]))
|
|
self.assertFalse(decide([pr(1, merged=5 * minute), pr(2, labels=["skip-release"], merged=minute)]), "the settled one waits for nothing")
|
|
self.assertFalse(decide([pr(1)], paused=True))
|
|
self.assertTrue(decide([], paused=True, force=True))
|
|
|
|
def test_notes(self):
|
|
prs = [pr(5, "Nether portals: vanilla timing and linking (fixes #94)", issues=[94]),
|
|
pr(6, "Controllers: gilrs input layer and gameplay", labels=["feature"], author="cosmic"),
|
|
pr(7, "Release folder in Rust", labels=["skip-release"]),
|
|
pr(8, "Website footer", paths=[])]
|
|
title, body = train.notes("game", "v2026.10.15.1", prs, ["- Sound cuts out after an hour."])
|
|
self.assertEqual(title, "Horizon 2026.10.15.1: Controllers, nether portals")
|
|
self.assertIn("### New\n- Controllers: gilrs input layer and gameplay (#6)", body)
|
|
self.assertIn("### Fixed\n- Nether portals: vanilla timing and linking (#5, fixes #94)", body)
|
|
self.assertIn("### Known issues\n- Sound cuts out after an hour.", body)
|
|
self.assertNotIn("#7", body)
|
|
self.assertNotIn("#8", body)
|
|
self.assertIn("Thanks to @cosmic.", body)
|
|
self.assertLess(body.index("### New"), body.index("### Fixed"))
|
|
title, _ = train.notes("game", "v2026.10.15.2", [pr(9, "Horizon's mark", labels=["feature"]),
|
|
pr(10, "Game menus", labels=["feature"])], [])
|
|
self.assertEqual(title, "Horizon 2026.10.15.2: Game menus, Horizon's mark", "proper nouns keep their capital")
|
|
|
|
def test_a_long_release_is_named_after_its_first_titles(self):
|
|
prs = [pr(n, f"Change number {n} with a fairly long title") for n in range(1, 10)]
|
|
title, _ = train.notes("game", "v2026.10.15", prs, [])
|
|
self.assertTrue(title.startswith("Horizon 2026.10.15: Change number 9"), title)
|
|
self.assertTrue(title.endswith("more fixes"), title)
|
|
self.assertLess(len(title), 100)
|
|
|
|
def test_a_feature_split_over_several_pull_requests_is_named_once(self):
|
|
prs = [pr(1, "Controllers: rumble", labels=["feature"]), pr(2, "Controllers: menus", labels=["feature"]),
|
|
pr(3, "Maces: smash attacks"), pr(4, "Fix stuck keys when opening Options")]
|
|
title, _ = train.notes("game", "v2026.10.15", prs, [])
|
|
self.assertEqual(title, "Horizon 2026.10.15: Controllers, stuck keys when opening Options, maces")
|
|
self.assertEqual(train.summary([], [pr(n, f"Change number {n} with a fairly long title") for n in range(4)]),
|
|
"Change number 3 with a fairly long title and 3 more fixes")
|
|
|
|
def test_launcher_notes(self):
|
|
title, body = train.notes("launcher", "launcher-v0.2.0", [pr(3, "Launcher: say the self-update succeeded")], [])
|
|
self.assertEqual(title, "Horizon launcher 0.2.0")
|
|
self.assertIn("### Fixed\n- Launcher: say the self-update succeeded (#3)", body)
|
|
self.assertNotIn("Known issues", body)
|
|
|
|
def test_known_issues_are_the_bullets_of_the_file(self):
|
|
with tempfile.TemporaryDirectory() as temp:
|
|
path = Path(temp, "known-issues.md")
|
|
path.write_text("# Known issues\n\nSome text.\n\n- One.\n- Two, with **bold**.\n continued\n")
|
|
self.assertEqual(train.known_issues(path), ["- One.", "- Two, with **bold**."])
|
|
self.assertEqual(train.known_issues(Path(temp, "missing.md")), [])
|
|
|
|
|
|
if __name__ == "__main__":
|
|
unittest.main()
|