Skip to content

Added method which converts signatureDate to UTC timezone#1344

Merged
sosnovsky merged 16 commits intomasterfrom
tests/issue-1329
Jan 28, 2022
Merged

Added method which converts signatureDate to UTC timezone#1344
sosnovsky merged 16 commits intomasterfrom
tests/issue-1329

Conversation

@fcvakintos
Copy link
Contributor

This PR contains changes for checkSignatureDateValue method, convert date to UTC timezone
close #1329


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 sosnovsky as a code owner January 27, 2022 15:20

await mockApi.withMockedApis(async () => {
const times = await driver.getDeviceTime();
console.log('times', times);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it needed or was just for debug purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, i will remove it, sorry

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sosnovsky can we skip test run after this update?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Why skip it? I think it should pass without any issues

@sosnovsky sosnovsky enabled auto-merge (squash) January 28, 2022 09:20
@sosnovsky sosnovsky merged commit 2409286 into master Jan 28, 2022
@sosnovsky sosnovsky deleted the tests/issue-1329 branch January 28, 2022 10:11
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.

set custom timezone for simulator in ui tests

3 participants