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

Added the milestone#3 of native tesseract libraries #1089

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

dileping
Copy link
Contributor

Milestone Delivery Checklist

  • The milestone-delivery-template.md has been copied and updated.
  • This pull request is being made by the same account as the accepted application.
  • I have disclosed any and all sources of reused code in the submitted repositories and have done my due diligence to meet its license requirements.
  • In case of acceptance, an invoice must be submitted and the payment will be transferred to the Polkadot/fiat account provided in the application.
  • The delivery is according to the Guidelines for Milestone Deliverables.

Link to the application pull request: w3f/Grants-Program#1856

@keeganquigley keeganquigley self-assigned this Jan 10, 2024
@keeganquigley
Copy link
Contributor

Sorry for the delay @dileping many of us were on vacation over the holidays. I'm starting this evaluation now!

Copy link
Contributor

@keeganquigley keeganquigley left a comment

Choose a reason for hiding this comment

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

Thanks for the delivery @dileping everything is working well, I have three questions:

  • Are there any Kotlin unit tests you can point me to or otherwise how can I check code coverage?
  • The delivery states that "the rest of the libraries in siblings directories (both rust and java), even though are essential, are mostly private and contain code that serve the public APIs to function." Can you further explain what this means? Our current policy states that "all code produced as part of a grant must be open-sourced, and it must also not rely on closed-source software for full functionality."
  • The delivery states that the article will be emailed privately. Could you send this to [email protected]? Thanks!

@dileping
Copy link
Contributor Author

Hi @keeganquigley,

Thanks for starting the review process. Please see my comments below.

Thanks for the delivery @dileping everything is working well, I have three questions:

  • Are there any Kotlin unit tests you can point me to or otherwise how can I check code coverage?
    The tests are rather done in the form of applications due to the nature of the software (the communication between Apps in Android can hardly be tested with unit tests). As for the test applications, here are some:
  1. Covers client APIs with Test protocol: https://github.com/tesseract-one/Tesseract.android/tree/main/examples/native/app
  2. Covers wallet APIs with Test protocol: https://github.com/tesseract-one/Tesseract.android/tree/main/examples/native/wallet
  3. Covers client APIs with the Substrate protocol: https://github.com/tesseract-one/polkachat.kotlin
  4. Covers wallet APIs with the Substrate protocol: https://github.com/tesseract-one/dev-wallet.kotlin

Also, I'd suggest checking out the following:

  1. https://github.com/tesseract-one/Tesseract.android/tree/main/java/client-transport-ipc
  2. https://github.com/tesseract-one/Tesseract.android/tree/main/java/service-transport-ipc
  3. https://github.com/tesseract-one/Tesseract.android/tree/main/java/common-transport-ipc
    The above uses the API for transport development and implements Android IPC using the new APIs.
  • The delivery states that "the rest of the libraries in siblings directories (both rust and java), even though are essential, are mostly private and contain code that serve the public APIs to function." Can you further explain what this means? Our current policy states that "all code produced as part of a grant must be open-sourced, and it must also not rely on closed-source software for full functionality."

Private means that they are "private" from the perspective of how the code is structured. Like a private function vs a public function. The code is fully open source, though, and is licensed under Apache 2.0. Please check it within the mentioned directories.

  • The delivery states that the article will be emailed privately. Could you send this to [email protected]? Thanks!
    Of course, I will take care of it ASAP.

@keeganquigley
Copy link
Contributor

Ah thanks for the explanations @dileping that makes sense. I was just concerned when you mentioned "private" so wanted to double-check. You can find my evaluation here. Everything works as expected. Once the article is sent so I can check it over, we should be good to go to accept the delivery :)

@keeganquigley keeganquigley merged commit 38c5ef0 into w3f:master Jan 17, 2024
6 checks passed
Copy link

🪙 Please fill out the invoice form in order to initiate the payment process. Thank you!

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.

2 participants