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

feat: tests for android & ios #23

Merged
merged 42 commits into from
Aug 26, 2024
Merged

feat: tests for android & ios #23

merged 42 commits into from
Aug 26, 2024

Conversation

mhochsto
Copy link
Contributor

No description provided.

mhochsto and others added 18 commits August 2, 2024 08:29
@mhochsto
chore: initial test setup
54ea0eb
@mhochsto
chore: added test cases for web and Kotlin Crypto Module
a36b677
@mhochsto @0x7061
chore: initial test setup
7cba9f1
@mhochsto @0x7061
chore: added test cases for web and Kotlin Crypto Module
a62e001
@0x7061
chore: fix linting, type errors
8eb07d0
@axi92
build: add test execution in workflows
4446c12
@mhochsto
chore: added crypto ios tests
1192749
@mhochsto
Merge branch 'feature/tests' of github.com:evva-sfw/abrevva-react-nat…
fa5323b 
…ive into feature/tests

# Conflicts:
#	src/index.test.tsx
@mhochsto
chore: added ble tests for swift
0f41299
@mhochsto
chore: added some testcases for android ble
26f7cf3
@axi92
chore: update README
0ab52df
@mhochsto
Update README.md
b8a5521 
We don"t need any sources anymore :)
@mhochsto
chore: changed function names to non-backtick-names
b18baa4
@mhochsto
Merge commit main into feature/tests
f5aa33d
@mhochsto
chore: resolved issues from previous merge
e0c8495
@mhochsto
chore: added startNotification() testcase
574d330
@mhochsto
chore: README.md from main
b050077
@mhochsto
chore: merged latest changes from main
9848ff5
@mhochsto mhochsto changed the title feature: tests for ios & android feat: tests for ios & android Aug 14, 2024
@mhochsto mhochsto requested review from 0x7061 and axi92 August 14, 2024 09:02
0x7061
0x7061 previously approved these changes Aug 14, 2024
View reviewed changes
@axi92
Copy link
Member

axi92 commented Aug 14, 2024 •
edited
Loading

@mhochsto do you know how we could execute the swift tests via cli?
We already test the javascript context with yarn test and kotlin with the android build.
But iOS is not executed yet in the workflows.
Would be nice to have them executed as well.

axi92
axi92 requested changes Aug 14, 2024
View reviewed changes
Copy link
Member

@axi92 axi92 left a comment •
edited by 0x7061
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO:

  • swift test execution in workflow

@axi92 axi92 assigned axi92 and mhochsto and unassigned axi92 Aug 19, 2024
axi92
axi92 reviewed Aug 23, 2024
View reviewed changes
@0x7061 0x7061 assigned 0x7061 and unassigned mhochsto Aug 23, 2024
0x7061
0x7061 previously approved these changes Aug 23, 2024
View reviewed changes
@axi92 axi92 requested a review from 0x7061 August 25, 2024 08:22
@0x7061 0x7061 merged commit 99cadcd into main Aug 26, 2024
9 checks passed
@0x7061 0x7061 deleted the feature/tests branch August 26, 2024 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@axi92 axi92 axi92 approved these changes

@0x7061 0x7061 0x7061 approved these changes

Assignees

@0x7061 0x7061

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mhochsto @axi92 @0x7061