Update to Selenium v.4.36.0 +semver: feature

[mialeska]

No description provided.

[coderabbitai]

Walkthrough

Bumps NuGet package versions in Aquality.Selenium.Core and its test project. Core updates NLog and Selenium packages. Tests update Appium.WebDriver, NUnit3TestAdapter, and Microsoft.NET.Test.Sdk. No source code, API, or control-flow changes.

Changes

Cohort / File(s)	Summary
Core project dependency updates Aquality.Selenium.Core/src/Aquality.Selenium.Core/Aquality.Selenium.Core.csproj	Updated package versions: NLog 6.0.4 → 6.0.5, Selenium.Support 4.35.0 → 4.36.0, Selenium.WebDriver 4.35.0 → 4.36.0.
Test project dependency updates Aquality.Selenium.Core/tests/Aquality.Selenium.Core.Tests/Aquality.Selenium.Core.Tests.csproj	Updated package versions: Appium.WebDriver 8.0.0 → 8.0.1, NUnit3TestAdapter 5.1.0 → 5.2.0, Microsoft.NET.Test.Sdk 17.14.1 → 18.0.0.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Update to Selenium v.4.35.0 +semver: feature #131 — Similar csproj package version bumps across core and tests, including Selenium updates.
• Update Selenium to v4.33.0 +semver:feature #128 — Updates Selenium.WebDriver and Selenium.Support versions in the same core project file.
• Update to Selenium v.4.31.0 +semver:feature #126 — Bumps Selenium-related dependencies in Aquality.Selenium.Core.csproj.

Poem

I thump my paws at version cheer,
Fresh carrots of NuGet near! 🥕
NLog nibble, Selenium hop,
Tests get spruced—flip, skip, bop.
With tidy refs and whiskers bright,
I merge this patch and bound to night.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description Check	❓ Inconclusive	No pull request description was provided, leaving reviewers without context or rationale for the dependency updates and making it unclear how the changes fit into the overall project goals.	Please add a brief description summarizing which packages were upgraded, any relevant motivations or compatibility considerations, and how these updates impact the project.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title clearly identifies the main change as updating Selenium to version 4.36.0 and concisely conveys the primary purpose of the pull request without extraneous details.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch update-selenium-to-4.36.0

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6589113 and b774aeb.

📒 Files selected for processing (2)

• Aquality.Selenium.Core/src/Aquality.Selenium.Core/Aquality.Selenium.Core.csproj (1 hunks)
• Aquality.Selenium.Core/tests/Aquality.Selenium.Core.Tests/Aquality.Selenium.Core.Tests.csproj (1 hunks)

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: mialeska
PR: aquality-automation/aquality-selenium-core-dotnet#130
File: Aquality.Selenium.Core/tests/Aquality.Selenium.Core.Tests/Aquality.Selenium.Core.Tests.csproj:28-28
Timestamp: 2025-07-17T12:58:10.272Z
Learning: In the aquality-selenium-core-dotnet project, the Appium.WebDriver upgrade from version 7.2.0 to 8.0.0 was successfully completed without requiring any code changes in the test suite, as confirmed by passing CI pipeline checks.

📚 Learning: 2025-07-17T12:58:10.272Z

Learnt from: mialeska
PR: aquality-automation/aquality-selenium-core-dotnet#130
File: Aquality.Selenium.Core/tests/Aquality.Selenium.Core.Tests/Aquality.Selenium.Core.Tests.csproj:28-28
Timestamp: 2025-07-17T12:58:10.272Z
Learning: In the aquality-selenium-core-dotnet project, the Appium.WebDriver upgrade from version 7.2.0 to 8.0.0 was successfully completed without requiring any code changes in the test suite, as confirmed by passing CI pipeline checks.

Applied to files:

• Aquality.Selenium.Core/tests/Aquality.Selenium.Core.Tests/Aquality.Selenium.Core.Tests.csproj
• Aquality.Selenium.Core/src/Aquality.Selenium.Core/Aquality.Selenium.Core.csproj

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: aquality-automation.aquality-selenium-core-dotnet (test Analyse code with SonarQube)
• GitHub Check: aquality-automation.aquality-selenium-core-dotnet (test Run tests)

🔇 Additional comments (1)

Aquality.Selenium.Core/tests/Aquality.Selenium.Core.Tests/Aquality.Selenium.Core.Tests.csproj (1)

38-38: Ensure .NET SDK ≥8.0.4 on CI/IDE
Microsoft.NET.Test.Sdk 18.0.0 requires .NET 8.0.4 or later. Confirm your build agents and local environments meet this minimum and rerun the full test suite.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud