Skip to content

Conversation

@bepro-bot
Copy link
Contributor

Describe the feature or project
Develop the unit tests

Bounty
50k BEPRO

Who are we
BetBlock

sgoia added 6 commits December 1, 2021 01:00
…etworkfactory-test

# resolved conflicts:
#	src/index.js
#	tests/index.js
tests index file fixed, added all  test cases/files.
-fixed test index.js file, added all test cases.
-sablier branch was removed as it has its own pull request.
…etworkfactory-test

# resolved conflicts:
#	tests/index.js

[PULL]
updated latest changes from master, removed sablier branch as it has its own pull request.
removed migrations/2_deploy_sablier.js file related to sablier branch.
@Ruiub3i
Copy link
Contributor

Ruiub3i commented Dec 3, 2021

Tests are failing (see the Github Log @sgoia )

-removed sablier json interface files dependencies as they are in its own branch.
@bepro-bot
Copy link
Contributor Author

@moshmage reviewed this with the following message:

@Ruiub3i lgtm

@sgoia
Copy link
Contributor

sgoia commented Feb 8, 2022

@moshmage reviewed this with the following message:

@Ruiub3i lgtm

guys I will fix the conflicts, other merges broke this PR, will notify when ready.

…o i98-networkfactory-test

# Conflicts resolved:
#	src/models/Sablier/Sablier.js
#	tests/index.js
#	tests/sablier/admin/TakeEarnings.js
#	tests/sablier/admin/UpdateFee.js
#	tests/sablier/effects/compoundingStream/CancelCompoundingStream.js
#	tests/sablier/effects/compoundingStream/CreateCompoundingStream.js
#	tests/sablier/effects/compoundingStream/WithdrawFromCompoundingStream.js
#	tests/sablier/effects/stream/CancelStream.js
#	tests/sablier/effects/stream/CreateStream.js
#	tests/sablier/effects/stream/WithdrawFromStream.js
#	tests/sablier/sablier.behavior.js
#	tests/sablier/sablier.js
#	tests/sablier/view/BalanceOf.js
#	tests/sablier/view/DeltaOf.js
#	tests/sablier/view/GetCompoundingStream.js
#	tests/sablier/view/GetEarnings.js
#	tests/sablier/view/GetStream.js
#	tests/sablier/view/InterestOf.js
#	tests/sablier/view/IsCompoundingStream.js
@sgoia
Copy link
Contributor

sgoia commented Feb 9, 2022

@moshmage reviewed this with the following message:
@Ruiub3i lgtm

guys I will fix the conflicts, other merges broke this PR, will notify when ready.

@moshmage conflicts resolved, all checks have passed.

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.

3 participants