v30.3-roots.1 · Commit 37: [PATCH 37/68] fix(release): honor the configured Roots version tag
ci/test/test_release_version.py
Before: ci/test/test_release_version.py · After: ci/test/test_release_version.py
File metadata
index ac3254d2d80a73bbdc1599c606c311e1b10e0602..58d7cb6324765eee95a71a52880ec5871a132cec 100755
Download original patch · Release notes
Diff rows 1–59 of 59. Line numbers refer to the original before/after files.
+ Added− RemovedUnmarked lines are context.
Scroll the code area horizontally to read long lines. The two line-number columns refer to before and after.
ci/test/test_release_version.py
CHANGED
|
@@ -6,6 +6,7 @@
|
|
|
6
6
|
from pathlib import Path
|
|
7
7
|
import re
|
|
8
8
|
import subprocess
|
|
9
|
+
import tempfile
|
|
9
10
|
import unittest
|
|
10
11
|
|
|
11
12
|
|
|
@@ -16,6 +17,52 @@ RELEASE_WORKFLOW = ROOT / ".github/workflows/release.yml"
|
|
|
16
17
|
|
|
17
18
|
|
|
18
19
|
class ReleaseVersionTest(unittest.TestCase):
|
|
20
|
+
def configure_release_version(self, tag=None):
|
|
21
|
+
build_config = BUILD_CONFIG.read_text(encoding="utf-8")
|
|
22
|
+
version_start = re.search(r'^set\(CLIENT_VERSION_STRING ', build_config, re.MULTILINE)
|
|
23
|
+
self.assertIsNotNone(version_start, "Release version configuration must be executable CMake")
|
|
24
|
+
version_block = build_config[version_start.start():].split('#=============================', 1)[0]
|
|
25
|
+
with tempfile.TemporaryDirectory(prefix="roots-release-version-") as directory:
|
|
26
|
+
source = Path(directory)
|
|
27
|
+
metadata = "\n".join(
|
|
28
|
+
re.search(rf"set\({name} [^)]+\)", build_config).group(0)
|
|
29
|
+
for name in ("CLIENT_VERSION_MAJOR", "CLIENT_VERSION_MINOR", "CLIENT_VERSION_RC")
|
|
30
|
+
)
|
|
31
|
+
(source / "CMakeLists.txt").write_text(
|
|
32
|
+
"cmake_minimum_required(VERSION 3.22)\n"
|
|
33
|
+
"project(ReleaseVersionTest LANGUAGES NONE)\n"
|
|
34
|
+
+ metadata + "\n" + version_block
|
|
35
|
+
+ f'configure_file("{ROOT / "cmake/bitcoin-build-config.h.in"}" '
|
|
36
|
+
'"${CMAKE_BINARY_DIR}/bitcoin-build-config.h" @ONLY)\n',
|
|
37
|
+
encoding="utf-8",
|
|
38
|
+
)
|
|
39
|
+
command = ["cmake", "-S", str(source), "-B", str(source / "build")]
|
|
40
|
+
if tag is not None:
|
|
41
|
+
command.append(f"-DCLIENT_VERSION_TAG={tag}")
|
|
42
|
+
result = subprocess.run(command, capture_output=True, text=True)
|
|
43
|
+
header = source / "build/bitcoin-build-config.h"
|
|
44
|
+
return result, header.read_text(encoding="utf-8") if header.exists() else ""
|
|
45
|
+
|
|
46
|
+
def test_release_tag_sets_compiled_version_without_git_metadata(self):
|
|
47
|
+
for tag in ("v30.3-roots.2", "v30.3-roots.4"):
|
|
48
|
+
with self.subTest(tag=tag):
|
|
49
|
+
result, header = self.configure_release_version(tag)
|
|
50
|
+
self.assertEqual(result.returncode, 0, result.stderr)
|
|
51
|
+
self.assertIn(f'#define CLIENT_VERSION_STRING "{tag[1:]}"', header)
|
|
52
|
+
self.assertNotIn("Manually-specified variables were not used", result.stderr)
|
|
53
|
+
|
|
54
|
+
def test_release_tag_rejects_wrong_base_and_invalid_roots_version(self):
|
|
55
|
+
for tag in ("v30.0-roots.4", "v30x3-roots.4", "v30.3-roots.0", "v30.3-knots.4", ""):
|
|
56
|
+
with self.subTest(tag=tag):
|
|
57
|
+
result, _ = self.configure_release_version(tag)
|
|
58
|
+
self.assertNotEqual(result.returncode, 0)
|
|
59
|
+
self.assertIn("CLIENT_VERSION_TAG must be a Roots release tag", result.stderr)
|
|
60
|
+
|
|
61
|
+
def test_untagged_build_keeps_default_roots_version(self):
|
|
62
|
+
result, header = self.configure_release_version()
|
|
63
|
+
self.assertEqual(result.returncode, 0, result.stderr)
|
|
64
|
+
self.assertIn('#define CLIENT_VERSION_STRING "30.3-roots.1"', header)
|
|
65
|
+
|
|
19
66
|
def test_version_and_release_tag_name_agree(self):
|
|
20
67
|
build_config = BUILD_CONFIG.read_text(encoding="utf-8")
|
|
21
68
|
self.assertIn("set(CLIENT_VERSION_MAJOR 30)", build_config)
|