Skip to content

Commit ef16be0

Browse files
author
Geobert Quach
committed
fix(tests): normalizing EOL management
Closes #241 add editorconfig add gitattributes
1 parent 8003c11 commit ef16be0

39 files changed

+607
-2
lines changed

‎.editorconfig

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
root = true
2+
3+
[*]
4+
end_of_line = lf
5+
insert_final_newline = true

‎.gitattributes

+2
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
# force LF on checkout to avoid rustfmt messing up the EOL
2+
* text eol=lf

‎Cargo.lock

+7
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎Cargo.toml

+1
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ serde = "1.0"
4343
serde_yaml = "0.7"
4444
serde_json = "1.0"
4545
toml = "0.4.0"
46+
normalize-line-endings = "0.2.2"
4647

4748
[dependencies.sass-rs]
4849
version = "0.2"

‎src/cobalt_model/files.rs

+3-1
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,12 @@ use std::fs;
22
use std::io::Read;
33
use std::io::Write;
44
use std::path;
5+
use std::iter::FromIterator;
56

67
use ignore::Match;
78
use ignore::gitignore::{Gitignore, GitignoreBuilder};
89
use walkdir::{WalkDir, DirEntry};
9-
10+
use normalize_line_endings::normalized;
1011
use error::Result;
1112

1213
pub struct FilesBuilder {
@@ -189,6 +190,7 @@ pub fn read_file<P: AsRef<path::Path>>(path: P) -> Result<String> {
189190
let mut file = fs::File::open(path.as_ref())?;
190191
let mut text = String::new();
191192
file.read_to_string(&mut text)?;
193+
let text = String::from_iter(normalized(text.chars()));
192194
Ok(text)
193195
}
194196

‎src/lib.rs

+2
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,8 @@ extern crate lazy_static;
8585
#[macro_use]
8686
extern crate serde;
8787

88+
extern crate normalize_line_endings;
89+
8890
pub use cobalt::build;
8991
pub use error::Error;
9092
pub use cobalt_model::Config;

‎src/template.rs

-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ use std::path;
33
use std::result;
44

55
use liquid;
6-
76
use error::*;
87
use cobalt_model;
98
use cobalt_model::files;

‎tests/fixtures/excerpts_CRLF/.editorconfig

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
[*]
2+
end_of_line = crlf
3+
insert_final_newline = true

‎tests/fixtures/excerpts_CRLF/.gitattributes

+2
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
# force tests to keep their Windows EOL
2+
* text eol=crlf

‎tests/fixtures/excerpts_CRLF/_cobalt.yml

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
source: "."
2+
site:
3+
title: cobalt blog

‎tests/fixtures/excerpts_CRLF/_includes/post.liquid

+6
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
<div>
2+
<h2>{{ page.title }}</h2>
3+
<p>
4+
{{ page.content }}
5+
</p>
6+
</div>

‎tests/fixtures/excerpts_CRLF/_layouts/default.liquid

+15
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
<!DOCTYPE html>
2+
<html>
3+
<head>
4+
<title>{{ page.title }}</title>
5+
</head>
6+
<body>
7+
<div>
8+
{% if page.collection == "posts" %}
9+
{% include 'post.liquid' %}
10+
{% else %}
11+
{{ page.content }}
12+
{% endif %}
13+
</div>
14+
</body>
15+
</html>

‎tests/fixtures/excerpts_CRLF/index.liquid

+15
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
layout: default.liquid
2+
---
3+
<div >
4+
<h2>Blog!</h2>
5+
<!--<br />-->
6+
<div>
7+
{% for post in collections.posts.pages %}
8+
<div>
9+
<h4>{{post.title}}</h4>
10+
<h4><a href="{{post.permalink}}">{{ post.title }}</a></h4>
11+
{{ post.excerpt }}
12+
</div>
13+
{% endfor %}
14+
</div>
15+
</div>

‎tests/fixtures/excerpts_CRLF/posts/post-1.md

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
layout: default.liquid
2+
3+
title: First block is an excerpt
4+
published_date: 2016-01-14 21:00:30 -0500
5+
---
6+
7+
# This is our first Post!
8+
9+
Welcome to the first post ever on cobalt.rs!

‎tests/fixtures/excerpts_CRLF/posts/post-10.md

+12
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
layout: default.liquid
2+
3+
title: Reference-style links immediately above first block
4+
published_date: 2017-02-11 04:50:21 -0000
5+
---
6+
7+
[20]: /0
8+
[21]: /1
9+
[22]: /2
10+
[23]: /3
11+
References are placed immediately above the first paragraph:
12+
[20][] [21][] [22][] [23][]

‎tests/fixtures/excerpts_CRLF/posts/post-11.md

+16
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
layout: default.liquid
2+
3+
title: Reference-style links immediately below first block do not resolve
4+
published_date: 2017-02-11 04:51:34 -0000
5+
---
6+
7+
References are placed immediately below the first paragraph:
8+
[30][] [31][] [32][] [33][]
9+
[30]: /0
10+
[31]: /1
11+
[32]: /2
12+
[33]: /3
13+
14+
Note that this is at it should be, as the markdown implementations I've tried
15+
(including CommonMark) do not render the above links (when the post is rendered
16+
individually).

‎tests/fixtures/excerpts_CRLF/posts/post-12.liquid

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
layout: default.liquid
2+
3+
title: Reference-style links in a non-markdown post do not resolve
4+
published_date: 2017-02-11 04:48:11 -0000
5+
---
6+
7+
References are placed below the first paragraph, and are separated from the
8+
first paragraph by a single blank line:
9+
[40][] [41][] [42][] [43][]
10+
11+
[40]: /0
12+
[41]: /1
13+
[42]: /2
14+
[43]: /3

‎tests/fixtures/excerpts_CRLF/posts/post-2.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
layout: default.liquid
2+
3+
title: An empty post means an empty excerpt
4+
published_date: 2016-01-14 21:01:30 -0500
5+
---

‎tests/fixtures/excerpts_CRLF/posts/post-3.md

+10
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
layout: default.liquid
2+
3+
title: Explicit `excerpt`
4+
published_date: 2016-01-14 21:02:30 -0500
5+
excerpt: Is in `markdown`
6+
---
7+
8+
# This is our third Post!
9+
10+
Welcome to the third post ever on cobalt.rs!

‎tests/fixtures/excerpts_CRLF/posts/post-4.md

+13
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
layout: default.liquid
2+
3+
title: Custom excerpt separator
4+
published_date: 2016-01-14 21:03:30 -0500
5+
excerpt_separator: <!-- more -->
6+
---
7+
8+
# Custom excerpt separator
9+
10+
Welcome to the 4th post on cobalt.rs!
11+
<!-- more -->
12+
13+
Something below the separator.

‎tests/fixtures/excerpts_CRLF/posts/post-5.md

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
layout: default.liquid
2+
3+
title: Both excerpt and excerpt separator are there
4+
published_date: 2016-01-14 21:04:30 -0500
5+
excerpt: "`excerpt` wins"
6+
excerpt_separator: <!-- more -->
7+
---
8+
9+
# Both excerpt and excerpt separator are there
10+
11+
Welcome to the 5th post on cobalt.rs!
12+
<!-- more -->
13+
14+
Something below the separator.

‎tests/fixtures/excerpts_CRLF/posts/post-6.liquid

+13
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
layout: default.liquid
2+
3+
title: Implicit excerpts work only in markdown
4+
published_date: 2016-01-14 21:05:30 -0500
5+
excerpt_separator: <!-- more -->
6+
---
7+
8+
<h1>Custom excerpt separator</h1>
9+
10+
<p>Welcome to the 6th post on cobalt.rs!</p>
11+
<!-- more -->
12+
13+
<p>Something below the separator.</p>

‎tests/fixtures/excerpts_CRLF/posts/post-7.liquid

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
layout: default.liquid
2+
3+
title: Explicit excerpts work even in liquid
4+
published_date: 2016-01-14 21:06:30 -0500
5+
excerpt: <strong>explicit</strong> excerpt
6+
excerpt_separator: <!-- more -->
7+
---
8+
9+
<h1>Custom excerpt separator</h1>
10+
11+
<p>Welcome to the 7th post on cobalt.rs!</p>
12+
<!-- more -->
13+
14+
<p>Something below the separator.</p>

‎tests/fixtures/excerpts_CRLF/posts/post-8.md

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
layout: default.liquid
2+
3+
title: Reference-style links above first block and separated by blank line resolve
4+
published_date: 2017-02-11 04:45:01 -0000
5+
---
6+
7+
[00]: /0
8+
[01]: /1
9+
[02]: /2
10+
[03]: /3
11+
12+
References-style links are placed above the first paragraph, and are separated
13+
from the first paragraph by a single blank line:
14+
[00][] [01][] [02][] [03][]

‎tests/fixtures/excerpts_CRLF/posts/post-9.md

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
layout: default.liquid
2+
3+
title: Reference-style links below first block and separated by blank line resolve
4+
published_date: 2017-02-11 04:48:11 -0000
5+
---
6+
7+
References are placed below the first paragraph, and are separated from the
8+
first paragraph by a single blank line:
9+
[10][] [11][] [12][] [13][]
10+
11+
[10]: /0
12+
[11]: /1
13+
[12]: /2
14+
[13]: /3

‎tests/mod.rs

+5
Original file line numberDiff line numberDiff line change
@@ -238,6 +238,11 @@ pub fn excerpts() {
238238
run_test("excerpts").unwrap();
239239
}
240240

241+
#[test]
242+
pub fn excerpts_crlf() {
243+
run_test("excerpts_CRLF").unwrap();
244+
}
245+
241246
#[test]
242247
pub fn posts_in_subfolder() {
243248
run_test("posts_in_subfolder").unwrap();

0 commit comments

Comments
 (0)