Skip to content

Commit

Permalink
Fix use of display().to_str in find_tests
Browse files Browse the repository at this point in the history
  • Loading branch information
sammykim committed Nov 18, 2013
1 parent e98ddef commit 120284d
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/test/harness/contenttest/contenttest.rs
Expand Up @@ -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()) );
}

Expand Down

5 comments on commit 120284d

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from jdm
at sammykim@120284d

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = c5ecc0d

Please sign in to comment.