Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Add Embedding support * Excluding tests files from the typescript type checking
- Loading branch information
Showing
23 changed files
with
1,625 additions
and
75 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,195 @@ | ||
| package main | ||
|
|
||
| import ( | ||
| "encoding/json" | ||
| "strings" | ||
| "testing" | ||
|
|
||
| "github.com/mattermost/mattermost-server/v5/model" | ||
| "github.com/mattermost/mattermost-server/v5/plugin" | ||
| "github.com/mattermost/mattermost-server/v5/plugin/plugintest" | ||
| "github.com/stretchr/testify/mock" | ||
| "github.com/stretchr/testify/require" | ||
| ) | ||
|
|
||
| func TestCommandHelp(t *testing.T) { | ||
| p := Plugin{ | ||
| configuration: &configuration{ | ||
| JitsiURL: "http://test", | ||
| }, | ||
| } | ||
| apiMock := plugintest.API{} | ||
| defer apiMock.AssertExpectations(t) | ||
|
|
||
| p.SetAPI(&apiMock) | ||
|
|
||
| helpText := strings.Replace(`###### Mattermost Jitsi Plugin - Slash Command Help | ||
| * |/jitsi| - Create a new meeting | ||
| * |/jitsi [topic]| - Create a new meeting with specified topic | ||
| * |/jitsi help| - Show this help text | ||
| * |/jitsi settings| - View your current user settings for the Jitsi plugin | ||
| * |/jitsi settings [setting] [value]| - Update your user settings (see below for options) | ||
| ###### Jitsi Settings: | ||
| * |/jitsi settings embedded [true/false]|: When true, Jitsi meeting is embedded as a floating window inside Mattermost. When false, Jitsi meeting opens in a new window. | ||
| * |/jitsi settings naming_scheme [words/uuid/mattermost/ask]|: Select how meeting names are generated with one of these options: | ||
| * |words|: Random English words in title case (e.g. PlayfulDragonsObserveCuriously) | ||
| * |uuid|: UUID (universally unique identifier) | ||
| * |mattermost|: Mattermost specific names. Combination of team name, channel name and random text in public and private channels; personal meeting name in direct and group messages channels. | ||
| * |ask|: The plugin asks you to select the name every time you start a meeting`, "|", "`", -1) | ||
|
|
||
| apiMock.On("SendEphemeralPost", "test-user", &model.Post{ | ||
| UserId: "test-user", | ||
| ChannelId: "test-channel", | ||
| Message: helpText, | ||
| }).Return(nil) | ||
| response, err := p.ExecuteCommand(&plugin.Context{}, &model.CommandArgs{UserId: "test-user", ChannelId: "test-channel", Command: "/jitsi help"}) | ||
| require.Equal(t, response, &model.CommandResponse{}) | ||
| require.Nil(t, err) | ||
| } | ||
|
|
||
| func TestCommandSetings(t *testing.T) { | ||
| p := Plugin{ | ||
| configuration: &configuration{ | ||
| JitsiURL: "http://test", | ||
| JitsiEmbedded: false, | ||
| JitsiNamingScheme: "mattermost", | ||
| }, | ||
| } | ||
|
|
||
| response, err := p.ExecuteCommand(&plugin.Context{}, &model.CommandArgs{UserId: "test-user", ChannelId: "test-channel", Command: "jitsi help"}) | ||
| require.Equal(t, response, &model.CommandResponse{}) | ||
| require.Nil(t, err) | ||
|
|
||
| tests := []struct { | ||
| name string | ||
| command string | ||
| output string | ||
| newConfig *UserConfig | ||
| }{ | ||
| { | ||
| name: "set valid setting with valid value", | ||
| command: "/jitsi settings embedded true", | ||
| output: "Jitsi settings updated", | ||
| newConfig: &UserConfig{Embedded: true, NamingScheme: "mattermost"}, | ||
| }, | ||
| { | ||
| name: "set valid setting with invalid value (embedded)", | ||
| command: "/jitsi settings embedded yes", | ||
| output: "Invalid `embedded` value, use `true` or `false`.", | ||
| newConfig: nil, | ||
| }, | ||
| { | ||
| name: "set valid setting with invalid value (naming_scheme)", | ||
| command: "/jitsi settings naming_scheme yes", | ||
| output: "Invalid `naming_scheme` value, use `ask`, `english-titlecase`, `uuid` or `mattermost`.", | ||
| newConfig: nil, | ||
| }, | ||
| { | ||
| name: "set invalid setting", | ||
| command: "/jitsi settings other true", | ||
| output: "Invalid config field, use `embedded` or `naming_scheme`.", | ||
| newConfig: nil, | ||
| }, | ||
| { | ||
| name: "set invalid number of parameters", | ||
| command: "/jitsi settings other", | ||
| output: "Invalid settings parameters\n", | ||
| newConfig: nil, | ||
| }, | ||
| { | ||
| name: "get current user settings", | ||
| command: "/jitsi settings", | ||
| output: "###### Jitsi Settings:\n* Embedded: `false`\n* Naming Scheme: `mattermost`", | ||
| newConfig: nil, | ||
| }, | ||
| } | ||
|
|
||
| for _, tt := range tests { | ||
| t.Run(tt.name, func(t *testing.T) { | ||
| apiMock := plugintest.API{} | ||
| defer apiMock.AssertExpectations(t) | ||
| p.SetAPI(&apiMock) | ||
| apiMock.On("KVGet", "config_test-user", mock.Anything).Return(nil, nil) | ||
| apiMock.On("SendEphemeralPost", "test-user", &model.Post{ | ||
| UserId: "test-user", | ||
| ChannelId: "test-channel", | ||
| Message: tt.output, | ||
| }).Return(nil) | ||
| if tt.newConfig != nil { | ||
| b, _ := json.Marshal(tt.newConfig) | ||
| apiMock.On("KVSet", "config_test-user", b).Return(nil) | ||
| var nilMap map[string]interface{} | ||
| apiMock.On("PublishWebSocketEvent", configChangeEvent, nilMap, &model.WebsocketBroadcast{UserId: "test-user"}) | ||
| } | ||
| response, err := p.ExecuteCommand(&plugin.Context{}, &model.CommandArgs{UserId: "test-user", ChannelId: "test-channel", Command: tt.command}) | ||
| require.Equal(t, response, &model.CommandResponse{}) | ||
| require.Nil(t, err) | ||
| }) | ||
| } | ||
| } | ||
|
|
||
| func TestCommandStartMeeting(t *testing.T) { | ||
| p := Plugin{ | ||
| configuration: &configuration{ | ||
| JitsiURL: "http://test", | ||
| }, | ||
| } | ||
|
|
||
| t.Run("meeting without topic and ask configuration", func(t *testing.T) { | ||
| apiMock := plugintest.API{} | ||
| defer apiMock.AssertExpectations(t) | ||
| p.SetAPI(&apiMock) | ||
|
|
||
| apiMock.On("SendEphemeralPost", "test-user", mock.MatchedBy(func(post *model.Post) bool { | ||
| return post.Props["attachments"].([]*model.SlackAttachment)[0].Text == "Select type of meeting you want to start" | ||
| })).Return(nil) | ||
| apiMock.On("GetUser", "test-user").Return(&model.User{Id: "test-user"}, nil) | ||
| apiMock.On("GetChannel", "test-channel").Return(&model.Channel{Id: "test-channel"}, nil) | ||
| apiMock.On("GetUser", "test-user").Return(&model.User{Id: "test-user"}, nil) | ||
| b, _ := json.Marshal(UserConfig{Embedded: false, NamingScheme: "ask"}) | ||
| apiMock.On("KVGet", "config_test-user", mock.Anything).Return(b, nil) | ||
| config := model.Config{} | ||
| config.SetDefaults() | ||
| apiMock.On("GetConfig").Return(&config, nil) | ||
|
|
||
| response, err := p.ExecuteCommand(&plugin.Context{}, &model.CommandArgs{UserId: "test-user", ChannelId: "test-channel", Command: "/jitsi"}) | ||
| require.Equal(t, response, &model.CommandResponse{}) | ||
| require.Nil(t, err) | ||
| }) | ||
|
|
||
| t.Run("meeting without topic and no ask configuration", func(t *testing.T) { | ||
| apiMock := plugintest.API{} | ||
| defer apiMock.AssertExpectations(t) | ||
| p.SetAPI(&apiMock) | ||
|
|
||
| apiMock.On("CreatePost", mock.MatchedBy(func(post *model.Post) bool { | ||
| return strings.HasPrefix(post.Props["meeting_link"].(string), "http://test/") | ||
| })).Return(&model.Post{}, nil) | ||
| apiMock.On("GetUser", "test-user").Return(&model.User{Id: "test-user"}, nil) | ||
| apiMock.On("GetChannel", "test-channel").Return(&model.Channel{Id: "test-channel"}, nil) | ||
| apiMock.On("GetUser", "test-user").Return(&model.User{Id: "test-user"}, nil) | ||
| apiMock.On("KVGet", "config_test-user", mock.Anything).Return(nil, nil) | ||
|
|
||
| response, err := p.ExecuteCommand(&plugin.Context{}, &model.CommandArgs{UserId: "test-user", ChannelId: "test-channel", Command: "/jitsi"}) | ||
| require.Equal(t, response, &model.CommandResponse{}) | ||
| require.Nil(t, err) | ||
| }) | ||
|
|
||
| t.Run("meeting with topic", func(t *testing.T) { | ||
| apiMock := plugintest.API{} | ||
| defer apiMock.AssertExpectations(t) | ||
| p.SetAPI(&apiMock) | ||
|
|
||
| apiMock.On("CreatePost", mock.MatchedBy(func(post *model.Post) bool { return post.Props["meeting_link"] == "http://test/topic" })).Return(&model.Post{}, nil) | ||
| apiMock.On("GetUser", "test-user").Return(&model.User{Id: "test-user"}, nil) | ||
| apiMock.On("GetChannel", "test-channel").Return(&model.Channel{Id: "test-channel"}, nil) | ||
| apiMock.On("GetUser", "test-user").Return(&model.User{Id: "test-user"}, nil) | ||
| apiMock.On("KVGet", "config_test-user", mock.Anything).Return(nil, nil) | ||
|
|
||
| response, err := p.ExecuteCommand(&plugin.Context{}, &model.CommandArgs{UserId: "test-user", ChannelId: "test-channel", Command: "/jitsi topic"}) | ||
| require.Equal(t, response, &model.CommandResponse{}) | ||
| require.Nil(t, err) | ||
| }) | ||
|
|
||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.