2 use rustc::hir::intravisit;
3 use rustc::session::{self, config, DiagnosticOutput};
4 use rustc::util::common::ErrorReported;
5 use rustc_data_structures::sync::Lrc;
6 use rustc_feature::UnstableFeatures;
7 use rustc_interface::interface;
8 use rustc_span::edition::Edition;
9 use rustc_span::source_map::SourceMap;
10 use rustc_span::symbol::sym;
11 use rustc_span::{BytePos, FileName, Pos, Span, DUMMY_SP};
12 use rustc_target::spec::TargetTriple;
14 use std::io::{self, Write};
16 use std::path::PathBuf;
17 use std::process::{self, Command, Stdio};
20 use syntax::with_globals;
21 use tempfile::Builder as TempFileBuilder;
24 use crate::clean::Attributes;
25 use crate::config::Options;
26 use crate::html::markdown::{self, ErrorCodes, Ignore, LangString};
28 #[derive(Clone, Default)]
29 pub struct TestOptions {
30 /// Whether to disable the default `extern crate my_crate;` when creating doctests.
31 pub no_crate_inject: bool,
32 /// Whether to emit compilation warnings when compiling doctests. Setting this will suppress
33 /// the default `#![allow(unused)]`.
34 pub display_warnings: bool,
35 /// Additional crate-level attributes to add to doctests.
36 pub attrs: Vec<String>,
39 pub fn run(options: Options) -> i32 {
40 let input = config::Input::File(options.input.clone());
42 let crate_types = if options.proc_macro_crate {
43 vec![config::CrateType::ProcMacro]
45 vec![config::CrateType::Dylib]
48 let sessopts = config::Options {
49 maybe_sysroot: options.maybe_sysroot.clone(),
50 search_paths: options.libs.clone(),
52 cg: options.codegen_options.clone(),
53 externs: options.externs.clone(),
54 unstable_features: UnstableFeatures::from_environment(),
55 lint_cap: Some(::rustc::lint::Level::Allow),
56 actually_rustdoc: true,
57 debugging_opts: config::DebuggingOptions { ..config::basic_debugging_options() },
58 edition: options.edition,
59 target_triple: options.target.clone(),
60 ..config::Options::default()
63 let mut cfgs = options.cfgs.clone();
64 cfgs.push("doc".to_owned());
65 cfgs.push("doctest".to_owned());
66 let config = interface::Config {
68 crate_cfg: interface::parse_cfgspecs(cfgs),
74 diagnostic_output: DiagnosticOutput::Default,
76 crate_name: options.crate_name.clone(),
77 lint_caps: Default::default(),
79 override_queries: None,
80 registry: rustc_driver::diagnostics_registry(),
83 let mut test_args = options.test_args.clone();
84 let display_warnings = options.display_warnings;
86 let tests = interface::run_compiler(config, |compiler| {
87 compiler.enter(|queries| {
88 let lower_to_hir = queries.lower_to_hir()?;
90 let mut opts = scrape_test_config(lower_to_hir.peek().0.krate());
91 opts.display_warnings |= options.display_warnings;
92 let enable_per_target_ignores = options.enable_per_target_ignores;
93 let mut collector = Collector::new(
94 queries.crate_name()?.peek().to_string(),
98 Some(compiler.source_map().clone()),
100 enable_per_target_ignores,
103 let mut global_ctxt = queries.global_ctxt()?.take();
105 global_ctxt.enter(|tcx| {
106 let krate = tcx.hir().krate();
107 let mut hir_collector = HirCollector {
108 sess: compiler.session(),
109 collector: &mut collector,
111 codes: ErrorCodes::from(
112 compiler.session().opts.unstable_features.is_nightly_build(),
115 hir_collector.visit_testable("".to_string(), &krate.attrs, |this| {
116 intravisit::walk_crate(this, krate);
120 let ret: Result<_, ErrorReported> = Ok(collector.tests);
124 .expect("compiler aborted in rustdoc!");
126 test_args.insert(0, "rustdoctest".to_string());
131 Some(testing::Options::new().display_output(display_warnings)),
137 // Look for `#![doc(test(no_crate_inject))]`, used by crates in the std facade.
138 fn scrape_test_config(krate: &::rustc::hir::Crate) -> TestOptions {
139 use syntax::print::pprust;
142 TestOptions { no_crate_inject: false, display_warnings: false, attrs: Vec::new() };
144 let test_attrs: Vec<_> = krate
147 .filter(|a| a.check_name(sym::doc))
148 .flat_map(|a| a.meta_item_list().unwrap_or_else(Vec::new))
149 .filter(|a| a.check_name(sym::test))
151 let attrs = test_attrs.iter().flat_map(|a| a.meta_item_list().unwrap_or(&[]));
154 if attr.check_name(sym::no_crate_inject) {
155 opts.no_crate_inject = true;
157 if attr.check_name(sym::attr) {
158 if let Some(l) = attr.meta_item_list() {
160 opts.attrs.push(pprust::meta_list_item_to_string(item));
169 /// Documentation test failure modes.
171 /// The test failed to compile.
173 /// The test is marked `compile_fail` but compiled successfully.
174 UnexpectedCompilePass,
175 /// The test failed to compile (as expected) but the compiler output did not contain all
176 /// expected error codes.
177 MissingErrorCodes(Vec<String>),
178 /// The test binary was unable to be executed.
179 ExecutionError(io::Error),
180 /// The test binary exited with a non-zero exit code.
182 /// This typically means an assertion in the test failed or another form of panic occurred.
183 ExecutionFailure(process::Output),
184 /// The test is marked `should_panic` but the test binary executed successfully.
196 as_test_harness: bool,
197 runtool: Option<String>,
198 runtool_args: Vec<String>,
199 target: TargetTriple,
201 mut error_codes: Vec<String>,
204 ) -> Result<(), TestFailure> {
205 let (test, line_offset) = match panic::catch_unwind(|| {
206 make_test(test, Some(cratename), as_test_harness, opts, edition)
208 Ok((test, line_offset)) => (test, line_offset),
209 Err(cause) if cause.is::<errors::FatalErrorMarker>() => {
210 // If the parser used by `make_test` panicked due to a fatal error, pass the test code
211 // through unchanged. The error will be reported during compilation.
214 Err(cause) => panic::resume_unwind(cause),
217 // FIXME(#44940): if doctests ever support path remapping, then this filename
218 // needs to be the result of `SourceMap::span_to_unmapped_path`.
219 let path = match filename {
220 FileName::Real(path) => path.clone(),
221 _ => PathBuf::from(r"doctest.rs"),
225 Temp(tempfile::TempDir),
230 fn path(&self) -> &std::path::Path {
232 DirState::Temp(t) => t.path(),
233 DirState::Perm(p) => p.as_path(),
238 let outdir = if let Some(mut path) = options.persist_doctests {
241 filename.to_string().rsplit('/').next().unwrap().replace(".", "_"),
244 std::fs::create_dir_all(&path).expect("Couldn't create directory for doctest executables");
249 TempFileBuilder::new()
250 .prefix("rustdoctest")
252 .expect("rustdoc needs a tempdir"),
255 let output_file = outdir.path().join("rust_out");
257 let rustc_binary = options
261 .unwrap_or_else(|| rustc_interface::util::rustc_path().expect("found rustc"));
262 let mut compiler = Command::new(&rustc_binary);
263 compiler.arg("--crate-type").arg("bin");
264 for cfg in &options.cfgs {
265 compiler.arg("--cfg").arg(&cfg);
267 if let Some(sysroot) = options.maybe_sysroot {
268 compiler.arg("--sysroot").arg(sysroot);
270 compiler.arg("--edition").arg(&edition.to_string());
271 compiler.env("UNSTABLE_RUSTDOC_TEST_PATH", path);
272 compiler.env("UNSTABLE_RUSTDOC_TEST_LINE", format!("{}", line as isize - line_offset as isize));
273 compiler.arg("-o").arg(&output_file);
275 compiler.arg("--test");
277 for lib_str in &options.lib_strs {
278 compiler.arg("-L").arg(&lib_str);
280 for extern_str in &options.extern_strs {
281 compiler.arg("--extern").arg(&extern_str);
283 compiler.arg("-Ccodegen-units=1");
284 for codegen_options_str in &options.codegen_options_strs {
285 compiler.arg("-C").arg(&codegen_options_str);
287 for debugging_option_str in &options.debugging_options_strs {
288 compiler.arg("-Z").arg(&debugging_option_str);
291 compiler.arg("--emit=metadata");
293 compiler.arg("--target").arg(target.to_string());
296 compiler.stdin(Stdio::piped());
297 compiler.stderr(Stdio::piped());
299 let mut child = compiler.spawn().expect("Failed to spawn rustc process");
301 let stdin = child.stdin.as_mut().expect("Failed to open stdin");
302 stdin.write_all(test.as_bytes()).expect("could write out test sources");
304 let output = child.wait_with_output().expect("Failed to read stdout");
306 struct Bomb<'a>(&'a str);
307 impl Drop for Bomb<'_> {
309 eprint!("{}", self.0);
313 let out = str::from_utf8(&output.stderr).unwrap();
314 let _bomb = Bomb(&out);
315 match (output.status.success(), compile_fail) {
317 return Err(TestFailure::UnexpectedCompilePass);
321 if !error_codes.is_empty() {
322 error_codes.retain(|err| !out.contains(err));
324 if !error_codes.is_empty() {
325 return Err(TestFailure::MissingErrorCodes(error_codes));
330 return Err(TestFailure::CompileError);
341 if let Some(tool) = runtool {
342 cmd = Command::new(tool);
343 cmd.arg(output_file);
344 cmd.args(runtool_args);
346 cmd = Command::new(output_file);
350 Err(e) => return Err(TestFailure::ExecutionError(e)),
352 if should_panic && out.status.success() {
353 return Err(TestFailure::UnexpectedRunPass);
354 } else if !should_panic && !out.status.success() {
355 return Err(TestFailure::ExecutionFailure(out));
363 /// Transforms a test into code that can be compiled into a Rust binary, and returns the number of
364 /// lines before the test code begins.
368 /// This function uses the compiler's parser internally. The parser will panic if it encounters a
369 /// fatal error while parsing the test.
372 cratename: Option<&str>,
373 dont_insert_main: bool,
376 ) -> (String, usize) {
377 let (crate_attrs, everything_else, crates) = partition_source(s);
378 let everything_else = everything_else.trim();
379 let mut line_offset = 0;
380 let mut prog = String::new();
382 if opts.attrs.is_empty() && !opts.display_warnings {
383 // If there aren't any attributes supplied by #![doc(test(attr(...)))], then allow some
384 // lints that are commonly triggered in doctests. The crate-level test attributes are
385 // commonly used to make tests fail in case they trigger warnings, so having this there in
386 // that case may cause some tests to pass when they shouldn't have.
387 prog.push_str("#![allow(unused)]\n");
391 // Next, any attributes that came from the crate root via #![doc(test(attr(...)))].
392 for attr in &opts.attrs {
393 prog.push_str(&format!("#![{}]\n", attr));
397 // Now push any outer attributes from the example, assuming they
398 // are intended to be crate attributes.
399 prog.push_str(&crate_attrs);
400 prog.push_str(&crates);
402 // Uses libsyntax to parse the doctest and find if there's a main fn and the extern
403 // crate already is included.
404 let (already_has_main, already_has_extern_crate, found_macro) = with_globals(edition, || {
405 use errors::emitter::EmitterWriter;
407 use rustc_parse::maybe_new_parser_from_source_str;
408 use rustc_span::source_map::FilePathMapping;
409 use syntax::sess::ParseSess;
411 let filename = FileName::anon_source_code(s);
412 let source = crates + &everything_else;
414 // Any errors in parsing should also appear when the doctest is compiled for real, so just
415 // send all the errors that libsyntax emits directly into a `Sink` instead of stderr.
416 let cm = Lrc::new(SourceMap::new(FilePathMapping::empty()));
417 let emitter = EmitterWriter::new(box io::sink(), None, false, false, false, None, false);
418 // FIXME(misdreavus): pass `-Z treat-err-as-bug` to the doctest parser
419 let handler = Handler::with_emitter(false, None, box emitter);
420 let sess = ParseSess::with_span_handler(handler, cm);
422 let mut found_main = false;
423 let mut found_extern_crate = cratename.is_none();
424 let mut found_macro = false;
426 let mut parser = match maybe_new_parser_from_source_str(&sess, filename, source) {
429 for mut err in errs {
433 return (found_main, found_extern_crate, found_macro);
438 match parser.parse_item() {
441 if let ast::ItemKind::Fn(..) = item.kind {
442 if item.ident.name == sym::main {
448 if !found_extern_crate {
449 if let ast::ItemKind::ExternCrate(original) = item.kind {
450 // This code will never be reached if `cratename` is none because
451 // `found_extern_crate` is initialized to `true` if it is none.
452 let cratename = cratename.unwrap();
455 Some(name) => found_extern_crate = name.as_str() == cratename,
456 None => found_extern_crate = item.ident.as_str() == cratename,
462 if let ast::ItemKind::Mac(..) = item.kind {
467 if found_main && found_extern_crate {
479 (found_main, found_extern_crate, found_macro)
482 // If a doctest's `fn main` is being masked by a wrapper macro, the parsing loop above won't
483 // see it. In that case, run the old text-based scan to see if they at least have a main
484 // function written inside a macro invocation. See
485 // https://github.com/rust-lang/rust/issues/56898
486 let already_has_main = if found_macro && !already_has_main {
489 let comment = line.find("//");
490 if let Some(comment_begins) = comment { &line[0..comment_begins] } else { line }
492 .any(|code| code.contains("fn main"))
497 // Don't inject `extern crate std` because it's already injected by the
499 if !already_has_extern_crate && !opts.no_crate_inject && cratename != Some("std") {
500 if let Some(cratename) = cratename {
501 // Make sure its actually used if not included.
502 if s.contains(cratename) {
503 prog.push_str(&format!("extern crate {};\n", cratename));
509 // FIXME: This code cannot yet handle no_std test cases yet
510 if dont_insert_main || already_has_main || prog.contains("![no_std]") {
511 prog.push_str(everything_else);
513 let returns_result = everything_else.trim_end().ends_with("(())");
514 let (main_pre, main_post) = if returns_result {
516 "fn main() { fn _inner() -> Result<(), impl core::fmt::Debug> {",
517 "}\n_inner().unwrap() }",
520 ("fn main() {\n", "\n}")
522 prog.extend([main_pre, everything_else, main_post].iter().cloned());
526 debug!("final doctest:\n{}", prog);
531 // FIXME(aburka): use a real parser to deal with multiline attributes
532 fn partition_source(s: &str) -> (String, String, String) {
533 #[derive(Copy, Clone, PartialEq)]
534 enum PartitionState {
539 let mut state = PartitionState::Attrs;
540 let mut before = String::new();
541 let mut crates = String::new();
542 let mut after = String::new();
544 for line in s.lines() {
545 let trimline = line.trim();
547 // FIXME(misdreavus): if a doc comment is placed on an extern crate statement, it will be
548 // shunted into "everything else"
550 PartitionState::Attrs => {
551 state = if trimline.starts_with("#![")
552 || trimline.chars().all(|c| c.is_whitespace())
553 || (trimline.starts_with("//") && !trimline.starts_with("///"))
555 PartitionState::Attrs
556 } else if trimline.starts_with("extern crate")
557 || trimline.starts_with("#[macro_use] extern crate")
559 PartitionState::Crates
561 PartitionState::Other
564 PartitionState::Crates => {
565 state = if trimline.starts_with("extern crate")
566 || trimline.starts_with("#[macro_use] extern crate")
567 || trimline.chars().all(|c| c.is_whitespace())
568 || (trimline.starts_with("//") && !trimline.starts_with("///"))
570 PartitionState::Crates
572 PartitionState::Other
575 PartitionState::Other => {}
579 PartitionState::Attrs => {
580 before.push_str(line);
581 before.push_str("\n");
583 PartitionState::Crates => {
584 crates.push_str(line);
585 crates.push_str("\n");
587 PartitionState::Other => {
588 after.push_str(line);
589 after.push_str("\n");
594 debug!("before:\n{}", before);
595 debug!("crates:\n{}", crates);
596 debug!("after:\n{}", after);
598 (before, after, crates)
602 fn add_test(&mut self, test: String, config: LangString, line: usize);
603 fn get_line(&self) -> usize {
606 fn register_header(&mut self, _name: &str, _level: u32) {}
609 pub struct Collector {
610 pub tests: Vec<testing::TestDescAndFn>,
612 // The name of the test displayed to the user, separated by `::`.
614 // In tests from Rust source, this is the path to the item
615 // e.g., `["std", "vec", "Vec", "push"]`.
617 // In tests from a markdown file, this is the titles of all headers (h1~h6)
618 // of the sections that contain the code block, e.g., if the markdown file is
631 // the `names` vector of that test will be `["Title", "Subtitle"]`.
636 enable_per_target_ignores: bool,
640 source_map: Option<Lrc<SourceMap>>,
641 filename: Option<PathBuf>,
650 source_map: Option<Lrc<SourceMap>>,
651 filename: Option<PathBuf>,
652 enable_per_target_ignores: bool,
659 enable_per_target_ignores,
668 fn generate_name(&self, line: usize, filename: &FileName) -> String {
669 format!("{} - {} (line {})", filename, self.names.join("::"), line)
672 pub fn set_position(&mut self, position: Span) {
673 self.position = position;
676 fn get_filename(&self) -> FileName {
677 if let Some(ref source_map) = self.source_map {
678 let filename = source_map.span_to_filename(self.position);
679 if let FileName::Real(ref filename) = filename {
680 if let Ok(cur_dir) = env::current_dir() {
681 if let Ok(path) = filename.strip_prefix(&cur_dir) {
682 return path.to_owned().into();
687 } else if let Some(ref filename) = self.filename {
688 filename.clone().into()
690 FileName::Custom("input".to_owned())
695 impl Tester for Collector {
696 fn add_test(&mut self, test: String, config: LangString, line: usize) {
697 let filename = self.get_filename();
698 let name = self.generate_name(line, &filename);
699 let cratename = self.cratename.to_string();
700 let opts = self.opts.clone();
701 let edition = config.edition.unwrap_or(self.options.edition.clone());
702 let options = self.options.clone();
703 let runtool = self.options.runtool.clone();
704 let runtool_args = self.options.runtool_args.clone();
705 let target = self.options.target.clone();
706 let target_str = target.to_string();
708 debug!("creating test {}: {}", name, test);
709 self.tests.push(testing::TestDescAndFn {
710 desc: testing::TestDesc {
711 name: testing::DynTestName(name.clone()),
712 ignore: match config.ignore {
714 Ignore::None => false,
715 Ignore::Some(ref ignores) => ignores.iter().any(|s| target_str.contains(s)),
717 // compiler failures are test failures
718 should_panic: testing::ShouldPanic::No,
719 allow_fail: config.allow_fail,
720 test_type: testing::TestType::DocTest,
722 testfn: testing::DynTestFn(box move || {
741 if let Err(err) = res {
743 TestFailure::CompileError => {
744 eprint!("Couldn't compile the test.");
746 TestFailure::UnexpectedCompilePass => {
747 eprint!("Test compiled successfully, but it's marked `compile_fail`.");
749 TestFailure::UnexpectedRunPass => {
750 eprint!("Test executable succeeded, but it's marked `should_panic`.");
752 TestFailure::MissingErrorCodes(codes) => {
753 eprint!("Some expected error codes were not found: {:?}", codes);
755 TestFailure::ExecutionError(err) => {
756 eprint!("Couldn't run the test: {}", err);
757 if err.kind() == io::ErrorKind::PermissionDenied {
758 eprint!(" - maybe your tempdir is mounted with noexec?");
761 TestFailure::ExecutionFailure(out) => {
762 let reason = if let Some(code) = out.status.code() {
763 format!("exit code {}", code)
765 String::from("terminated by signal")
768 eprintln!("Test executable failed ({}).", reason);
770 // FIXME(#12309): An unfortunate side-effect of capturing the test
771 // executable's output is that the relative ordering between the test's
772 // stdout and stderr is lost. However, this is better than the
773 // alternative: if the test executable inherited the parent's I/O
774 // handles the output wouldn't be captured at all, even on success.
776 // The ordering could be preserved if the test process' stderr was
777 // redirected to stdout, but that functionality does not exist in the
778 // standard library, so it may not be portable enough.
779 let stdout = str::from_utf8(&out.stdout).unwrap_or_default();
780 let stderr = str::from_utf8(&out.stderr).unwrap_or_default();
782 if !stdout.is_empty() || !stderr.is_empty() {
785 if !stdout.is_empty() {
786 eprintln!("stdout:\n{}", stdout);
789 if !stderr.is_empty() {
790 eprintln!("stderr:\n{}", stderr);
796 panic::resume_unwind(box ());
802 fn get_line(&self) -> usize {
803 if let Some(ref source_map) = self.source_map {
804 let line = self.position.lo().to_usize();
805 let line = source_map.lookup_char_pos(BytePos(line as u32)).line;
806 if line > 0 { line - 1 } else { line }
812 fn register_header(&mut self, name: &str, level: u32) {
813 if self.use_headers {
814 // We use these headings as test names, so it's good if
815 // they're valid identifiers.
820 if (i == 0 && rustc_lexer::is_id_start(c))
821 || (i != 0 && rustc_lexer::is_id_continue(c))
828 .collect::<String>();
830 // Here we try to efficiently assemble the header titles into the
831 // test name in the form of `h1::h2::h3::h4::h5::h6`.
833 // Suppose that originally `self.names` contains `[h1, h2, h3]`...
834 let level = level as usize;
835 if level <= self.names.len() {
836 // ... Consider `level == 2`. All headers in the lower levels
837 // are irrelevant in this new level. So we should reset
838 // `self.names` to contain headers until <h2>, and replace that
839 // slot with the new name: `[h1, name]`.
840 self.names.truncate(level);
841 self.names[level - 1] = name;
843 // ... On the other hand, consider `level == 5`. This means we
844 // need to extend `self.names` to contain five headers. We fill
845 // in the missing level (<h4>) with `_`. Thus `self.names` will
846 // become `[h1, h2, h3, "_", name]`.
847 if level - 1 > self.names.len() {
848 self.names.resize(level - 1, "_".to_owned());
850 self.names.push(name);
856 struct HirCollector<'a, 'hir> {
857 sess: &'a session::Session,
858 collector: &'a mut Collector,
859 map: &'a hir::map::Map<'hir>,
863 impl<'a, 'hir> HirCollector<'a, 'hir> {
864 fn visit_testable<F: FnOnce(&mut Self)>(
867 attrs: &[ast::Attribute],
870 let mut attrs = Attributes::from_ast(self.sess.diagnostic(), attrs);
871 if let Some(ref cfg) = attrs.cfg {
872 if !cfg.matches(&self.sess.parse_sess, Some(&self.sess.features_untracked())) {
877 let has_name = !name.is_empty();
879 self.collector.names.push(name);
882 attrs.collapse_doc_comments();
883 attrs.unindent_doc_comments();
884 // The collapse-docs pass won't combine sugared/raw doc attributes, or included files with
885 // anything else, this will combine them for us.
886 if let Some(doc) = attrs.collapsed_doc_value() {
887 self.collector.set_position(attrs.span.unwrap_or(DUMMY_SP));
888 markdown::find_testable_code(
892 self.collector.enable_per_target_ignores,
899 self.collector.names.pop();
904 impl<'a, 'hir> intravisit::Visitor<'hir> for HirCollector<'a, 'hir> {
905 fn nested_visit_map<'this>(&'this mut self) -> intravisit::NestedVisitorMap<'this, 'hir> {
906 intravisit::NestedVisitorMap::All(&self.map)
909 fn visit_item(&mut self, item: &'hir hir::Item) {
910 let name = if let hir::ItemKind::Impl(.., ref ty, _) = item.kind {
911 self.map.hir_to_pretty_string(ty.hir_id)
913 item.ident.to_string()
916 self.visit_testable(name, &item.attrs, |this| {
917 intravisit::walk_item(this, item);
921 fn visit_trait_item(&mut self, item: &'hir hir::TraitItem) {
922 self.visit_testable(item.ident.to_string(), &item.attrs, |this| {
923 intravisit::walk_trait_item(this, item);
927 fn visit_impl_item(&mut self, item: &'hir hir::ImplItem) {
928 self.visit_testable(item.ident.to_string(), &item.attrs, |this| {
929 intravisit::walk_impl_item(this, item);
933 fn visit_foreign_item(&mut self, item: &'hir hir::ForeignItem) {
934 self.visit_testable(item.ident.to_string(), &item.attrs, |this| {
935 intravisit::walk_foreign_item(this, item);
941 v: &'hir hir::Variant,
942 g: &'hir hir::Generics,
945 self.visit_testable(v.ident.to_string(), &v.attrs, |this| {
946 intravisit::walk_variant(this, v, g, item_id);
950 fn visit_struct_field(&mut self, f: &'hir hir::StructField) {
951 self.visit_testable(f.ident.to_string(), &f.attrs, |this| {
952 intravisit::walk_struct_field(this, f);
956 fn visit_macro_def(&mut self, macro_def: &'hir hir::MacroDef) {
957 self.visit_testable(macro_def.name.to_string(), ¯o_def.attrs, |_| ());