sweetpad icon indicating copy to clipboard operation
sweetpad copied to clipboard

Full integration with test pane

Open rpedretti opened this issue 1 year ago • 4 comments

This PR adds:

  • Read the test plan for the test build scheme and add to test explorer
  • Handle file change, creation and deletion to update the test explorer
  • Fix the commands to run the tests
  • The load should handle both SPM and non-SPM since the tests are reading from the test plan
  • Remove overall test result since we update all from logs

This pr fixes:

  • Getting code blocks from class

To do:

  • Mark tests as skipped if they are skipped on test plan

rpedretti avatar Nov 04 '24 17:11 rpedretti

😍 thanks for this pr. I need some time to review and test all changes, but at the first glance looks pretty good

hyzyla avatar Nov 04 '24 20:11 hyzyla

Any movement on this?

cameroncooke avatar Nov 23 '24 17:11 cameroncooke

Where is this at.. can we get it merged if we fix the conflicts?

bryan1anderson avatar Jan 24 '25 16:01 bryan1anderson

@hyzyla I am currently also looking into testing improvements, would be good to know what the plan is with this PR. I think it's a good approach and would be nice to get it merged, but I think we need to combine it with other approaches. I tested it with a complex project of mine with multiple targets, SPM modules, etc. and it didn't detect the tests properly. I can look into it if it can be fixed, but I think it's because autocreated test plans are not saved to disk and we cannot read them in this way.

I am currently looking into an approach using xcodebuild --enumerate-tests, maybe combining both makes sense. I think it will take a bit until I have something mergeable, but I'm just not sure if I should open a second PR about tests now or not.

BlackWolf avatar May 13 '25 08:05 BlackWolf