MacOs

[simontom]

Guys, shall this work on MacOs as well?

System.Exception : Could not find a diff program to use
at Assent.Reporters.DiffReporter.Report(String receivedFile, String approvedFile)
at Test.Infrastructure.Approvals.AssentDebugRepoter1.Report(String receivedFile, String approvedFile) in /Users/user_/GIT/das/Packages/TestInfrastructure/src/Approvals/AssentDebugRepoter.cs:line 17 at Assent.Engine1.RunReporter(IConfiguration1 configuration, T recieved, String receivedFileName, String approvedFileName) at Assent.Engine1.Execute(IConfiguration`1 configuration, TestMetadata metadata, T recieved)
at Assent.Extensions.Assent(Object testFixture, String recieved, Configuration configuration, String testName, String filePath)
at Test.Infrastructure.Approvals.AssentExtensions.AssentVerify(Object testFixture, String recieved, Configuration configuration, String testName, String filePath) in /Users/user_/GIT/das/Packages/TestInfrastructure/src/Approvals/AssentExtensions.cs:line 15
at Prodaas.Test.CommonTests.ProdaasRequestsQueriesEventsCompatibility.ListOfQueries() in /Users/user_/GIT/das/Test/CommonTests/RequestsQueriesEventsCompatibility.cs:line 37
at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

[droyad]

Yes it does. What Diff program and version do you use?

Out of the box ones are here.

Otherwise there are various ways to choose a Diff program

[simontom]

Interesting, can't make it working. I use Rider, have KDiff3 as well

Test:

Request dump is the current state (string type)

Helper assent method:

List of reporters:

DebugReporter:

[droyad]

Ah, looking at https://github.com/droyad/Assent/blob/master/src/Assent/Reporters/DiffPrograms/KDiff3DiffProgram.cs only the Windows paths are there.

See https://github.com/droyad/Assent/blob/master/src/Assent/Reporters/DiffPrograms/BeyondCompareDiffProgram.cs on how Windows vs Linux config works

The Environment Variable or direct configuration approach (write your own KDiff3Provider) should work.

[simontom]

Aaaah, OKay, that`s the reason then. I should set up some Environment Variable to make e.g. RiderDiff working, right?

[droyad]

Yes, or create it and register it in code:

var programs = new[] {
                    new AnotherDiffProgram()
                }
                .Concat(DiffReporter.DefaultDiffPrograms)
                .ToArray();
var reporter = new DiffReporter(programs);
configuration.UsingReporter(reporter);

Happy to add it to the default implementations once you have it working

[simontom]

That's it:

private class MacRiderDiffProgram : IDiffProgram
{
    public bool Launch(string receivedFile, string approvedFile)
    {
        if (!IsRiderAvailable())
        {
            return false;
        }

        Process.Start(new ProcessStartInfo("rider", CreateProcessStartArgs(receivedFile, approvedFile)))?.WaitForExit();
        return true;
    }

    private static string CreateProcessStartArgs(string receivedFile, string approvedFile)
    {
        return $"\"diff\" \"--wait\" \"{receivedFile}\" \"{approvedFile}\"";
    }

    private static bool IsRiderAvailable()
    {
        try
        {
            using var process = Process.Start(new ProcessStartInfo("which", "rider"));
            process?.WaitForExit();
            return process?.ExitCode == 0;
        }
        catch
        {
            return false;
        }
    }
}