Skip to content

Commit b05f019

Browse files
committed
Fix some test changes
1 parent d74ac50 commit b05f019

3 files changed

Lines changed: 13 additions & 8 deletions

File tree

test/Services/UnitTest/Ordering/Application/IdentifierCommandHandlerTest.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ private CreateOrderCommand FakeOrderRequest(Dictionary<string, object> args = nu
7171
{
7272
return new CreateOrderCommand(
7373
null,
74+
userId: args != null && args.ContainsKey("userId") ? (string)args["userId"] : null,
7475
city: args != null && args.ContainsKey("city") ? (string)args["city"] : null,
7576
street: args != null && args.ContainsKey("street") ? (string)args["street"] : null,
7677
state: args != null && args.ContainsKey("state") ? (string)args["state"] : null,

test/Services/UnitTest/Ordering/Application/NewOrderCommandHandlerTest.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,13 +66,14 @@ private Buyer FakeBuyer()
6666

6767
private Order FakeOrder()
6868
{
69-
return new Order(new Address("street", "city", "state", "country", "zipcode"), 1, "12", "111", "fakeName", DateTime.Now.AddYears(1));
69+
return new Order(new Guid().ToString(), new Address("street", "city", "state", "country", "zipcode"), 1, "12", "111", "fakeName", DateTime.Now.AddYears(1));
7070
}
7171

7272
private CreateOrderCommand FakeOrderRequestWithBuyer(Dictionary<string, object> args = null)
7373
{
7474
return new CreateOrderCommand(
7575
null,
76+
userId: args != null && args.ContainsKey("userId") ? (string)args["userId"] : null,
7677
city: args != null && args.ContainsKey("city") ? (string)args["city"] : null,
7778
street: args != null && args.ContainsKey("street") ? (string)args["street"] : null,
7879
state: args != null && args.ContainsKey("state") ? (string)args["state"] : null,

test/Services/UnitTest/Ordering/Domain/OrderAggregateTest.cs

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,8 @@ public void Invalid_units_setting()
9696
[Fact]
9797
public void Add_new_Order_raises_new_event()
9898
{
99-
//Arrange
99+
//Arrange
100+
var userId = new Guid();
100101
var street = "fakeStreet";
101102
var city = "FakeCity";
102103
var state = "fakeState";
@@ -110,7 +111,7 @@ public void Add_new_Order_raises_new_event()
110111
var expectedResult = 1;
111112

112113
//Act
113-
var fakeOrder = new Order(new Address(street, city, state, country, zipcode), cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
114+
var fakeOrder = new Order(userId.ToString(), new Address(street, city, state, country, zipcode), cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
114115

115116
//Assert
116117
Assert.Equal(fakeOrder.DomainEvents.Count, expectedResult);
@@ -119,7 +120,8 @@ public void Add_new_Order_raises_new_event()
119120
[Fact]
120121
public void Add_event_Order_explicitly_raises_new_event()
121122
{
122-
//Arrange
123+
//Arrange
124+
var userId = new Guid();
123125
var street = "fakeStreet";
124126
var city = "FakeCity";
125127
var state = "fakeState";
@@ -133,8 +135,8 @@ public void Add_event_Order_explicitly_raises_new_event()
133135
var expectedResult = 2;
134136

135137
//Act
136-
var fakeOrder = new Order(new Address(street, city, state, country, zipcode), cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
137-
fakeOrder.AddDomainEvent(new OrderStartedDomainEvent(fakeOrder,cardTypeId,cardNumber,cardSecurityNumber,cardHolderName,cardExpiration));
138+
var fakeOrder = new Order(userId.ToString(), new Address(street, city, state, country, zipcode), cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
139+
fakeOrder.AddDomainEvent(new OrderStartedDomainEvent(fakeOrder, userId.ToString(), cardTypeId,cardNumber,cardSecurityNumber,cardHolderName,cardExpiration));
138140
//Assert
139141
Assert.Equal(fakeOrder.DomainEvents.Count, expectedResult);
140142
}
@@ -143,6 +145,7 @@ public void Add_event_Order_explicitly_raises_new_event()
143145
public void Remove_event_Order_explicitly()
144146
{
145147
//Arrange
148+
var userId = new Guid();
146149
var street = "fakeStreet";
147150
var city = "FakeCity";
148151
var state = "fakeState";
@@ -153,8 +156,8 @@ public void Remove_event_Order_explicitly()
153156
var cardSecurityNumber = "123";
154157
var cardHolderName = "FakeName";
155158
var cardExpiration = DateTime.Now.AddYears(1);
156-
var fakeOrder = new Order(new Address(street, city, state, country, zipcode), cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
157-
var @fakeEvent = new OrderStartedDomainEvent(fakeOrder, cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
159+
var fakeOrder = new Order(userId.ToString(), new Address(street, city, state, country, zipcode), cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
160+
var @fakeEvent = new OrderStartedDomainEvent(fakeOrder, userId.ToString(), cardTypeId, cardNumber, cardSecurityNumber, cardHolderName, cardExpiration);
158161
var expectedResult = 1;
159162

160163
//Act

0 commit comments

Comments
 (0)