python icon indicating copy to clipboard operation
python copied to clipboard

add more smoke tests

Open mkinney opened this issue 4 years ago • 5 comments

Need help on adding more smoke tests for the following:

  1. --ch-enable and --ch-disable: ~~Need help with how to use and verify these settings are applied~~ EDIT: This has been resolved.

  2. How do you test --no-atime

Note: These require at least two connected devices. 3) Test --reply: How do you test this? 4) Test --dest DEST: How do you test this? (looks like #146 might have info about how to setup/test) 5) Test --sendping: ~~How do you test this?~~ EDIT: This has been resolved.

Note: These are probably hard to test. 6) Test --gpio*: What hardware is needed to test this? Can I test on HeltecV21? 7) Test --ble 8) Test --noproto Do we want a smoke test on this?

mkinney avatar Dec 10 '21 18:12 mkinney

After looking at the code, I think the --ch-enable and --ch-disable needed some code changes.

mkinney avatar Dec 11 '21 02:12 mkinney

So, number 1 above is no longer a question with PR#157. The other items 2-8 need input.

mkinney avatar Dec 11 '21 04:12 mkinney

The --no-atime option makes no sense to me.

Perhaps it should be renamed to --send-position?

mkinney avatar Dec 12 '21 21:12 mkinney

Looks like to test --reply, open up one terminal session with --reply and then on another terminal do a --sendtext. The first terminal should simply show the value received.

mkinney avatar Dec 30 '21 15:12 mkinney

Looks like info about the SerialPlugin can be found in the source: https://github.com/meshtastic/Meshtastic-device/blob/master/src/plugins/esp32/SerialPlugin.cpp#L12

mkinney avatar Jan 18 '22 06:01 mkinney