Skip to content

Conversation

@CapDev12
Copy link
Member

@CapDev12 CapDev12 commented Sep 2, 2019

Core - it-test: Fork resolving on downloading

@CapDev12 CapDev12 changed the title Two integration tests EC-749 Two integration tests Sep 2, 2019
@CapDev12 CapDev12 changed the title EC-749 Two integration tests EC-749 Fork resolving on downloading Sep 2, 2019
@CapDev12 CapDev12 changed the title EC-749 Fork resolving on downloading EC-749 Core - it-test: Fork resolving on downloading Sep 2, 2019
capdev added 4 commits September 2, 2019 18:37
add connectOnlyWithKnownPeers to Config
add connectOnlyWithKnownPeers to Config
add test Late node should sync with the first of two nodes
add test All nodes should go to first chain
@CapDev12 CapDev12 changed the title EC-749 Core - it-test: Fork resolving on downloading EC-749 Sep 3, 2019
import encry.EncryApp
import encry.settings.RESTApiSettings

case class NodeApiRoute(restApiSettings: RESTApiSettings)(implicit val context: ActorRefFactory)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure, that creating such a route is a good idea

Copy link
Member Author

@CapDev12 CapDev12 Oct 1, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is necessary for the correct node restart
So as not to damage the database files

We can refuse the route and take a chance

@GRYE
Copy link
Contributor

GRYE commented Sep 30, 2019

Снимок экрана 2019-09-30 в 15 35 36

Tests does not run when I try to execute them

@CapDev12
Copy link
Member Author

CapDev12 commented Oct 1, 2019

Tests does not run when I try to execute them

If you run tests selectively, you must first run
$>sbt docker
in project it
for container image rebuild

@GRYE
Copy link
Contributor

GRYE commented Oct 1, 2019

So, I tried to run it as follows:

project it
clean
compile
docker
testOnly *NodeRestartTest

But it fails anyway

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