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

Update game loop test logic to handle expected Arduino response #234

Open
nashirj opened this issue Apr 11, 2022 · 2 comments
Open

Update game loop test logic to handle expected Arduino response #234

nashirj opened this issue Apr 11, 2022 · 2 comments
Labels
General Software Team Game loop, web app, GUI, etc

Comments

@nashirj
Copy link
Member

nashirj commented Apr 11, 2022

In testfiles, in addition to the outgoing pi msg, we should specify the expected Arduino response. This will allow smoother automated testing.

To this end, it might be cleaner to separate out test file runner logic from the main game loop.

Also, the opcode generator GUI should be updated to include the expected response from the Arduino (where possible; some instructions like requesting current electromagnet position do not have a static response msg, but others do).

@nashirj nashirj added the General Software Team Game loop, web app, GUI, etc label Apr 11, 2022
@nashirj
Copy link
Member Author

nashirj commented Apr 17, 2022

image

@nashirj
Copy link
Member Author

nashirj commented Apr 22, 2022

As part of this PR, fix logic so that the message is sent verbatim; don't parse the test message and then send a message accordingly, just send it directly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Software Team Game loop, web app, GUI, etc
Projects
None yet
Development

No branches or pull requests

1 participant