talksik/tonic
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

chore: Refactor bootstrap test (#1216)

Browse Source 
* chore: Remove unnecessary conversions to string in bootstrap tests

* chore: Simplify assertion in bootstrap tests
This commit is contained in:
tottoto
2023-01-05 04:19:28 +09:00
committed by GitHub
parent cc42d1f88c
commit c772a786a9
3 changed files with 9 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tonic-health/tests/bootstrap.rs
+3 -5
View File
@@ -13,7 +13,7 @@ fn bootstrap() {
.build_client(true)
.build_server(true)
.build_transport(false)
.out_dir(format!("{}", out_dir.display()))
.out_dir(&out_dir)
.compile(iface_files, dirs)
.unwrap();
@@ -21,11 +21,9 @@ fn bootstrap() {
.arg("diff")
.arg("--exit-code")
.arg("--")
.arg(format!("{}", out_dir.display()))
.arg(&out_dir)
.status()
.unwrap();
if !status.success() {
panic!("You should commit the protobuf files");
}
assert!(status.success(), "You should commit the protobuf files");
}
tonic-reflection/tests/bootstrap.rs
+3 -5
View File
@@ -13,7 +13,7 @@ fn bootstrap() {
.build_client(true)
.build_server(true)
.build_transport(false)
.out_dir(format!("{}", out_dir.display()))
.out_dir(&out_dir)
.compile(iface_files, dirs)
.unwrap();
@@ -21,11 +21,9 @@ fn bootstrap() {
.arg("diff")
.arg("--exit-code")
.arg("--")
.arg(format!("{}", out_dir.display()))
.arg(&out_dir)
.status()
.unwrap();
if !status.success() {
panic!("You should commit the protobuf files");
}
assert!(status.success(), "You should commit the protobuf files");
}
tonic-types/tests/bootstrap.rs
+3 -5
View File
@@ -10,7 +10,7 @@ fn bootstrap() {
.join("generated");
tonic_build::configure()
.out_dir(format!("{}", out_dir.display()))
.out_dir(&out_dir)
.file_descriptor_set_path(out_dir.join("types.bin"))
.compile(iface_files, dirs)
.unwrap();
@@ -19,11 +19,9 @@ fn bootstrap() {
.arg("diff")
.arg("--exit-code")
.arg("--")
.arg(format!("{}", out_dir.display()))
.arg(&out_dir)
.status()
.unwrap();
if !status.success() {
panic!("You should commit the protobuf files");
}
assert!(status.success(), "You should commit the protobuf files");
}