FIX integrations are now listed in users

Itiviti · Aug 22, 2016 · ed61f2b · ed61f2b · redcraft · Aug 22, 2016
1 parent 987e2ae
commit ed61f2b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/sources/src/main/java/com/ullink/slack/simpleslackapi/impl/SlackJSONSessionStatusParser.java b/sources/src/main/java/com/ullink/slack/simpleslackapi/impl/SlackJSONSessionStatusParser.java
@@ -81,9 +81,11 @@ void parse() throws ParseException {
             for (Object jsonObject : integrationsJson)
             {
                 JSONObject jsonIntegration = (JSONObject) jsonObject;
+                SlackUser slackUser = SlackJSONParsingUtils.buildSlackUser(jsonIntegration);
                 SlackIntegration slackIntegration = SlackJSONParsingUtils.buildSlackIntegration(jsonIntegration);
                 LOGGER.debug("slack integration found : " + slackIntegration.getId());
                 integrations.put(slackIntegration.getId(), slackIntegration);
+                users.put(slackUser.getId(), slackUser);
             }
         }
 

diff --git a/.../src/test/java/com/ullink/slack/simpleslackapi/impl/TestSlackJSONSessionStatusParser.java b/.../src/test/java/com/ullink/slack/simpleslackapi/impl/TestSlackJSONSessionStatusParser.java
@@ -26,7 +26,7 @@ public void testParsingSessionDescription() throws Exception
         parser.parse();
 
         assertThat(parser.getChannels()).containsOnlyKeys("CHANNELID1", "CHANNELID2", "CHANNELID3", "GROUPID1", "DIM01");
-        assertThat(parser.getUsers()).containsOnlyKeys("USERID1","USERID2","USERID3","USERID4","BOTID1","BOTID2");
+        assertThat(parser.getUsers()).containsOnlyKeys("USERID1","USERID2","USERID3","USERID4","BOTID1","BOTID2","INTEGRATION1","INTEGRATION2");
         assertThat(parser.getWebSocketURL()).isEqualTo("wss://mywebsocketurl");
         assertThat(parser.getUsers().get("USERID1").getTimeZone()).isEqualTo("Europe/Amsterdam");
         assertThat(parser.getUsers().get("USERID1").getTimeZoneLabel()).isEqualTo("Central European Summer Time");