Todd Smith

Results 21 comments of Todd Smith

+1 This project looks dead. Tons of work in 2015, then another bump in 2016. It sounds great though but it's a gamble to use for production if it's dead.

I'm still going through it but it looks like the accessing the form from behind the UI isn't working. You can access the form directly using the `testapp` fixture, getting...

We may want to wrap that whole create new sensor routine into it's own function like we did with the `login_user`. Maybe give it some default arguments so that we...

Here is the [ID field change](https://github.com/kizniche/Mycodo/blob/57e32b4f737266ee6e1c955cdec1e5b1ac70c12e/mycodo/mycodo_flask/templates/pages/sensor.html#L21) I made to get this to work: ``

My guess is that `flaskutils.db_retrieve_table` is returning an empty list. If you mark the `sensor = flaskutils.db_retrieve_table(current_app.config['MYCODO_DB_PATH'], Sensor)` line with a red [debugger entry point](https://www.jetbrains.com/help/pycharm/2016.1/debugging.html) you can step through the...

Found it. One of several errors actually. [`flaskutils`](https://github.com/kizniche/Mycodo/blob/57e32b4f737266ee6e1c955cdec1e5b1ac70c12e/mycodo/flaskutils.py#L1851) hard codes the `MYCODO_DB_PATH` into the session_scope call. The test application is using a throwaway db and this function is trying to...

This needs to use the `current_app.config['MYCODO_DB_PATH']` trick. There are tons of them in this module so they all need to change.

yeah lets fix the tests. I see some issues here that we need to resolve early and maybe getting the tests up and running will flush out more issues that...

Plenty of time. Enjoy your trip

Related to #131 I'm putting this in here since it pops up in the installer and is DB structure related. I get errors when running `setup.sh` [at the initial DB...