Conversation
this.fire('changeState', { sectionA: {} }); | ||
this.fire('changeState', { | ||
sectionA: { | ||
metadata: null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it should be setting a component to null as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem.
Test PASSed. |
With Rick's comment 👍 Even without that fix the QA was good. |
search: 'apache2' } | ||
}, sectionB: {}}; | ||
var changeState = { | ||
sectionA: { metadata: null } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't you need to show the component being reset here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Components being reset elsewhere are already tested--this is just testing that metadata can be reset safely.
Test FAILed. |
Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge |
Fix the charm browser's Home button search * Set's metadata for sectionA to null on `EVT_SEARCH_GOHOME` * Adds test to test_ui_state ensuring `metadata: null` results in navigating back to '/' from a search QA with and without the `il` flag
Fix the charm browser's Home button search
EVT_SEARCH_GOHOME
metadata: null
results in navigating back to '/' from a searchQA with and without the
il
flag