1218679 - Integration tests for memory tool

Status: RESOLVED FIXED

(DevTools :: Memory, defect)

Description

[Jordan Santell [:jsantell] [@jsantell]]

We need to add browser mochitests for some common workflows, using simulated clicks, etc.

* Taking a snapshot, changing breakdown
* Turning on recording allocations, take a snapshot, invert
* Something with multiple snapshots
* In th event of an error, can take more snapshots (error can be simulated via action creator, possibly similar to bug 1218678

We need this before shipping to ensure robustness outside of our xpcshell tests.

Comment 1

[Jordan Santell [:jsantell] [@jsantell]]

Attachment: 1218679-integration-memory.patch

Handles the breakdown changes, taking snapshots. Still needs something for inverting the tree, probably recording allocations from other bugs, error handling. This is just the core though, necessary.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c9953fb83919

Comment 2

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Comment on attachment 8679661 [details] [diff] [review]
1218679-integration-memory.patch

Review of attachment 8679661 [details] [diff] [review]:
-----------------------------------------------------------------

v nice

::: devtools/client/memory/constants.js
@@ +29,5 @@
>  // Fired to toggle tree inversion on or off.
>  actions.TOGGLE_INVERTED = "toggle-inverted";
>  
> +// Fired to set a new breakdown.
> +actions.SET_BREAKDOWN = "set-breakdown";

lol

::: devtools/client/memory/initializer.js
@@ +37,5 @@
>    });
>  }
>  
>  function destroy () {
>    return Task.spawn(function*(){});

Should we set the globals to null here?

::: devtools/client/memory/test/browser/browser_memory-breakdowns-01.js
@@ +1,5 @@
> +/* Any copyright is dedicated to the Public Domain.
> +   http://creativecommons.org/publicdomain/zero/1.0/ */
> +
> +/**
> + * Tests that the heap tree renders rows based on the breakdown

nit: period

::: devtools/client/memory/test/browser/head.js
@@ +66,5 @@
>      finish();
>    });
>  }
> +
> +function waitUntilState (store, predicate) {

Can you make a test-utils.js module so we can share this between test suites?

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/5f17f4325f31

Comment 4

[Carsten Book [:Tomcat]]

sorry had to back this out in https://treeherder.mozilla.org/#/jobs?repo=fx-team&revision=8edcaa544e24 since one of this changes caused test failures on windows and OS X like https://treeherder.mozilla.org/logviewer.html#?job_id=5401492&repo=fx-team

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/ae06ef1f37db

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ae06ef1f37db