Tweaks for special characters in names failing tests
This commit is contained in:
parent
2779bd5f88
commit
9b381ebbbc
|
|
@ -17,7 +17,7 @@ class EventFactory extends Factory
|
||||||
public function definition(): array
|
public function definition(): array
|
||||||
{
|
{
|
||||||
return [
|
return [
|
||||||
'name' => $this->faker->unique()->name(),
|
'name' => $this->faker->unique()->firstName,
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -73,7 +73,7 @@ it('shows school data', function () {
|
||||||
foreach ($this->schools as $school) {
|
foreach ($this->schools as $school) {
|
||||||
$response->assertSeeInOrder([
|
$response->assertSeeInOrder([
|
||||||
'td',
|
'td',
|
||||||
$school->name,
|
e($school->name),
|
||||||
'/td',
|
'/td',
|
||||||
'td',
|
'td',
|
||||||
$invoiceDataService->getGrandTotal($school->id),
|
$invoiceDataService->getGrandTotal($school->id),
|
||||||
|
|
|
||||||
|
|
@ -47,7 +47,7 @@ it('shows ensemble data', function () {
|
||||||
foreach ($events as $event) {
|
foreach ($events as $event) {
|
||||||
foreach ($event->ensembles as $ensemble) {
|
foreach ($event->ensembles as $ensemble) {
|
||||||
$response->assertSeeInOrder([
|
$response->assertSeeInOrder([
|
||||||
$event->name, '<td', $ensemble->id, '</td>', '<td', e($ensemble->name), '</td>',
|
e($event->name), '<td', $ensemble->id, '</td>', '<td', e($ensemble->name), '</td>',
|
||||||
], false);
|
], false);
|
||||||
}
|
}
|
||||||
$response->assertSee($event->name);
|
$response->assertSee($event->name);
|
||||||
|
|
@ -71,7 +71,7 @@ it('shows a delete option for ensembles with no students seated', function () {
|
||||||
$response = get((route('admin.ensembles.index')));
|
$response = get((route('admin.ensembles.index')));
|
||||||
$response->assertOk();
|
$response->assertOk();
|
||||||
$response->assertSee(route('admin.ensembles.destroy', $noSeatsEnsemble), false);
|
$response->assertSee(route('admin.ensembles.destroy', $noSeatsEnsemble), false);
|
||||||
#$response->assertDontSee(route('admin.ensembles.destroy', $seatsEnsemble), false); // TODO figure out how to test for a delete form that does not also see an edit form
|
//$response->assertDontSee(route('admin.ensembles.destroy', $seatsEnsemble), false); // TODO figure out how to test for a delete form that does not also see an edit form
|
||||||
});
|
});
|
||||||
it('allows an administrator to delete an ensemble while no entries are seated', function () {
|
it('allows an administrator to delete an ensemble while no entries are seated', function () {
|
||||||
// Arrange
|
// Arrange
|
||||||
|
|
@ -119,7 +119,7 @@ it('allows an administrator to update an ensemble', function () {
|
||||||
$ensemble = Ensemble::factory()->create();
|
$ensemble = Ensemble::factory()->create();
|
||||||
$newData = [
|
$newData = [
|
||||||
'name' => 'New Ensemble Name',
|
'name' => 'New Ensemble Name',
|
||||||
'code' => 'NEWC',
|
'code' => 'newC',
|
||||||
];
|
];
|
||||||
// Act & Assert
|
// Act & Assert
|
||||||
actAsAdmin();
|
actAsAdmin();
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue