ClojureScript

remove the full inst literal test - adds 10 seconds to test runs when testing all the major JS engines

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Declined
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Activity

Hide
David Nolen added a comment - - edited

To be honest this mostly affects people doing performance work who run the tests against all three JS engines. The tests execute fairly quickly under V8, but they take a long time under JavaScriptCore and SpiderMonkey. I understand that the full tests are desireable - would be nice if there was a way to run :all or :core benchmarks, or something like that.

Show
David Nolen added a comment - - edited To be honest this mostly affects people doing performance work who run the tests against all three JS engines. The tests execute fairly quickly under V8, but they take a long time under JavaScriptCore and SpiderMonkey. I understand that the full tests are desireable - would be nice if there was a way to run :all or :core benchmarks, or something like that.
Hide
David Nolen added a comment -

This test is still very slow but necessary.

Show
David Nolen added a comment - This test is still very slow but necessary.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: