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

setup basic unit testing for lib/core/control/automaton.ex #5

Open
upstarter opened this issue Feb 9, 2020 · 4 comments
Open

setup basic unit testing for lib/core/control/automaton.ex #5

upstarter opened this issue Feb 9, 2020 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation testing

Comments

@upstarter
Copy link
Owner

upstarter commented Feb 9, 2020

Setup a good skeleton with elixir testing best practices would be a big step forward for the project.

See the BTSK (behavior tree starter kit) for examples of whats important to test for the abstract stuff in the API. That would be a great starting point.

@upstarter upstarter created this issue from a note in The Automata Project (To do) Feb 9, 2020
@upstarter upstarter added the good first issue Good for newcomers label Feb 9, 2020
@Atul9
Copy link
Collaborator

Atul9 commented Feb 9, 2020

Hi @upstarter I would like to take this up.

@upstarter upstarter added documentation Improvements or additions to documentation and removed good first issue Good for newcomers labels Feb 9, 2020
@upstarter upstarter moved this from To do to In progress in The Automata Project Feb 13, 2020
@upstarter upstarter changed the title TESTING: setup basic unit testing for lib/core/behavior.ex TESTING: setup basic unit testing for lib/core/nodeex Feb 13, 2020
@upstarter upstarter self-assigned this Feb 13, 2020
@upstarter upstarter changed the title TESTING: setup basic unit testing for lib/core/nodeex TESTING: setup basic unit testing for lib/core/node.ex Feb 14, 2020
@upstarter
Copy link
Owner Author

upstarter commented Feb 14, 2020

Hey @Atul9, sorry it took so long to get back to you. I am still working on getting a simple version working. Right now it runs with iex -S mix, but only ticking the trees that do nothing but log. Next is to get a very simple sequence running, then we're off to the races! If you want to have a go at testing the existing simple functionality, just make a PR.

@Atul9
Copy link
Collaborator

Atul9 commented Feb 14, 2020

@upstarter ok, I am familiar with adding function test but new to adding macro test, I am trying it out. I will work on it this weekend & If I need help with macro test I will ask about it in slack.

@Atul9
Copy link
Collaborator

Atul9 commented Feb 17, 2020

Hi @upstarter I am not feeling well. I need to take rest, hence someone else can take this up.

@upstarter upstarter changed the title TESTING: setup basic unit testing for lib/core/node.ex TESTING: setup basic unit testing for lib/core/control/automaton.ex Feb 22, 2020
@upstarter upstarter moved this from In progress to To do in The Automata Project Mar 12, 2020
@upstarter upstarter changed the title TESTING: setup basic unit testing for lib/core/control/automaton.ex setup basic unit testing for lib/core/control/automaton.ex May 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation testing
Projects
Development

No branches or pull requests

2 participants