apache / cordova-paramedic

Apache Cordova - Paramedic
https://cordova.apache.org/
Apache License 2.0
36 stars 53 forks source link

ci(gh-actions): add test strategy matrix #216

Closed erisu closed 3 years ago

erisu commented 3 years ago

Motivation, Context & Description

This PR adds the test strategy matrix:

Android:

All tests uses Java 8 except for API 30. API >=30 will use Java 11.

iOS:

Starting April 26, 2021, all iOS and iPadOS apps submitted to the App Store must be built with Xcode 12 and the iOS 14 SDK.

Because of this requirement by Apple, we will only be testing against the latest Xcode release. We will be using the latest-stable flag which excludes beta.

Additionally, since iOS versioning is slightly different, when a new version is released, we will need to manually update the configuration files.

./conf/pr/local/ios-<VERSION>.config.json

For Plugin Repos:

In general, these workflow configs should be able to be copy to each plugin with no change required.

Testing

Checklist

erisu commented 3 years ago

IF we test @nightly we would have to think about what to cover. I believe it would also need to be on a separate workflow and separate PR.

We don't want nightly tests failures preventing PRs. Maybe run as a cron job and not related to PRs or pushes.

Then there are also nightly versions of Cordova-CLI, Cordova-Platforms, and also OS platforms.

At this time, I think it is not something we should consider. I am not against the idea but don't think it is related to this PR. For sure something to investigate after getting our plugins passing again.