adding helper function to create a timestamp

[flothjl]

Addresses #570

• Adds and exports a helper function to create timestamps to use in dwn messages.

[codesandbox]

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

[codecov-commenter]

Codecov Report

Merging #583 (609b77b) into main (fb08183) will decrease coverage by 0.12%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #583      +/-   ##
==========================================
- Coverage   97.81%   97.69%   -0.12%     
==========================================
  Files          66       66              
  Lines        7964     7987      +23     
  Branches     1157     1155       -2     
==========================================
+ Hits         7790     7803      +13     
- Misses        168      178      +10     
  Partials        6        6              

Files	Coverage Δ
src/index.ts	100.00% <100.00%> (ø)
src/utils/time.ts	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

[csuwildcat]

Does the method allow for calling with no parameters and returning an auto-generated timestamp too?

[flothjl]

@csuwildcat currently no. There is another helper method getCurrentTimeInHighPrecision

[LiranCohen]

@csuwildcat when you say auto-generated do you mean of the current time as @flothjl mentioned or a random timestamp for testing?

We do have a crude randomTimestamp in the TestDataGenerator class. Maybe the intent of these can be combined?

https://github.com/TBD54566975/dwn-sdk-js/blob/main/tests/utils/test-data-generator.ts#L834-L844

[csuwildcat]

I meant getting the current time in this format. I know the records create/update code does this automatically if omitted, just might be worth it exposing auto-generated/- filled timestamps if you call with no params or less than full params. Could be a waste of time though, because the most important thing is just having a function to cast values into the right format.

[LiranCohen]

@csuwildcat ah yeah, in that case I think keeping the current getCurrentTimeInHighPrecision makes more sense, that way the two different functions have clearer intents without having the optional values.

[flothjl]

@LiranCohen good call out on randomTimestamp. I updated randomTimestamp to use the new helper method.

[thehenrytsai]

🙏 🔥 🎖️