Skip to content

Commit 6f5c3bd

Browse files
committed
Modernise and fix warnings
1 parent b5ffc32 commit 6f5c3bd

File tree

4 files changed

+13
-8
lines changed

4 files changed

+13
-8
lines changed

Patterns/Observer/SpecialsSubject.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@ namespace Observer
44
{
55
public class SpecialsSubject : ISubject
66
{
7-
public delegate void Callback (string s);
8-
public string SubjectState {get; set;}
7+
public delegate void Callback(string s);
8+
public required string SubjectState { get; set; }
99

10-
private List<IObserver> _observers = new List<IObserver>();
10+
private readonly List<IObserver> _observers = [];
1111

1212
public void Attach(IObserver observer)
1313
{

Patterns/Proxy/Entries.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ public bool Delete(int id)
1414
{
1515
return _products.Remove(id);
1616
}
17-
public IProductInfo Get(int id)
17+
public IProductInfo? Get(int id)
1818
{
1919
if (!_products.ContainsKey(id))
2020
{

Patterns/Proxy/IEntries.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,6 @@ namespace Proxy
44
public interface IEntries
55
{
66
bool Delete(int id);
7-
IProductInfo Get(int id);
7+
IProductInfo? Get(int id);
88
}
99
}

Tests/ObserverTest.cs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,13 @@ public class ObserverShould
1111
public void UpdateObserverOnce_WhenSubjectHasSpecials()
1212
{
1313
// Arrange
14-
var subject = new SpecialsSubject();
14+
var subject = new SpecialsSubject
15+
{
16+
SubjectState = "Footwear Sale"
17+
};
1518
var mockObserver = new Mock<IObserver>();
1619
// Act
1720
subject.Attach(mockObserver.Object);
18-
subject.SubjectState = "Footwear Sale";
1921
subject.Notify();
2022
// Assert
2123
mockObserver.Verify(observer => observer.Update(subject), Times.Once());
@@ -25,7 +27,10 @@ public void UpdateObserverOnce_WhenSubjectHasSpecials()
2527
public void NotCallUpdate_WhenObserverNotAttachedToSubject()
2628
{
2729
// Arrange
28-
var subject = new SpecialsSubject();
30+
var subject = new SpecialsSubject
31+
{
32+
SubjectState = "Footwear Sale"
33+
};
2934
var mockObserver = new Mock<IObserver>();
3035
// Act
3136
subject.SubjectState = "Footwear Sale";

0 commit comments

Comments
 (0)