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

Improve test coverage #4

Open
DAddYE opened this issue May 5, 2011 · 10 comments
Open

Improve test coverage #4

DAddYE opened this issue May 5, 2011 · 10 comments
Assignees

Comments

@DAddYE
Copy link
Member

DAddYE commented May 5, 2011

  1. Create a new model
  2. Create the admin page
  3. Create the frontend page

@nesquena, @achiu any other idea?

@ghost ghost assigned nesquena May 5, 2011
@nesquena
Copy link
Member

nesquena commented May 5, 2011

Can we test development reloading, i think so.
Creating a controller and model and visiting the controller and checking data is installed.
Creating two subapps and testing that session data can be passed between them in a freshly generated project

@DAddYE
Copy link
Member Author

DAddYE commented May 5, 2011

Let's go @nesquena start to improve it ;)

@nesquena
Copy link
Member

nesquena commented May 5, 2011

Yep I plan to start adding more tests to this, as we all should as time permits.

@nesquena
Copy link
Member

nesquena commented May 5, 2011

More ideas, let's have a sinatra app with module included (helpers) and then make sure for every template engine that rendering a template with a partial works.

@DAddYE
Copy link
Member Author

DAddYE commented May 6, 2011

@nesquena extend this: https://github.com/padrino/single-apps then we can add test for every thing ;)

@DAddYE
Copy link
Member Author

DAddYE commented May 16, 2011

Now tests are more more fast.

Now tests are in rspec for a better output and log backtrace

Added spec for single-apps, @nesquena @achiu ? Can we daily improve our coverage? Requires no more than 10 minutes write few lines.

@DAddYE
Copy link
Member Author

DAddYE commented May 16, 2011

I forgot a big mention! Now every thing use our internal padrino reloader, I think there is no way better than this to check that every thing works like a charm!

@nesquena
Copy link
Member

Awesome, great work man!

@dariocravero
Copy link

See padrino/padrino-framework#920.

@wikimatze
Copy link
Member

Can we delete this repository? Integration testing is done on travis.

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

No branches or pull requests

4 participants