Use lookahead.error() (#142)

KiChjang · web-flow · commit 7e942b82d31d · 2022-02-14T09:47:45.000Z
* Use lookahead.error()

* Create UI test for invalid scale-info attributes
diff --git a/derive/src/attr.rs b/derive/src/attr.rs
@@ -250,8 +250,7 @@ impl Parse for ScaleInfoAttr {
             let capture_docs = input.parse()?;
             Ok(Self::CaptureDocs(capture_docs))
         } else {
-            Err(input
-                .error("Expected one of: `bounds`, `skip_type_params` or `capture_docs"))
+            Err(lookahead.error())
         }
     }
 }
diff --git a/test_suite/tests/ui/fail_with_invalid_scale_info_attrs.rs b/test_suite/tests/ui/fail_with_invalid_scale_info_attrs.rs
@@ -0,0 +1,11 @@
+use scale_info::TypeInfo;
+use scale::Encode;
+
+#[derive(TypeInfo, Encode)]
+#[scale_info(foo)]
+struct InvalidKeywordInScaleInfoAttr {
+    a: u8,
+    b: u16,
+}
+
+fn main() {}
diff --git a/test_suite/tests/ui/fail_with_invalid_scale_info_attrs.stderr b/test_suite/tests/ui/fail_with_invalid_scale_info_attrs.stderr
@@ -0,0 +1,5 @@
+error: expected one of: `bounds`, `skip_type_params`, `capture_docs`
+ --> tests/ui/fail_with_invalid_scale_info_attrs.rs:5:14
+  |
+5 | #[scale_info(foo)]
+  |              ^^^

Original file line number	Diff line number	Diff line change
`@@ -250,8 +250,7 @@ impl Parse for ScaleInfoAttr {`
`250`	`250`	`let capture_docs = input.parse()?;`
`251`	`251`	`Ok(Self::CaptureDocs(capture_docs))`
`252`	`252`	`} else {`
`253`		`- Err(input`
`254`		- .error("Expected one of: `bounds`, `skip_type_params` or `capture_docs"))
	`253`	`+ Err(lookahead.error())`
`255`	`254`	`}`
`256`	`255`	`}`
`257`	`256`	`}`