-
Notifications
You must be signed in to change notification settings - Fork 19
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
e2e tests for Java SDK #24
Comments
Burned 5SP in PR41 setting up CI and writing tests. Left to do is configuring maven to treat unit tests and integration tests differently, then commit |
Didn't get to this in PR42, moving to PR43 |
@gkc For testing purposes, is there an atSign that we use and are okay with publishing the keys to GitHub? |
No, we use GitHub secrets. I've got a branch with some integration tests that I need to merge which you can build on |
Will close out early this week in PR44 |
Gentle ping |
Moving to backlog until Java-using customers rev up again |
Describe the bug
no e2e tests at present
Expected behavior
Tests of sharing data between at signs, and associated notification delivery and receipt. Tests should run as GitHub action on PR create / PR branch push / merge to trunk
The text was updated successfully, but these errors were encountered: