Skip to content
This repository has been archived by the owner on Jan 1, 2025. It is now read-only.

WIP add test to ensure wallet will start up with pluto instances #82

Closed
wants to merge 1 commit into from

Conversation

mixmix
Copy link
Collaborator

@mixmix mixmix commented Feb 1, 2024

I'm getting up to speed again (slooow).

The problem I saw was that the SDK was not starting up with my pluto instance. It was complaining about getAllMediators not being a function on this.pluto somewhere deep in the SDK ...

I thought it would be good to have an integration test that checks that these pluto instances will in fact play well with SDK. There's some tests which check you can instantiate, but we need tests which check that the SDK will start()

This is a sketch... I'm struggling with some Typescript here

@mixmix mixmix requested a review from elribonazo February 1, 2024 23:36
@elribonazo
Copy link
Collaborator

@mixmix great!!

TestSuite though is the suite that all the storages should pass and it should be agnostic to the SDK. I would propose instead of placing this test inside the test-suite to place this test inside database tests.

There in Database, we do have the SDK tests with the SDK schemas, models and queries.

Hope this helps clarify :) u will not have this import issues either in Database because it already has all what it needs.

@elribonazo elribonazo closed this Jun 15, 2024
@mixmix mixmix deleted the integration_test branch June 16, 2024 22:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants