Merge pull request elastic#7513 from ycombinator/emit-new-status-on-c…

…hange

Emit new state + message as well, on status change

src/server/status/__tests__/status.js

@@ -21,15 +21,15 @@ describe('Status class', function () {
   it('emits change when the status is set', function (done) {
     let status = serverStatus.createForPlugin(plugin);
 
-    status.once('change', function (prev, prevMsg) {
-      expect(status.state).to.be('green');
-      expect(status.message).to.be('GREEN');
-      expect(prev).to.be('uninitialized');
-
-      status.once('change', function (prev, prevMsg) {
-        expect(status.state).to.be('red');
-        expect(status.message).to.be('RED');
-        expect(prev).to.be('green');
+    status.once('change', function (prevState, prevMsg, newState, newMsg) {
+      expect(newState).to.be('green');
+      expect(newMsg).to.be('GREEN');
+      expect(prevState).to.be('uninitialized');
+
+      status.once('change', function (prevState, prevMsg, newState, newMsg) {
+        expect(newState).to.be('red');
+        expect(newMsg).to.be('RED');
+        expect(prevState).to.be('green');
         expect(prevMsg).to.be('GREEN');
 
         done();

src/server/status/status.js

@@ -82,8 +82,8 @@ states.all.forEach(function (state) {
       return;
     }
 
-    this.emit(state.id, previous, previousMsg);
-    this.emit('change', previous, previousMsg);
+    this.emit(state.id, previous, previousMsg, this.state, this.message);
+    this.emit('change', previous, previousMsg, this.state, this.message);
   };
 });