Skip to content

added new spec for check recipients in offline mode - #1432

Merged
tomholub merged 5 commits into
masterfrom
test/issue-1420
Mar 23, 2022
Merged

tomholub merged 5 commits into
masterfrom
test/issue-1420

Conversation

@fcvakintos

@fcvakintos fcvakintos commented Mar 21, 2022

Copy link
Copy Markdown
Contributor

This PR contains new spec for checking recipients in offline mode, added changes for attester-endpints file(added new method for returning error)

close #1420


Tests

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@fcvakintos
fcvakintos requested a review from tomholub as a code owner March 21, 2022 14:13

@tomholub tomholub left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, please see comments for improvements

Comment thread appium/api-mocks/apis/attester/attester-endpoints.ts Outdated
Comment thread appium/api-mocks/apis/attester/attester-endpoints.ts Outdated
Comment thread appium/tests/data/index.ts Outdated
Comment thread appium/tests/specs/mock/setup/CheckRecipientsInOfflineMode.spec.ts Outdated
@fcvakintos
fcvakintos requested a review from tomholub March 22, 2022 10:23
Comment thread appium/api-mocks/apis/attester/attester-endpoints.ts Outdated
Comment thread appium/tests/data/index.ts
Comment thread appium/tests/specs/mock/setup/CheckRecipientsInOfflineMode.spec.ts

@tomholub tomholub left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Much better. Thanks!

@tomholub
tomholub enabled auto-merge (squash) March 22, 2022 16:43
@fcvakintos

Copy link
Copy Markdown
Contributor Author

@tomholub I found issue
app crash when user click on "Reply all" this email "Message with cc and multiple recipients and attachment"

Screen.Recording.2022-03-23.at.14.39.51.mov

@fcvakintos

Copy link
Copy Markdown
Contributor Author

@tomholub can i disable this test?

@fcvakintos

Copy link
Copy Markdown
Contributor Author

@tomholub ok, sure

@tomholub
tomholub merged commit 1f1f70b into master Mar 23, 2022
@tomholub
tomholub deleted the test/issue-1420 branch March 23, 2022 17:41
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.

test for using local active contact if failed to fetch

3 participants