Fix defaults in onlyargs_derive (#34)

- Fixes default for floats and integers.
- Fixes struct fields named name (just don't name your field arg_name_ and you'll be ok).
- Adds regression tests.

onlyargs_derive/Cargo.toml

@@ -13,6 +13,13 @@ license = "MIT"
 [lib]
 proc-macro = true
 
+[[test]]
+name = "compile_and_fail"
+path = "compile_tests/compiler.rs"
+
 [dependencies]
 myn = "0.2.1"
 onlyargs = { version = "0.1", path = ".." }
+
+[dev-dependencies]
+trybuild = "1"

onlyargs_derive/compile_tests/compiler.rs

@@ -0,0 +1,41 @@
+#[test]
+fn compile_tests() {
+    let t = trybuild::TestCases::new();
+    t.pass("compile_tests/default_bool_false.rs");
+    t.pass("compile_tests/default_bool_true.rs");
+    t.pass("compile_tests/default_f32.rs");
+    t.pass("compile_tests/default_f64.rs");
+    t.pass("compile_tests/default_i8.rs");
+    t.pass("compile_tests/default_i128.rs");
+    t.pass("compile_tests/default_isize.rs");
+    // TODO: Negatives are not supported yet!
+    // t.pass("compile_tests/default_negative_i8.rs");
+    // t.pass("compile_tests/default_negative_i128.rs");
+    // t.pass("compile_tests/default_negative_isize.rs");
+    t.pass("compile_tests/default_osstring.rs");
+    t.pass("compile_tests/default_pathbuf.rs");
+    t.pass("compile_tests/default_string.rs");
+    t.pass("compile_tests/default_u8.rs");
+    t.pass("compile_tests/default_u128.rs");
+    t.pass("compile_tests/default_usize.rs");
+
+    t.pass("compile_tests/positional_f32.rs");
+    t.pass("compile_tests/positional_f64.rs");
+    t.pass("compile_tests/positional_i8.rs");
+    t.pass("compile_tests/positional_i128.rs");
+    t.pass("compile_tests/positional_isize.rs");
+    t.pass("compile_tests/positional_osstring.rs");
+    t.pass("compile_tests/positional_pathbuf.rs");
+    t.pass("compile_tests/positional_string.rs");
+    t.pass("compile_tests/positional_u8.rs");
+    t.pass("compile_tests/positional_usize.rs");
+
+    t.pass("compile_tests/empty.rs");
+    t.pass("compile_tests/optional.rs");
+    t.pass("compile_tests/struct_doc_comment.rs");
+    t.pass("compile_tests/struct_footer.rs");
+
+    t.compile_fail("compile_tests/conflicting_short_name.rs");
+    t.pass("compile_tests/manual_short_name.rs");
+    t.pass("compile_tests/ignore_short_name.rs");
+}

onlyargs_derive/compile_tests/conflicting_short_name.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    min: i32,
+    max: i32,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/conflicting_short_name.stderr

@@ -0,0 +1,5 @@
+error: Only one short arg is allowed. `-m` also used on field `min`
+ --> compile_tests/conflicting_short_name.rs:4:5
+  |
+4 |     max: i32,
+  |     ^^^

onlyargs_derive/compile_tests/default_bool_false.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(false)]
+    maybe: bool,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_bool_true.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(true)]
+    maybe: bool,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_f32.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42.123)]
+    width: f32,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_f64.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42.123)]
+    width: f64,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_i128.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42)]
+    width: i128,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_i8.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42)]
+    width: i8,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_isize.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42)]
+    width: isize,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_negative_i128.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(-42)]
+    width: i128,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_negative_i8.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(-42)]
+    width: i8,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_negative_isize.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(-42)]
+    width: isize,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_osstring.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default("foo bar")]
+    name: std::ffi::OsString,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_pathbuf.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default("./foo/bar")]
+    path: std::path::PathBuf,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_string.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default("foo bar")]
+    name: String,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_u128.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42)]
+    width: u128,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_u8.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42)]
+    width: u8,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/default_usize.rs

@@ -0,0 +1,7 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    #[default(42)]
+    width: usize,
+}
+
+fn main() {}

onlyargs_derive/compile_tests/empty.rs

@@ -0,0 +1,4 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {}
+
+fn main() {}

onlyargs_derive/compile_tests/ignore_short_name.rs

@@ -0,0 +1,9 @@
+#[derive(Debug, onlyargs_derive::OnlyArgs)]
+struct Args {
+    min: i32,
+
+    #[long]
+    max: i32,
+}
+
+fn main() {}