Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Catalina tests #32

Merged
merged 27 commits into from
Feb 4, 2021
Merged

Catalina tests #32

merged 27 commits into from
Feb 4, 2021

Conversation

kyle-n
Copy link
Owner

@kyle-n kyle-n commented Jan 27, 2021

Updates test suite to run Essayist test app on Catalina-based systems since GitHub has removed Big Sur environments from public preview (#31).

  • macOS version of Essayist uses AppKit delegate lifecycle instead of SwiftUI (Big Sur-only)
  • macOS version uses storyboards to launch
  • iOS/macOS versions of Essayist now have an icon because some strange errors on tests referenced icons
  • UIKit tests use a special method that tries multiple times to launch the app, solving a Catalina simulator bug
  • testAutocorrectionTypeModifier() tests editor content instead of whole-app visual snapshots (this is much less brittle)
  • ci.yml targets macOS-latest, aka Catalina.

You do not have to worry about these changes affecting the build size of any app using Essayist. Files under /Tests are not included in builds.

@kyle-n kyle-n merged commit 51fec64 into main Feb 4, 2021
@kyle-n kyle-n deleted the catalina-tests branch February 4, 2021 22:17
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