Also send session groups when logging Gatling event

[pcbue]

Description

Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

• Relevant Issues : Enhancement: log groups in perf events #1467
• Relevant PRs : (optional)
• Type of change :
  • New feature
  • Bug fix for existing feature
  • Code quality improvement
  • Addition or Improvement of tests
  • Addition or Improvement of documentation

[ptrthomas]

@pcbue thank you for this. LGTM but I'd like to summon @theathlete3141 to have a look in case this has any other impact

[theathlete3141]

I wasn't actually aware of the "groups" functionality in Gatling, but this looks right. I expect that session.groups is just an empty list if you don't make use of groups when creating the scenario.

[ptrthomas]

thanks @theathlete3141 let me loop in @sixdouglas as I just remembered he is familiar with this part of the code as well

[pcbue]

I expect that session.groups is just an empty list if you don't make use of groups when creating the scenario.

Yes, indeed, session.groups resolve to Nil (== List.empty) if not used (or no groups still encountered) in the scenario

[sixdouglas]

It seems for me too.
Didn't know Gatling group functionality either, though.