fix: handle multiline bodies properly

Author: marionebl

source/rules/footer-leading-blank.js

@@ -6,11 +6,16 @@ export default (parsed, when) => {
 
 	const negated = when === 'never';
 
-	// get complete body split into lines
-	const lines = (parsed.raw || '').split(/\r|\n/).slice(2);
+	const count = (parsed.body || '').split(/\r|\n/).length;
+
+	// get complete message split into lines
+	const lines = (parsed.raw || '')
+		.split(/\r|\n/)
+		.slice(count + 1);
+
 	const [leading] = lines;
 
-	// check if the first line of body is empty
+	// check if the first line of footer is empty
 	const succeeds = leading === '';
 
 	return [

source/rules/footer-leading-blank.test.js

@@ -110,19 +110,19 @@ test('with blank line before footer should succeed for "always"', t => {
 	t.is(actual, expected);
 });
 
-test.failing('with blank line before footer and multiline body should succeed for empty keyword', t => {
+test('with blank line before footer and multiline body should succeed for empty keyword', t => {
 	const [actual] = footerLeadingBlank(parsed.withMulitLine);
 	const expected = true;
 	t.is(actual, expected);
 });
 
-test.failing('with blank line before footer and multiline body should fail for "never"', t => {
+test('with blank line before footer and multiline body should fail for "never"', t => {
 	const [actual] = footerLeadingBlank(parsed.withMulitLine, 'never');
 	const expected = false;
 	t.is(actual, expected);
 });
 
-test.failing('with blank line before footer and multiline body should succeed for "always"', t => {
+test('with blank line before footer and multiline body should succeed for "always"', t => {
 	const [actual] = footerLeadingBlank(parsed.withMulitLine, 'always');
 	const expected = true;
 	t.is(actual, expected);