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

Added missing AddressService api call #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nemo83
Copy link

@nemo83 nemo83 commented Oct 22, 2023

Hello,

there's an interesting api call on the blockfrost api for addresses that was missing in the java lib.

I've implemented it and seems to be working.

Just not sure how to write a reliable test. Do I need to setup something on preview/preprod and test w/ a specific project id?

Thanks

@satran004
Copy link
Collaborator

@nemo83 Looks like integration tests are still pointing to LEGACY TESTNET URL which needs to be changed. But tests with PREPROD makes sense.

@nemo83
Copy link
Author

nemo83 commented Oct 24, 2023

Just tried to use preprod and no tests pass 😭
By the looks of it, tests are pointing specifically to (legacy) testnet or using legacy testnet data.

I wouldn't want this to block merging this PR.

Thoughts?

@satran004
Copy link
Collaborator

satran004 commented Oct 24, 2023

@nemo83, Let's just add a test class with preprod data for this api. But we will ignore the test run result for now.
Later we need to update all other tests to use preprod data.

@mmahut Can you please help to update the Project Id from Legacy testnet to Preprod for this repo ? Please use the same key as legacy testnet.

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