Skip to content

Commit 94dda53

Browse files
authored
add test targets, fix tests (#66)
I just realized that this is not downstream and things are not taken care of for you: these tests never ran. Enabled rust tests, fixed problems.
1 parent 8644ce6 commit 94dda53

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

kj-rs/BUILD.bazel

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
load("//tools/bazel:rust_cxx_bridge.bzl", "rust_cxx_bridge")
2-
load("@rules_rust//rust:defs.bzl", "rust_library")
2+
load("@rules_rust//rust:defs.bzl", "rust_library", "rust_test")
33

44
cc_library(
55
name = "kj-rs-lib",
@@ -30,6 +30,12 @@ rust_library(
3030
],
3131
)
3232

33+
rust_test(
34+
name = "kj-rs_test",
35+
crate = "kj-rs",
36+
edition = "2024",
37+
)
38+
3339
rust_cxx_bridge(
3440
name = "bridge",
3541
src = "lib.rs",

kj-rs/date.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -98,15 +98,15 @@ mod tests {
9898
}
9999

100100
#[test]
101-
fn test_from_nanoseconds() {
102-
let date = KjDate::from_nanoseconds(1000000000);
103-
assert_eq!(date.nanoseconds(), 1000000000);
101+
fn test_from() {
102+
let date = KjDate::from(1_000_000_000);
103+
assert_eq!(date.nanoseconds(), 1_000_000_000);
104104
}
105105

106106
#[test]
107107
fn test_ordering() {
108-
let earlier = KjDate::from_nanoseconds(1000);
109-
let later = KjDate::from_nanoseconds(2000);
108+
let earlier = KjDate::from(1000);
109+
let later = KjDate::from(2000);
110110
assert!(earlier < later);
111111
}
112112

@@ -118,8 +118,8 @@ mod tests {
118118

119119
#[test]
120120
fn test_system_time_conversion() {
121-
let date = KjDate::from_nanoseconds(1000000000);
122-
let system_time = date.to_system_time();
121+
let date = KjDate::from(1_000_000_000);
122+
let system_time: SystemTime = date.into();
123123
let converted_back = KjDate::from(system_time);
124124
assert_eq!(date, converted_back);
125125
}

0 commit comments

Comments
 (0)