Restructuring to Eliminate "ChapterMain()" about essentialcsharp HOT 6 CLOSED

Everything should be completed as you requested. I ran the tests and get the same results pre vs post restructuring, so I'm hoping I put it back together correctly. I'm going to sleep on it before submitting the pull request, but its current state is on my fork https://github.com/nlundby/EssentialCSharp/tree/v7.0

from essentialcsharp.

Wow Neal... that was fast. Thanks!

Can you please identify what tests are failing for you. On Windows, all the tests are passing for me. Thanks!

from essentialcsharp.

Pull request submitted. These two tests were failing for me on both Mac and Windows:

Listing15.14.Tests.cs(31,57): error CS1501: No overload for method 'ExpectLike' takes 3 arguments [./EssentialCSharp/src/Chapter15.Tests/Chapter15.Tests.csproj]
Listing14.12.ProjectionWithSystem.Linq.Enumerable.Select.Tests.cs(40,36): error CS1501: No overload for method 'IsLike' takes 2 arguments [./src/Chapter14.Tests/Chapter14.Tests.csproj]

A fix was submitted in my pull request.

from essentialcsharp.

Is it possible that you haven't updated your sub modules so you have and old version of test too,s by chance? (Or, alternatively, I didn't update the sub module pointer to point to the most recent TestTools perhaps?)

from essentialcsharp.

Bingo! All is well. Yes, I believe the submodule pointer does not point to the most recent TestTools. I added some steps to the Readme to ensure the latest are downloaded. Unfortunately this did not resolve the mac(and now verified linux) test fail on listing08.16.

My 15.14 and 14.12 changes have been retracted in the pull request.

from essentialcsharp.

From the comments and looking at the current state of the code it sounds like a PR was made against this and these changes are live.

from essentialcsharp.