Skip to content

[7.x] Make testing route method more robust #500

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Aug 1, 2022
Merged

[7.x] Make testing route method more robust #500

merged 7 commits into from
Aug 1, 2022

Conversation

arthurkirkosa
Copy link
Collaborator

The new method allows you to send an Model object as the $path and it allows having an action class beside the query array

@arthurkirkosa arthurkirkosa requested a review from binaryk August 1, 2022 07:58
@binaryk binaryk changed the title Make testing route method more robust [7.x] Make testing route method more robust Aug 1, 2022
binaryk
binaryk requested changes Aug 1, 2022
View reviewed changes
@arthurkirkosa arthurkirkosa requested a review from binaryk August 1, 2022 12:31
@binaryk binaryk merged commit 310d91b into 7.x Aug 1, 2022
@binaryk
Copy link
Collaborator

binaryk commented Aug 1, 2022

Thank you!

@binaryk binaryk deleted the testing-route branch August 1, 2022 12:42
binaryk added a commit that referenced this pull request Sep 15, 2022
@binaryk @daniel-banciulea
@ilichkaya @eloquentcoder @arthurkirkosa @ttungbmt @yondifon
7.x (#491)
0b5b708 
* fix: support ukraine

* Clean up & upgrading php + laravel (#464)

* fix: cleaning the repository method

* Fix styling

* fix: wip

* fix: wip

* fix: wip

* fix: wip

* fix: wip

* fix: wip

* feat: drop laravel 8 support

* fix: wip

* Fix styling

* fix: wip

* Fix styling

* fix: refactoring matches

* fix: wip

* Fix styling

* fix: wip

* fix: wip

* Larastan (#461)

* fix: support ukraine

* adding larastan

* wip

* Fix styling

* Fix styling

* fix: wip

* Fix styling

* fix: wip

* fix: php 8.1

* fix: cover windows tests

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: delete unused

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: route method

* Fix styling

* fix: roadmap

* fix: Make sure any action isn't permitted unless the Model Policy exists

* fix: wip

* Fix styling

* fix: Block requests without policy (#466)

* fix: Block requests without policy

* Fix styling

* fix: wip

* fix: wip

* fix: wip

* fix: tests

* fix: config

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: Adding package tools and fixing related bug (#467)

* Tests 2 (#468)

* fix: use route helper

* Fix styling

* fix: tests refactoring

* Fix styling

* fix: route key

* fix: wip

* Fix styling

* fix: phpunit config

* fix: coverage

* fix: factories

* Fix styling

* fix: wip

* fix: wip

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: wip

* fix: wip

* fix: wip

* Custom logs (#469)

* fix: assertables & prototypes

* Fix styling

* fix: observer for logs

* Fix styling

* fix: wip

* Fix styling

* fix: custom logs

* fix: wip

* Fix styling

* fix: sideeffect

* Fix styling

* fix: wip

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* With eager loading (#470)

* fix: adding with for the show

* fix: cleanup responses using helper

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: List restify routes through artisan command (#471)

* fix: List restify routes through artisan command

* Fix styling

* fix: wip

* fix: tests

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: wip

* fix: helpers for action logs

* Fix styling

* fix: fix eager fields without key in include

* fix: wip

* fix: custom serializer (#472)

* fix: custom serializer

* Fix styling

* fix: serializer docs

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* Fix styling

* Relationships nested (#473)

* fix: eager loading nested relationships

* Fix styling

* fix: wip

* fix: eager loading nested relationships

* fix: wip

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: don't check related if no query (#475)

* Fix after bulk method calls (#476)

* fix after bulk method calls

* fix number of calls

* revert returns

* Fix styling

* Keep model attributes for deletedBulk method. (#477)

* fix after bulk method calls

* fix number of calls

* revert returns

* keep model attributes

* fix: fix psalm and tests (#478)

* fix: fix psalm and tests

* fix: wip

* fix: wip

* Performance (#474)

* fix: catch issues

* Fix styling

* fix: ensure eager loading works

* Fix styling

* fix: performance improvements

* Fix styling

* fix: related performance improvements

* Fix styling

* fix: drop cast support

* Fix styling

* fix: optimize current repository search

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* Recursive related (#479)

* fix: collection

* Fix styling

* fix: tests

* fix: wip

* Fix styling

* fix: recursive

* Fix styling

* fix: recursive related including columns

* fix: typo

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* Feedback related (#480)

* fix: feedback from related pr

* Fix styling

* fix: wip

* fix: wip

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* Dynamic meta (#481)

* fix: configurable meta render

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: formatting (#482)

* Docs 7x (#483)

* fix: repository generator improvements & sanctum middleware scaffolding

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* Docs 7x (#484)

* fix: repository generator improvements & sanctum middleware scaffolding

* feat: [7.x] Eager fields recognize pattern of the key to retrieve repository.

* Fix styling

* fix: updating docs for v7

* fix: merge

* Fix styling

* fix: wip

* fix: wip

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* Last formatting (#485)

* fix: wip

* fix: wip

* fix: [7.x] Load routes from the application service provider so the d… (#487)

* fix: [7.x] Load routes from the application service provider so the developer could disable it.

* Fix styling

* fix: wip

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: [7.x] Loading routes before middleware stack.

* fix: wip

* Fix styling

* fix: Do not resolve current repository from the singleton.

* Fix styling

* Singleton issues (#492)

* fix: [7.x] Singleton issues fixes.

* Fix styling

* fix: wip

* Fix styling

* fix: wip

* fix: tests

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: publish restify before routes

* AUTH - Actions  (#490)

* Update index.md

* Update index.md

* Update actions.md

* Update actions.md

* Update fields.md

* Update getters.md

* Update relations.md

* Update repositories-advanced.md

* Update repositories.md

* Update serializer.md

* Update rest-methods.md

* Update index.md

* Update index.md

* Update index.md

* Update quickstart.md

* Update authentication.md

* Update authorization.md

* Update profile.md

* fix: docs

* Fix styling

* fix: readme

* fix: quickstart docs

* Update quickstart.md (#493)

Updated the quickstart docs to correct the typo **creates** an abstract `app/Restif/Repository.php` to **creates** an abstract `app/Restify/Repository.php`

* Update README.md

* [7.x] Nested relationships using the same parent (#494)

* fix: [7.x] Singleton issues fixes.

* Fix styling

* fix: wip

* Fix styling

* fix: wip

* fix: tests

* fix: fixing assoc for eager loading

* Fix styling

* fix: wip parent relationship

* Fix styling

* fix: wip

* Fix styling

* fix: fixing recursive issue

* Fix styling

* fix: upgrading guideline

* fix: wip

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: wip

* [7.x] Cache policies (#496)

* fix: [7.x] cache the authorized repository policy

* Fix styling

* fix: caching policies

* Fix styling

* fix: docs

* fix: docs

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: [7.x] Action log observer fix. (#497)

* fix: fixing cache enable option (#498)

* [7.x] Make testing route method more robust (#500)

* Make testing route method more robust

* Fix tests

* Fix tests

* Fix styling

* Fix tests

* Fix tests

* Move action as 3rd param

Co-authored-by: arthurkirkosa <arthurkirkosa@users.noreply.github.com>

* fix: [7.x] chaning docs category (#499)

* [7.x] Fix authorization http code (#501)

* fix: [7.x] Use 401 unathorized http code.

* fix: wip

* fix: adding test

* fix: wip

* Fix styling

* fix: wip

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: ensure testing helpers have test coverage (#502)

* fix: ensure testing helpers have test coverage

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* [7.x] Do not load singleton DTO in tests (#503)

* fix: do not load routes twice in tests and don't run singleton for related in tests

* Fix styling

* fix: wip

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* fix: upgrading

* feat: making the policy meta method protected (#504)

* Fix Gate::check - show - not working (#505)

* fix scout when search input is null (#506)

Fix when using ElasticSearch: Elastic\Elasticsearch\Exception\ClientResponseException: 400 Bad Request: {"error":{"root_cause":[{"type":"parsing_exception","reason":"[query_string] unknown token [VALUE_NULL] after [query]","line":1,"col":35}],"type":"parsing_exception","reason":"[query_string] unknown token [VALUE_NULL] after [query]","line":1,"col":35},"status":400}

* fix: fixing public index method (#513)

* Serialize One model (#510)

* fix: listing routes in console (#514)

* fix: listing routes in console

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

* Fix styling

Co-authored-by: binaryk <binaryk@users.noreply.github.com>
Co-authored-by: Daniel Bănciulea <daniel.banciulea@systeady.com>
Co-authored-by: ilichkaya <109801809+ilichkaya@users.noreply.github.com>
Co-authored-by: Patrick Obafemi <eloquentintech@gmail.com>
Co-authored-by: Arthur Kirkosa <arthur.kirkosa@me.com>
Co-authored-by: arthurkirkosa <arthurkirkosa@users.noreply.github.com>
Co-authored-by: Trương Thanh Tùng <ttungbmt@gmail.com>
Co-authored-by: ndifon <malico.yong@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@binaryk binaryk Awaiting requested review from binaryk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arthurkirkosa @binaryk