handle subexpressions for set delete

[ben-schwen]

Follows #7500
Closes #7507

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.04%. Comparing base (ba7a1b3) to head (7a73e19).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #7508   +/-   ##
=======================================
  Coverage   99.04%   99.04%           
=======================================
  Files          87       87           
  Lines       16704    16706    +2     
=======================================
+ Hits        16544    16546    +2     
  Misses        160      160           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[aitap]

So it does happen after all. LGTM, thanks!

[ben-schwen]

So it does happen after all. LGTM, thanks!

Yes and R6 classes is a common enough case to not ignore it!

[github-actions]

No obvious timing issues in HEAD=set_delete_subexpression

Generated via commit 7a73e19

Download link for the artifact containing the test results: ↓ atime-results.zip

Task	Duration
R setup and installing dependencies	5 minutes and 9 seconds
Installing different package versions	11 minutes and 49 seconds
Running and plotting the test cases	5 minutes and 9 seconds