Merge pull request #619 from no-chris/no-chris/issue588

Do not repeat chords in copied sections if `autoRepeatChords` is false

packages/chord-mark/src/renderer/components/renderSong.js

@@ -206,7 +206,9 @@ export default function renderSong(
 
 	function shouldRepeatLines(line) {
 		const shouldSkipAutoRepeatChordLine =
-			line.isFromAutoRepeatChords && !autoRepeatChords;
+			!autoRepeatChords &&
+			(line.isFromAutoRepeatChords ||
+				(line.type === lineTypes.CHORD && line.isFromSectionCopy));
 
 		const shouldSkipSectionMultiplyLine =
 			line.isFromSectionMultiply && !expandSectionMultiply;

packages/chord-mark/tests/unit/renderer/components/renderSong.spec.js

@@ -132,6 +132,48 @@ verseLine`;
 		expect(toText(rendered)).toBe(expected);
 	});
 
+	test('should not repeat chords when autoRepeatChords === false', () => {
+		const input = `#v
+A B
+verseLine
+#v
+#v`;
+		const expected = `Verse 1
+|A     |B     |
+verseLine
+Verse 2
+verseLine
+Verse 3
+verseLine`;
+		const rendered = renderSongText(input, {
+			expandSectionCopy: true,
+			autoRepeatChords: false,
+		});
+		expect(toText(rendered)).toBe(expected);
+	});
+
+	test('should not repeat chords when autoRepeatChords === false, even if the section contains only chords', () => {
+		const input = `#v
+A B
+C D
+
+#v
+
+#v`;
+		const expected = `Verse 1
+|A     |B     |
+|C     |D     |
+
+Verse 2
+
+Verse 3`;
+		const rendered = renderSongText(input, {
+			expandSectionCopy: true,
+			autoRepeatChords: false,
+		});
+		expect(toText(rendered)).toBe(expected);
+	});
+
 	test('should only repeat section label when expandSectionCopy === false', () => {
 		const input = `#v
 A B