Skip to content

Refactor DownloadInteractor#79

Merged
marciniwanicki merged 3 commits into
mainfrom
mi/refactor
Sep 8, 2024
Merged

Refactor DownloadInteractor#79
marciniwanicki merged 3 commits into
mainfrom
mi/refactor

Conversation

@marciniwanicki
Copy link
Copy Markdown
Contributor

@marciniwanicki marciniwanicki commented Sep 8, 2024

  • Refactored the interactor to be able to unit test it
  • Introduced new RunLoop type that is capable of holding the process alive and allows to terminate it gracefully without relying on system exit()

Test Plan:

  • Ensure all CI checks pass
  • Run curie download -p file.ipsw locally, verify the download process works without any issues and can be interrupted

@marciniwanicki marciniwanicki merged commit e33fba9 into main Sep 8, 2024
@marciniwanicki marciniwanicki deleted the mi/refactor branch September 8, 2024 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant