-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
ci: Add code integration testing to repo #1569
Merged
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
a8b5df0
ci: Add code integration testing to repo
cbaker6 280fd32
Update ci.yml
cbaker6 8ca1fbe
Update ci.yml
cbaker6 ad04c21
Update ci.yml
cbaker6 8b72d4f
Update ci.yml
cbaker6 9443a44
Update ci.yml
cbaker6 1fc4e91
fix UI tapping in tests
cbaker6 aecbd06
Merge remote-tracking branch 'refs/remotes/origin/updatedCI'
cbaker6 aafb665
fix exit button
cbaker6 ae03244
fix more catalog tests
cbaker6 11c18f1
fix catalog slider tests
cbaker6 fb12261
fix all broken and add missing tests
cbaker6 9b6e1b7
add onboarding
cbaker6 18f7eef
update ci
cbaker6 37ece6c
update CFBundleVersion
cbaker6 e3f9ed8
test bundle version
cbaker6 4c590fa
adjust build version numbers
cbaker6 45c5ec8
rename ci file
cbaker6 aeb3cb0
revert back to 14
cbaker6 8b58151
add name to ci build
cbaker6 bed3df0
revert catalog
cbaker6 ea67b2e
revert
cbaker6 b12ca27
nit
cbaker6 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Update ci.yml
- Loading branch information
commit 9443a44433fa141fb16effcb6edb97c1586f6b02
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets keep this specific for testing ResearchKit only for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want me to move all of the
ORKCatalog
test updates out of this PR? I've tested the changes locally and they all pass nowThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, keeping the catalog app as is for now would be best.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted, moved to #1570