[Bug]: Unable to ignore csharpier formatting within case statement group

[ngbrown]

Description

When formatting my code, I have some verbatim strings where the first line is being indented in a way I don't want and adding // csharpier-ignore or // csharpier-ignore-start doesn't skip over over that block.

I can't use raw string literals because this file is in project still targeting .NET 4.7.2 and raw string literals are not available in C# 7.3. The project is also still opened in Visual Studio 2019, which only supports up to C# 9.0, not C# 11, where string literals were introduced.

Steps to Reproduce

Have a file with the following content:

namespace format_test;

public class Class1
{
    public void IHaveAConfiguration(string value)
    {
        switch (value)
        {
            case "aoeuao":
                // csharpier-ignore-start
                var table = TableFromString(
@"			| Field         | Value |
			| ValueA        | 1     |
			| ValueB        | AC08  |"
                );
                // csharpier-ignore-end
                break;
        }

        // ... do things with table
    }

    private static string TableFromString(string tableString)
    {
        // ... implementation to convert string to table
        return tableString; // Placeholder return
    }
}

Run csharpier on it.

Expected Behavior

Using // csharpier-ignore should work, and so should the // csharpier-ignore-start and // csharpier-ignore-end pair.

Actual Behavior

When ran through csharpier the first line of the string becomes indented and making the table no longer line up like this:

            // csharpier-ignore-start
            var table = CommonStepHelpers.TableFromString(
                @"			| Field         | Value |
			| ValueA        | 1     |
			| ValueB        | AC08  |"
            );
            // csharpier-ignore-end

[belav]

I'll get this fixed for the next release but wanted to point out that you can use newer versions of c# with .net framework. We are on net48 at work and have been sticking to the latest c# for a while now. There were a couple things more recently that required PolySharp but I'm pretty sure raw strings just worked. I'm assuming you can do the same with net472.

[ngbrown]

Thanks @belav.

Raw string literals were introduced in C# 11. I didn't mention this, but because of the WinForms editor being only 64-bit in VS 2022 and greater, we still have some projects where we also try to keep them compatible with VS2019. VS2019 only supports up to C# 9.

[belav]

VS2019 only supports up to C# 9.
Ah yeah, I forgot that VS is somewhat tied to the dotnet version.

I haven't been able to reproduce the problem with verbatim strings, are you on an older version of csharpier perhaps? I've tried various versions of adding/removing indents on the verbatim string and csharpier doesn't touch any of those lines from what I've seen.

[ngbrown]

I was using version 1.2.6.

I was trying to pull out a minimum example, but the original code is actually in a switch statement. I played around with the example, and see that CSharpier works on my originally provided example, but it fails when in a case section (without a separate block) in a switch statement.

The csharpier-ignore-start/csharpier-ignore-end pair work when outside of the switch statement and block, but not when inside. I would actually rather use a single line // csharpier-ignore type comment, and that currently doesn't work inside the switch statement.

I updated the example in the issue description to what I have actually tested.

[ngbrown]

I added a failing test case, then looked into the why, and managed to also resolve the issue. See PR #1831.

[belav]

Thanks for tracking this down!