Skip to content

Add tests Chapter6 - v8 #87

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Aug 28, 2019
Merged

Add tests Chapter6 - v8 #87

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
3d06659
Remove space from Listing06.11. Add tests 6,8,11
COsborn2 Jul 25, 2019
b301ec8
Remove space from GetName Listing14. Fix Listing20 property, visibility.
COsborn2 Jul 25, 2019
b5ed427
Fix property for Listing22
COsborn2 Jul 25, 2019
7f8cf0d
Add test 22
COsborn2 Jul 25, 2019
fe4768d
Make Listing31 public. Add tests 26,27,29,31
COsborn2 Jul 25, 2019
ee5adbe
Change visibility of Listings12,13,31. Add test to Listing 13
COsborn2 Aug 9, 2019
c10b619
Change visibility of classes. Add Tests 16,32,33,37
COsborn2 Aug 9, 2019
5abf3bf
Change visibility of Program. Add test for Listing 46
COsborn2 Aug 9, 2019
56c2fb3
Attempt to resolve ambiguity on Expect type
COsborn2 Aug 9, 2019
8737aa0
Apply suggestions from code review
COsborn2 Aug 9, 2019
9007480
Update based off code review
COsborn2 Aug 14, 2019
0b1702a
Add assertions to Listing6.13 test
COsborn2 Aug 15, 2019
bc3349b
Change to WriteLine for consistency between Listing5.2 && 5.9
COsborn2 Aug 27, 2019
c41b6ce
Cleanup file after creation
COsborn2 Aug 27, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove space from GetName Listing14. Fix Listing20 property, visibility. 
- Tests added for 14,17,18,19,20
  • Loading branch information
@COsborn2
COsborn2 committed Aug 27, 2019
commit b301ec81c2091515af6cbf58a384d19a22c8cb21
2 changes: 1 addition & 1 deletion src/Chapter06/Listing06.14.DataRetrievalFromAFile.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ class Employee

public string GetName()
{
return $"{ FirstName } { LastName }";
return $"{ FirstName } { LastName }";
}

public void SetName(string newFirstName, string newLastName)
Expand Down
4 changes: 2 additions & 2 deletions src/Chapter06/Listing06.20.ProvidingPropertyValidation.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

namespace AddisonWesley.Michaelis.EssentialCSharp.Chapter06.Listing06_20
{
class Employee
public class Employee
{
// ...
public void Initialize(
Expand All @@ -17,7 +17,7 @@ public void Initialize(
// LastName property
public string LastName
{
get => _FirstName;
get => _LastName;
set
{
// Validate LastName assignment
Expand Down