Skip to content

Commit

Permalink
Fix FaaS serialization and MockApmServer DTOs
Browse files Browse the repository at this point in the history
  • Loading branch information
stevejgordon committed Dec 5, 2024
1 parent 4394389 commit 51fbbea
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/Elastic.Apm/Model/Transaction.cs
Original file line number Diff line number Diff line change
Expand Up @@ -473,6 +473,7 @@ public string Name
/// <summary>
/// Contains data related to FaaS (Function as a Service) events.
/// </summary>
[JsonPropertyName("faas")]
public Faas FaaS { get; set; }

/// <summary>
Expand Down
1 change: 1 addition & 0 deletions test/Elastic.Apm.Tests.MockApmServer/FaasDto.cs
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ namespace Elastic.Apm.Tests.MockApmServer;

public class FaasDto
{
[JsonPropertyName("cold_start")]
public bool ColdStart { get; set; }

public string Execution { get; set; }
Expand Down
2 changes: 1 addition & 1 deletion test/Elastic.Apm.Tests.MockApmServer/TransactionDto.cs
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,10 @@ internal class TransactionDto : ITimedDto

public string Type { get; set; }


[JsonPropertyName("dropped_spans_stats")]
public List<DroppedSpanStatsDto> DroppedSpanStats { get; set; }

[JsonPropertyName("faas")]
public FaasDto FaaS { get; set; }

public override string ToString() => new ToStringBuilder(nameof(TransactionDto))
Expand Down

0 comments on commit 51fbbea

Please sign in to comment.