Skip to content
Permalink
Browse files

Fix use of display().to_str in find_tests

  • Loading branch information...
sammykim committed Nov 15, 2013
1 parent e98ddef commit 120284d186546d356b5c83638399565323e1b110
Showing with 1 addition and 2 deletions.
  1. +1 −2 src/test/harness/contenttest/contenttest.rs
@@ -68,8 +68,7 @@ fn test_options(config: Config) -> TestOpts {

fn find_tests(config: Config) -> ~[TestDescAndFn] {
let mut files = list_dir_path(&Path::new(config.source_dir));
// FIXME (#1094): not the right way to transform a path
files.retain( |file| file.display().to_str().ends_with(".html") );
files.retain(|file| file.extension_str() == Some("html") );
return files.map(|file| make_test(file.display().to_str()) );
}

5 comments on commit 120284d

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Nov 18, 2013

saw approval from jdm
at sammykim@120284d

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Nov 18, 2013

merging sammykim/servo/issue-1197 = 120284d into auto

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Nov 18, 2013

sammykim/servo/issue-1197 = 120284d merged ok, testing candidate = c5ecc0d

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo replied Nov 18, 2013

fast-forwarding master to auto = c5ecc0d

Please sign in to comment.
You can’t perform that action at this time.