Skip to content

Commit 9df2033

Browse files
fix: add otlp test changes
1 parent 54030d3 commit 9df2033

File tree

3 files changed

+16
-10
lines changed

3 files changed

+16
-10
lines changed

tests/OpenTelemetry/Formatters/SummitAttendeeBadgeAuditLogFormatterTest.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ private function createMockSubject(): mixed
3737
$mockOrder = Mockery::mock('models\summit\SummitOrder');
3838
$mockOrder->shouldReceive('getSummit')->andReturn($mockSummit);
3939

40+
$mockOwner = Mockery::mock('models\summit\SummitAttendee');
41+
$mockOwner->shouldReceive('getFirstName')->andReturn('John');
42+
$mockOwner->shouldReceive('getSurname')->andReturn('Doe');
43+
4044
$mockTicket = Mockery::mock('models\summit\SummitAttendeeTicket');
4145
$mockTicket->shouldReceive('getId')->andReturn(self::TICKET_ID);
4246
$mockTicket->shouldReceive('getNumber')->andReturn(self::TICKET_NUMBER);
4347
$mockTicket->shouldReceive('getOrder')->andReturn($mockOrder);
48+
$mockTicket->shouldReceive('getOwner')->andReturn($mockOwner);
4449

4550
$mock = Mockery::mock('models\summit\SummitAttendeeBadge');
4651
$mock->shouldReceive('getId')->andReturn(self::BADGE_ID);
@@ -58,8 +63,8 @@ public function testSubjectCreationAuditMessage(): void
5863

5964
$this->assertNotNull($result);
6065
$this->assertStringContainsString('created', $result);
61-
$this->assertStringContainsString(self::TICKET_NUMBER, $result);
62-
$this->assertStringContainsString((string)self::TICKET_ID, $result);
66+
$this->assertStringContainsString('John Doe', $result);
67+
$this->assertStringContainsString((string)self::BADGE_ID, $result);
6368
}
6469

6570
public function testSubjectUpdateAuditMessage(): void

tests/OpenTelemetry/Formatters/SummitAttendeeNoteAuditLogFormatterTest.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,8 @@ private function createMockSubject(): mixed
3939
$mockAttendee->shouldReceive('getId')->andReturn(self::ATTENDEE_ID);
4040
$mockAttendee->shouldReceive('getEmail')->andReturn(self::ATTENDEE_EMAIL);
4141
$mockAttendee->shouldReceive('getSummit')->andReturn($mockSummit);
42+
$mockAttendee->shouldReceive('getFirstName')->andReturn('Jane');
43+
$mockAttendee->shouldReceive('getSurname')->andReturn('Smith');
4244

4345
$mock = Mockery::mock('models\summit\SummitAttendeeNote');
4446
$mock->shouldReceive('getId')->andReturn(self::NOTE_ID);
@@ -56,8 +58,8 @@ public function testSubjectCreationAuditMessage(): void
5658

5759
$this->assertNotNull($result);
5860
$this->assertStringContainsString('created', $result);
59-
$this->assertStringContainsString(self::ATTENDEE_EMAIL, $result);
60-
$this->assertStringContainsString((string)self::ATTENDEE_ID, $result);
61+
$this->assertStringContainsString('Jane Smith', $result);
62+
$this->assertStringContainsString((string)self::NOTE_ID, $result);
6163
}
6264

6365
public function testSubjectUpdateAuditMessage(): void

tests/OpenTelemetry/Formatters/SummitAttendeeTicketAuditLogFormatterTest.php

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,14 @@ private function createMockSubject(): mixed
3939
$mockSummit = Mockery::mock('models\summit\Summit');
4040
$mockSummit->shouldReceive('getName')->andReturn(self::SUMMIT_NAME);
4141

42-
$mockOwner = Mockery::mock('models\main\Member');
43-
$mockOwner->shouldReceive('getEmail')->andReturn(self::OWNER_EMAIL);
42+
$mockOwner = Mockery::mock('models\summit\SummitAttendee');
43+
$mockOwner->shouldReceive('getFirstName')->andReturn('John');
44+
$mockOwner->shouldReceive('getSurname')->andReturn('Doe');
4445

4546
$mockOrder = Mockery::mock('models\summit\SummitOrder');
4647
$mockOrder->shouldReceive('getId')->andReturn(self::ORDER_ID);
4748
$mockOrder->shouldReceive('getNumber')->andReturn(self::ORDER_NUMBER);
4849
$mockOrder->shouldReceive('getSummit')->andReturn($mockSummit);
49-
$mockOrder->shouldReceive('getOwner')->andReturn($mockOwner);
5050

5151
$mockTicketType = Mockery::mock('models\summit\SummitTicketType');
5252
$mockTicketType->shouldReceive('getName')->andReturn(self::TICKET_TYPE);
@@ -57,6 +57,7 @@ private function createMockSubject(): mixed
5757
$mock->shouldReceive('getOrder')->andReturn($mockOrder);
5858
$mock->shouldReceive('getTicketType')->andReturn($mockTicketType);
5959
$mock->shouldReceive('getStatus')->andReturn(self::TICKET_STATUS);
60+
$mock->shouldReceive('getOwner')->andReturn($mockOwner);
6061

6162
return $mock;
6263
}
@@ -69,10 +70,8 @@ public function testSubjectCreationAuditMessage(): void
6970

7071
$this->assertNotNull($result);
7172
$this->assertStringContainsString('created', $result);
72-
$this->assertStringContainsString(self::TICKET_NUMBER, $result);
73+
$this->assertStringContainsString('John Doe', $result);
7374
$this->assertStringContainsString((string)self::TICKET_ID, $result);
74-
$this->assertStringContainsString(self::TICKET_TYPE, $result);
75-
$this->assertStringContainsString(self::ORDER_NUMBER, $result);
7675
}
7776

7877
public function testSubjectUpdateAuditMessage(): void

0 commit comments

Comments
 (0)