Marshalling nested JS Date to rb Time not working

[kliuless]

Related to #12

require 'bundler/inline'
gemfile do
  gem 'mini_racer', '= 0.2.4'
end

ctx = MiniRacer::Context.new
ctx.eval( <<~JSEVAL )
  var extractDate = (nestedObj) => nestedObj.abc[0].constructor.name;
JSEVAL

# PASS: returns "Date" (rb => js conversion happens even when `Time` is nested)
p ctx.call('extractDate', {abc: [Time.now]})

# PASS: returns `Time` (js => rb conversion happens when a `Date` isn't nested)
p ctx.eval(%q|new Date()|).class

# FAIL: returns `String` (js => rb conversion doesn't happen for nested `Date`s)
p ctx.eval(%q|( {abc: new Date()} )|)['abc'].class

[SamSaffron]

@seanmakesgames can you have a look?

[seanmakesgames]

yeah. I can take a look.

[seanmakesgames]

Sorry for the lack of comms on this one. It's been a full couple weeks. I was able to take a look at it, looks like this case is traveling down a different code path which I'm less familiar with and so need some more time to dig through it. In the meantime, if you marshal it with a proc, it should work as expected. (you can take a look at https://github.com/discourse/mini_racer/blob/master/test/mini_racer_test.rb#L214 for an example)

Not sure if that works for you, but worth a shot. :)

[kliuless]

No rush - this probably isn't something I'll need in the near future. I discovered the issue because I'm using the rb => js nested conversion (which works perfectly), and was curious to see whether it worked in the other direction.

[tisba]

Might be related to #136