Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add App::url() method tests #2109

Merged
merged 68 commits into from
Sep 17, 2023
Merged
Changes from 1 commit
Commits
Show all changes
68 commits
Select commit Hold shift + click to select a range
7fdbbe7
Fix PrettyUrl legitimate path
abbadon1334 Aug 23, 2023
ba90e32
Add Unit tests for App::url
abbadon1334 Aug 25, 2023
f2dd8f3
add default excepted page and extension
abbadon1334 Aug 25, 2023
6713d61
fix 7.4 array merge with keys / spread operator
abbadon1334 Aug 25, 2023
04720c5
Add more test cases for App:url
abbadon1334 Aug 25, 2023
80b8fca
Reorganize code
abbadon1334 Aug 25, 2023
27d9cb4
Improve converage
abbadon1334 Aug 26, 2023
67f0594
Improve App::url tests
abbadon1334 Aug 28, 2023
b2810b8
Skip $useRequestUrl=true due to REQUEST_URI
abbadon1334 Aug 28, 2023
3997714
Change REQUEST_URI to Psr7 methods
abbadon1334 Aug 28, 2023
4348ff1
Allow $useRequestUrl=true
abbadon1334 Aug 28, 2023
e4c3428
clean up
abbadon1334 Aug 28, 2023
7895ec9
Improve error message
abbadon1334 Aug 28, 2023
f1fefac
CS Fix
abbadon1334 Aug 28, 2023
308cc87
Merge branch 'develop' into add_tests_app_url
abbadon1334 Sep 3, 2023
23a5a6b
Add App::$urlBuildingPage to customize url building page
abbadon1334 Sep 4, 2023
f3f099f
Comment out App::url failing tests
abbadon1334 Sep 4, 2023
3c36038
Merge branch 'develop' into fix_pretty_url_legitimate_path
abbadon1334 Sep 4, 2023
534cff4
Avoid change of global var $_SERVER
abbadon1334 Sep 4, 2023
a01c3bb
Fix missing request in App for Unit tests
abbadon1334 Sep 4, 2023
62f370b
Comment out not working tests
abbadon1334 Sep 4, 2023
645e0aa
Strong type for $urlBuildingXxx vars
abbadon1334 Sep 8, 2023
7022361
Use CreateApp
abbadon1334 Sep 8, 2023
0a1bd7f
Remove message from Assert
abbadon1334 Sep 8, 2023
95b7d1e
Cs Fix
abbadon1334 Sep 8, 2023
7bd47a2
Raise Nyholm/psr7 from 1.4 to 1.6
abbadon1334 Sep 8, 2023
d5b4224
Merge branch 'develop' into add_tests_app_url
mvorisek Sep 9, 2023
47d6032
DEBUG REVERT ALL
mvorisek Sep 9, 2023
d4a0810
introduce App::$urlBuildingIndexPage prop
mvorisek Sep 9, 2023
89e3efa
drop protected App::getRequestUrl() method
mvorisek Sep 9, 2023
b490c1f
replace $_SERVER['REQUEST_URI'] with PSR7 request
mvorisek Sep 9, 2023
4d0e4c7
fix cs
mvorisek Sep 9, 2023
8b3bd58
original AppTest changes
mvorisek Sep 9, 2023
fccada4
no manual assert context dump
mvorisek Sep 9, 2023
b14f5fe
allow to create test App with defaults
mvorisek Sep 9, 2023
c3de25c
always use test App in tests
mvorisek Sep 9, 2023
a4c193f
Merge branch 'develop'
mvorisek Sep 9, 2023
84cfbd0
Merge PR 2096
mvorisek Sep 9, 2023
547f2f4
remaining reverted changes from PR 2096
mvorisek Sep 9, 2023
9353bba
Merge develop
mvorisek Sep 9, 2023
b80942d
Merge branch 'develop'
mvorisek Sep 10, 2023
c9d06c6
Merge branch 'develop'
mvorisek Sep 14, 2023
bae7f69
Merge branch 'develop'
mvorisek Sep 14, 2023
9cff373
enable commented tests
mvorisek Sep 14, 2023
75efe8c
silence failing tests
mvorisek Sep 14, 2023
2c2b8ca
add wrongly removed check_unset_page test
mvorisek Sep 14, 2023
965926e
REVERT ALL (reset to develop)
mvorisek Sep 14, 2023
1fe9677
Merge branch 'develop'
mvorisek Sep 15, 2023
1d3df06
reland reverted changes in minimal fashion
mvorisek Sep 15, 2023
637f66a
Revert nonse App:url() path build
mvorisek Sep 15, 2023
426addb
improve grammar
mvorisek Sep 17, 2023
10d8773
uncomment all tests
mvorisek Sep 17, 2023
384f8e3
simplify original URL build testing
mvorisek Sep 17, 2023
38d3729
refactor all providers to yield
mvorisek Sep 17, 2023
018a1ba
improve phpdoc
mvorisek Sep 17, 2023
5d7fd8b
improve wording
mvorisek Sep 17, 2023
7b35daf
rework testing
mvorisek Sep 17, 2023
e1dc916
unset sticky is deleting extra
mvorisek Sep 17, 2023
4e4fc23
str_ends_with is better
mvorisek Sep 17, 2023
92ccde8
do not add ext if URL ends with "/"
mvorisek Sep 17, 2023
49abe58
rm failing tests
mvorisek Sep 17, 2023
0f638de
revert test until fixed by 2095 PR
mvorisek Sep 17, 2023
65b4c3c
minor CS
mvorisek Sep 17, 2023
9c7407c
scheme/host is not strictly required
mvorisek Sep 17, 2023
1e411bf
fix typo
mvorisek Sep 17, 2023
0cece36
more tests
mvorisek Sep 17, 2023
ffab2a0
if body was not covered
mvorisek Sep 17, 2023
ac138d4
minor cs
mvorisek Sep 17, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
scheme/host is not strictly required
mvorisek committed Sep 17, 2023
commit 9c7407cfc6295a98a642b1fcca2f84bb76d817bc
2 changes: 1 addition & 1 deletion tests/AppTest.php
Original file line number Diff line number Diff line change
@@ -120,7 +120,7 @@ public function provideUrlCases(): iterable
*/
public function testUrl(string $requestUrl, array $appStickyGetArguments, array $page, array $extraRequestUrlArgs, string $exceptedUrl): void
{
$request = (new Psr17Factory())->createServerRequest('GET', 'http://xxx' . $requestUrl);
$request = (new Psr17Factory())->createServerRequest('GET', $requestUrl);

$app = $this->createApp([
'request' => $request,