Skip to content

Fix test inconsistency in group utils#2626

Closed
ozh wants to merge 9 commits intomasterfrom
issue2621
Closed

Fix test inconsistency in group utils#2626
ozh wants to merge 9 commits intomasterfrom
issue2621

Conversation

@ozh
Copy link
Copy Markdown
Member

@ozh ozh commented Mar 24, 2020

Simplify logic of tests in NextDecimal_Tests to only test what is supposed to be tested.

Should fix #2621

ozh and others added 9 commits March 22, 2020 17:56
This all boils down to redirecting to $_SERVER['REQUEST_URI'] after successful login
- no dying
- no output if we're on cli
Co-Authored-By: Léo Colombaro <git@colombaro.fr>
Co-Authored-By: Léo Colombaro <git@colombaro.fr>
Do not rely on HOSTNAME, HTTP_HOST or whatever, just use YOURLS_SITE
Simplify logic of tests in NextDecimal_Tests to only test what is supposed to be tested.
Should fix #2621
@ozh
Copy link
Copy Markdown
Member Author

ozh commented Mar 24, 2020

Huh. I think I'm mixing two branches. Oh well... :)

@ozh ozh mentioned this pull request Mar 24, 2020
@ozh ozh closed this Mar 24, 2020
@LeoColomb LeoColomb deleted the issue2621 branch March 28, 2020 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tests inconsistency in group utils

1 participant