From 87f3e536a405529d4fd493a7a2ec28c132f08c6b Mon Sep 17 00:00:00 2001 From: rrokkam Date: Sun, 19 Jul 2020 12:38:24 -0700 Subject: [PATCH] Risky: Move tests into mod.rs Usual Rust practice is to put tests in the same file they are testing. I don't see any magic in the texttests that would require tests and code to be in separate files, but I am not entirely sure, so I added the risky tag. --- rust/src/gildedrose/mod.rs | 13 ++++++++++++- rust/src/gildedrose/test.rs | 10 ---------- 2 files changed, 12 insertions(+), 11 deletions(-) delete mode 100644 rust/src/gildedrose/test.rs diff --git a/rust/src/gildedrose/mod.rs b/rust/src/gildedrose/mod.rs index 0c1bc16c..43565e4b 100644 --- a/rust/src/gildedrose/mod.rs +++ b/rust/src/gildedrose/mod.rs @@ -78,4 +78,15 @@ impl GildedRose { } #[cfg(test)] -mod test; +mod tests { + use super::{GildedRose, Item}; + + #[test] + pub fn foo() { + let items = vec![Item::new(String::from("foo"), 0, 0)]; + let mut rose = GildedRose::new(items); + rose.update_quality(); + + assert_eq!("fixme", rose.items[0].name); + } +} diff --git a/rust/src/gildedrose/test.rs b/rust/src/gildedrose/test.rs deleted file mode 100644 index 1883cc40..00000000 --- a/rust/src/gildedrose/test.rs +++ /dev/null @@ -1,10 +0,0 @@ -use super::{GildedRose, Item}; - -#[test] -pub fn foo() { - let items = vec![Item::new(String::from("foo"), 0, 0)]; - let mut rose = GildedRose::new(items); - rose.update_quality(); - - assert_eq!("fixme", rose.items[0].name); -}