[Pls send halp!] Cleaning up the test output

♻️ ♻️ ♻️ Please consider the environment before printing this Github issue. ♻️ ♻️ ♻️

Currently, the test output is very noisy – even for green builds. This makes it very difficult find any actual failures, especially on small screens and on phones.

Let’s try to clean it up!

Here are the useless output I have found based on this test run. Leave a comment about what you are looking into (🔒) so others won’t duplicate the effort.

A lot of these probably have the same root cause (for example, because of how the default Ember.onError works) and probably require some refactoring in the test infrastructure.

• Build phase
  • 'container' is imported by tmp/rollup-cache_path-I6PjJI6s.tmp/run_loop.js, but could not be resolved – treating it as an external dependency
  • 'ember-debug' is imported by tmp/rollup-cache_path-l0Z4sygU.tmp/index.js, but could not be resolved – treating it as an external dependency
  • 'assert' is imported from external module '@glimmer/util' but never used
  • 'assert' and 'SERIALIZATION_FIRST_NODE_STRING' are imported from external module '@glimmer/util' but never used (🔒 @James-Byrne)
• Test phase
  • WARNING: Library "magic" is already registered with Ember.
  • console.trace
  • DEPRECATION: Should not throw [deprecation id: test] (🔒 @Draggha )
  • (node:6689) Warning: Possible EventEmitter memory leak detected. * listeners added. Use emitter.setMaxListeners() to increase limit (🔒 @lukecoy )
  • (node:6689) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: *): Error: Navigation Timeout Exceeded: 900ms exceeded
  • Error: Element .does-not-exist not found. (🔒 @KamiKillertO )
  • Error: Catch me (🔒 @KamiKillertO )
  • Error: the error (🔒 @KamiKillertO )
  • Testing paused. Use resumeTest() to continue.
  • The library version banner on app boot (🔒 @Draggha )

    DEBUG: -------------------------------
    DEBUG: Ember  : *
    DEBUG: jQuery : *
    DEBUG: -------------------------------
    

  • Error while processing route: * (🔒 @allthesignals )
  • More context objects were passed than there are dynamic segments for the route: stink-bomb `
  • WARNING: Binding style attributes may introduce cross-site scripting vulnerabilities; please ensure that values being bound are properly escaped. For more information, including how to disable this warning, see https://emberjs.com/deprecations/v1.x/#toc_binding-style-attributes. Style affected: * (🔒 @KamiKillertO )

[acorncom]

Thanks folks, I’ve updated the issue accordingly

[chancancode]

Looks like a lot of these are addressed, I am okay closing the tracking issue and filing new ones as needed. I opened #17016 and #17017. Looks like a few of these still have outstanding PRs waiting for reviews also.