Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #8 fixes #4 fixes #5 - Implementing .NET test suite for libskycoin-dotnet #11

Merged
merged 77 commits into from
Dec 10, 2018

Conversation

m4yk3ldev
Copy link

@m4yk3ldev m4yk3ldev commented Oct 16, 2018

Fixes #8
Fixes #5
Fixes #4

Changes:

  • Tests for cipher.
  • Tests for coin
  • Tests for fee
  • Tests for droplet
  • Tests for param

stdevEclipse and others added 30 commits August 4, 2018 19:15
…ailures: 0, Inconclusive: 0, Time: 0,0694558 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
… 0, Failures: 0, Inconclusive: 0, Time: 6,0212747 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…, Failures: 0, Inconclusive: 0, Time: 12,9174782 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…rors: 0, Failures: 0, Inconclusive: 0, Time: 12,9590975 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…y1305 Tests run: 31, Errors: 0, Failures: 0, Inconclusive: 0, Time: 16,2105772 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…Failures: 0, Inconclusive: 0, Time: 12,9019926 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…Failures: 0, Inconclusive: 0, Time: 12,9913398 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…ummary_Handle Tests run: 37, Errors: 1, Failures: 0, Inconclusive: 0, Time: 13,2049951 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
Maykel Arias Torres added 9 commits October 16, 2018 20:05
…Errors: 0, Failures: 0, Inconclusive: 0, Time: 18,7982045 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…5, Errors: 0, Failures: 0, Inconclusive: 0, Time: 17,0322507 seconds
… Failures: 0, Inconclusive: 0, Time: 17,4740162 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…, Inconclusive: 0, Time: 15,9607057 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…s: 0, Inconclusive: 0, Time: 18,8853374 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
…res: 0, Inconclusive: 0, Time: 15,39145 seconds

  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
@olemis
Copy link
Collaborator

olemis commented Nov 28, 2018

Is this missing tests for param ?

}
}

// %rename(SKY_params_GetDistributionAddresses) CSharp_skycoin_SKY_params_GetDistributionAddresses;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why comments ?

Copy link
Author

Choose a reason for hiding this comment

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

I already deleted it

@@ -0,0 +1,1233 @@
/* Code generated by cmd/cgo; DO NOT EDIT. */
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use the header file in skycoin/skycoin submodule .

Copy link
Collaborator

Choose a reason for hiding this comment

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

... and remove this file .

Copy link
Author

Choose a reason for hiding this comment

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

Copy link
Author

Choose a reason for hiding this comment

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

gopath/src/github.com/skycoin/skycoin/include/libskycoin.h:465: Error: Syntax error - possibly a missing semicolon.

Copy link
Author

Choose a reason for hiding this comment

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

typedef float _Complex GoComplex64;
typedef double _Complex GoComplex128;

@@ -0,0 +1,56 @@

Copy link
Collaborator

Choose a reason for hiding this comment

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

Use the definitions in skycoin/skycoin submodule , and delete this file .

Copy link
Author

Choose a reason for hiding this comment

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

I already deleted it

@olemis olemis changed the title Fixes #8 Implementing .NET test suite for libskycoin-dotnet Fixes #8 fixes #4 - Implementing .NET test suite for libskycoin-dotnet Nov 30, 2018
@olemis olemis changed the title Fixes #8 fixes #4 - Implementing .NET test suite for libskycoin-dotnet Fixes #8 fixes #4 fixes #5 - Implementing .NET test suite for libskycoin-dotnet Nov 30, 2018
@olemis olemis merged commit ad8aa23 into develop Dec 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants