Move Next line in QuickLoop

[mint-dewit]

About the Contributor

This pull request is posted on behalf of the BBC.

Type of Contribution

This is a: Feature Improvement

Current Behavior

Moving the next line using keyboard shortcuts or from Blueprints will ignore any QuickLoop markings.

New Behavior

Moving the next line using keyboard shortcuts or from Blueprints will keep the Next line inside of the QuickLoop bounds unless explicitly told to ignore them.

Testing

No additional unit tests have been added.

Affected areas

QuickLoop and any moveNext actions

Other Information

As reviewed by @Julusian in bbc#31

[codecov]

Codecov Report

Attention: Patch coverage is 35.76642% with 88 lines in your changes missing coverage. Please review.

Project coverage is 55.71%. Comparing base (8c5c933) to head (b87cd9f).
Report is 64 commits behind head on release52.

Files with missing lines	Patch %	Lines
...ker/src/playout/model/services/QuickLoopService.ts	4.47%	64 Missing ⚠️
...orker/src/blueprints/context/OnSetAsNextContext.ts	12.50%	7 Missing ⚠️
.../job-worker/src/blueprints/context/adlibActions.ts	12.50%	7 Missing ⚠️
...c/playout/model/implementation/PlayoutModelImpl.ts	30.00%	7 Missing ⚠️
meteor/server/api/userActions.ts	0.00%	3 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##           release52    #1354      +/-   ##
=============================================
- Coverage      56.47%   55.71%   -0.76%     
=============================================
  Files            401      384      -17     
  Lines          71984    68508    -3476     
  Branches        4551     4203     -348     
=============================================
- Hits           40650    38171    -2479     
+ Misses         31233    30204    -1029     
- Partials         101      133      +32     

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