1017029 - [appmgr v2] Validation is not triggered after the file manifest has changed

Status: RESOLVED FIXED

(DevTools Graveyard :: WebIDE, defect)

Comment 1

[Paul Rouget [:paul]]

Attachment: v1

Small fix, big test.

Comment 2

[Alexandre Poirot [:ochameau] [PTO back August 21]]

Comment on attachment 8430091 [details] [diff] [review]
v1

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

Can you ensure running this test on try before landing?
The previous try run you gave on bug 1007218 didn't seem to be running webide test:
  https://tbpl.mozilla.org/?tree=Try&rev=b27aaed038bc

::: browser/devtools/webide/test/test_manifestUpdate.html
@@ +35,5 @@
> +
> +            let project = win.AppManager.selectedProject;
> +
> +            let deferred = promise.defer();
> +            win.UI.projecteditor.once("onEditorCreated", deferred.resolve);

What about registering the listener before the call that is going to dispatch the event (importPackagedApp?)?
Is it Task-proof?

@@ +59,5 @@
> +            let json = JSON.parse(data);
> +            is(json.name, "xxx", "manifest written on disc");
> +
> +            // Make the manifest invalid on disk
> +            delete json.name

miss `;`

Comment 3

[Paul Rouget [:paul]]

(In reply to Alexandre Poirot (:ochameau) from comment #2)
> Comment on attachment 8430091 [details] [diff] [review]
> v1
> 
> Review of attachment 8430091 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Can you ensure running this test on try before landing?
> The previous try run you gave on bug 1007218 didn't seem to be running
> webide test:
>   https://tbpl.mozilla.org/?tree=Try&rev=b27aaed038bc

I'll make sure all the tests pass in bug 1007057 (not required to land this patch).

> ::: browser/devtools/webide/test/test_manifestUpdate.html
> @@ +35,5 @@
> > +
> > +            let project = win.AppManager.selectedProject;
> > +
> > +            let deferred = promise.defer();
> > +            win.UI.projecteditor.once("onEditorCreated", deferred.resolve);
> 
> What about registering the listener before the call that is going to
> dispatch the event (importPackagedApp?)?
> Is it Task-proof?

projecteditor is not available earlier.

> @@ +59,5 @@
> > +            let json = JSON.parse(data);
> > +            is(json.name, "xxx", "manifest written on disc");
> > +
> > +            // Make the manifest invalid on disk
> > +            delete json.name
> 
> miss `;`

Ok.

Comment 4

[Paul Rouget [:paul]]

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

Comment 5

[Paul Rouget [:paul]]

Damn, I pushed and forgot to add the ';'. I don't want to push again just for this. I'll add the ';' in bug 1016924.

Comment 6

[Ed Morley [:emorley]]

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