Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Detect projects before running multi_scan #363

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tejassharma96
Copy link

@tejassharma96 tejassharma96 commented Nov 15, 2021

Checklist

  • I've run bundle exec rake from the root directory to see all new and existing tests pass
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.

Motivation and Context

Fixes #362

Description

Run detect_projects on the options before starting multi_scan

@tejassharma96
Copy link
Author

@lyndsey-ferguson this does cause all the multi_scan_manager/runner tests to fail since it stops at each test and says "Couldn't automatically detect the project file, please provide a path:". Any ideas on a way to update those to support this?

@tejassharma96
Copy link
Author

I updated each test in that folder with the path to the AtomicBoy workspace. This appears to be how they do it when testing scan in fastlane too: https://github.com/fastlane/fastlane/blob/master/scan/spec/runner_spec.rb

This fixed the test issues.

@tejassharma96
Copy link
Author

Would love to get some eyes on this if possible

@lyndsey-ferguson
Copy link
Owner

Hello @tejassharma96 , I want to apologize that I have not responded to this PR. A family health issue is requiring me to narrow my focus on the essentials and I don't have the time I need to focus on the plugin more than a minute. If you can get some other contributors to this project to review this, I would consider merging it.

@lyndsey-ferguson lyndsey-ferguson added the needs more analysis This issue or PR needs more analysis as it may not align with intent of plugin label Jul 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs more analysis This issue or PR needs more analysis as it may not align with intent of plugin
Projects
test_center_µ
  
To do
Development

Successfully merging this pull request may close these issues.

Trying to run with testplans fails unless you specify a workspace
2 participants