Codeception / Codeception/module-webdriver

Deprecate either `seeInSource()` or `seeInPageSource()`

Open
#19 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
PHP
Stars
39
Forks
25
PR merge metrics
No merged PRs in 30d

Description

[`WebDriver::seeInSource()`](https://github.com/Codeception/module-webdriver/blob/master/src/Codeception/Module/WebDriver.php#L946) and [`WebDriver::seeInPageSource()`](https://github.com/Codeception/module-webdriver/blob/master/src/Codeception/Module/WebDriver.php#L966) both claim to do the same, but differ internally. Which one is the current/recommended? => Deprecate the other! (i.e. keep it as alias till v5.0, then remove it)

Contributor guide

No contributing guide indexed for this repository

Research direction

Compare WebDriver::seeInSource() and WebDriver::seeInPageSource() in src/Codeception/Module/WebDriver.php at the referenced lines, then determine which behavior is current and recommended. Done means the decision is made and the other method is marked for deprecation while remaining an alias until v5.0.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
testing
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.