Skip to content

Circuit#5

Open
JohnDoePlusPlus wants to merge 11 commits intolabs42io:developfrom
JohnDoePlusPlus:feature/circuit
Open

Circuit#5
JohnDoePlusPlus wants to merge 11 commits intolabs42io:developfrom
JohnDoePlusPlus:feature/circuit

Conversation

@JohnDoePlusPlus
Copy link

No description provided.

Copy link
Member

@dimadeveatii dimadeveatii left a comment

Choose a reason for hiding this comment

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

I like the implementation. Please cover also with unit tests each file.

alex added 2 commits November 14, 2019 11:41
implement factories as class (not function)
move each factory to its entity folde
change isPromise function
add tests for CircuitState and its factory
and tests for Policies and their factory
alex added 4 commits November 15, 2019 14:19
change isPromise to isPromiseLike
add more tests
remove from factories one line functions
add spies
rename policy methods
CircuitState separate code in methods
CircuitState separate code in methods
and rename removeExecution method to registerCall
alex added 2 commits November 18, 2019 11:15
add unit test (now everything is tested)
change implementation of CircuitState and ArgumentsCircuitStateStorage
to avoid memory leaks
improve unit tests
alex and others added 2 commits November 22, 2019 19:03
improve tests
improve all factories
adjust unit tests for factories
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