refactored roi_ranges out of the model

[ashleymeigh2]

Issue

Closes #2297

Description

roi_range removed from model
refactored presenter to no longer use roi_range from model

Testing

Updated unit tests to verify correct renaming behavior and proper exceptions for invalid operations.
Manually tested renaming ROIs in the UI to ensure changes are reflected correctly.

[coveralls]

coverage: 73.6% (-0.02%) from 73.622%
when pulling c319ca7 on 2297_remove_range_from_model
into 3f099d2 on main.

[samtygier-stfc]

Looking good. There is one test that is no longer needed.

[samtygier-stfc]

When I do a 2D binned RITS export I get

2025-01-20 15:47:41,181 [mantidimaging.gui.dialogs.async_task.task:L53] ERROR: Failed to execute task: SpectrumViewerWindowModel.save_rits_images() missing 1 required positional argument: 'roi'
Traceback (most recent call last):
  File "/home/sam/git/mantidimaging/mantidimaging/gui/dialogs/async_task/task.py", line 50, in run
    self.result = call_with_known_parameters(self.task_function, **self.kwargs)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/sam/git/mantidimaging/mantidimaging/core/utility/func_call.py", line 12, in call_with_known_parameters
    return func(**ka)
           ^^^^^^^^^^
TypeError: SpectrumViewerWindowModel.save_rits_images() missing 1 required positional argument: 'roi'

[samtygier-stfc]

Review task. Check that the binned export uses the ROI specified by the user. For example

Should output 4 files. This is currently broken on main, where it seems to be using the whole area and outputting thousands of files.

[samtygier-stfc]

Looking good. RITS export modes are working