[input-output-hk/cardano-sl] [DEVOPS-819] WIP blacklist testing (#2879)

parsonsmatt commented on this pull request.

> @@ -610,20 +610,6 @@ runAction wc action = do

NoOp -> pure ()

— lastAction .= action

— — Let’s print it out to JSON
— walletState <- get - - log "==================================================================" - log . decodeUtf8 . encodePretty $ diff (toJSON previousWalletState) (toJSON walletState) - log "==================================================================" - - -- increment successful actions - log "Success!" - successActions <>= [action]

Why is this removed?

> @@ -38,8 +38,8 @@ import Cardano.Wallet.Client (ClientError (..), Response (..), Servant
import Pos.Core (getCoin, mkCoin, unsafeAddCoin, unsafeSubCoin)
import qualified Pos.Wallet.Web.ClientTypes.Types as V0

-import Error
-import Types
+import Error ( WalletTestError(..), showConstr )
+import Types (WalletState, ActionProbabilities, WalletTestMode, wallets, getWeight, lastAction, successActions, transactions, addresses, walletsPass, accounts, nonGenesisWallets, actionsNum, Action (..))

For internal modules that we use heavily, adding explicit imports only adds noise IMO. This is also way over the 80 char column limit

Добавить комментарий