Why are lines wrapped where they are in infer_struct001e.fs and infer_struct002e.fs

[jsoref]

These files have not been changed since they were initially published.

fsharp/tests/fsharpqa/Source/Conformance/ObjectOrientedTypeDefinitions/TypeKindInference/infer_struct001e.fs

Lines 7 to 8 in be6352d

	//infer_interface001e.fs(16,6): error FS0191: The kind of the type specified by its attributes does not match the kind impl
	//ied by its definition.

-- fwiw, this first line appears to be the longest // line that isn't a //< line in this directory

fsharp/tests/fsharpqa/Source/Conformance/ObjectOrientedTypeDefinitions/TypeKindInference/infer_struct001e.fs

Lines 10 to 11 in be6352d

	//infer_interface001e.fs(22,6): error FS0191: The kind of the type specified by its attributes does not match the kind imp
	//lied by its definition.

-- fwiw, the first line here is precisely one character shorter than the first line of the previous block

fsharp/tests/fsharpqa/Source/Conformance/ObjectOrientedTypeDefinitions/TypeKindInference/infer_struct002e.fs

Lines 8 to 9 in be6352d

	//infer_struct002e.fs(13,6): error FS0191: The kind of the type specified by its attributes does not match the kind implie
	//d by its definition.

-- fwiw, this first line is just as long as the first line of the previous block (which makes it one character shorter than the first block)

• If the lines don't need to be wrapped at all, then I'd propose unwrapping them and adjusting their line numbers
• If the lines do need to be wrapped, but don't need to be wrapped at the precise wrap points (which seem very arbitrary), then I'd propose changing the wrap points to be at word boundaries instead of at the seemingly arbitrary mid-word points.

[vzarytovskii]

I suppose if wrapping needs to be there, it depends on terminal width set by tests. Usually, we don't touch tests under fsharpqa unless needed, we want to migrate all of them to the ComponentTests suite, which has a much better framework for assertions and baseline.

[jsoref]

I don't see how the terminal width can change between line 7 and line 10...

That said, if your general policy is to not touch these files, I can ignore them. These things just look kinda odd, and fixing them to be slightly less odd wouldn't be terribly hard (it isn't hard to avoid messing up the line count -- just move letters so that the wrap is before/after a word...)

[vzarytovskii]

Wait, the lines you're referring to are likely not used in framework, only ones included in <Expects> tag.

[jsoref]

So does that mean you'd be amenable to reflowing them?

[vzarytovskii]

Sure, or delete them.

[jsoref]

opened #16188