RailsBridge

Sarah	hi Zach!

Zach M.	Hey Sarah

Sarah	ok, I figured out my git problem :)

Zach M.	That's funny that you're patching an OpenLazlo plugin. I remember working with non-Rails OpenLazlo years ago

Sarah	u ready to dive in?

Zach M.	Sure, where do you need me to start?

Sarah	so, I just checked in some code that makes the specs run

Sarah	<a href="http://github.com/ultrasaurus/openlaszlo_plugin/commit/26fbaf78a9d5759a732c8cffc4cbb840003236e3" target="_blank">http://github.com/ultrasaurus/openlaszlo_p…</a>

Sarah	I dont' really understand why I needed to require 'action_controller' and friends in order to use assert_select (via RSpec's have_tag)

Zach M.	Yah

Zach M.	I was just going to ask that

Zach M.	

<a href="/room/221003/paste/132377425" target="_blank">View paste</a>

<br /><pre><code>+require 'action_controller'

0

+require 'action_controller/assertions/selector_assertions'

0

+include ActionController::Assertions::SelectorAssertions</code></pre>

Zach M.	Those don't seem necessary

Zach M.	lemme clone the code

Sarah	k

Sarah	the error I was getting before is written up here:  <a href="http://www.ruby-forum.com/topic/187635#new" target="_blank">http://www.ruby-forum.com/topic/187635#new</a>

Zach M.	Hrm, I may know the probelm

Sarah	the failure happened in Rails 2.3.2 but not 2.1

Zach M.	hold on a sec, lemme situate my browser windows

Sarah	k

Zach M.	Interesting

Sarah	do tell

Zach M.	What's the difference between the plugin_spec_helper and spec_helper?

Zach M.	It looks like plugin_spec_helper is grabbing the spec of the Rails dir it's installed into?

Sarah	it wants the spec_helper from the Rails app

Sarah	I don't know why

Sarah	a colleague said he thought it had something to do with view testing

Zach M.	So you'll have to forgive me, I haven't written a ton of plugins. But the ones I've always written, I test it apart from a Rails install

Zach M.	A good example would be webrat

Zach M.	<a href="http://github.com/brynary/webrat/tree/master" target="_blank">http://github.com/brynary/webrat/tree/master</a>

Zach M.	openlaszlo_applet_specs.rb and swfobject_view_helper_spec.rb are the two main specs right? The others are just for environment setup?

Zach M.	It's interesting the way osteele set up the spec directory: <a href="http://github.com/osteele/openlaszlo_plugin/tree/bfe0bef882d23055d9fe6aa80447de26e14eedeb/spec" target="_blank">http://github.com/osteele/openlaszlo_plugi…</a>

Zach M.	Sorry, I'm trying to dig around the code to try and get a bit acquainted with it

Zach M.	This stuff may be needed:

Zach M.	

<a href="/room/221003/paste/132378497" target="_blank">View paste</a>

<br /><pre><code># just enough infrastructure to get 'assert_select' to work

require 'action_controller'

require 'action_controller/assertions/selector_assertions'

include ActionController::Assertions::SelectorAssertions</code></pre>

Zach M.	Because it's only pulling in rspec

Zach M.	and rspec doesn't have assert_select, but rather rspec-rails does

Sarah	has left the room 

Sarah	has entered the room

Zach M.	Welcome back :)

Sarah	not sure what happened there

Zach M.	heh, luckily, you can read the chat history

Sarah	I don't see any history

Zach M.	Oh no?

Zach M.	shoot

Sarah	I thought that was the point of campfire!

Zach M.	Try this: <a href="http://simplechatter.campfirenow.com/room/221003/transcript/2009/05/24" target="_blank">http://simplechatter.campfirenow.com/room/…</a>

Zach M.	Does that work?

Sarah A.	has entered the room

Sarah A.	hello

Zach M.	Hey

Zach M.	Having troubles?

Sarah A.	trying different browser

Zach M.	Ah

Zach M.	Here's a link to the transcripts: <a href="http://simplechatter.campfirenow.com/room/221003/transcript/2009/05/24" target="_blank">http://simplechatter.campfirenow.com/room/…</a>

Sarah A.	whenever I hit return, it would re-direct

Zach M.	Wierd

Sarah A.	that link required an email & password

Zach M.	bah

Zach M.	Here:

Zach M.	

<a href="/room/221003/paste/132378842" target="_blank">View paste</a>

 <span class="number_of_lines">(<span>16 more lines</span>)</span> <br /><pre><code>a colleague said he thought it had something to do with view testing

Zach M. 	

So you'll have to forgive me, I haven't written a ton of plugins. But the ones I've always written, I test it apart from a Rails install

Zach M. 	

A good example would be webrat

Zach M. 	

http://github.com/brynary/webrat/tree/master

Zach M. 	

openlaszlo_applet_specs.rb and swfobject_view_helper_spec.rb are the two main specs right? The others are just for environment setup?

Zach M. 	

It's interesting the way osteele set up the spec directory: http://github.com/osteele/openlaszlo_plugi…

May 24 	

1:10 PM

Zach M. 	

Sorry, I'm trying to dig around the code to try and get a bit acquainted with it

...</code></pre>

Zach M.	(Kinda nasty)

Sarah A.	I have the rspec-rails gem installed 

Sarah A.	used to be a plugin when this plugin was written

Sarah A.	is there a way for me to just include it

Sarah A.	or something

Zach M.	Hrm. So conceptually, I'm thinking of this in 2 ways

Zach M.	You have you're rails install that this is embedded into

Sarah A.	btw:  openlaszlo_applet_specs.rb and swfobject_view_helper_spec.rb are the main ones I'm using, but not the main point of the plugin

Zach M.	(and that rails install may have dependencies installed)

Zach M.	And you have your plugin with it's specs and possibly dependencies

Sarah A.	ok

Zach M.	What you're doing is embedding the plugin in order to test it. And therefore the Rails environment (with all it's dependencies) isn't loaded up

Zach M.	I guess my first question is, what are you wanting to accomplish?

Sarah A.	applet_view_helper.rb has a bug

Zach M.	Ah

Sarah A.	line 48 should read

Sarah A.	path = applet_path(source)

Zach M.	Gotcha

Sarah A.	I want to create a spec that fails

Sarah A.	and then fix the bug and have it pass

Zach M.	is there current coverage for that method?

Sarah A.	nope

Sarah A.	:)

Ad	Upgrade your Campfire account and say good-bye to ads. You'll also get more disk space, more chatters, SSL (on Plus and Premium plans), the ability to upload your own logo, and more. <strong>UPGRADE SPECIAL:</strong> Upgrade today and save $10 on your first month. Just enter coupon code CHATAD on the upgrade screen. To upgrade click the Account tab above.

	<p><small>Ads are only shown on free accounts. Upgrade to go ad free.</small></p>

Zach M.	hehe

Zach M.	Ok, so there's 3 options:

Zach M.	1) You submit the github CODE change (without the specs)

Zach M.	2) You keep working in the spec layout in order to create a new spec for that helper/method that fails/passes with your code change

Zach M.	3) You revamp the way rspec is being used to test, and also include the spec for that code change

Zach M.	#3 is going to take the most effort, but you probably won't be as frustrated

Sarah A.	I don't mind doing #3... I have 2 goals

Sarah A.	1) I will be using and living with this plugin for a while, so I want to be happy with it and comfortable making changes

Sarah	has left the room 

Sarah A.	2) I want to learn best practices for developing plugins and using RSpec, since we have tons of plugin in my day-job and we're moving to cucumber/Rspec in our new coding

Zach M.	Cool

Sarah A.	so

Zach M.	Ok, so if we started down the path of #3, I'd see a few things we'd need to do

Zach M.	On a highlevel, I should be able to checkout the plugin code and run the specs (without it being embedded in a Rails project)

Sarah A.	that would be very nice

Zach M.	(I may need to install some gems or deps to do that, but it's not dependent on the Rails install)

Sarah A.	but, can we test views without rails?

Zach M.	Second, I'd see the spec folder revamping quite a bit

Zach M.	So, you can still use rails and rails testing infrastructure without it being embedded in a Rails install

Zach M.	lemme grab a link from the Webrat plugin

Sarah A.	cool

Zach M.	Interesting, Webrat has changed quite a bit of their stuff

Zach M.	I'm not quite sure if you NEED to assert using view tests, but you may have to

Zach M.	One way you could do that would be to require rails as a gem. So it'll have the same effects as if it were embedded, but you don't have that "wrapper codebase"

Zach M.	You're already kinda doing that

Zach M.	

<a href="/room/221003/paste/132379925" target="_blank">View paste</a>

<br /><pre><code># just enough infrastructure to get 'assert_select' to work

require 'action_controller'

require 'action_controller/assertions/selector_assertions'

include ActionController::Assertions::SelectorAssertions</code></pre>

Sarah A.	well... the bug I'm fixing happens when you add ?debug=true onto the page URL to get the OpenLaszlo app to compile with its embedded debugger

Zach M.	That may actually be grabbing the rubygems version of ActionController, not the wrapper code

Sarah A.	right

Zach M.	This is an interesting problem. I haven't had to test Helper code in a plugin before

Zach M.	Oh, you know what would be a good project to check out. RadiantCMS. They do a bit more messaging to create extensions, but I wonder if there may be some gems on how to setup the rspec and rspec-rails environment in order to test helpers

Zach M.	<a href="http://github.com/radiant/radiant/tree/master" target="_blank">http://github.com/radiant/radiant/tree/master</a>

Zach M.	Lemme see if I can gist some extension code i've written in the past

Sarah A.	cool

Sarah A.	I need to go away for 15 minutes and feed my kid breakfast

Sarah A.	can I ping you when I'm back?

Zach M.	Not a problem. I may be around.

Zach M.	One last thing

Zach M.	It looks like Radiant is doing the same thing as the "Embed inside rails" method

Zach M.	

<a href="/room/221003/paste/132380249" target="_blank">View paste</a>

<br /><pre><code>unless defined? RADIANT_ROOT

  ENV["RAILS_ENV"] = "test"

  require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../")}/config/boot"

  require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../")}/config/environment"

end

require 'spec/rails'</code></pre>

Zach M.	bummer

Zach M.	I'll check a bit more on this codebase, and maybe I can shoot you an email that may help you out

Zach M.	take care

Sarah A.	yeah, if you can send me some pointers on stuff to read

Sarah A.	then I can start getting up to speed, if you have to take off 

Zach M.	Sounds good

Sarah A.	I'm back

Zach M.	Right on

Zach M.	Trying to make headway

Zach M.	Something is odd

Zach M.	Btw, in case you're curious. Radiant uses the same methodology: <a href="https://gist.github.com/cf7dff6a2620993654b8" target="_blank">https://gist.github.com/cf7dff6a2620993654b8</a>

Zach M.	(Except they embed rails and rspec differently)

Zach M.	But I created a new rails install just to make sure I'm not going insane:

Zach M.	

<a href="/room/221003/paste/132382462" target="_blank">View paste</a>

<br /><pre><code>require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')

describe DogHelper do

  it "should bark" do

    html = helper.bark

    html.should have_tag("script")

  end

end

</code></pre>

Zach M.	That fails but not on the assert_select deal

Sarah A.	did you make that as a plugin/spec or regular spec?

Zach M.	regular spec

Zach M.	perhaps I should create a new plugin/spec too

Sarah A.	well, it is a good idea to try stuff as a regular spec to test assumptions... I'll try that too

Zach M.	Yea, I'm a big "test assumptions" guy. Especially when things seem wacky

Zach M.	Something is plain funky

Zach M.	Real helper spec:

Zach M.	

<a href="/room/221003/paste/132383016" target="_blank">View paste</a>

<br /><pre><code>require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')

describe DogHelper do

  it "should bark" do

    html = helper.bark

    html.should have_tag("script")

  end

end

</code></pre>

Zach M.	And it's spec helper:

Zach M.	

<a href="/room/221003/paste/132383031" target="_blank">View paste</a>

<br /><pre><code># This file is copied to ~/spec when you run 'ruby script/generate rspec'

# from the project root directory.

ENV["RAILS_ENV"] ||= 'test'

require File.dirname(__FILE__) + "/../config/environment" unless defined?(RAILS_ROOT)

require 'spec/autorun'

require 'spec/rails'

Spec::Runner.configure do |config|

  # If you're not using ActiveRecord you should remove these

  # lines, delete config/database.yml and disable :active_record

  # in your config/boot.rb

  config.use_transactional_fixtures = true

  config.use_instantiated_fixtures  = false

  config.fixture_path = RAILS_ROOT + '/spec/fixtures/'

end

</code></pre>

Zach M.	Plugin helper:

Zach M.	(Er plugin helper spec):

Zach M.	

<a href="/room/221003/paste/132383052" target="_blank">View paste</a>

<br /><pre><code>require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')

describe DogHelper do

  it "should bark" do

    html = helper.bark

    html.should have_tag("script")

  end

end

</code></pre>

Zach M.	And it's spec_helper:

Zach M.	require File.dirname(__FILE__) + '/../../../../spec/spec_helper'

Zach M.	

<a href="/room/221003/paste/132383078" target="_blank">View paste</a>

<br /><pre><code>require File.dirname(__FILE__) + '/../../../../spec/spec_helper'

</code></pre>

Sarah A.	is the have_tag part of webrat?

Zach M.	Getting same results with this:

Zach M.	

<a href="/room/221003/paste/132383220" target="_blank">View paste</a>

<br /><pre><code>require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')

describe DogHelper do

  include DogHelper

  it "should bark" do

    # html = helper.bark

    html = bark

    html.should have_tag("script")

  end

end

</code></pre>

Zach M.	Nah, it's part of rspec-rails

Zach M.	it's the thing calling assert_select

Zach M.	This spec code is giving me:

Zach M.	

<a href="/room/221003/paste/132383276" target="_blank">View paste</a>

<br /><pre><code>cairhien:helpers zmoazeni$ ruby dog_helper_spec.rb 

F

1)

NoMethodError in 'DogHelper should bark'

undefined method `assert_select' for #<ActiveSupport::TestCase::Subclass_1:0x22ba5dc>

dog_helper_spec.rb:9:

dog_helper_spec.rb:3:

</code></pre>

Sarah A.	that's what I got before

Zach M.	I may have an idea

Sarah A.	do we need to require 'rspec-rails'

Sarah A.	or something?

Zach M.	If you put a print statement in the have_tag.rb, you'll see it's included

Zach M.	and that's inside rspec-rails

Sarah A.	well, at least I know it's not just crazy for me

Zach M.	I don't think RSpec knows it's a helper spec

Zach M.	Rspec loads the matchers based on what Type of spec it is Model, Controller, Helpers, Views

Sarah A.	The Rspec Book does not yet have a chapter on Rails helpers

Zach M.	That's why it's necessary to do:

Zach M.	

<a href="/room/221003/paste/132383445" target="_blank">View paste</a>

<br /><pre><code>describe DogHelper do

  include DogHelper</code></pre>

Zach M.	and not just:

Zach M.	describe DogHelper do

Sarah A.	did you write a DogHelper?

Ad	Campfire is just one of many simple and easy to use web-based tools designed by <a href="http://www.37signals.com" target="_blank">37signals</a>.

	<p><small>Ads are only shown on free accounts. Upgrade to go ad free.</small></p>

Zach M.	Yea

Zach M.	Bingo bango

Zach M.	

<a href="/room/221003/paste/132383504" target="_blank">View paste</a>

<br /><pre><code>require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')

describe DogHelper, :type => :helper do

  it "should bark" do

    html = helper.bark

    html.should have_tag("script")

  end

end

</code></pre>

Zach M.	That works

Sarah A.	?

Sarah A.	ah

Zach M.	And that fixes the swfobject spec

Zach M.	Btw, this is the spec I have:

Zach M.	

<a href="/room/221003/paste/132383609" target="_blank">View paste</a>

 <span class="number_of_lines">(<span>40 more lines</span>)</span> <br /><pre><code>require File.expand_path(File.dirname(__FILE__) + '/plugin_spec_helper')

require File.expand_path(File.dirname(__FILE__) + "/../lib/swfobject_view_helper")

# include ActionView::Helpers

describe ActionView::Helpers::SwfObjectHelper, :type => :helper do

  # include ActionView::Helpers::SwfObjectHelper

  describe :swfobject_tag do

    # before(:each) do

    #   ENV['RAILS_ENV'] = 'production'

    #   self.stub!(:compute_public_path).and_return('/applets/url.swf')

    # end

    # it "should call swfobject.embedSWF" do

...</code></pre>

Zach M.	See I commented out the include, and I qualified the Spec (that way we don't have to include ActionView::Helpers

Zach M.	And this is the plugin_spec_helper:

Zach M.	

<a href="/room/221003/paste/132383716" target="_blank">View paste</a>

<br /><pre><code>begin

  require File.dirname(__FILE__) + '/../../../../spec/spec_helper'

rescue LoadError

  puts "You need to install rspec in your base app"

  exit

end

# # just enough infrastructure to get 'assert_select' to work

# require 'action_controller'

# require 'action_controller/assertions/selector_assertions'

# include ActionController::Assertions::SelectorAssertions</code></pre>

Zach M.	(You don't need that infrastructure

Zach M.	)

Sarah A.	I tried it here and it worked for me too!

Zach M.	I think the problem was, the infrastructure was there. RSpec didn't allow you to use it for that spec

Sarah A.	ah

Zach M.	One last thing:

Zach M.	This is dangerous:

Zach M.	

<a href="/room/221003/paste/132383798" target="_blank">View paste</a>

<br /><pre><code> # before(:each) do

    #   ENV['RAILS_ENV'] = 'production'

    #   self.stub!(:compute_public_path).and_return('/applets/url.swf')

    # end</code></pre>

Sarah A.	where is that?

Zach M.	Rails tests expect they are running in the "test" environment

Zach M.	Line 11ish of swfobject_view_helper.rb

Zach M.	I see there's some logic in the helper that switches based on whether it's in Production or Development

Zach M.	Instead of the Helper switching on ENV["RAILS_ENV"] I'd shove that into a small class like this:

Zach M.	

<a href="/room/221003/paste/132384028" target="_blank">View paste</a>

<br /><pre><code>class EnvChecker

  def production?

    ENV['RAILS_ENV'] == 'production'

  end

  def development?

    ENV['RAILS_ENV'] == 'development'

  end

end</code></pre>

Zach M.	Then your "before(:each)" can stub that EnvChecker

Zach M.	Does that make sense?

Sarah A.	almost

Sarah A.	so... in the helper, use the EnvChecker class

Zach M.	yea

Sarah A.	then change the EnvChecker behavior (using mock?) in the before

Zach M.	yep

Sarah A.	rather than swithing the real env

Sarah A.	cool

Zach M.	I'll throw you a gist

Sarah A.	where would yu put the EnvChecker code?

Zach M.	somewhere in lib maybe

Zach M.	Lib may need some new directory cleanup. "helpers", "support" I dunno

Sarah A.	Lemme see if I can write it and run it by you

Zach M.	<a href="https://gist.github.com/71a30f8d26928dfe9663" target="_blank">https://gist.github.com/71a30f8d26928dfe9663</a>

Zach M.	That may be a bit clearer

Sarah A.	yes.  thanks.

Zach M.	Is there anything else you need help on?

Sarah A.	yes...where do I find the docs for have_tag

Zach M.	hehe, hrm

Sarah A.	and for Rspec in general

Zach M.	This is rspec itself: <a href="http://rspec.rubyforge.org/rspec/1.2.6/" target="_blank">http://rspec.rubyforge.org/rspec/1.2.6/</a>

Zach M.	But the have_tag is in rspec-rails

Sarah A.	right

Zach M.	<a href="http://rspec.rubyforge.org/rspec-rails/1.2.6/" target="_blank">http://rspec.rubyforge.org/rspec-rails/1.2.6/</a>

Zach M.	have_tag: <a href="http://rspec.rubyforge.org/rspec-rails/1.2.6/classes/Spec/Rails/Matchers.html#M000064" target="_blank">http://rspec.rubyforge.org/rspec-rails/1.2…</a>

Zach M.	It's not all that clear though because it refers to this documentation: <a href="http://api.rubyonrails.org/classes/ActionController/Assertions/SelectorAssertions.html#M000547" target="_blank">http://api.rubyonrails.org/classes/ActionC…</a>

Sarah A.	I'm trying to understand line 36

Sarah A.	of swfobject_view_helper_spec.rb

Sarah A.	what /"8" means

Sarah A.	is that a regex?

Zach M.	Ah

Zach M.	it's actually:

Zach M.	/"8", \{/

Zach M.	yea

Sarah A.	this test fails

Zach M.	the regex starts and ends with //

Sarah A.	right

Zach M.	so the code says: "There is a 'script' tag and inside that there is the text '"8", {'"

Sarah A.	so... it looks like it is failing because the "  doesn't match ' in the code

Sarah A.	also the version number is wrong

Sarah A.	or rather, not detailed enough

Zach M.	yep

Sarah A.	the default is: '8,0,24,0'

Sarah A.	but maybe that comes out with double-quotes in the code

Sarah A.	yup

Zach M.	I get knocked on a lot for debugging this way, but I'm a huge "print debugger": 

Zach M.	

<a href="/room/221003/paste/132385236" target="_blank">View paste</a>

<br /><pre><code>    it "should default to version 8" do

      html = swfobject_tag("url")

      puts html

      html.should have_tag('script', /"8", \{/)

    end</code></pre>

Sarah A.	well... just did 'view source' on the running app :)

Zach M.	heh

Zach M.	this is what running the test tells me:

Zach M.	

<a href="/room/221003/paste/132385295" target="_blank">View paste</a>

<br /><pre><code>&lt;div id="url"&gt;

            <p>Requires the Flash plugin.  If the plugin is already installed, click <a href="?detectflash=false">here</a>.</p>

          </div>

          <script type="text/javascript">//<![CDATA[

            swfobject.embedSWF("/applets/url.swf", "url", "100%", "100%", "8,0,24,0", "/expressInstall.swf", {"lzproxied": false}, {"scale": "noscale"});

          //]]>

          </script>

</code></pre>

Zach M.	I think it goes back to being a "Verify Assumptions"

Zach M.	er

Sarah A.	yup.. the code definitely specifes:  '8,0,24,0'

Zach M.	I think it goes back to me being a "Verify Assumptions" coder

Sarah A.	I forget what is special about that version

Sarah A.	I'll just update the spec to match the code

Sarah A.	ok... now for my "real" question

Zach M.	haha

Sarah A.	I've created an example group (i guess they are called that) for applet_view_helper..

Sarah A.	and I have an example that passes

Sarah A.	now I want to write the failing example

Sarah A.	I need to test the view when I pass ?debug=true in the page that embeds the view

Sarah A.	does that make sense?

Zach M.	Sorta

Zach M.	So I'm guessing "?debug=true" will be a query param going to openlaszlo?

Sarah A.	so here's the use case:

Ad	From 37signals, the people who bring you Campfire: <a href="https://gettingreal.37signals.com" target="_blank">Getting Real, the book</a>. Discover the smarter, easier, faster way to build a successful web-based application. 171 pages.

	<p><small>Ads are only shown on free accounts. Upgrade to go ad free.</small></p>

Sarah A.	I have an app that manages a list of videos

Sarah A.	normally I would see a video detail page with this URL

Sarah A.	<a href="http://localhost:3000/videos/1" target="_blank">http://localhost:3000/videos/1</a>

Sarah A.	however, if I specify this URL:  <a href="http://localhost:3000/videos/1?debug=true" target="_blank">http://localhost:3000/videos/1?debug=true</a>

Sarah A.	then the plugin will compile a different version of the app and name it whatever-debug.swf and refer to that URL

Sarah A.	(in the OpenLaszlo dev env ?debug=true will do the same thing, but the plugin will compile the swf in non-proxied mode, if you want to know the OL details)

Sarah A.	is that any clearer?

Zach M.	This may start getting outside my knowledge of how you integrate OL with Rails

Sarah A.	well.. the Rails question is:

Zach M.	I'm assuming that <a href="http://localhost:3000/videos/1" target="_blank">http://localhost:3000/videos/1</a> hits some regular controller, and in the view it uses helpers that point to tho OL server to compile, return the flash?

Sarah A.	yeah

Sarah A.	that's what the applet_tag helper does

Zach M.	gotcha

Sarah A.	it calls:

Sarah A.	

<a href="/room/221003/paste/132385859" target="_blank">View paste</a>

<br /><pre><code>            if params.include?('debug')

</code></pre>

Zach M.	So my high level concern would be, what if the application wanted to use "debug" ?

Sarah A.	so in the test, I need to send the param

Sarah A.	it can include that in the url string or as a canvas attribute inside the LZX

Sarah A.	but that would be rare

Sarah A.	usually you just want to use debug mode temporarily in development

Zach M.	Gotcha. Hrm. It seems like that param should be more obscure, or configurable

Zach M.	like olz_debug I dunno

Sarah A.	well, it is nice to have it be the same as it would be if you were deploying an OpenLaszlo server

Sarah A.	in production you would either not deploy the server (which is what I typically do) ...

Sarah A.	or you would lock it down to not allow this

Zach M.	Ok

Sarah A.	so... I need to know how to send the param in RSpec

Sarah A.	and I wasn't sure where to even look for that

Zach M.	So it doesn't sound like it'd have anything to do with the view right? Because you don't want conditional debugs littering your view. It sounds like it's the helper that would interpet 'params.include?("debug")'

Zach M.	I think you may be able to set the params in the "before", and if not, you can always use stubs to fake it out

Sarah A.	yes it does, (line 49)

Zach M.	Lemme check something

Zach M.	ok:

Zach M.	

<a href="/room/221003/paste/132386281" target="_blank">View paste</a>

<br /><pre><code>require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')

describe DogHelper do

  it "should bark" do

    params[:debug] = "foo"

    html = helper.bark

    html.should have_tag("script")

  end

end

</code></pre>

Zach M.	

<a href="/room/221003/paste/132386289" target="_blank">View paste</a>

<br /><pre><code>module DogHelper

  def bark

    p params

    "foo"

  end

end

</code></pre>

Sarah A.	that is exactly it

Sarah A.	ok... let me digest all this for a bit

Zach M.	hehe notta problem

Sarah A.	can I send you some code to review ?

Zach M.	Sure

Sarah A.	when I get further along...

Zach M.	I may not be able to get it to a point to run it

Sarah A.	thanks so much!

Zach M.	Hope you have a great weekend :)