something wrong with set_timeout in specs about opal-jquery HOT 6 CLOSED

Hi Jared. When things run async in javascript, we can't have a top level wrapper that will catch issues in all browsers. There is also an instance async method which you can use which will catch any errors:

describe "some async magic" do
  async "it might raise an error" do
    something_happens_later do
      async { raise "yo" }
    end
  end
end

Wrapping things inside that async block will make things work as they are meant to.

from opal-jquery.

Thanks for the helpful tip...doesn't really explain my problem however because I didn't write any of the code that's not working right, particularly the strange set_timeout behavior. :)

from opal-jquery.

The timeout behaviour is intentional. Maybe it should be renamed though to avoid confusion withsetTimeout

from opal-jquery.

Thanks Adam, that put me on the right path. I renamed set_timeout to delay (since set_timeout was actually just an alias of delay) and changed the values. I also fixed the other issue you mentioned by putting async { } inside the delay blocks instead of wrapping them.

from opal-jquery.

Just to follow up, the reason that method deals with seconds instead of milliseconds is that it feels a bit more consistent with other ruby/rails libs:

# ruby corelib
sleep(1)    # => 1 second

# rails monkey patching
12.seconds.to_i   # => 12

from opal-jquery.

Yeah, I think it just threw me seeing "set_timeout" since that has such a specific usage in Javascript. And apparently I wasn't the only one confused since the tests weren't working. Less confusing to call it delay. :) Thanks again for the tips.

from opal-jquery.