-
Notifications
You must be signed in to change notification settings - Fork 11
Remove unused functions and increase test coverage #31
Conversation
test/datatable.jl
Outdated
|
||
@testset "misc" begin | ||
dt = DataTable(A = 'A':'C') | ||
@test sprint(dump, dt) == "DataTables.DataTable 3 observations of 1 variables\n A: Array{Char}((3,))\n 1: Char A\n 2: Char B\n 3: Char C\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use triple quotes for this kind of multiline string:
"""DataTables.DataTable 3 observations of 1 variables
A: Array{Char}((3,))
1: Char A
2: Char B
3: Char C
"""
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The leading newline is chomped out I believe, so I think you can add a line break between """
and D
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it took me some time to get it right, but this one should work. OTOH leading spaces are removed when they align with the quotes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, TIL.
I've redone the commits here to be 4 logical units. |
Thanks! |
No description provided.