Skip to content
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

Update docs: Remind users they may need to add tests #2051

Open
cloudwheels opened this issue Nov 18, 2018 · 2 comments
Open

Update docs: Remind users they may need to add tests #2051

cloudwheels opened this issue Nov 18, 2018 · 2 comments

Comments

@cloudwheels
Copy link

cloudwheels commented Nov 18, 2018

Description / Steps to reproduce / Feature proposal

  • Work through the steps to build todoList on top of todos

Current Behavior

  • If the todoList tutorial is pulled from the repo / using lb4 example it will include tests.
  • Tests are not automatically created if starting with the todo example and building todoList using lb4 controller etc

Expected Behavior

@Yaty Yaty added the Docs label Nov 18, 2018
@bajtos
Copy link
Member

bajtos commented Nov 22, 2018

I agree we should improve lb4 tooling to help users to write tests for their code too. I think the following issue is pretty much covering your second point: #885 "lb4 controller" should prepare test infrastructure

Let's keep this issue focused on short-term improvements of our docs, i.e. Update docs: Remind users they may need to add tests.

@cloudwheels would you like to contribute this change yourself?

@cloudwheels cloudwheels changed the title Docs: working through todoList tutorial after todos rather than starting from lb4 example does not create tests Update docs: Remind users they may need to add tests Nov 22, 2018
@cloudwheels
Copy link
Author

Updated title to reflect your suggestions as basis for straightforward PR - yes can take this :)
Will get myself organised to contrib asap, may need a day or two if that's OK to balance things with the real world (mucho tiempo aqui!) but no longer...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants