merge with development #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* RocksDbRepository reach 42% cover * Rename abstract class ICrypto to Crypto. I'm sure this was once an interface but it's not anymore, threfore should not carry the prefix "I" that identify the interfaces. * fix some merge problems * Increase Server tests coverage to 94%. Black listed peers try to connect test * fix merge issues * * Remove protected fields from TestBase and replace with overload usage of the method RandomInt. * Small clean up on the UtCrypto * Create UtBlockchain and create first tests. more tests will be added soon! * fix issues from the merge * Adding testing information to the team Contributing page. * Small CleanUp the test class with properties that aren't been used anymore. * UtBlockChain: InitializeBlockchain_IsGenesisBlock_InitializeComplete * Firsts UtBlockProcessor tests * AddBlock tests added * * UtBlockProcessor: Run test * Fix an inverted if logic. The code only enter in the While loop if the cancellation was requested and should be the other way around. * Cleanup * Replace BlockProcessor dependency from Prompt and replace with IBlockPool * Remove unused properties from BlockProcessor * ContainsBlock method in BlockProcessor signature changed to Private because is not used anywhere else but internally * UtBlockProcessor - Loop test: When the received block is not the expected next. * Remove comment * Remove async modifier from methods that aren't implemented * . * Merge branch 'development' of https://github.com/aboimpinto/neo-sharp into development * Add test class, #301 * Write tests for Add method. #301 * Refactor the Block / BlockHeader in Signed and Unsigned objects (mutable and immutable). The intenstion of this refactor is to bring better readability of the code and remove logic code from the DTO. * Implement the UpdateHash methods in the creation of the Signed<object> * Signed and Unsigned strucuture return GenenisBlock * Draft of the BinaryCustomSeralization of TransactionBase * Revert "Remove stack logs (#321)" This reverts commit 34c676a. * Add BlockOperationsManager and WitnessOperationManager that will be resposable to Sign and Verify the block and the witness * Initial work, removing logic from the model objects and add it into Witness. Transaction, BlockHeader and Block OperationManager classes. In this classes the verify method will be implemented. #306, #307 * Cleanup and remove code that was not merged. * regreassion test for the case where the current block is null. This happen when the node start without genesis block. * Remove dependencies from BlockProcessor and remove BlockHeader references. BlockProcessor use only Block object. #350, #351 * now the Persist consider the genesis block for the case of start a node in a clean environment
…ist (#414) - Reviewed all outstanding TODO markers in task list and added associated github issue and referenced the associated issue number inline.
* Add entire containers folder * Add entire containers folder * init * corrected the build command * Documented each Dockerfile * Delete ttt * Deleted wrong folder * Code re-write, to reduce complexity using just one Dockerfile. Making possible to compile the Node using source from master or development branch * Removing deprecated folder * Removing deprecated folder * Added travis to build a docker image * adding continer image build * correcting Dockerfile path * pushing Docker container to registry * pushing Docker container to registry * pushing Docker container to registry * correcting auth env variables for docker registry * correcting auth env variables for docker registry * deleted wrong travis file * changed default first command to be run after container starts * changed default folder for builder stage * build an SDK image and also push to registry * build an SDK and a runtime image and also push to registry * build is not builder tag :) * set env vars * run coverage tests inside SDK container * corrected bug in docker run * corrected bug docker run twice * escape * bash command * remove ruby language * add script * new travis standard build stages * new travis standard build stages * new travis standard build stages * all tags and branched to deploy scripts * define branch deploy * all_branches * bug bash -c * remove dup deploy * use --password-stdin for security reasons * changed instructions to update with CoZ official docker registry * changed docker hub to cityofzion for origin deploy * docker hub env REPO commented * Corrected break lines * travis conditional * travis conditional1 * travis conditional * travis conditional * minor fix with repo push * Revert "minor fix with repo push" This reverts commit baa376e. * insert condition REPO to not build if it's a forked repo * fix bug, no $ * Condition to run only on CityofZion/neo-sharp repo * docker conmpose to run node with neo-private-netwwork * fix peers to point to default neo-privnet peers * Integrate local-faucet in the multi-container ecosystem * docker login and push * More verbose and aligned
* Add entire containers folder * Add entire containers folder * init * corrected the build command * Documented each Dockerfile * Delete ttt * Deleted wrong folder * Code re-write, to reduce complexity using just one Dockerfile. Making possible to compile the Node using source from master or development branch * Removing deprecated folder * Removing deprecated folder * Added travis to build a docker image * adding continer image build * correcting Dockerfile path * pushing Docker container to registry * pushing Docker container to registry * pushing Docker container to registry * correcting auth env variables for docker registry * correcting auth env variables for docker registry * deleted wrong travis file * changed default first command to be run after container starts * changed default folder for builder stage * build an SDK image and also push to registry * build an SDK and a runtime image and also push to registry * build is not builder tag :) * set env vars * run coverage tests inside SDK container * corrected bug in docker run * corrected bug docker run twice * escape * bash command * remove ruby language * add script * new travis standard build stages * new travis standard build stages * new travis standard build stages * all tags and branched to deploy scripts * define branch deploy * all_branches * bug bash -c * remove dup deploy * use --password-stdin for security reasons * changed instructions to update with CoZ official docker registry * changed docker hub to cityofzion for origin deploy * docker hub env REPO commented * Corrected break lines * travis conditional * travis conditional1 * travis conditional * travis conditional * minor fix with repo push * Revert "minor fix with repo push" This reverts commit baa376e. * insert condition REPO to not build if it's a forked repo * fix bug, no $ * Condition to run only on CityofZion/neo-sharp repo * docker conmpose to run node with neo-private-netwwork * fix peers to point to default neo-privnet peers * Integrate local-faucet in the multi-container ecosystem * docker login and push * More verbose and aligned * correct env vars decoding syntax errors
* Add entire containers folder * Add entire containers folder * init * corrected the build command * Documented each Dockerfile * Delete ttt * Deleted wrong folder * Code re-write, to reduce complexity using just one Dockerfile. Making possible to compile the Node using source from master or development branch * Removing deprecated folder * Removing deprecated folder * Added travis to build a docker image * adding continer image build * correcting Dockerfile path * pushing Docker container to registry * pushing Docker container to registry * pushing Docker container to registry * correcting auth env variables for docker registry * correcting auth env variables for docker registry * deleted wrong travis file * changed default first command to be run after container starts * changed default folder for builder stage * build an SDK image and also push to registry * build an SDK and a runtime image and also push to registry * build is not builder tag :) * set env vars * run coverage tests inside SDK container * corrected bug in docker run * corrected bug docker run twice * escape * bash command * remove ruby language * add script * new travis standard build stages * new travis standard build stages * new travis standard build stages * all tags and branched to deploy scripts * define branch deploy * all_branches * bug bash -c * remove dup deploy * use --password-stdin for security reasons * changed instructions to update with CoZ official docker registry * changed docker hub to cityofzion for origin deploy * docker hub env REPO commented * Corrected break lines * travis conditional * travis conditional1 * travis conditional * travis conditional * minor fix with repo push * Revert "minor fix with repo push" This reverts commit baa376e. * insert condition REPO to not build if it's a forked repo * fix bug, no $ * Condition to run only on CityofZion/neo-sharp repo * docker conmpose to run node with neo-private-netwwork * fix peers to point to default neo-privnet peers * Integrate local-faucet in the multi-container ecosystem * Downgrade travis dotnet version to 2.1.300 * change env vars
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.