Add a note for tests with modular setup #579

Merged
merged 2 commits into from Dec 13, 2012

Projects

None yet

3 participants

@acco
Contributor
acco commented Oct 4, 2012

Found this because I switched between the two midway through development.

README suggests that for setup you need to have this in every test:

 def app
   Sinatra::Application
 end

But if you're using the modular setup, you must use the class name of your app instead.

@blambeau

Your note actually appears inside the code block. It should probably be unindented. Or maybe move it as a ruby comment in the app method (?).

@acco
Contributor
acco commented Oct 8, 2012

@blambeau Whoops, thanks! I'm unfamiliar with where Sinatra users tend to get their information. If you feel it would be preferable to have this in the app method I'm happy to move it.

@rkh rkh merged commit c40fc9f into sinatra:master Dec 13, 2012

1 check failed

default The Travis build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment