Apply suggestions from code review
Co-authored-by: Ewelina Lasowy <56546832+EwelinaLasowy@users.noreply.github.com>
This commit is contained in:
parent
8726b81a66
commit
83e003ddbb
@ -50,7 +50,7 @@ class Controller extends SlackController
|
|||||||
);
|
);
|
||||||
|
|
||||||
return Response::create($this->request)
|
return Response::create($this->request)
|
||||||
->withText(":x: Polecenie `/{$this->request->command} {$this->request->text}` jest niepoprawna:")
|
->withText(":x: Polecenie `/{$this->request->command} {$this->request->text}` jest niepoprawne:")
|
||||||
->withAttachments($errors->all());
|
->withAttachments($errors->all());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -38,7 +38,7 @@ class SendDailySummaryToSlackTest extends TestCase
|
|||||||
Http::assertSentCount(1);
|
Http::assertSentCount(1);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testCommandDoesntSendIfWeekend(): void
|
public function testCommandDoesntSendMessageIfWeekend(): void
|
||||||
{
|
{
|
||||||
$weekend = Carbon::create(2022, 4, 23);
|
$weekend = Carbon::create(2022, 4, 23);
|
||||||
$this->assertTrue($weekend->isWeekend());
|
$this->assertTrue($weekend->isWeekend());
|
||||||
@ -51,7 +51,7 @@ class SendDailySummaryToSlackTest extends TestCase
|
|||||||
Http::assertNothingSent();
|
Http::assertNothingSent();
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testCommandDoesntSendIfHoliday(): void
|
public function testCommandDoesntSendMessageIfHoliday(): void
|
||||||
{
|
{
|
||||||
$holiday = Holiday::factory(["date" => Carbon::create(2022, 4, 22)])->create();
|
$holiday = Holiday::factory(["date" => Carbon::create(2022, 4, 22)])->create();
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user