5 60m 0 DEV 60m-ac1f5531 156-166 Proper declaration format is expected. It would be nice to start each constructor argument on its own line with proper intend new WithTwoClassParamAndIntParam( new WithClassParam( new EmptyObject() ), new WithClassAndIntParams( 42, new EmptyObject() ), 42 ) tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 71 5 0 DEV 5-cd4cb888 11-20 60m/DEV Implement proper formatting. Base puzzle with all rules for references. The rules are following: 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. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 72 5 0 DEV 5-aba79164 24-24 60m/DEV Implement rule: if there is new operator in arguments, all arguments should be properly formatted. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 73 5 0 DEV 5-fee780ba 28-28 60m/DEV Implement rule: spaces between arguments are required: (0, 0, 1) instead of (0,0,1). tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 74 5 0 DEV 5-d0ce5b4b 32-32 60m/DEV Implement rule: Lambdas are always from new string. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 75 5 0 DEV 5-1eddb7ad 36-36 60m/DEV Implement rule: Indention is 4 spaces. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 76 5 0 DEV 5-de36cb8b 40-40 60m/DEV Implement rule: Inner properties should be from separate lines. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 150 76 0 DEV 76-b1461939 31-31 60m/DEV Add recursive parsing for assignments. tests/ObjectToTest.UnitTests/WellFormattedCodeTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 151 76 0 DEV 76-ab64b10e 55-55 60m/DEV Fix this. tests/ObjectToTest.UnitTests/InstantiationStatementTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 152 76 0 DEV 76-962767e2 12-22 60m/DEV Add rules into documentation. The rules are following: 0. API should be compatible with old one. 1. if there is new operator in arguments, all arguments should be from new lines. 2. spaces between arguments are required if arguments list are in one line: (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. Each argument on separate line. 8. Dictionary should be formatted with separate set of rules. Each pair on separate line. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 153 76 0 DEV 76-41d35ef3 26-26 60m/DEV Implement rule: if there is an inline instantiation (new operator) in arguments, all arguments should be properly formatted. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 154 76 0 DEV 76-d0ce5b4b 30-30 60m/DEV Implement rule: Lambdas are always from new string. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 155 76 0 DEV 76-de36cb8b 34-34 60m/DEV Implement rule: Inner properties should be from separate lines. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 156 76 0 DEV 76-8f445bc5 38-38 60m/DEV Implement rule: All arrays should be placed with proper formatting. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 168 156 0 DEV 156-9de53b53 44-44 60m/DEV InstantiationStatement is no capable to recognize anonymous array tests/ObjectToTest.UnitTests/InstantiationStatementTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 157 76 0 DEV 76-3fa6ea22 42-42 60m/DEV Implement rule: Dictionary should be formatted with separate set of rules. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 158 76 0 DEV 76-8f4cc3a8 10-10 60m/DEV Add assignment with type definition. src/CodeFormatting/Syntax/Statements/Assignment/PossibleAssignments.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 159 76 0 DEV 76-7d061f5d 14-14 60m/DEV Add assignment to variable. src/CodeFormatting/Syntax/Statements/Assignment/PossibleAssignments.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 77 5 0 DEV 5-47d4c793 44-44 60m/DEV Implement rule: Lines bigger than 80 characters should be properly formatted. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 78 5 0 DEV 5-8f445bc5 48-48 60m/DEV Implement rule: All arrays should be placed with proper formatting. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 79 5 0 DEV 5-3fa6ea22 52-52 60m/DEV Implement rule: Dictionary should be formatted with separate set of rules. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 80 5 0 DEV 5-38b4cde6 56-56 60m/DEV In ToTestWellFormattedTests: tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 84 80 0 DEV 80-2d20ab77 25-25 60m/DEV Describe formatting rules in readme file. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs @Stanislau Stanislau_Kanaplianik@epam.com 85 80 0 DEV 80-788167ac 20-22 60m/DEV Define approach for formatting. Possible option: IDecompiledCodeAsString|string object.ToString(IFormat), where IFormat can be abstract interface with implementation for each formatting rule. src/Extensions/ObjectExtensions.cs Stanislau Stanislau_Kanaplianik@epam.com 91 85 0 DEV 85-ab01c3f2 4-6 60m/DEV Decompose code formatting to create syntax tree from code. Better to parse code and format it properly. It isolates existing functionality. Probably need to invest some time to find existing ability to create syntax tree from code. src/CodeFormatting/WellFormattedCode.cs Stanislau Stanislau_Kanaplianik@epam.com 96 91 0 DEV 91-86347da6 9-10 60m/DEV Define necessary properties Most likely: new [TypeDefinition]([Arguments]) {[InlinePropertiesAssignment]} src/CodeFormatting/Syntax/IInstantiationStatement.cs Stanislau Stanislau_Kanaplianik@epam.com 97 91 0 DEV 91-757bcd9b 9-10 60m/DEV Create an implementation for this. It should be responsible for splitting code to statements and providing statements of the right type. src/CodeFormatting/Syntax/ISyntaxTree.cs Stanislau Stanislau_Kanaplianik@epam.com 100 97 0 DEV 97-c8441ae2 10-10 60m/DEV Implement different statements and register them here. src/CodeFormatting/Syntax/Implementation/PossibleCodeStatements.cs Stanislau Stanislau_Kanaplianik@epam.com 103 100 0 DEV 100-1403ee85 11-11 60m/DEV Implement instantiation statement parsing. src/CodeFormatting/Syntax/Implementation/InstantiationStatement.cs Stanislau Stanislau_Kanaplianik@epam.com 105 103 0 DEV 103-1449c44a 10-10 60m/DEV Parse arguments. src/CodeFormatting/Syntax/Implementation/Arguments.cs Stanislau Stanislau_Kanaplianik@epam.com 106 103 0 DEV 103-6f662a60 10-10 60m/DEV Parse properties. src/CodeFormatting/Syntax/Implementation/PropertyAssignments.cs Stanislau Stanislau_Kanaplianik@epam.com 110 106 0 DEV 106-e47b7527 9-9 60m/DEV Add lambdas support. src/CodeFormatting/Syntax/Implementation/PossibleArguments.cs Stanislau Stanislau_Kanaplianik@epam.com 111 106 0 DEV 106-ab2b5e46 10-10 60m/DEV Adjust project structure. src/CodeFormatting/Syntax/SyntaxTree.cs Stanislau Stanislau_Kanaplianik@epam.com 112 106 0 DEV 106-7b8c4b95 24-24 60m/DEV Change split implementation to consider closures, lambdas and inline initializations. src/CodeFormatting/Syntax/SyntaxTree.cs Stanislau Stanislau_Kanaplianik@epam.com 113 106 0 DEV 106-5af0cf4e 29-29 60m/DEV Refactoring for all logic to be moved from constructor into lazy initializations or lazy executables. src/CodeFormatting/Syntax/SyntaxTree.cs Stanislau Stanislau_Kanaplianik@epam.com 107 103 0 DEV 103-aca9e7c5 11-11 60m/DEV Think about array assignment. src/CodeFormatting/Syntax/Implementation/InstantiationStatement.cs Stanislau Stanislau_Kanaplianik@epam.com 98 91 0 DEV 91-b3afd652 4-4 60m/DEV Create hierarchy for code formatting based on ISyntaxTree. src/CodeFormatting/WellFormattedCode.cs Stanislau Stanislau_Kanaplianik@epam.com 124 98 0 DEV 98-39e62256 212-212 60m/DEV This should work but is is not. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Stanislau Stanislau_Kanaplianik@epam.com 125 98 0 DEV 98-65555348 23-23 60m/DEV Rework infrastructure for code formatting. src/CodeFormatting/WellFormattedCode.cs Stanislau Stanislau_Kanaplianik@epam.com 137 125 0 DEV 125-962767e2 13-23 60m/DEV Add rules into documentation. The rules are following: 0. API should be compatible with old one. 1. if there is new operator in arguments, all arguments should be from new lines. 2. spaces between arguments are required if arguments list are in one line: (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. Each argument on separate line. 8. Dictionary should be formatted with separate set of rules. Each pair on separate line. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Stanislau Stanislau_Kanaplianik@epam.com 138 125 0 DEV 125-b1d7e48e 11-11 60m/DEV Use Insert 0 instead of add. src/CodeFormatting/Formatting/Format.cs Stanislau Stanislau_Kanaplianik@epam.com 139 125 0 DEV 125-614e355d 17-17 60m/DEV Rename Override to just ForArrayOf. src/CodeFormatting/Formatting/Format.cs Stanislau Stanislau_Kanaplianik@epam.com 140 125 0 DEV 125-88fc4eb7 12-12 60m/DEV Implement invocation statement. src/CodeFormatting/Syntax/Statements/PossibleCodeStatements.cs Stanislau Stanislau_Kanaplianik@epam.com 141 125 0 DEV 125-cfa6c346 16-16 60m/DEV Implement assignment statement. src/CodeFormatting/Syntax/Statements/PossibleCodeStatements.cs Stanislau Stanislau_Kanaplianik@epam.com 142 125 0 DEV 125-d8a4f619 9-9 60m/LEAD IInstantiationStatement is ICodeStatement and IArgument at the same time. src/CodeFormatting/Syntax/Contracts/IInstantiationStatement.cs Stanislau Stanislau_Kanaplianik@epam.com 160 98 0 DEV 98-016d53c1 199-199 60m/DEV This should work but it is not. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 92 85 0 DEV 85-c9c8d051 21-22 60m/ARCH: Can we extend ObjectAsConstructor to be able to check if object created with issues? It is needed to define whether we need to format the code at all. src/Extensions/ObjectExtensions.cs Stanislau Stanislau_Kanaplianik@epam.com 81 5 0 DEV 5-a4cd6989 62-62 60m/DEV Implement a test to check rule: Lines bigger than 80 characters should be properly formatted.. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 82 5 60 DEV 5-f769c879 489-489 Adjust this test with formatting rules. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 86 5 0 DEV 5-9186b939 11-21 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. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs @Stanislau Stanislau_Kanaplianik@epam.com 143 5 0 DEV 5-41d35ef3 27-27 60m/DEV Implement rule: if there is an inline instantiation (new operator) in arguments, all arguments should be properly formatted. tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs Stanislau Stanislau_Kanaplianik@epam.com 6 60m 0 DEV 60m-213b64df 197-197 Make WithGenericArgument test to be green. Generic type constructors are not supported at the moment. Need to add the support. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 7 60m 0 DEV 60m-1ef53fb8 209-209 Make WithIEnumerableInt test to be green. Collection argument type constructors are not supported at the moment. Need to add the support. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 8 60m 0 DEV 60m-ac3331af 230-230 Make IncorrectArgumentsClass test to be green. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 11 60m 0 DEV 60m-90fcc3c8 256-258 Make CircularReferenceDetection test to be green. Now the circular references between the objects are not detected. It would be nice to fix the issue tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 37 11 60 DEV 11-7e3ee3ee 306-308 Make ComplexCircularReferenceDetection test to be green. Now the circular references between the objects are not detected. It would be nice to fix the issue tests/ObjectToTest.UnitTests/ToTestTests.cs @MaxShustov max.shustov@hotmail.com 38 11 60 DEV 11-e081f816 86-89 Make CircularReferenceInitProperties test to be green. Now the circular references not initialized properly and does not detect that shared object has circular reference in its fields or properties. It would nice to fix the test. tests/ObjectToTest.UnitTests/ObjectSharedArgumentsTests.cs Maksym Shustov max.shustov@hotmail.com 46 38 60 DEV 38-1a440fb4 13-14 Make HasCircularReference test to be green. HasCircularReference always returns false but the method should be able tests/ObjectToTest.UnitTests/ObjectExtensionsTests.cs Maksym Shustov max.shustov@hotmail.com 47 38 60 DEV 38-547e639d 34-36 Make HasComplexCircularReference test to be green. HasCircularReference method should be able to detect the situation when object has a circular reference but not direct. Instead it uses the object that uses the other one which uses the original in its field or property. tests/ObjectToTest.UnitTests/ObjectExtensionsTests.cs Maksym Shustov max.shustov@hotmail.com 12 60m 0 DEV 60m-ac170357 274-276 Make TheSameObjectDetection test to be green. Now the same object detection is not implemented. It would be nice to fix the issue. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 18 12 60 DEV 12-d42c7aa9 13-18 Improve the test: Ideally the test should compare the result that returned by Argument method with some manually created argument that wraps the original object. Assert.Equal( new Argument(user), new ObjectSharedArguments(withUser).Argument(user) ); tests/ObjectToTest.UnitTests/ObjectSharedArgumentsTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 19 12 60 DEV 12-615eb024 22-34 The method should try to find the same argument inside the target object and check if its the shared one. It means the argument is used more than 1 time inside the target object fileds or properties: The example below shows sych case when user object is the shared one. var user = new User("user name"); var withUser = new WithUserArgument( user, new WithUserPublicProperty { User = user } ); The Test: ObjectSharedArgumentsTests.SharedArgument should be green src/ObjectSharedArguments.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 21 12 60 DEV 12-77aedd72 37-37 ObjectSharedArguments should be able to support nulls for shared objects. tests/ObjectToTest.UnitTests/ObjectSharedArgumentsTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 22 12 60 DEV 12-2adcea6a 70-84 ObjectSharedArguments class should return as a string the piepce of code that initializes the shared object arguments. For example: var user = new User("user name"); var withUser = new WithUserArgument( user, new WithUserPublicProperty { User = user } ); ToString method should return: var user = new User("user name"); If object has no shared objects in its internal state the method should return empty string. tests/ObjectToTest.UnitTests/ObjectSharedArgumentsTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 23 12 60 DEV 12-bf97e2bc 29-30 Make a code refactoring for this property. The property is too big and it looks like Loops for Fields and Properties are pretty the same. src/ObjectSharedArguments.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 25 12 60 DEV 12-25ff0d1d 54-55 Make a code refactoring for this method. The method is too big and complex. It looks like the Loops for Fields and Properties are pretty the same. src/ObjectSharedArguments.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 15 30m 0 DEV 30m-3d61cad0 242-242 Make CtorWithDictionaryIntString test to be green. Dictionary argument type constructors are not supported at the moment. Need to add the support. tests/ObjectToTest.UnitTests/ToTestTests.cs @MaxShustov max.shustov@hotmail.com 20 60m 0 ARCH 60m-92217446 10-11 Need to create Interface/Class diagram that describes the solution on high level Ideally this diagram should be put into Docs folder of the repository and referenced in Readme file. src/ObjectAsConstructor.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 63 20 0 DEV 20-65f0b286 15-15 60m/ARCH Need to complete the diagram ObjectToTest/docs/ObjectToTest.drawio src/ObjectAsConstructor.cs @MaxShustov max.shustov@hotmail.com 29 60m 0 DEV 60m-de6b1700 309-310 Make SingletonAsArgument test to be green. Now the singletons are not detected. It would be nice to fix the issue tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 35 60m 0 LEAD 60m-c0ddede6 340-341 Warning comment when object has an internal state that can not be initialized through the constructor. It means that the state of an object has been changed after its been created and it's happened through the method or event tests/ObjectToTest.UnitTests/ToTestTests.cs @MaxShustov max.shustov@hotmail.com 43 60m 0 DEV 60m-833141ed 245-246 Make CtorWithFuncArgument test to be green. Now constructor with Func arguments are not supported. Need to be fixed tests/ObjectToTest.UnitTests/ToTestTests.cs @MaxShustov max.shustov@hotmail.com 52 43 60 DEV 43-e812cfe8 21-22 Make FuncCtor test to be green. Now DelegateConstructor does not support delegate body generation. tests/ObjectToTest.UnitTests/DelegateConstructorTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 53 43 60 DEV 43-7268d76e 37-38 Make FuncCtorWith1Param test to be green. Now DelegateConstructor does not support delegate body generation. tests/ObjectToTest.UnitTests/DelegateConstructorTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 54 43 60 DEV 43-1f7b68b6 86-87 Make ActionCtorWith2ParamsAndBody test to be green. Now DelegateConstructor does not support delegate body generation. tests/ObjectToTest.UnitTests/DelegateConstructorTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 44 60m 0 DEV 60m-1c83a860 260-261 Make WithActionArgument test to be green. Now constructor with Action arguments are not supported. Need to be fixed tests/ObjectToTest.UnitTests/ToTestTests.cs @MaxShustov max.shustov@hotmail.com 57 60m 0 DEV 60m-52d1da88 261-262 Make OtherObjectMethodReferenceAsArgument test to be green. Now DelegateConstructor does not support object method as reference. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 64 60m 0 ARCH 60m-ed924176 17-17 Need to implement object dependencies tracing logic. src/Core/ObjectDependenciesTrace.cs @MaxShustov max.shustov@hotmail.com 89 64 0 DEV 64-b89c1043 15-17 60m/DEV InvalidConstructor returns Arguments as empty list The class should be able to calculate the target object constructor and return the list of its arguments tests/ObjectToTest.UnitTests/InvalidConstructorTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 90 64 0 DEV 64-52c16d09 34-35 60m/DEV ObjectDependenciesTrace does not support recursive analyzes for invalid constructor arguments. The test should be fixed and recursive analyzes implemented. tests/ObjectToTest.UnitTests/ObjectDependenciesTraceTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 116 60m 0 DEV 60m-13211a65 283-285 Make SharedOtherObjectMethodsReferenceAsArgument test to be green. Now DelegateConstructor does not support shared objects. DelegateConstructor should be able to generate code with shared arguments in such cases. tests/ObjectToTest.UnitTests/ToTestTests.cs @MaxShustov max.shustov@hotmail.com 122 116 0 DEV 116-50c12a8f 91-91 60m/DEV ObjectSharedArguments can not initialize the target of function delegate. tests/ObjectToTest.UnitTests/ObjectSharedArgumentsTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 162 122 0 DEV 122-1de999dd 6-11 60m/DEV SharedObjects summary is a little bit difficult for quick understanding. Need to rephrase it The entity finds the objects that are part of current object internal state but also used in more than one object entity. https://github.com/DenisZhukovski/ObjectToTest/wiki/Shared-Objects-Detection </summary> src/Core/SharedObjects.cs Eugene Puntus kapral.20.it@gmail.com 119 60m 0 LEAD 60m-29085618 454-455 Warning comment when object has an internal state that can not be initialized through the constructor. It means that the state of an object has been changed after its been created and it's happened through the method or event. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 120 60m 0 ARCH 60m-a60ec9c0 12-17 ObjectAsConstructor class has to be structurally fixed to support logging as decorator. Now when the class does not work as expected its not an easy one to understand how the logic works. Ideally we should be able to wrap the original object with Logged one to be able to see the whole execution process. It seems it was a bad idea to use extension method for some crucial pieces of code. src/ObjectAsConstructor.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 127 60m 0 DEV 60m-c03a3736 426-427 When constructor argument is shared object the name and return type is not correct because of the custom object class name which does not fir the argument name and type tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 130 60m 0 DEV 60m-99c1dd43 449-449 When shared object is used several times its children also considered as shared objects tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 134 60m 0 DEV 60m-ef372b2f 284-286 Make SharedArgumentWhenSeveralTimesAsCtorArgument test to be green. Now when object is used several times as the same ctor argument it's not detected as shared object. The issue need ot be fixed. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 146 134 0 DEV 134-2e67643f 99-100 60m/DEV Make SeveralTimesCtorArgumentAsShared test to be green. The problem is that SharedObjects is not implemented properly. tests/ObjectToTest.UnitTests/SharedObjectsTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 166 60m 0 ARCH 60m-6081b6c6 61-69 SyntaxTree ToString should show the syntax hierarchy. Something similar to the example 'new TimeSpan(18, 17, 34, 24, 5)': - InstantiationStatement: TimeSpan - Arguments: 5 - Argument: Int32(18) - Argument: Int32(17) - Argument: Int32(34) - Argument: Int32(24) - Argument: Int32(5) tests/ObjectToTest.UnitTests/SyntaxTreeTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 185 180 0 DEV 180-470c4360 124-124 60m/DEV SharedObjects should be empty for HttpClient. The test should be green. tests/ObjectToTest.UnitTests/SharedObjectsTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 186 180 0 DEV 180-a57c516e 517-517 60m/DEV HttpClient should be recreated properly, The test should be green. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 187 180 0 DEV 180-828a6e9f 12-12 60m/DEV ObjectsUsageGraph should be empty for HttpClient. The test should be green. tests/ObjectToTest.UnitTests/ObjectsUsageGraphTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 188 180 0 DEV 180-64fe7295 202-202 60m/DEV ObjectSharedArguments should be empty for HttpClient. The test should be green. tests/ObjectToTest.UnitTests/ObjectSharedArgumentsTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 189 180 0 DEV 180-b1140a42 55-55 60m/DEV SharedCircularProperties should be empty for HttpClient. The test should be green. tests/ObjectToTest.UnitTests/SharedCircularPropertiesTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 60m 0 DEV 60m-9aa5e31b 556-556 Object With NotImplemented properties should be able to instantiate. tests/ObjectToTest.UnitTests/ToTestTests.cs @DenisZhukovski 46710314+DenisZhukovski@users.noreply.github.com 195 60m 0 DEV 60m-0d2c1876 568-568 Object With Type properties should be able to instantiate. tests/ObjectToTest.UnitTests/ToTestTests.cs Denis Zhukovski 46710314+DenisZhukovski@users.noreply.github.com 196