Skip to content

Commit 4dbf305

Browse files
committed
Fix mocks for select index
1 parent 855f06a commit 4dbf305

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

dotnet/test/support/UI/SelectTests.cs

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ public void CanSetSingleOptionSelectedByIndex()
169169

170170
webElement.SetupGet<string>(_ => _.TagName).Returns("select");
171171
webElement.Setup(_ => _.GetDomAttribute(It.Is<string>(x => x == "multiple"))).Returns((string)null);
172-
option1.Setup<string>(_ => _.GetDomAttribute(It.IsAny<string>())).Returns("2");
172+
option1.Setup<string>(_ => _.GetDomProperty(It.IsAny<string>())).Returns("2");
173173
option1.SetupGet<bool>(_ => _.Selected).Returns(false);
174174
option1.SetupGet<bool>(_ => _.Enabled).Returns(true);
175175
option1.Setup(_ => _.Click());
@@ -178,7 +178,7 @@ public void CanSetSingleOptionSelectedByIndex()
178178
new SelectElement(webElement.Object).SelectByIndex(2);
179179
option1.Verify(_ => _.Selected, Times.Once);
180180
option1.Verify(_ => _.Click(), Times.Once);
181-
option1.Verify(_ => _.GetDomAttribute(It.IsAny<string>()), Times.Once);
181+
option1.Verify(_ => _.GetDomProperty(It.IsAny<string>()), Times.Once);
182182
webElement.Verify(_ => _.FindElements(It.IsAny<By>()), Times.Once);
183183
}
184184

@@ -247,11 +247,11 @@ public void CanSetMultipleOptionSelectedByIndex()
247247

248248
webElement.SetupGet<string>(_ => _.TagName).Returns("select");
249249
webElement.Setup(_ => _.GetDomAttribute(It.Is<string>(x => x == "multiple"))).Returns("true");
250-
option1.Setup<string>(_ => _.GetDomAttribute(It.IsAny<string>())).Returns("1");
250+
option1.Setup<string>(_ => _.GetDomProperty(It.IsAny<string>())).Returns("1");
251251
option1.SetupGet<bool>(_ => _.Selected).Returns(false);
252252
option1.SetupGet<bool>(_ => _.Enabled).Returns(true);
253253
option1.Setup(_ => _.Click());
254-
option2.Setup<string>(_ => _.GetDomAttribute(It.IsAny<string>())).Returns("2");
254+
option2.Setup<string>(_ => _.GetDomProperty(It.IsAny<string>())).Returns("2");
255255
option2.SetupGet<bool>(_ => _.Selected).Returns(false);
256256
option2.SetupGet<bool>(_ => _.Enabled).Returns(true);
257257
option2.Setup(_ => _.Click());
@@ -260,10 +260,10 @@ public void CanSetMultipleOptionSelectedByIndex()
260260
new SelectElement(webElement.Object).SelectByIndex(2);
261261
option1.Verify(_ => _.Selected, Times.Never);
262262
option1.Verify(_ => _.Click(), Times.Never);
263-
option1.Verify(_ => _.GetDomAttribute(It.IsAny<string>()), Times.Once);
263+
option1.Verify(_ => _.GetDomProperty(It.IsAny<string>()), Times.Once);
264264
option2.Verify(_ => _.Selected, Times.Once);
265265
option2.Verify(_ => _.Click(), Times.Once);
266-
option2.Verify(_ => _.GetDomAttribute(It.IsAny<string>()), Times.Once);
266+
option2.Verify(_ => _.GetDomProperty(It.IsAny<string>()), Times.Once);
267267
webElement.Verify(_ => _.FindElements(It.IsAny<By>()), Times.Once);
268268
}
269269

@@ -314,13 +314,13 @@ public void CanDeselectSingleOptionSelectedByIndex()
314314

315315
webElement.SetupGet<string>(_ => _.TagName).Returns("select");
316316
webElement.Setup(_ => _.GetDomAttribute(It.Is<string>(x => x == "multiple"))).Returns("true");
317-
option1.Setup<string>(_ => _.GetDomAttribute(It.IsAny<string>())).Returns("2");
317+
option1.Setup<string>(_ => _.GetDomProperty(It.IsAny<string>())).Returns("2");
318318
option1.SetupGet<bool>(_ => _.Selected).Returns(true);
319319
option1.Setup(_ => _.Click());
320320
webElement.Setup(_ => _.FindElements(It.IsAny<By>())).Returns(new ReadOnlyCollection<IWebElement>(options)).Verifiable();
321321

322322
new SelectElement(webElement.Object).DeselectByIndex(2);
323-
option1.Verify(_ => _.GetDomAttribute(It.IsAny<string>()), Times.Once);
323+
option1.Verify(_ => _.GetDomProperty(It.IsAny<string>()), Times.Once);
324324
option1.Verify(_ => _.Selected, Times.Once);
325325
option1.Verify(_ => _.Click(), Times.Once);
326326
webElement.Verify(_ => _.FindElements(It.IsAny<By>()), Times.Once);
@@ -382,19 +382,19 @@ public void CanDeselectMultipleOptionSelectedByIndex()
382382

383383
webElement.SetupGet<string>(_ => _.TagName).Returns("select");
384384
webElement.Setup(_ => _.GetDomAttribute(It.Is<string>(x => x == "multiple"))).Returns("true");
385-
option1.Setup<string>(_ => _.GetDomAttribute(It.IsAny<string>())).Returns("1");
385+
option1.Setup<string>(_ => _.GetDomProperty(It.IsAny<string>())).Returns("1");
386386
option1.SetupGet<bool>(_ => _.Selected).Returns(true);
387387
option1.Setup(_ => _.Click());
388-
option2.Setup<string>(_ => _.GetDomAttribute(It.IsAny<string>())).Returns("2");
388+
option2.Setup<string>(_ => _.GetDomProperty(It.IsAny<string>())).Returns("2");
389389
option2.SetupGet<bool>(_ => _.Selected).Returns(true);
390390
option2.Setup(_ => _.Click());
391391
webElement.Setup(_ => _.FindElements(It.IsAny<By>())).Returns(new ReadOnlyCollection<IWebElement>(options)).Verifiable();
392392

393393
new SelectElement(webElement.Object).DeselectByIndex(2);
394-
option1.Verify(_ => _.GetDomAttribute(It.IsAny<string>()), Times.Once);
394+
option1.Verify(_ => _.GetDomProperty(It.IsAny<string>()), Times.Once);
395395
option1.Verify(_ => _.Selected, Times.Never);
396396
option1.Verify(_ => _.Click(), Times.Never);
397-
option2.Verify(_ => _.GetDomAttribute(It.IsAny<string>()), Times.Once);
397+
option2.Verify(_ => _.GetDomProperty(It.IsAny<string>()), Times.Once);
398398
option2.Verify(_ => _.Selected, Times.Once);
399399
option2.Verify(_ => _.Click(), Times.Once);
400400
webElement.Verify(_ => _.FindElements(It.IsAny<By>()), Times.Once);

0 commit comments

Comments
 (0)