Several tests have been disabled, and need to be re-enabled.
There are some issues tho, such as the fact that
- the
cli_runner in conftest no longer is a click.CliRunner instance as it used to be, but a subprocess wrapper. I would recommend to use click.CliRunner instance, since it allows for far greater flexibility, among which checking logs.
- Some other fixtures have also been modified (i.e.
dummy_agent_tim), which now returns a useless boolean instead of the path, so test that check for example whether the protocol spec is part of the scaffolded protocol components' README fails.
Several tests have been disabled, and need to be re-enabled.
There are some issues tho, such as the fact that
cli_runnerin conftest no longer is aclick.CliRunnerinstance as it used to be, but a subprocess wrapper. I would recommend to use click.CliRunner instance, since it allows for far greater flexibility, among which checking logs.dummy_agent_tim), which now returns a useless boolean instead of the path, so test that check for example whether the protocol spec is part of the scaffolded protocol components' README fails.