msglist [nfc]: s/maybeUpdateMessage/handleUpdateMessageEvent/

Author: chrisbobbe

lib/model/message.dart

@@ -94,7 +94,7 @@ class MessageStoreImpl with MessageStore {
 
   void handleUpdateMessageEvent(UpdateMessageEvent event) {
     for (final view in _messageListViews) {
-      view.maybeUpdateMessage(event); // TODO update mainly in [messages] instead
+      view.handleUpdateMessageEvent(event); // TODO update mainly in [messages] instead
     }
   }
 

lib/model/message_list.dart

@@ -483,7 +483,7 @@ class MessageListView with ChangeNotifier, _MessageSequence {
   /// TODO(#150): Handle message moves.
   // NB that when handling message moves (#150), recipient headers
   // may need updating, and consequently showSender too.
-  void maybeUpdateMessage(UpdateMessageEvent event) {
+  void handleUpdateMessageEvent(UpdateMessageEvent event) {
     final idx = _findMessageWithId(event.messageId);
     if (idx == -1)  {
       return;

test/model/message_list_test.dart

@@ -297,7 +297,7 @@ void main() {
     });
   });
 
-  group('maybeUpdateMessage', () {
+  group('handleUpdateMessageEvent', () {
     test('update a message', () async {
       final originalMessage = eg.streamMessage(
         content: "<p>Hello, world</p>");
@@ -317,7 +317,7 @@ void main() {
         ..flags.not((it) => it.deepEquals(updateEvent.flags))
         ..isMeMessage.not((it) => it.equals(updateEvent.isMeMessage!));
 
-      model.maybeUpdateMessage(updateEvent);
+      model.handleUpdateMessageEvent(updateEvent);
       checkNotifiedOnce();
       check(model).messages.single
         ..identicalTo(message)
@@ -337,7 +337,7 @@ void main() {
       await prepare();
       await prepareMessages(foundOldest: true, messages: [originalMessage]);
 
-      model.maybeUpdateMessage(updateEvent);
+      model.handleUpdateMessageEvent(updateEvent);
       checkNotNotified();
       check(model).messages.single
         ..content.equals(originalMessage.content)
@@ -359,7 +359,7 @@ void main() {
       await prepareMessages(foundOldest: true, messages: [originalMessage]);
       final message = model.messages.single;
 
-      model.maybeUpdateMessage(updateEvent);
+      model.handleUpdateMessageEvent(updateEvent);
       checkNotifiedOnce();
       check(model).messages.single
         ..identicalTo(message)
@@ -813,17 +813,17 @@ void main() {
     await store.handleEvent(MessageEvent(id: 0, message: streamMessage(14)));
     checkNotifiedOnce();
 
-    // Then test maybeUpdateMessage, where a header is and remains needed…
+    // Then test handleUpdateMessageEvent, where a header is and remains needed…
     UpdateMessageEvent updateEvent(Message message) => eg.updateMessageEditEvent(
       message, renderedContent: '${message.content}<p>edited</p>',
     );
-    model.maybeUpdateMessage(updateEvent(model.messages.first));
+    model.handleUpdateMessageEvent(updateEvent(model.messages.first));
     checkNotifiedOnce();
-    model.maybeUpdateMessage(updateEvent(model.messages[model.messages.length - 2]));
+    model.handleUpdateMessageEvent(updateEvent(model.messages[model.messages.length - 2]));
     checkNotifiedOnce();
 
     // … and where it's not.
-    model.maybeUpdateMessage(updateEvent(model.messages.last));
+    model.handleUpdateMessageEvent(updateEvent(model.messages.last));
     checkNotifiedOnce();
 
     // Then test reassemble.