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.

Sign up for GitHub

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

Jump to bottom

ci(android): run native android tests #18

Closed
wants to merge 118 commits into from
Closed

ci(android): run native android tests #18

wants to merge 118 commits into from

Conversation

Santi92
Copy link
Contributor

@Santi92 Santi92 commented Sep 30, 2021 •
edited by danielgomezrico
Loading

Fix #22

@cdmunoz
First commit
b5abc01
@cdmunoz
Update flutter and sdk versions
58a02fe
@cdmunoz
Add iterable dependency for Android
8bc8911
@cdmunoz
Add initialize for iterable
c6b4e2c
@cdmunoz
Add init for iterable
bc9c7b1
@cdmunoz
Add setEmail and setUserId methods
f17f061
@cdmunoz
Remove platform version example
43195c6
@cdmunoz
Add user id to example
d87dbf3
@cdmunoz
Add track event
b290d75
@cdmunoz
Update plugin unit test content
72d6bac
@cdmunoz
Add unit tests
ae4e796
@cdmunoz
Format unit tests code
47be593
@cdmunoz
Add register for push method
ce7e329
@cdmunoz
Add firebase to iterable sdk dependency
0b65d58
@cdmunoz
Refactor to receive push integration name as initialize param
f5a885d
@cdmunoz
Remove unused dependency
67c8790
@cdmunoz
Add unregister for push to sign out from iterable
7f2ff6a
@cdmunoz
Update unit tests for sign out
d817d03
@cdmunoz
Update unit tests for sign out
6bc77b5
@cdmunoz
Add firebase configuration to run example successfully
98833b9
@cdmunoz
Update plsugin README file
dc69e1f
@cdmunoz
Update README.md
c17ff15
@cdmunoz
Update README.md
45e500b
@cdmunoz
Add example README update
c704207
@cdmunoz
Update README with what is missing for plugin
b4afaad
@cdmunoz
Setup Iterable for iOS
df950fa
@cdmunoz
Add initialize method for iOS
d8ad13b
@cdmunoz
Add setEmail and setUserId for iOS
4cfc211
@cdmunoz
Add track events to iOS
8db0117
@cdmunoz
Add result to each handle case in iOS plugin part
9635965
danielgomezrico and others added 21 commits January 17, 2022 18:31
@danielgomezrico
ci(Automation): fix github action is broken for android (#38)
8d87cd7 
* Add any file to see if CI works

* Fix android builds

* Fix yaml

* Cleanup CI

* Fix concurrency setup

* Cleanup naming

* Fix yaml for build
@Santi92 @danielgomezrico
Create android.yml
fa92450
@Santi92 @danielgomezrico
a litte Change
791fdfa
@Santi92 @danielgomezrico
Refactor location file android.yml
38f5ed2
@Santi92 @danielgomezrico
Fix in syntax
f63a41b
@Santi92 @danielgomezrico
New step for navigation a gradlew
86a453a
@Santi92 @danielgomezrico
Dele
15c2d31
@Santi92 @danielgomezrico
Fix path
9094bf8
@Santi92 @danielgomezrico
Download Gradlew
8236dd3
@Santi92 @danielgomezrico
Android gradlew
b6fde1c
@danielgomezrico
feat(CI, Dart step): remove unused dart step (#20)
43c0a2e
@danielgomezrico
ci(Test Action): avoid redundant jobs (#25)
b807f6d 
* Cancel old jobs and rename the job

* Rename job name to show the specific test running
@danielgomezrico
ci(Build action): run a build for every PR (#24)
031dd8c 
* Add build action

* Cancel old jobs on every push

* Fix web is not compiling for example

* Fix working directory

* Improve naming

* Remove web build since it is not yet supported
@Santi92 @danielgomezrico
Run test
70b9513
@Santi92 @danielgomezrico
Run Only job
31f0e23
@Santi92 @danielgomezrico
Fix syntax
fcc0ae8
@danielgomezrico
Go to android folder
1cf4e88
@danielgomezrico
Try with build-root-directory for android
9652254
@danielgomezrico
Refactor ci to have flutter version fixed
4a65064
@danielgomezrico
Add gradlew script
b0f18d2
@danielgomezrico
Add missing file
fadbf24
@danielgomezrico
Copy link
Contributor

I tried some things and got no good results, for some reason the android side of the flutter plugin does not generate a build version on which we could run the tests 🤔

@danielgomezrico danielgomezrico changed the title ci(android): refactor CI setup to run tests on Android and build with fixed flutter versions ci(android): run native android tests Jan 18, 2022
@danielgomezrico danielgomezrico added the help wanted Extra attention is needed label Jan 28, 2022
@mixergtz mixergtz closed this Feb 11, 2022
@danielgomezrico
Copy link
Contributor

We are not doing an effort here to make it work so, any PRs are welcome 🤓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielgomezrico danielgomezrico danielgomezrico approved these changes

@cdmunoz cdmunoz cdmunoz approved these changes

@juansoos juansoos Awaiting requested review from juansoos

Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CI] Run unit tests for Android implementation on CI
5 participants
@Santi92 @danielgomezrico @cdmunoz @mixergtz @Gabriel-Azevedo