[frontend] Change "Add teams..." dialog into "Modify teams..." and alter behaviour (#2112)

[RomuDeuxfois]

Proposed changes

• "Add teams..." controls in Inject definitions is changed to "Modify teams"
• The Modify teams dialog allows for adding or removing teams to/from inject
• Allowed team list is scoped down to the teams added to a scenario/simulation accordingly
  • If no teams were added to a scenario or simulation, no teams are available to add to injects

Related issues

• Fix Modifying targets in an inject is not working properly #2112

Checklist

• I consider the submitted work as finished
• I tested the code for its functionality
• I wrote test cases for the relevant uses case
• I added/update the relevant documentation (either on github or on notion)
• Where necessary I refactored code to improve the overall quality
• For bug fix -> I implemented a test that covers the bug

Further comments

Based on @antoinemzs work

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 35.12%. Comparing base (43019c7) to head (1e673dc).
Report is 3 commits behind head on release/1.11.0.

Files with missing lines	Patch %	Lines
...src/main/java/io/openbas/utils/InjectionUtils.java	0.00%	4 Missing ⚠️
...va/io/openbas/rest/asset/endpoint/EndpointApi.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@                Coverage Diff                @@
##             release/1.11.0    #2127   +/-   ##
=================================================
  Coverage             35.11%   35.12%           
  Complexity             1418     1418           
=================================================
  Files                   535      536    +1     
  Lines                 16187    16184    -3     
  Branches               1037     1036    -1     
=================================================
  Hits                   5684     5684           
+ Misses                10209    10206    -3     
  Partials                294      294           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[antoinemzs]

The scope looks to have increased from the issue's description, just to be clear, this solves the issue and introduces a similar behaviour for Target Assets ?

[RomuDeuxfois]

The scope looks to have increased from the issue's description, just to be clear, this solves the issue and introduces a similar behaviour for Target Assets ?

Exactly, I realized that we also had to align the behavior with assets and asset groups in order to have something consistent for the user.