[Telemetry Tables] Don't mutate configuration if object is not able to be persisted

[jvigliotta]

Closes #7625
VIPEROMCT-428

Describe your changes:

All Submissions:

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• Is this a notable change that will require a special callout in the release notes? For example, will this break compatibility with existing APIs or projects that consume these plugins?

Author Checklist

• Changes address original issue?
• Tests included and/or updated with changes?
• Has this been smoke tested?
• Have you associated this PR with a type: label? Note: this is not necessarily the same as the original issue.
• Have you associated a milestone with this PR? Note: leave blank if unsure.
• Testing instructions included in associated issue OR is this a dependency/testcase change?

Reviewer Checklist

• Changes appear to address issue?
• Reviewer has tested changes by following the provided instructions?
• Changes appear not to be breaking changes?
• Appropriate automated tests included?
• Code style and in-line documentation are appropriate?

[codecov]

Codecov Report

Attention: Patch coverage is 75.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 56.16%. Comparing base (b2b0837) to head (f72ddf7).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7626      +/-   ##
==========================================
- Coverage   56.58%   56.16%   -0.43%     
==========================================
  Files         672      672              
  Lines       27128    27131       +3     
  Branches     2635     2635              
==========================================
- Hits        15351    15238     -113     
- Misses      11450    11566     +116     
  Partials      327      327              

Flag	Coverage Δ
e2e-full	23.57% <0.00%> (-24.14%)	⬇️
e2e-stable	59.93% <50.00%> (+0.03%)	⬆️
unit	49.11% <75.00%> (+<0.01%)	⬆️

Files	Coverage Δ
...gins/telemetryTable/TelemetryTableConfiguration.js	63.75% <75.00%> (+0.11%)	⬆️

... and 11 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b2b0837...f72ddf7. Read the comment docs.

[davetsay]

Good change.
I think this is worthy of an e2e test, but the change looks good.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 75.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 56.32%. Comparing base (311ad0b) to head (6d53e85).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7626      +/-   ##
==========================================
- Coverage   56.64%   56.32%   -0.32%     
==========================================
  Files         672      672              
  Lines       27122    27125       +3     
  Branches     2632     2632              
==========================================
- Hits        15362    15277      -85     
- Misses      11432    11520      +88     
  Partials      328      328              

Flag	Coverage Δ
e2e-full	23.63% <0.00%> (-18.41%)	⬇️
e2e-stable	60.35% <50.00%> (-0.06%)	⬇️
unit	49.08% <75.00%> (-0.05%)	⬇️

Files	Coverage Δ
...gins/telemetryTable/TelemetryTableConfiguration.js	63.75% <75.00%> (+0.11%)	⬆️

... and 12 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 311ad0b...6d53e85. Read the comment docs.

[ozyx]

Test on openmct-yamcs side