Skip to content

Update unit tests websocket#2468

Merged
thewhaleking merged 4 commits intostagingfrom
fix/thewhaleking/update-unit-tests-websocket
Nov 25, 2024
Merged

Update unit tests websocket#2468
thewhaleking merged 4 commits intostagingfrom
fix/thewhaleking/update-unit-tests-websocket

Conversation

@thewhaleking
Copy link
Contributor

Updates unit tests to use the mocked websocket that is now used for subtensor instantiation.

@thewhaleking thewhaleking force-pushed the fix/thewhaleking/update-unit-tests-websocket branch from 31d6829 to cafba0c Compare November 25, 2024 13:51
@basfroman
Copy link
Collaborator

I've tested this changes and them are solving the real-chain-call problem partially. We still have 39 unit-tests broken without the connection.
image
But I believe those tests are old one and doesn't related with websockt update.
Anyway we need to create the issue for to fix this behavior.

@thewhaleking thewhaleking merged commit e147653 into staging Nov 25, 2024
@thewhaleking thewhaleking deleted the fix/thewhaleking/update-unit-tests-websocket branch November 25, 2024 20:03
@ibraheem-abe ibraheem-abe mentioned this pull request Nov 26, 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