Merge pull request #1285 from amihaiemil/1281

#1281 updated messages
self-xdsd · Nov 14, 2021 · 012d1ec · 012d1ec
2 parents 52f2e11 + 82930ca
commit 012d1ec
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/self-core-impl/src/test/java/com/selfxdsd/core/managers/ConfusedTestCase.java b/self-core-impl/src/test/java/com/selfxdsd/core/managers/ConfusedTestCase.java
@@ -73,7 +73,8 @@ public void returnsStepForConfused() {
         );
 
         Mockito.verify(comments).post("> " + "gibberish" + "\n\n"
-            + "@mihai didn't understand that, sorry.");
+            + "@mihai I'm afraid I didn't understand that, sorry. "
+            + "Say \"commands\" for a full list of things I understand.");
     }
 
     /**

diff --git a/self-core-impl/src/test/java/com/selfxdsd/core/managers/DeregisterTestCase.java b/self-core-impl/src/test/java/com/selfxdsd/core/managers/DeregisterTestCase.java
@@ -152,8 +152,8 @@ public void authorCanNotDeregisterTask(){
         Mockito.verify(issue.comments(), Mockito.times(1))
             .post("> Deregister please!\n\n"
                 + "@john-dev you don't have the appropriate role "
-                + "to remove this task.\n\n"
-                + "Only users with PO or ARCH roles are allowed.");
+                + "to remove this task from scope.\n\n"
+                + "Only users with PO or ARCH roles are allowed to do that.");
     }
 
     /**

diff --git a/self-core-impl/src/test/resources/responses_en.properties b/self-core-impl/src/test/resources/responses_en.properties
@@ -33,11 +33,11 @@ resigned.comment=@%s ok, I'll find someone else to take care of it.
 cannotResign.comment=@%s this task is not assigned to you at the moment.\
                      Only the task's assignee can ask for resignation.
 deregister.comment=@%s ok, I've removed this task from scope. I'm not managing it anymore.
-cannotDeregister.comment=@%s you don't have the appropriate role to remove this task.\n\n\
-                         Only users with PO or ARCH roles are allowed.
+cannotDeregister.comment=@%s you don't have the appropriate role to remove this task from scope.\n\n\
+                         Only users with PO or ARCH roles are allowed to do that.
 taskNotRegistered.comment=@%s this ticket is not registered as a task, therefore I'm not working on it \
                           at the moment.\n\n\
-                          If you want me to take care of it, assign it to me or say "register".
+                          If you want me to take care of it, just say "add" or "register".
 taskNotRegisteredTicketClosed.comment=@%s this ticket is not registered as a task, therefore I'm not working on it at the moment.\n\n\
                                       If you want me to take care of it, please reopen it. However, reopening Issues is not encouraged -- \
                                       it's better if you open new tickets instead.
@@ -46,7 +46,7 @@ taskNotAssigned.comment=@%s this ticket is registered as a task, I will assign i
 taskIsAssigned.comment=@%s this ticket is assigned to @%s since ``%s`` and it should be closed \
                        until ``%s``. Estimation is ``%s`` minutes.
 mustBeContributor.comment=@%s only project contributors can give this command.
-misunderstand.comment=@%s didn't understand that, sorry.
+misunderstand.comment=@%s I'm afraid I didn't understand that, sorry. Say "commands" for a full list of things I understand.
 taskAlreadyRegistered.comment=@%s this issue is already registered as a task.
 taskRegistered.comment=@%s I've just registered this ticket as a task and will assign it to someone soon. Thanks!
 taskInvoiced.comment=@%s thank you for resolving this ticket. I've just added it to your active invoice. You can always \