Skip to content
This repository has been archived by the owner on Oct 13, 2022. It is now read-only.

"/blog/articles/{Str::random(8}"を直す #156

Open
Takeno-hito opened this issue Mar 27, 2021 · 2 comments
Open

"/blog/articles/{Str::random(8}"を直す #156

Takeno-hito opened this issue Mar 27, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@Takeno-hito
Copy link
Member

Takeno-hito commented Mar 27, 2021 •
edited
Loading

BlogArticleTest::testShowNotfound で

$this->get("/blog/articles/{Str::random(8}");

と書いてるし括弧閉じ付けても結局正しいテストにはならない(実際は明らかに404が出て落ちるんで実はどうでも良かったりもするけど

$this->get("/blog/articles/".Str::random(8));

おそらくこれが正しい + 他の所も軒並みそのままになってるのでせっかくなら直してほしい
@Takeno-hito Takeno-hito added the bug Something isn't working label Mar 27, 2021
@Takeno-hito
Copy link
Member Author

Uploading image.png…

@Takeno-hito
Copy link
Member Author

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Takeno-hito