Skip to content

Commit dd5b14c

Browse files
authored
Fix mis-named property in results analyzer run summary. (#201)
The request count only includes failed requests.
1 parent 071a532 commit dd5b14c

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

src/ResultsAnalyzer/Analyze/Main.fs

+1-2
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,7 @@ let main (args:AnalyzeArgs) =
123123
let runDataMap = failedByResponseCode |> Map.ofSeq
124124
let runSummary =
125125
{
126-
RunSummary.requestsCount = failedStatusRequestResponsePairs |> Seq.length
127-
sequencesCount = 0
126+
RunSummary.failedRequestsCount = failedStatusRequestResponsePairs |> Seq.length
128127
RunSummary.bugCount = match runDataMap |> Map.tryFind BugResponseCode with
129128
| Some x -> x |> Seq.length
130129
| None -> 0

src/ResultsAnalyzer/Analyze/Types.fs

+2-5
Original file line numberDiff line numberDiff line change
@@ -63,11 +63,8 @@ type RequestExecutionSummary =
6363

6464
type RunSummary =
6565
{
66-
/// Total number of executed requests
67-
requestsCount : int
68-
69-
/// Total number of executed sequences
70-
sequencesCount : int
66+
/// Total number of requests that were classified as a failure (e.g. 40x or 50x)
67+
failedRequestsCount : int
7168

7269
/// The number of bugs found. Note: for now, this will be
7370
/// the same as the number of 'BugResponseCode' errors above.

0 commit comments

Comments
 (0)