1502131 - Updated and re-enable old debugger tests in shared

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement, P3)

Description

[David Walsh :davidwalsh]

A handful of tests were temporarily disabled and moved to the devtools/client/shared/tests directory so that we could remove the old debugger and elevate the new debugger:

browser_dbg_promises-allocation-stack.js
browser_dbg_promises-chrome-allocation-stack.js
browser_dbg_promises-fulfillment-stack.js
browser_dbg_promises-rejection-stack.js
browser_dbg_terminate-on-tab-close.js

Along with assets:

doc_promise-get-allocation-stack.html
doc_promise-get-fulfillment-stack.html
doc_promise-get-rejection-stack.html
doc_terminate-on-tab-close.html

We should update the code in those tests so that we can reenable them.  The utility file they used was:

https://hg.mozilla.org/mozilla-central/file/37d240a1d498/devtools/client/debugger/test/mochitest/head.js

Comment 1

[Chujun Lu]

Hey David, I searched for the test file names and found that they are now in the devtools/server/tests/browser directory. In devtools/server/tests/browser/browser.ini, those tests aren't disabled. I'm wondering if this bug is still valid, or do I miss anything? Thanks!

Comment 2

[David Walsh :davidwalsh]

It looks like this is done! Awesome!