search

uclibs / ucrate

Scholar@UC: University of Cincinnati's self-submission institutional repository
https://scholar.uc.edu
Other
5 stars 3 forks source link

Modifies rails_helper to use required_version. #1114

Closed scherztc closed 8 months ago

scherztc commented 8 months ago

Fixes #1072

Present short summary (50 characters or less)

Replaced the .version with .required_version for the Selenium Webdriver configuration in the rails_helper.rb

Janell-Huyck commented 8 months ago

Explanation from ChatGPT of this change:

Objective: Ensuring uniformity in Chromedriver usage across all development and CI/CD environments, leading to more consistent test results.

Change Explained: required_version = pins the Chromedriver to a specific version ('72.0.3626.69'), unlike version = which could lead to automatic updates to newer versions. This strict pinning is crucial to avoid test discrepancies caused by divergent driver versions.

Impact: Stabilizes the test suite, especially for tests that may rely on specific functionalities of Chromedriver '72.0.3626.69'.

Comparison: Previously, version = might have allowed for driver updates, which could introduce inconsistencies. The required_version = ensures a fixed version, enhancing test reliability.

Janell-Huyck commented 8 months ago

Looks good! 🚀