chore: add tests for not_strict

bpaf_derive/src/field_tests.rs

@@ -485,6 +485,19 @@ fn strict_positional_named_fields() {
     };
     assert_eq!(input.to_token_stream().to_string(), output.to_string());
 }
+
+#[test]
+fn not_strict_positional_named_fields() {
+    let input: NamedField = parse_quote! {
+        #[bpaf(positional("ARG"), not_strict)]
+        name: String
+    };
+    let output = quote! {
+        ::bpaf::positional::<String>("ARG").not_strict()
+    };
+    assert_eq!(input.to_token_stream().to_string(), output.to_string());
+}
+
 #[test]
 fn optional_named_pathed() {
     let input: NamedField = parse_quote! {

tests/positionals.rs

@@ -144,3 +144,17 @@ fn strictly_positional() {
     let r = parser.run_inner(&["--"]).unwrap_err().unwrap_stderr();
     assert_eq!(r, "expected `A`, pass `--help` for usage information");
 }
+
+#[test]
+fn not_strictly_positional() {
+    let parser = positional::<String>("A").not_strict().to_options();
+
+    let r = parser.run_inner(&["a"]).unwrap();
+    assert_eq!(r, "a");
+
+    let r = parser.run_inner(&["--", "a"]).unwrap_err().unwrap_stderr();
+    assert_eq!(r, "expected `A` to be on the left side of `--`");
+
+    let r = parser.run_inner(&["--"]).unwrap_err().unwrap_stderr();
+    assert_eq!(r, "expected `A`, pass `--help` for usage information");
+}