Skip to content

Commit 9b2e9ac

Browse files
authored
feat: Jump selenium from 4.0.0 to 4.4.0 (#517)
* Target .NET Framework 4.8 * test:Fix ClipboardTests * Update Selenium from 4.0.0 to 4.4.0. Removal of Obsolete AddAdditionalCapability Implement GetDomProperty * Removal of GetDomProperty
1 parent e97f197 commit 9b2e9ac

File tree

5 files changed

+7
-17
lines changed

5 files changed

+7
-17
lines changed

src/Appium.Net/Appium.Net.csproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@
4646
<PrivateAssets>all</PrivateAssets>
4747
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
4848
</PackageReference>
49-
<PackageReference Include="Selenium.Support" Version="4.0.0" />
50-
<PackageReference Include="Selenium.WebDriver" Version="4.0.0" />
49+
<PackageReference Include="Selenium.Support" Version="4.4.0" />
50+
<PackageReference Include="Selenium.WebDriver" Version="4.4.0" />
5151
</ItemGroup>
5252

5353
<ItemGroup>

src/Appium.Net/Appium/AppiumElement.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,9 +155,10 @@ public override string GetCssValue(string propertyName) => CacheValue(
155155
() => base.GetCssValue(propertyName)
156156
)?.ToString();
157157

158-
public override string GetProperty(string propertyName) => CacheValue(
158+
//TODO: Add Integrations tests
159+
public string GetProperty(string propertyName) => CacheValue(
159160
"property/" + propertyName,
160-
() => base.GetProperty(propertyName)
161+
() => base.GetDomProperty(propertyName)
161162
)?.ToString();
162163

163164
protected virtual object CacheValue(string key, Func<object> getter)

src/Appium.Net/Appium/AppiumOptions.cs

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -82,17 +82,6 @@ public void AddAdditionalAppiumOption(string optionName, object optionValue)
8282
this.additionalAppiumOptions[name] = optionValue;
8383
}
8484

85-
/// <summary>
86-
/// Add new capabilities
87-
/// </summary>
88-
/// <param name="capabilityName">Capability name</param>
89-
/// <param name="capabilityValue">Capabilities value, which cannot be null or empty</param>
90-
[Obsolete("Use the the AddAdditionalAppiumOption method for adding additional options")]
91-
public override void AddAdditionalCapability(string capabilityName, object capabilityValue)
92-
{
93-
this.AddAdditionalAppiumOption(capabilityName, capabilityValue);
94-
}
95-
9685
public override void AddAdditionalOption(string optionName, object optionValue)
9786
{
9887
throw new NotImplementedException("Use the the AddAdditionalAppiumOption method for adding additional options");

src/Appium.Net/Appium/Service/Options/OptionCollector.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ public OptionCollector AddCapabilities(AppiumOptions options)
6868

6969
foreach (var item in result)
7070
{
71-
this.options.AddAdditionalCapability(item.Key, item.Value);
71+
this.options.AddAdditionalAppiumOption(item.Key, item.Value);
7272
}
7373
}
7474

test/integration/ServerTests/StartingAppLocallyTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ public void CheckThatServiseIsNotRunWhenTheCreatingOfANewSessionIsFailed()
133133
? //it will be a cause of error
134134
Caps.GetAndroidUIAutomatorCaps(Apps.Get("androidApiDemos"))
135135
: Caps.GetAndroidUIAutomatorCaps(Apps.Get("androidApiDemos"));
136-
capabilities.AddAdditionalAppiumOption(MobileCapabilityType.DeviceName, "iPhone Simulator");
136+
capabilities.DeviceName = "iPhone Simulator";
137137
capabilities.AddAdditionalAppiumOption(MobileCapabilityType.PlatformName, MobilePlatform.IOS);
138138

139139
var builder = new AppiumServiceBuilder();

0 commit comments

Comments
 (0)