per group tickets

rngadam · rngadam · commit b2997cf2b163 · 2017-02-03T18:39:22.000+08:00
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -168,7 +168,7 @@ module.exports = {
         "no-throw-literal": "error",
         "no-trailing-spaces": "error",
         "no-undef-init": "error",
-        "no-undefined": "error",
+        "no-undefined": "off",
         "no-underscore-dangle": "error",
         "no-unmodified-loop-condition": "error",
         "no-unneeded-ternary": "error",
diff --git a/support.js b/support.js
@@ -70,23 +70,37 @@ const actions = [
     reply: (message, output) => JSON.stringify(output, null, 4),
 },
 {
-    action: (data, _message, username) => ({
-            'tickets': data.tickets,
-            'username': username.toLowerCase()
+    action: (data, message, username) => ({
+            'tickets': ticket.filterTicketsAnd(
+                username ? ticket.userNameFilter(
+                    data.tickets, username) : data.tickets
+            , {
+                'roomName': message.roomName,
+                'status': 'open'
+            }),
     }),
     regexp: /todo ?(\w*)/i,
-    reply: (message, output) => ticket.showTickets(
-        output.tickets, ['open'], output.username),
+    reply: (message, output) => ticket.showTickets(output.tickets),
 },
 {
-    action: (data, _message) => data.tickets,
+    action: (data, message) => ({
+            'tickets': ticket.filterTicketsOr(
+                ticket.userNameFilter(data.tickets, message.userName)
+            , {
+                status: 'open'
+            })
+    }),
     regexp: /mine/i,
-    reply: (message, output) => ticket.showTickets(output, ['open'], message.userName),
+    reply: (message, output) => ticket.showTickets(output.tickets),
 },
 {
-    action: (data, _message) => data.tickets,
+    action: (data, message) => ({
+            'tickets': ticket.filterTicketsOr(data.tickets, {
+                'roomName': message.roomName,
+            }),
+    }),
     regexp: /history/i,
-    reply: (message, output) => ticket.showTickets(output, ['open', 'closed']),
+    reply: (message, output) => ticket.showTickets(output.tickets),
 },
 {
     action: (data, _message, id) => ticket.forgetTicket(data.tickets, id),
diff --git a/test/support_test.js b/test/support_test.js
@@ -54,6 +54,40 @@ function twoTickets() {
     }
 }
 
+function threeTickets() {
+    return {
+        tickets: {
+            '1': {
+                assignee: null,
+                closed: null,
+                content: 'content',
+                id: '1',
+                requester: 'Ricky',
+                roomName: 'MyRoom',
+                status: 'open',
+            },
+            '2': {
+                assignee: null,
+                closed: null,
+                content: 'content',
+                id: '2',
+                requester: 'Dmitry',
+                roomName: 'MyRoom',
+                status: 'open',
+            },
+            '3': {
+                assignee: 'Ricky',
+                closed: null,
+                content: 'content',
+                id: '3',
+                requester: 'Dmitry',
+                roomName: 'OtherRoom',
+                status: 'open',
+            },
+            lastId: 3,
+        }
+    }
+}
 function emptyData() {
     return {
         tickets: {
@@ -166,6 +200,13 @@ describe('support', function() {
         assert.match(reply, /I don't understand the question/)
     })
 
+    it('TODO excludes other rooms ticket', function() {
+        const reply = support.process(threeTickets(), message('todo'))
+        assert.match(reply, /#1/)
+        assert.match(reply, /#2/)
+        assert.notMatch(reply, /#3/)
+    })
+
     it('inventory', function() {
         const data = emptyData()
         data.fs = {
