ToTestWellFormattedTests.cs

using System;
using System.Collections.Generic;
using ObjectToTest.UnitTests.Data;
using ObjectToTest.UnitTests.Models;
using Xunit;
using Xunit.Abstractions;

namespace ObjectToTest.UnitTests
{
    /*
     * @todo #5 60m/DEV Implement proper formatting. Base puzzle with all rules for references.
     * The rules are following:
     * 0. API should be compatible with old one.
     * 1. if there is new operator in arguments, all arguments should be properly formatted.
     * 2. spaces between arguments are required: (0, 0, 1) instead of (0,0,1).
     * 3. Lambdas are always from new string.
     * 4. Indention is 4 spaces.
     * 5. Inner properties should be from separate lines.
     * 6. Lines bigger than 80 characters should be properly formatted.
     * 7. All arrays should be placed with proper formatting.
     * 8. Dictionary should be formatted with separate set of rules.
    */

    /*
     * @todo #80 60m/DEV Describe formatting rules in readme file.
     */

    /*
    * @todo #5 60m/DEV Implement rule: if there is new operator in arguments, all arguments should be properly formatted.
    */

    /*
    * @todo #5 60m/DEV Implement rule: spaces between arguments are required: (0, 0, 1) instead of (0,0,1).
    */

    /*
    * @todo #5 60m/DEV Implement rule: Lambdas are always from new string.
    */

    /*
    * @todo #5 60m/DEV Implement rule: Indention is 4 spaces.
    */

    /*
    * @todo #5 60m/DEV Implement rule: Inner properties should be from separate lines.
    */

    /*
    * @todo #5 60m/DEV Implement rule: Lines bigger than 80 characters should be properly formatted.
    */

    /*
    * @todo #5 60m/DEV Implement rule: All arrays should be placed with proper formatting.
    */

    /*
    * @todo #5 60m/DEV Implement rule: Dictionary should be formatted with separate set of rules.
    */

    public class ToTestWellFormattedTests
    {
        private readonly ITestOutputHelper _output;

        public ToTestWellFormattedTests(ITestOutputHelper output)
        {
            _output = output;
        }
        
#pragma warning disable CS8604
        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ThrowArgumentNullException_ShouldBehaveAsNotFormatted()
        {
            object? obj = null;
            Assert.Throws<ArgumentNullException>(() => obj.ToTestWellFormatted());
        }
#pragma warning restore CS8604
        
        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void InnerPropertyShouldBeFromSeparateLine_StringPropertyInitializer()
        {
            Assert.Equal(
                "new WithOnePublicProperty()" +
                "{" +
                "    PropertyName = \"Test\"" +
                "}",
                new WithOnePublicProperty
                {
                    PropertyName = "Test"
                }.ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void InnerPropertyShouldBeFromSeparateLine_IntPropertyInitializer()
        {
            Assert.Equal(
                "new WithOnePublicIntProperty()" +
                "{" +
                "    PropertyName = 42" +
                "}",
                new WithOnePublicIntProperty
                {
                    PropertyName = 42
                }.ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void InnerPropertyShouldBeFromSeparateLine_MultipleValueTypesPropertyInitializer()
        {
            Assert.Equal(
                "new WithTwoProperties()" +
                "{" +
                "    IntProperty = 42," +
                "    StringProperty = \"Test\"" +
                "}",
                new WithTwoProperties
                {
                    IntProperty = 42,
                    StringProperty = "Test"
                }.ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ShortArgumentsWithoutNewPlacedInline_CtorWithIntArgumentForProperty()
        {
            Assert.Equal(
                "new WithOneParameterContructorAndPublicReadProperty(42)",
                new WithOneParameterContructorAndPublicReadProperty(42)
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ShortArgumentsWithoutNewPlacedInline_CtorWithIntArgumentForPrivateField()
        {
            Assert.Equal(
                "new WithOneParamAndPrivateField(42)",
                new WithOneParamAndPrivateField(42).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5, rule 2")]
        public void ShortArgumentsWithoutNewPlacedInline_CtorWithMultipleValueTypeArguments()
        {
            Assert.Equal(
                "new WithTwoParamOneFieldAndOneProperty(42, \"Test\")",
                new WithTwoParamOneFieldAndOneProperty(
                    42,
                    "Test"
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5, rule 6")]
        public void StringMoreThan80CharsShouldBeFormatted_CtorWithMultipleLongValueTypeArguments()
        {
            Assert.Equal(
                "new WithTwoParamOneFieldAndOneProperty(" +
                "    42," +
                "    \"Test Test Test Test Test Test Test Test Test Test Test Test Test Test Test Test\"" +
                ")",
                new WithTwoParamOneFieldAndOneProperty(
                    42,
                    "Test Test Test Test Test Test Test Test Test Test Test Test Test Test Test Test"
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_CtorWithReferenceTypeArgument()
        {
            Assert.Equal(
                "new WithClassParam(" +
                "    new EmptyObject()" +
                ")",
                new WithClassParam(new EmptyObject()).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_CtorWithMultipleComplexTypesArguments()
        {
            Assert.Equal(
                "new WithClassAndIntParams(" +
                "    42," +
                "    new EmptyObject()" +
                ")",
                new WithClassAndIntParams(
                    42,
                    new EmptyObject()
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5, rule 2")]
        public void ShortArgumentsShouldBeSeparatedBySpace_TimeSpanConstructor()
        {
            Assert.Equal(
                "new TimeSpan(18, 17, 34, 24, 5)",
                new TimeSpan(18, 17, 34, 24, 5)
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_CtorWithComplexDependencyArgument()
        {
            Assert.Equal(
                "new WithClassParamThatDependsOnClass(" +
                "    new WithClassParam(" +
                "        new EmptyObject()" +
                "    )" +
                ")",
                new WithClassParamThatDependsOnClass(
                    new WithClassParam(new EmptyObject())
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_CtorWithComplexDependencySeveralArguments()
        {
            Assert.Equal(
                "new WithTwoClassParamAndIntParam(" +
                "    new WithClassParam(" +
                "        new EmptyObject()" +
                "    )," +
                "    new WithClassAndIntParams(" +
                "        42," +
                "        new EmptyObject()" +
                "    )," +
                "    42" +
                ")",
                new WithTwoClassParamAndIntParam(
                    new WithClassParam(new EmptyObject()),
                    new WithClassAndIntParams(
                        42,
                        new EmptyObject()
                    ),
                    42
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void InnerPropertyShouldBeFromSeparateLine__And_ArgumentWithNewShouldBeFormatted__CtorWithComplexArgumentsAndProperties()
        {
            Assert.Equal(
                "new WithClassParamWithProp(" +
                "    new WithOnePublicProperty()" +
                "    {" +
                "        PropertyName = \"Test\"" +
                "    }," +
                "    42" +
                ")",
                new WithClassParamWithProp(
                    new WithOnePublicProperty { PropertyName = "Test" },
                    42
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void GenericArgumentsAreNotFormatted_CtorWithGenericArgument()
        {
            Assert.Equal(
                "new WithGenericArgument<IPrice>()",
                new WithGenericArgument<IPrice>().ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void GenericArgumentsAreNotFormatted_With2GenericArguments()
        {
            Assert.Equal(
                "new With2GenericArguments<IPrice,IUser>(" +
                "    new Price(" +
                "        10" +
                "    )" +
                ")",
                new With2GenericArguments<IPrice, IUser>(new Price(10))
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void GenericArgumentsAreNotFormatted_With3GenericArguments()
        {
            Assert.Equal(
                "new With3GenericArguments<int,decimal,string>()",
                new With3GenericArguments<int, decimal, string>()
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5, rule 2")]
        public void ShortArgumentsShouldBeSeparatedBySpace_And_ArgumentWithNewShouldBeFormatted_CtorWithStructArgument()
        {
            Assert.Equal(
                "new WithStructArgument(" +
                "    new Vector3(0, 0, 1)" +
                ")",
                new WithStructArgument(UnityEngine.Vector3.forward)
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ShortArgumentsWithoutNewPlacedInline_CtorWithEnumArgument()
        {
            Assert.Equal(
                "new WithEnumArgument(FlaggedEnum.Advanced)",
                new WithEnumArgument(FlaggedEnum.Advanced)
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void LambdasShouldBePlacedOnNewLine_CtorWithFuncArgument()
        {
            Assert.Equal(
                "new WithFuncArgument(" +
                "    () => 0" +
                ")",
                new WithFuncArgument(() => 0)
                    .ToTestWellFormatted()
                    .Log(_output)
            ); 
        }
        
        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void LambdasShouldBePlacedOnNewLine_CtorWithActionArgument()
        {
            Assert.Equal(
                "new WithActionArgument(" +
                "    pos => {}" +
                ")",
                new WithActionArgument((pos) => { })
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArraysShouldBeFormatted_CtorWithIEnumerableInt()
        {
            Assert.Equal(
                "new WithIEnumerableInt(" +
                "    new[] " +
                "    {" +
                "        1," +
                "        2," +
                "        4," +
                "        5" +
                "    }" +
                ")",
                new WithIEnumerableInt(new[] { 1, 2, 4, 5 })
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArraysShouldBeFormatted_CtorWithListInt()
        {
            Assert.Equal(
                "new WithListArgument(" +
                "    new List<int> " +
                "    {" +
                "        1," +
                "        2," +
                "        4," +
                "        5" +
                "    }" +
                ")",
                new WithListArgument(new List<int> { 1, 2, 4, 5 })
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_CtorWithEmptyListInt()
        {
            Assert.Equal(
                "new WithListArgument(" +
                "    new List<int>()" +
                ")",
                new WithListArgument(new List<int>())
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void DictionaryShouldBeFormatted_CtorWithDictionaryIntString()
        {
            Assert.Equal(
                "new WithDictionaryArgument(" +
                "    new Dictionary<int,string>" +
                "    {" +
                "        { 1, \"1\" }," +
                "        { 2, \"2\" }," +
                "        { 3, \"3\" }" +
                "    }" +
                ")",
                new WithDictionaryArgument(
                    new Dictionary<int,string> { { 1, "1" }, { 2, "2" }, { 3, "3" } }
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_CtorWithEmptyDictionaryIntString()
        {
            Assert.Equal(
                "new WithDictionaryArgument(" +
                "    new Dictionary<int,string>()" +
                ")",
                new WithDictionaryArgument(
                    new Dictionary<int, string>()
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_And_ShortArgumentsWithoutNewPlacedInline_CtorWithInterfaceArguments()
        {
            Assert.Equal(
                "new Foo(" +
                "    new Price(10)," +
                "    new User(\"User Name\")" +
                ")",
                new Foo(
                    new Price(10),
                    new User("User Name")
                ).ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_And_InnerPropertyShouldBeFromSeparateLine_TheSameObjectDetection()
        {
            var user = new User("user name");
            var withUser = new WithUserArgument(
                user,
                new WithUserPublicProperty
                {
                    User = user
                }
            );
            Assert.Equal(
                "var user = new User(\"user name\");" + Environment.NewLine +
                "new WithUserArgument(" +
                "    user," +
                "    new WithUserPublicProperty()" +
                "    {" +
                "        User = user" +
                "    }" +
                ")",
                withUser.ToTestWellFormatted().Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_SingletonAsSharedArgument()
        {
            Assert.Equal(
                "new SharedSingletons(" +
                "    new WithSingletonAndOtherArgument(" +
                "        SingletonClass.Instance," +
                "        new Price(10)" +
                "    )," +
                "    new WithSingletonArgument(SingletonClass.Instance)" +
                ")",
                new SharedSingletons(
                        new WithSingletonAndOtherArgument(SingletonClass.Instance, new Price(10)),
                        new WithSingletonArgument(SingletonClass.Instance)
                    ).ToTestWellFormatted()
                    .Log(_output)
            );
        }

        [Fact(Skip = "Need to be fixed in scope of puzzle #5")]
        public void ArgumentWithNewShouldBeFormatted_SingletonWithOtherArgument()
        {
            Assert.Equal(
                "new WithSingletonAndOtherArgument(" +
                "    SingletonClass.Instance," +
                "    new Price(10)" +
                ")",
                new WithSingletonAndOtherArgument(SingletonClass.Instance, new Price(10))
                    .ToTestWellFormatted()
                    .Log(_output)
            );
        }
    }
}