dedis / cothority_template Goto Github PK

Onet-structures changed a bit - reflect that in latest code.

Follow latest structure as proposed in the new skipchain-directory.

Improve the app a bit so that the documentation can profit from it.

The simulation-examples have some shortcomings:

• names Simulation and simulation are too close
• service-simulation doesn't implement Node
• Node should show how to retrieve the actual index that is about to be initialized
• simulation doesn't have a test

Closes #5
Closes #6

the previous php-call possible with whatsmyip.org does not work anymore. I propose we replace it with a simple service on dedis.ch that listens on a port. If a host sends a number to that port, the service will try to connect back to the host at the sent port-#. This avoids abuse of the system.

Currently, to run the simulation as a test you write

simul.Start("mySimul.toml")

while running it as a real simulation you NEED to do (otherwise only run localhost):

simul.Start()

The real simulation should accept toml file as well, or at least display an error message specifying that something's wrong. For example, no flags were working at all when using simul.Start("tomlFile") as a real simulation.

CreateProtocol-fix

Without the Suite field in a simulation file, CreateRoster fails.

With the move of the libraries to dedis/onet, change cothority_template to use onet instead of cothority.

I updated cothority to v3.3.1 today, and the following test failed:

--- FAIL: TestKeyValue_Invoke (2.41s)
    keyvalue_test.go:248: 
        	Error Trace:	keyvalue_test.go:248
        	            				keyvalue_test.go:101
        	Error:      	Expected nil, but got: tcp://127.0.0.1:57879 Contract keyValue got - instruction:
        	            	-- hash: 8ff3cc677f90e9d330a0433c0392c25d30feb580c150d7a0a88ddbced31862ef
        	            	-- instID: 25ebbbe71e9efceb2b078bbe2d6ec673482de74fc9310a20bcbb4b3f5e55f36f
        	            	-- action: invoke:keyValue.update
        	            	-- identities: [ed25519:d43007437e8330e6686374ea12ae25938faec1db2c175d3676e3de68b049a61c]
        	            	-- counters: [2]
        	            	-- signatures: 1
        	            	-error in getting constructor: keyValue and returned error: instruction verification failed: expression evaluated to false
        	Test:       	TestKeyValue_Invoke

Problem: Service-tests are incoherent:

• no test for api.go
• test in service_test.go is in fact a test for api.go

TODO:

• move test from service_test.go into `api_test.go
• add test in service_test.go

The template-protocol is so bad that it doesn't finish with more than 2 nodes :(

When attempting to build the project or run any of the setup scripts.
Example:
Running the command ./run_conode.sh local 3 the program responds with the error
omniledger/keyvalue.go not enough arguments in call to service.NewStateChange().....
I am greeted with the same response when running the command go get ./...