search

dictation-toolbox / aenea

Client-server library for using voice macros from Dragon NaturallySpeaking and Dragonfly on remote/non-windows hosts.
GNU Lesser General Public License v3.0
211 stars 62 forks source link

Use a different version of dragonfly on the readme https://github.com… #173

Closed dylan-chong closed 6 years ago

dylan-chong commented 6 years ago

…/Danesprite/dragonfly

Closes issue #167

dylan-chong commented 6 years ago

I also noticed there are tests but i have never found a way to run as there is no documentation

dylan-chong commented 6 years ago

By the way i just realise that there are extra keys added to dragonfly2, that won't work through aenea https://dragonfly2.readthedocs.io/en/latest/actions.html#key-names @calmofthestorm what you suggest we do about this?

calmofthestorm commented 6 years ago

I also noticed there are tests but i have never found a way to run as there is no documentation

Yeah, the testing story for Aenea is :/. Server just has integration tests with test-client.py. For the client-side tests (what I believe you're referring to here), there's dragonfly_mock.py which mocks out much of the Dragonfly stuff to allow me to run some simple unit tests, though I've only tried this on Linux. Unfortunately this does not let me test grammars.

To run these tests, go to aenea/client, add . to PYTHONPATH, and python test/test_*.py. Currently two have rotted it would seem. I should really set up CI for those, unlike the integration tests it should be quite straightforward since it's just pure Python running on linux. Need to add typeables to dragonfly_mock.py and figure out what the init errors are about.

I'll open an issue to bisect and fix this and add CI to prevent it from recurring. For this CL, don't worry about the tests:/

calmofthestorm commented 6 years ago

Thanks for taking the time to do this, implementing the decision from previous discussions on the mailing list and issue. It's good to see an active Dragonfly fork.