adding tests for changes in JP-3311 and minor fixes

[penaguerrero]

Resolves JP-3311

Closes #

This PR adds tests for changes made in https://github.com/spacetelescope/jwst/pull/8464

Checklist for PR authors (skip items if you don't have permissions or they are not applicable)

• added entry in CHANGES.rst within the relevant release section
• updated or added relevant tests
• updated relevant documentation
• added relevant milestone
• added relevant label(s)
• ran regression tests, post a link to the Jenkins job below.
  How to run regression tests on a PR
• All comments are resolved
• Make sure the JIRA ticket is resolved properly

[penaguerrero]

running reg test at https://plwishmaster.stsci.edu:8081/job/RT/job/JWST-Developers-Pull-Requests/1438/ -> cancelled

[penaguerrero]

reg test running: https://plwishmaster.stsci.edu:8081/job/RT/job/JWST-Developers-Pull-Requests/1439/

[codecov]

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 57.93%. Comparing base (781e0e0) to head (1a54d02).
Report is 294 commits behind head on master.

Files	Patch %	Lines
jwst/outlier_detection/outlier_detection_spec.py	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #8481      +/-   ##
==========================================
- Coverage   57.93%   57.93%   -0.01%     
==========================================
  Files         387      387              
  Lines       38839    38852      +13     
==========================================
+ Hits        22502    22508       +6     
- Misses      16337    16344       +7     

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

[emolter]

Code changes look good to me! Can you run a new regtest on Jenkins, at least for the tests that were previously failing (test_miri_lrs_slit_spec3 and test_nirspec_fs_spec3)?

[hbushouse]

Code changes seem fine to me and latest regtest run showed only 1 unrelated failure. So I think this is good to go.