Skip to content

Support long descriptions across multiple lines - #83

Merged
BNAndras merged 1 commit into
exercism:mainfrom
BNAndras:fix-multiline-test-description
Aug 29, 2026
Merged

Support long descriptions across multiple lines#83
BNAndras merged 1 commit into
exercism:mainfrom
BNAndras:fix-multiline-test-description

Conversation

@BNAndras

Copy link
Copy Markdown
Member

Related to https://forum.exercism.org/t/online-test-runner-fails-despite-passing-all-tests/77782 and exercism/coffeescript#412 when I broke up some long test description lines for relative-distance and satellite.

@BNAndras
BNAndras requested a review from a team as a code owner August 29, 2026 04:10
@BNAndras BNAndras added x:action/fix Fix an issue x:rep/tiny Tiny amount of reputation labels Aug 29, 2026
"message": null
}
]
} No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Trailing newline?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

All the golden tests are like that because we use JSON.stringify() to serialize the results.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It does feel a bit weird now that you point it out so I can make a separate PR adding trailing newlines.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When you load the files, you can strip trailing newlines 😁

@BNAndras
BNAndras merged commit a6e4a6f into exercism:main Aug 29, 2026
3 checks passed
@BNAndras
BNAndras deleted the fix-multiline-test-description branch August 29, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:action/fix Fix an issue x:rep/tiny Tiny amount of reputation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants