]> git.lizzy.rs Git - rust.git/blob - src/librustdoc/test.rs
Add comment about the problem, and use provided path if available
[rust.git] / src / librustdoc / test.rs
1 // Copyright 2013-2014 The Rust Project Developers. See the COPYRIGHT
2 // file at the top-level directory of this distribution and at
3 // http://rust-lang.org/COPYRIGHT.
4 //
5 // Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6 // http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7 // <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8 // option. This file may not be copied, modified, or distributed
9 // except according to those terms.
10
11 use std::collections::HashMap;
12 use std::env;
13 use std::ffi::OsString;
14 use std::io::prelude::*;
15 use std::io;
16 use std::path::{Path, PathBuf};
17 use std::panic::{self, AssertUnwindSafe};
18 use std::process::Command;
19 use std::rc::Rc;
20 use std::str;
21 use std::sync::{Arc, Mutex};
22
23 use testing;
24 use rustc_lint;
25 use rustc::hir;
26 use rustc::hir::intravisit;
27 use rustc::session::{self, CompileIncomplete, config};
28 use rustc::session::config::{OutputType, OutputTypes, Externs};
29 use rustc::session::search_paths::{SearchPaths, PathKind};
30 use rustc_metadata::dynamic_lib::DynamicLibrary;
31 use tempdir::TempDir;
32 use rustc_driver::{self, driver, Compilation};
33 use rustc_driver::driver::phase_2_configure_and_expand;
34 use rustc_metadata::cstore::CStore;
35 use rustc_resolve::MakeGlobMap;
36 use syntax::ast;
37 use syntax::codemap::CodeMap;
38 use syntax::feature_gate::UnstableFeatures;
39 use syntax_pos::{BytePos, DUMMY_SP, Pos, Span, FileName};
40 use errors;
41 use errors::emitter::ColorConfig;
42
43 use clean::Attributes;
44 use html::markdown::{self, RenderType};
45
46 #[derive(Clone, Default)]
47 pub struct TestOptions {
48     pub no_crate_inject: bool,
49     pub attrs: Vec<String>,
50 }
51
52 pub fn run(input_path: &Path,
53            cfgs: Vec<String>,
54            libs: SearchPaths,
55            externs: Externs,
56            mut test_args: Vec<String>,
57            crate_name: Option<String>,
58            maybe_sysroot: Option<PathBuf>,
59            render_type: RenderType,
60            display_warnings: bool,
61            linker: Option<PathBuf>)
62            -> isize {
63     let input = config::Input::File(input_path.to_owned());
64
65     let sessopts = config::Options {
66         maybe_sysroot: maybe_sysroot.clone().or_else(
67             || Some(env::current_exe().unwrap().parent().unwrap().parent().unwrap().to_path_buf())),
68         search_paths: libs.clone(),
69         crate_types: vec![config::CrateTypeDylib],
70         externs: externs.clone(),
71         unstable_features: UnstableFeatures::from_environment(),
72         lint_cap: Some(::rustc::lint::Level::Allow),
73         actually_rustdoc: true,
74         ..config::basic_options().clone()
75     };
76
77     let codemap = Rc::new(CodeMap::new(sessopts.file_path_mapping()));
78     let handler =
79         errors::Handler::with_tty_emitter(ColorConfig::Auto,
80                                           true, false,
81                                           Some(codemap.clone()));
82
83     let mut sess = session::build_session_(
84         sessopts, Some(input_path.to_owned()), handler, codemap.clone(),
85     );
86     let trans = rustc_driver::get_trans(&sess);
87     let cstore = Rc::new(CStore::new(trans.metadata_loader()));
88     rustc_lint::register_builtins(&mut sess.lint_store.borrow_mut(), Some(&sess));
89     sess.parse_sess.config =
90         config::build_configuration(&sess, config::parse_cfgspecs(cfgs.clone()));
91
92     let krate = panictry!(driver::phase_1_parse_input(&driver::CompileController::basic(),
93                                                       &sess,
94                                                       &input));
95     let driver::ExpansionResult { defs, mut hir_forest, .. } = {
96         phase_2_configure_and_expand(
97             &sess,
98             &cstore,
99             krate,
100             None,
101             "rustdoc-test",
102             None,
103             MakeGlobMap::No,
104             |_| Ok(()),
105         ).expect("phase_2_configure_and_expand aborted in rustdoc!")
106     };
107
108     let crate_name = crate_name.unwrap_or_else(|| {
109         ::rustc_trans_utils::link::find_crate_name(None, &hir_forest.krate().attrs, &input)
110     });
111     let opts = scrape_test_config(hir_forest.krate());
112     let mut collector = Collector::new(crate_name,
113                                        cfgs,
114                                        libs,
115                                        externs,
116                                        false,
117                                        opts,
118                                        maybe_sysroot,
119                                        Some(codemap),
120                                        None,
121                                        render_type,
122                                        linker);
123
124     {
125         let map = hir::map::map_crate(&sess, &*cstore, &mut hir_forest, &defs);
126         let krate = map.krate();
127         let mut hir_collector = HirCollector {
128             sess: &sess,
129             collector: &mut collector,
130             map: &map
131         };
132         hir_collector.visit_testable("".to_string(), &krate.attrs, |this| {
133             intravisit::walk_crate(this, krate);
134         });
135     }
136
137     test_args.insert(0, "rustdoctest".to_string());
138
139     testing::test_main(&test_args,
140                        collector.tests.into_iter().collect(),
141                        testing::Options::new().display_output(display_warnings));
142     0
143 }
144
145 // Look for #![doc(test(no_crate_inject))], used by crates in the std facade
146 fn scrape_test_config(krate: &::rustc::hir::Crate) -> TestOptions {
147     use syntax::print::pprust;
148
149     let mut opts = TestOptions {
150         no_crate_inject: false,
151         attrs: Vec::new(),
152     };
153
154     let test_attrs: Vec<_> = krate.attrs.iter()
155         .filter(|a| a.check_name("doc"))
156         .flat_map(|a| a.meta_item_list().unwrap_or_else(Vec::new))
157         .filter(|a| a.check_name("test"))
158         .collect();
159     let attrs = test_attrs.iter().flat_map(|a| a.meta_item_list().unwrap_or(&[]));
160
161     for attr in attrs {
162         if attr.check_name("no_crate_inject") {
163             opts.no_crate_inject = true;
164         }
165         if attr.check_name("attr") {
166             if let Some(l) = attr.meta_item_list() {
167                 for item in l {
168                     opts.attrs.push(pprust::meta_list_item_to_string(item));
169                 }
170             }
171         }
172     }
173
174     opts
175 }
176
177 fn run_test(test: &str, cratename: &str, filename: &FileName, line: usize,
178             cfgs: Vec<String>, libs: SearchPaths,
179             externs: Externs,
180             should_panic: bool, no_run: bool, as_test_harness: bool,
181             compile_fail: bool, mut error_codes: Vec<String>, opts: &TestOptions,
182             maybe_sysroot: Option<PathBuf>,
183             linker: Option<PathBuf>) {
184     // the test harness wants its own `main` & top level functions, so
185     // never wrap the test in `fn main() { ... }`
186     let (test, line_offset) = make_test(test, Some(cratename), as_test_harness, opts);
187     // FIXME(#44940): if doctests ever support path remapping, then this filename
188     // needs to be the result of CodeMap::span_to_unmapped_path
189     let input = config::Input::Str {
190         name: filename.to_owned(),
191         input: test.to_owned(),
192     };
193     let outputs = OutputTypes::new(&[(OutputType::Exe, None)]);
194
195     let sessopts = config::Options {
196         maybe_sysroot: maybe_sysroot.or_else(
197             || Some(env::current_exe().unwrap().parent().unwrap().parent().unwrap().to_path_buf())),
198         search_paths: libs,
199         crate_types: vec![config::CrateTypeExecutable],
200         output_types: outputs,
201         externs,
202         cg: config::CodegenOptions {
203             prefer_dynamic: true,
204             linker,
205             .. config::basic_codegen_options()
206         },
207         test: as_test_harness,
208         unstable_features: UnstableFeatures::from_environment(),
209         ..config::basic_options().clone()
210     };
211
212     // Shuffle around a few input and output handles here. We're going to pass
213     // an explicit handle into rustc to collect output messages, but we also
214     // want to catch the error message that rustc prints when it fails.
215     //
216     // We take our thread-local stderr (likely set by the test runner) and replace
217     // it with a sink that is also passed to rustc itself. When this function
218     // returns the output of the sink is copied onto the output of our own thread.
219     //
220     // The basic idea is to not use a default Handler for rustc, and then also
221     // not print things by default to the actual stderr.
222     struct Sink(Arc<Mutex<Vec<u8>>>);
223     impl Write for Sink {
224         fn write(&mut self, data: &[u8]) -> io::Result<usize> {
225             Write::write(&mut *self.0.lock().unwrap(), data)
226         }
227         fn flush(&mut self) -> io::Result<()> { Ok(()) }
228     }
229     struct Bomb(Arc<Mutex<Vec<u8>>>, Box<Write+Send>);
230     impl Drop for Bomb {
231         fn drop(&mut self) {
232             let _ = self.1.write_all(&self.0.lock().unwrap());
233         }
234     }
235     let data = Arc::new(Mutex::new(Vec::new()));
236     let codemap = Rc::new(CodeMap::new_doctest(
237         sessopts.file_path_mapping(), filename.clone(), line as isize - line_offset as isize
238     ));
239     let emitter = errors::emitter::EmitterWriter::new(box Sink(data.clone()),
240                                                       Some(codemap.clone()),
241                                                       false);
242     let old = io::set_panic(Some(box Sink(data.clone())));
243     let _bomb = Bomb(data.clone(), old.unwrap_or(box io::stdout()));
244
245     // Compile the code
246     let diagnostic_handler = errors::Handler::with_emitter(true, false, box emitter);
247
248     let mut sess = session::build_session_(
249         sessopts, None, diagnostic_handler, codemap,
250     );
251     let trans = rustc_driver::get_trans(&sess);
252     let cstore = Rc::new(CStore::new(trans.metadata_loader()));
253     rustc_lint::register_builtins(&mut sess.lint_store.borrow_mut(), Some(&sess));
254
255     let outdir = Mutex::new(TempDir::new("rustdoctest").ok().expect("rustdoc needs a tempdir"));
256     let libdir = sess.target_filesearch(PathKind::All).get_lib_path();
257     let mut control = driver::CompileController::basic();
258     sess.parse_sess.config =
259         config::build_configuration(&sess, config::parse_cfgspecs(cfgs.clone()));
260     let out = Some(outdir.lock().unwrap().path().to_path_buf());
261
262     if no_run {
263         control.after_analysis.stop = Compilation::Stop;
264     }
265
266     let res = panic::catch_unwind(AssertUnwindSafe(|| {
267         driver::compile_input(
268             trans,
269             &sess,
270             &cstore,
271             &None,
272             &input,
273             &out,
274             &None,
275             None,
276             &control
277         )
278     }));
279
280     let compile_result = match res {
281         Ok(Ok(())) | Ok(Err(CompileIncomplete::Stopped)) => Ok(()),
282         Err(_) | Ok(Err(CompileIncomplete::Errored(_))) => Err(())
283     };
284
285     match (compile_result, compile_fail) {
286         (Ok(()), true) => {
287             panic!("test compiled while it wasn't supposed to")
288         }
289         (Ok(()), false) => {}
290         (Err(()), true) => {
291             if error_codes.len() > 0 {
292                 let out = String::from_utf8(data.lock().unwrap().to_vec()).unwrap();
293                 error_codes.retain(|err| !out.contains(err));
294             }
295         }
296         (Err(()), false) => {
297             panic!("couldn't compile the test")
298         }
299     }
300
301     if error_codes.len() > 0 {
302         panic!("Some expected error codes were not found: {:?}", error_codes);
303     }
304
305     if no_run { return }
306
307     // Run the code!
308     //
309     // We're careful to prepend the *target* dylib search path to the child's
310     // environment to ensure that the target loads the right libraries at
311     // runtime. It would be a sad day if the *host* libraries were loaded as a
312     // mistake.
313     let mut cmd = Command::new(&outdir.lock().unwrap().path().join("rust_out"));
314     let var = DynamicLibrary::envvar();
315     let newpath = {
316         let path = env::var_os(var).unwrap_or(OsString::new());
317         let mut path = env::split_paths(&path).collect::<Vec<_>>();
318         path.insert(0, libdir.clone());
319         env::join_paths(path).unwrap()
320     };
321     cmd.env(var, &newpath);
322
323     match cmd.output() {
324         Err(e) => panic!("couldn't run the test: {}{}", e,
325                         if e.kind() == io::ErrorKind::PermissionDenied {
326                             " - maybe your tempdir is mounted with noexec?"
327                         } else { "" }),
328         Ok(out) => {
329             if should_panic && out.status.success() {
330                 panic!("test executable succeeded when it should have failed");
331             } else if !should_panic && !out.status.success() {
332                 panic!("test executable failed:\n{}\n{}\n",
333                        str::from_utf8(&out.stdout).unwrap_or(""),
334                        str::from_utf8(&out.stderr).unwrap_or(""));
335             }
336         }
337     }
338 }
339
340 /// Makes the test file. Also returns the number of lines before the code begins
341 pub fn make_test(s: &str,
342                  cratename: Option<&str>,
343                  dont_insert_main: bool,
344                  opts: &TestOptions)
345                  -> (String, usize) {
346     let (crate_attrs, everything_else) = partition_source(s);
347     let mut line_offset = 0;
348     let mut prog = String::new();
349
350     if opts.attrs.is_empty() {
351         // If there aren't any attributes supplied by #![doc(test(attr(...)))], then allow some
352         // lints that are commonly triggered in doctests. The crate-level test attributes are
353         // commonly used to make tests fail in case they trigger warnings, so having this there in
354         // that case may cause some tests to pass when they shouldn't have.
355         prog.push_str("#![allow(unused)]\n");
356         line_offset += 1;
357     }
358
359     // Next, any attributes that came from the crate root via #![doc(test(attr(...)))].
360     for attr in &opts.attrs {
361         prog.push_str(&format!("#![{}]\n", attr));
362         line_offset += 1;
363     }
364
365     // Now push any outer attributes from the example, assuming they
366     // are intended to be crate attributes.
367     prog.push_str(&crate_attrs);
368
369     // Don't inject `extern crate std` because it's already injected by the
370     // compiler.
371     if !s.contains("extern crate") && !opts.no_crate_inject && cratename != Some("std") {
372         if let Some(cratename) = cratename {
373             if s.contains(cratename) {
374                 prog.push_str(&format!("extern crate {};\n", cratename));
375                 line_offset += 1;
376             }
377         }
378     }
379
380     // FIXME (#21299): prefer libsyntax or some other actual parser over this
381     // best-effort ad hoc approach
382     let already_has_main = s.lines()
383         .map(|line| {
384             let comment = line.find("//");
385             if let Some(comment_begins) = comment {
386                 &line[0..comment_begins]
387             } else {
388                 line
389             }
390         })
391         .any(|code| code.contains("fn main"));
392
393     if dont_insert_main || already_has_main {
394         prog.push_str(&everything_else);
395     } else {
396         prog.push_str("fn main() {\n");
397         line_offset += 1;
398         prog.push_str(&everything_else);
399         prog = prog.trim().into();
400         prog.push_str("\n}");
401     }
402
403     info!("final test program: {}", prog);
404
405     (prog, line_offset)
406 }
407
408 // FIXME(aburka): use a real parser to deal with multiline attributes
409 fn partition_source(s: &str) -> (String, String) {
410     use std_unicode::str::UnicodeStr;
411
412     let mut after_header = false;
413     let mut before = String::new();
414     let mut after = String::new();
415
416     for line in s.lines() {
417         let trimline = line.trim();
418         let header = trimline.is_whitespace() ||
419             trimline.starts_with("#![");
420         if !header || after_header {
421             after_header = true;
422             after.push_str(line);
423             after.push_str("\n");
424         } else {
425             before.push_str(line);
426             before.push_str("\n");
427         }
428     }
429
430     (before, after)
431 }
432
433 pub struct Collector {
434     pub tests: Vec<testing::TestDescAndFn>,
435     // to be removed when hoedown will be definitely gone
436     pub old_tests: HashMap<String, Vec<String>>,
437
438     // The name of the test displayed to the user, separated by `::`.
439     //
440     // In tests from Rust source, this is the path to the item
441     // e.g. `["std", "vec", "Vec", "push"]`.
442     //
443     // In tests from a markdown file, this is the titles of all headers (h1~h6)
444     // of the sections that contain the code block, e.g. if the markdown file is
445     // written as:
446     //
447     // ``````markdown
448     // # Title
449     //
450     // ## Subtitle
451     //
452     // ```rust
453     // assert!(true);
454     // ```
455     // ``````
456     //
457     // the `names` vector of that test will be `["Title", "Subtitle"]`.
458     names: Vec<String>,
459
460     cfgs: Vec<String>,
461     libs: SearchPaths,
462     externs: Externs,
463     use_headers: bool,
464     cratename: String,
465     opts: TestOptions,
466     maybe_sysroot: Option<PathBuf>,
467     position: Span,
468     codemap: Option<Rc<CodeMap>>,
469     filename: Option<PathBuf>,
470     // to be removed when hoedown will be removed as well
471     pub render_type: RenderType,
472     linker: Option<PathBuf>,
473 }
474
475 impl Collector {
476     pub fn new(cratename: String, cfgs: Vec<String>, libs: SearchPaths, externs: Externs,
477                use_headers: bool, opts: TestOptions, maybe_sysroot: Option<PathBuf>,
478                codemap: Option<Rc<CodeMap>>, filename: Option<PathBuf>,
479                render_type: RenderType, linker: Option<PathBuf>) -> Collector {
480         Collector {
481             tests: Vec::new(),
482             old_tests: HashMap::new(),
483             names: Vec::new(),
484             cfgs,
485             libs,
486             externs,
487             use_headers,
488             cratename,
489             opts,
490             maybe_sysroot,
491             position: DUMMY_SP,
492             codemap,
493             filename,
494             render_type,
495             linker,
496         }
497     }
498
499     fn generate_name(&self, line: usize, filename: &FileName) -> String {
500         format!("{} - {} (line {})", filename, self.names.join("::"), line)
501     }
502
503     // to be removed once hoedown is gone
504     fn generate_name_beginning(&self, filename: &FileName) -> String {
505         format!("{} - {} (line", filename, self.names.join("::"))
506     }
507
508     pub fn add_old_test(&mut self, test: String, filename: FileName) {
509         let name_beg = self.generate_name_beginning(&filename);
510         let entry = self.old_tests.entry(name_beg)
511                                   .or_insert(Vec::new());
512         entry.push(test.trim().to_owned());
513     }
514
515     pub fn add_test(&mut self, test: String,
516                     should_panic: bool, no_run: bool, should_ignore: bool,
517                     as_test_harness: bool, compile_fail: bool, error_codes: Vec<String>,
518                     line: usize, filename: FileName, allow_fail: bool) {
519         let name = self.generate_name(line, &filename);
520         // to be removed when hoedown is removed
521         if self.render_type == RenderType::Pulldown {
522             let name_beg = self.generate_name_beginning(&filename);
523             let mut found = false;
524             let test = test.trim().to_owned();
525             if let Some(entry) = self.old_tests.get_mut(&name_beg) {
526                 found = entry.remove_item(&test).is_some();
527             }
528             if !found {
529                 eprintln!("WARNING: {} Code block is not currently run as a test, but will \
530                            in future versions of rustdoc. Please ensure this code block is \
531                            a runnable test, or use the `ignore` directive.",
532                           name);
533                 return
534             }
535         }
536         let cfgs = self.cfgs.clone();
537         let libs = self.libs.clone();
538         let externs = self.externs.clone();
539         let cratename = self.cratename.to_string();
540         let opts = self.opts.clone();
541         let maybe_sysroot = self.maybe_sysroot.clone();
542         let linker = self.linker.clone();
543         debug!("Creating test {}: {}", name, test);
544         self.tests.push(testing::TestDescAndFn {
545             desc: testing::TestDesc {
546                 name: testing::DynTestName(name),
547                 ignore: should_ignore,
548                 // compiler failures are test failures
549                 should_panic: testing::ShouldPanic::No,
550                 allow_fail,
551             },
552             testfn: testing::DynTestFn(box move || {
553                 let panic = io::set_panic(None);
554                 let print = io::set_print(None);
555                 match {
556                     rustc_driver::in_rustc_thread(move || {
557                         io::set_panic(panic);
558                         io::set_print(print);
559                         run_test(&test,
560                                  &cratename,
561                                  &filename,
562                                  line,
563                                  cfgs,
564                                  libs,
565                                  externs,
566                                  should_panic,
567                                  no_run,
568                                  as_test_harness,
569                                  compile_fail,
570                                  error_codes,
571                                  &opts,
572                                  maybe_sysroot,
573                                  linker)
574                     })
575                 } {
576                     Ok(()) => (),
577                     Err(err) => panic::resume_unwind(err),
578                 }
579             }),
580         });
581     }
582
583     pub fn get_line(&self) -> usize {
584         if let Some(ref codemap) = self.codemap {
585             let line = self.position.lo().to_usize();
586             let line = codemap.lookup_char_pos(BytePos(line as u32)).line;
587             if line > 0 { line - 1 } else { line }
588         } else {
589             0
590         }
591     }
592
593     pub fn set_position(&mut self, position: Span) {
594         self.position = position;
595     }
596
597     pub fn get_filename(&self) -> FileName {
598         if let Some(ref codemap) = self.codemap {
599             let filename = codemap.span_to_filename(self.position);
600             if let FileName::Real(ref filename) = filename {
601                 if let Ok(cur_dir) = env::current_dir() {
602                     if let Ok(path) = filename.strip_prefix(&cur_dir) {
603                         return path.to_owned().into();
604                     }
605                 }
606             }
607             filename
608         } else if let Some(ref filename) = self.filename {
609             filename.clone().into()
610         } else {
611             FileName::Custom("input".to_owned())
612         }
613     }
614
615     pub fn register_header(&mut self, name: &str, level: u32) {
616         if self.use_headers {
617             // we use these headings as test names, so it's good if
618             // they're valid identifiers.
619             let name = name.chars().enumerate().map(|(i, c)| {
620                     if (i == 0 && c.is_xid_start()) ||
621                         (i != 0 && c.is_xid_continue()) {
622                         c
623                     } else {
624                         '_'
625                     }
626                 }).collect::<String>();
627
628             // Here we try to efficiently assemble the header titles into the
629             // test name in the form of `h1::h2::h3::h4::h5::h6`.
630             //
631             // Suppose originally `self.names` contains `[h1, h2, h3]`...
632             let level = level as usize;
633             if level <= self.names.len() {
634                 // ... Consider `level == 2`. All headers in the lower levels
635                 // are irrelevant in this new level. So we should reset
636                 // `self.names` to contain headers until <h2>, and replace that
637                 // slot with the new name: `[h1, name]`.
638                 self.names.truncate(level);
639                 self.names[level - 1] = name;
640             } else {
641                 // ... On the other hand, consider `level == 5`. This means we
642                 // need to extend `self.names` to contain five headers. We fill
643                 // in the missing level (<h4>) with `_`. Thus `self.names` will
644                 // become `[h1, h2, h3, "_", name]`.
645                 if level - 1 > self.names.len() {
646                     self.names.resize(level - 1, "_".to_owned());
647                 }
648                 self.names.push(name);
649             }
650         }
651     }
652 }
653
654 struct HirCollector<'a, 'hir: 'a> {
655     sess: &'a session::Session,
656     collector: &'a mut Collector,
657     map: &'a hir::map::Map<'hir>
658 }
659
660 impl<'a, 'hir> HirCollector<'a, 'hir> {
661     fn visit_testable<F: FnOnce(&mut Self)>(&mut self,
662                                             name: String,
663                                             attrs: &[ast::Attribute],
664                                             nested: F) {
665         let mut attrs = Attributes::from_ast(self.sess.diagnostic(), attrs);
666         if let Some(ref cfg) = attrs.cfg {
667             if !cfg.matches(&self.sess.parse_sess, Some(&self.sess.features.borrow())) {
668                 return;
669             }
670         }
671
672         let has_name = !name.is_empty();
673         if has_name {
674             self.collector.names.push(name);
675         }
676
677         attrs.collapse_doc_comments();
678         attrs.unindent_doc_comments();
679         // the collapse-docs pass won't combine sugared/raw doc attributes, or included files with
680         // anything else, this will combine them for us
681         if let Some(doc) = attrs.collapsed_doc_value() {
682             if self.collector.render_type == RenderType::Pulldown {
683                 markdown::old_find_testable_code(&doc, self.collector,
684                                                  attrs.span.unwrap_or(DUMMY_SP));
685                 markdown::find_testable_code(&doc, self.collector,
686                                              attrs.span.unwrap_or(DUMMY_SP));
687             } else {
688                 markdown::old_find_testable_code(&doc, self.collector,
689                                                  attrs.span.unwrap_or(DUMMY_SP));
690             }
691         }
692
693         nested(self);
694
695         if has_name {
696             self.collector.names.pop();
697         }
698     }
699 }
700
701 impl<'a, 'hir> intravisit::Visitor<'hir> for HirCollector<'a, 'hir> {
702     fn nested_visit_map<'this>(&'this mut self) -> intravisit::NestedVisitorMap<'this, 'hir> {
703         intravisit::NestedVisitorMap::All(&self.map)
704     }
705
706     fn visit_item(&mut self, item: &'hir hir::Item) {
707         let name = if let hir::ItemImpl(.., ref ty, _) = item.node {
708             self.map.node_to_pretty_string(ty.id)
709         } else {
710             item.name.to_string()
711         };
712
713         self.visit_testable(name, &item.attrs, |this| {
714             intravisit::walk_item(this, item);
715         });
716     }
717
718     fn visit_trait_item(&mut self, item: &'hir hir::TraitItem) {
719         self.visit_testable(item.name.to_string(), &item.attrs, |this| {
720             intravisit::walk_trait_item(this, item);
721         });
722     }
723
724     fn visit_impl_item(&mut self, item: &'hir hir::ImplItem) {
725         self.visit_testable(item.name.to_string(), &item.attrs, |this| {
726             intravisit::walk_impl_item(this, item);
727         });
728     }
729
730     fn visit_foreign_item(&mut self, item: &'hir hir::ForeignItem) {
731         self.visit_testable(item.name.to_string(), &item.attrs, |this| {
732             intravisit::walk_foreign_item(this, item);
733         });
734     }
735
736     fn visit_variant(&mut self,
737                      v: &'hir hir::Variant,
738                      g: &'hir hir::Generics,
739                      item_id: ast::NodeId) {
740         self.visit_testable(v.node.name.to_string(), &v.node.attrs, |this| {
741             intravisit::walk_variant(this, v, g, item_id);
742         });
743     }
744
745     fn visit_struct_field(&mut self, f: &'hir hir::StructField) {
746         self.visit_testable(f.name.to_string(), &f.attrs, |this| {
747             intravisit::walk_struct_field(this, f);
748         });
749     }
750
751     fn visit_macro_def(&mut self, macro_def: &'hir hir::MacroDef) {
752         self.visit_testable(macro_def.name.to_string(), &macro_def.attrs, |_| ());
753     }
754 }