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

Cesride-wasm integration example e2e #149

Closed
wants to merge 4 commits into from

Conversation

daidoji
Copy link
Contributor

@daidoji daidoji commented Nov 29, 2023

This is a draft PR representing an e2e integration with cesride-wasm that corresponds with this PR in cesride:
WebOfTrust/cesride#155

It uses the dater implementation from cesride-wasm to execute tests in signify-ts that correspond to the test_dater() tests in keripy that have been transcribed into jest.

Note: todos are due to deviations from keripy/signify-ts calling conventions and the current cesride implementation. Also for future work is getting javascript options objects and destructuring to work so there's a better calling convention than new Dater(dts, undefined, undefined, ...

Implementation example of dater in signify-ts.

Right now it just uses a package dependency and a mixture of cesr
components from the cesride crate with some constants from signify-ts
itself.  This is just an example to show the integration works.  Also
doesn't have feature flags and the dependency is hardcoded into the
package.json.

dater tests are transcribed into jest from keripy test_dater()

Also some todos to be figured out later.
@daidoji
Copy link
Contributor Author

daidoji commented Nov 29, 2023

Note: Tests don't pass because I didn't work out the dependency issue on cesride-wasm.

@pfeairheller
Copy link
Member

What's the intention of this PR? Perhaps this work would be better on a fork if you just want a place to show how it could work.

@daidoji
Copy link
Contributor Author

daidoji commented Jan 18, 2024

@pfeairheller oh sorry, the intention was to demonstrate that cesride-wasm could be imported into signify-ts and would pass a unit test of js code proving interoperability. I'm not sure if I have the ability to create a branch on this repo but that would be fine by me if one was created and I can push this there.

@daidoji
Copy link
Contributor Author

daidoji commented Jan 29, 2024

Closed. Implementation exists on daidoji/signify-ts if future generations are interested.

@daidoji daidoji closed this Jan 29, 2024
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