-
Notifications
You must be signed in to change notification settings - Fork 129
tests: Add manual tests procedure. #120
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
Conversation
@microbit-mark I haven't finished writing these test cases, but could you start reviewing the content of what's currently in the PR? |
- A micro:bit with WebUSB-enabled DAPLink and access to a version of the | ||
Python Editor with WebUSB REPL | ||
|
||
- If running the tests on Windows 7 you might need the Mbed serial driver |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we link to https://os.mbed.com/docs/mbed-os/v5.11/tutorials/windows-serial-driver.html here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can assume the meat-and-bones test runner should have enough technical background to figure this out. I'd prefer not to add info that can get out-of-date and nobody will check in the future.
- A serial terminal program you are comfortable to connect to a micro:bit | ||
- The working microREPL installation (https://github.com/ntoll/microrepl) | ||
- Mu editor (https://codewith.mu/) | ||
- A micro:bit with WebUSB-enabled DAPLink and access to a version of the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we link to firmware update page here https://microbit.org/guide/firmware/ and say this is 0247+
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As with the previous one, I'd prefer to assume the person running the tests can figure this out themselves. This part is not essential either, as you only need one option from the list.
I have checked over the language and added comments. Looks good at this stage. |
Thanks Mark! I've added a commit with the review comment changes. |
@microbit-mark I think I've finished the current version of the Test Procedure, you can see the changes and additions in commit e623207. Could you check the latest changes and if you are happy with the content we can merge this to master. |
The changes look good to merge |
Thanks Mark! |
tests: Add manual tests procedure. (bbcmicrobit#120)
Co-authored-by: microbit-carlos <[email protected]>
This is still a work in progress.