You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
currently the config does not tell if a network is a local net or a remote net.
especially, a localnet can be deployed to become a remotenet.
in this case the localnet should not be used anymore.
the project structure defines 3 set of functions concerning networks::
localnet-xyz : interacts with local processes and local files - function can not be applied on a remotenet.
remotenet-xyz : interacts via SSH with remote nodes - functions can not be applied on a localnet
net-xyz: interacts with RPC only, functions can be applied to localnet and remotenets.
there is also testnet-xyz what means localnet in it's essence, but usually refers to integration tests for a specific feature.
this testnet naming pattern should be reworked, by either specifying features and restrictions that only apply to integration tests, or completly replaced with localnet.
Tests that are currently written for net suffer the inability to not spin up and shut down the 'localnet' - and are currently not suitable for automatic integration tests - that issue should be addressed.
The text was updated successfully, but these errors were encountered:
currently the config does not tell if a network is a local net or a remote net.
especially, a localnet can be deployed to become a remotenet.
in this case the localnet should not be used anymore.
the project structure defines 3 set of functions concerning networks::
there is also
testnet-xyz
what meanslocalnet
in it's essence, but usually refers to integration tests for a specific feature.this testnet naming pattern should be reworked, by either specifying features and restrictions that only apply to integration tests, or completly replaced with
localnet
.Tests that are currently written for
net
suffer the inability to not spin up and shut down the 'localnet' - and are currently not suitable for automatic integration tests - that issue should be addressed.The text was updated successfully, but these errors were encountered: