[MM-44765] Add API method for getting post's information from permalink (#21518)
* add api for getting post's information * add information about current user state Co-authored-by: Mattermod <mattermod@users.noreply.github.com>
Этот коммит содержится в:
Konstantinos Pittas
коммит произвёл
GitHub
GitHub
родитель
0c64252c9f
Коммит
3059abdd88
22
api4/post.go
22
api4/post.go
@@ -23,6 +23,7 @@ func (api *API) InitPost() {
|
||||
api.BaseRoutes.Posts.Handle("/ids", api.APISessionRequired(getPostsByIds)).Methods("POST")
|
||||
api.BaseRoutes.Posts.Handle("/ephemeral", api.APISessionRequired(createEphemeralPost)).Methods("POST")
|
||||
api.BaseRoutes.Post.Handle("/thread", api.APISessionRequired(getPostThread)).Methods("GET")
|
||||
api.BaseRoutes.Post.Handle("/info", api.APISessionRequired(getPostInfo)).Methods("GET")
|
||||
api.BaseRoutes.Post.Handle("/files/info", api.APISessionRequired(getFileInfosForPost)).Methods("GET")
|
||||
api.BaseRoutes.PostsForChannel.Handle("", api.APISessionRequired(getPostsForChannel)).Methods("GET")
|
||||
api.BaseRoutes.PostsForUser.Handle("/flagged", api.APISessionRequired(getFlaggedPostsForUser)).Methods("GET")
|
||||
@@ -1055,3 +1056,24 @@ func getFileInfosForPost(c *Context, w http.ResponseWriter, r *http.Request) {
|
||||
w.Header().Set(model.HeaderEtagServer, model.GetEtagForFileInfos(infos))
|
||||
w.Write(js)
|
||||
}
|
||||
|
||||
func getPostInfo(c *Context, w http.ResponseWriter, r *http.Request) {
|
||||
c.RequirePostId()
|
||||
if c.Err != nil {
|
||||
return
|
||||
}
|
||||
|
||||
info, appErr := c.App.GetPostInfo(c.AppContext, c.Params.PostId)
|
||||
if appErr != nil {
|
||||
c.Err = appErr
|
||||
return
|
||||
}
|
||||
|
||||
js, err := json.Marshal(info)
|
||||
if err != nil {
|
||||
c.Err = model.NewAppError("getPostInfo", "api.marshal_error", nil, "", http.StatusInternalServerError).Wrap(err)
|
||||
return
|
||||
}
|
||||
|
||||
w.Write(js)
|
||||
}
|
||||
|
||||
@@ -3343,6 +3343,262 @@ func TestPostReminder(t *testing.T) {
|
||||
require.Truef(t, caught, "User should have received %s event", model.WebsocketEventEphemeralMessage)
|
||||
}
|
||||
|
||||
func TestPostGetInfo(t *testing.T) {
|
||||
th := Setup(t).InitBasic()
|
||||
defer th.TearDown()
|
||||
client := th.Client
|
||||
sysadminClient := th.SystemAdminClient
|
||||
sysadminClient.AddTeamMember(th.BasicTeam.Id, th.SystemAdminUser.Id)
|
||||
|
||||
openChannel, _, err := client.CreateChannel(&model.Channel{TeamId: th.BasicTeam.Id, Type: model.ChannelTypeOpen, Name: "open-channel", DisplayName: "Open Channel"})
|
||||
require.NoError(t, err)
|
||||
sysadminClient.AddChannelMember(openChannel.Id, th.SystemAdminUser.Id)
|
||||
openPost, _, err := client.CreatePost(&model.Post{ChannelId: openChannel.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
privateChannel, _, err := sysadminClient.CreateChannel(&model.Channel{TeamId: th.BasicTeam.Id, Type: model.ChannelTypePrivate, Name: "private-channel", DisplayName: "Private Channel"})
|
||||
require.NoError(t, err)
|
||||
privatePost, _, err := sysadminClient.CreatePost(&model.Post{ChannelId: privateChannel.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
privateChannelBasicUser, _, err := client.CreateChannel(&model.Channel{TeamId: th.BasicTeam.Id, Type: model.ChannelTypePrivate, Name: "private-channel-basic-user", DisplayName: "Private Channel - Basic User"})
|
||||
require.NoError(t, err)
|
||||
privatePostBasicUser, _, err := client.CreatePost(&model.Post{ChannelId: privateChannelBasicUser.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
user3 := th.CreateUser()
|
||||
gmChannel, _, err := client.CreateGroupChannel([]string{th.BasicUser.Id, th.BasicUser2.Id, user3.Id})
|
||||
require.NoError(t, err)
|
||||
gmPost, _, err := client.CreatePost(&model.Post{ChannelId: gmChannel.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
dmChannel, _, err := client.CreateDirectChannel(th.BasicUser.Id, th.BasicUser2.Id)
|
||||
require.NoError(t, err)
|
||||
dmPost, _, err := client.CreatePost(&model.Post{ChannelId: dmChannel.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
openTeam, _, err := sysadminClient.CreateTeam(&model.Team{Type: model.TeamOpen, Name: "open-team", DisplayName: "Open Team"})
|
||||
require.NoError(t, err)
|
||||
openTeamOpenChannel, _, err := sysadminClient.CreateChannel(&model.Channel{TeamId: openTeam.Id, Type: model.ChannelTypeOpen, Name: "open-team-open-channel", DisplayName: "Open Team - Open Channel"})
|
||||
require.NoError(t, err)
|
||||
openTeamOpenPost, _, err := sysadminClient.CreatePost(&model.Post{ChannelId: openTeamOpenChannel.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
// Alt team is a team without the sysadmin in it.
|
||||
altOpenTeam, _, err := client.CreateTeam(&model.Team{Type: model.TeamOpen, Name: "alt-open-team", DisplayName: "Alt Open Team"})
|
||||
require.NoError(t, err)
|
||||
altOpenTeamOpenChannel, _, err := client.CreateChannel(&model.Channel{TeamId: altOpenTeam.Id, Type: model.ChannelTypeOpen, Name: "alt-open-team-open-channel", DisplayName: "Open Team - Open Channel"})
|
||||
require.NoError(t, err)
|
||||
altOpenTeamOpenPost, _, err := client.CreatePost(&model.Post{ChannelId: altOpenTeamOpenChannel.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
inviteTeam, _, err := sysadminClient.CreateTeam(&model.Team{Type: model.TeamInvite, Name: "invite-team", DisplayName: "Invite Team"})
|
||||
require.NoError(t, err)
|
||||
inviteTeamOpenChannel, _, err := sysadminClient.CreateChannel(&model.Channel{TeamId: inviteTeam.Id, Type: model.ChannelTypeOpen, Name: "invite-team-open-channel", DisplayName: "Invite Team - Open Channel"})
|
||||
require.NoError(t, err)
|
||||
inviteTeamOpenPost, _, err := sysadminClient.CreatePost(&model.Post{ChannelId: inviteTeamOpenChannel.Id})
|
||||
require.NoError(t, err)
|
||||
|
||||
testCases := []struct {
|
||||
name string
|
||||
team *model.Team
|
||||
hasJoinedTeam bool
|
||||
channel *model.Channel
|
||||
hasJoinedChannel bool
|
||||
post *model.Post
|
||||
client *model.Client4
|
||||
hasAccess bool
|
||||
}{
|
||||
// Open channel - Current Team
|
||||
{
|
||||
name: "Open post - Current team - Basic user",
|
||||
team: th.BasicTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: openChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: openPost,
|
||||
client: client,
|
||||
hasAccess: true,
|
||||
},
|
||||
{
|
||||
name: "Open post - Current team - Sysadmin user",
|
||||
team: th.BasicTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: openChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: openPost,
|
||||
client: sysadminClient,
|
||||
hasAccess: true,
|
||||
},
|
||||
|
||||
// Private channel - Current Team
|
||||
{
|
||||
name: "Private post by sysadmin - Current team - Basic user",
|
||||
team: th.BasicTeam,
|
||||
channel: privateChannel,
|
||||
post: privatePost,
|
||||
client: client,
|
||||
hasAccess: false,
|
||||
},
|
||||
{
|
||||
name: "Private post by sysadmin - Current team - Sysadmin user",
|
||||
team: th.BasicTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: privateChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: privatePost,
|
||||
client: sysadminClient,
|
||||
hasAccess: true,
|
||||
},
|
||||
{
|
||||
name: "Private post by basic user - Current team - Basic user",
|
||||
team: th.BasicTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: privateChannelBasicUser,
|
||||
hasJoinedChannel: true,
|
||||
post: privatePostBasicUser,
|
||||
client: client,
|
||||
hasAccess: true,
|
||||
},
|
||||
{
|
||||
name: "Private post by basic user - Current team - Sysadmin user",
|
||||
team: th.BasicTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: privateChannelBasicUser,
|
||||
hasJoinedChannel: false,
|
||||
post: privatePostBasicUser,
|
||||
client: sysadminClient,
|
||||
hasAccess: true,
|
||||
},
|
||||
|
||||
// GM channel
|
||||
{
|
||||
name: "GM post - Current team - Basic user",
|
||||
team: nil,
|
||||
channel: gmChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: gmPost,
|
||||
client: client,
|
||||
hasAccess: true,
|
||||
},
|
||||
{
|
||||
name: "GM post - Current team - Sysadmin user",
|
||||
team: nil,
|
||||
channel: gmChannel,
|
||||
post: gmPost,
|
||||
client: sysadminClient,
|
||||
hasAccess: false,
|
||||
},
|
||||
|
||||
// DM channel
|
||||
{
|
||||
name: "DM post - Current team - Basic user",
|
||||
team: nil,
|
||||
channel: dmChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: dmPost,
|
||||
client: client,
|
||||
hasAccess: true,
|
||||
},
|
||||
{
|
||||
name: "DM post - Current team - Sysadmin user",
|
||||
team: nil,
|
||||
channel: dmChannel,
|
||||
post: dmPost,
|
||||
client: sysadminClient,
|
||||
hasAccess: false,
|
||||
},
|
||||
|
||||
// Open channel - Open Team
|
||||
{
|
||||
name: "Open post - Open team - Basic user",
|
||||
team: openTeam,
|
||||
hasJoinedTeam: false,
|
||||
channel: openTeamOpenChannel,
|
||||
hasJoinedChannel: false,
|
||||
post: openTeamOpenPost,
|
||||
client: client,
|
||||
hasAccess: true,
|
||||
},
|
||||
{
|
||||
name: "Open post - Open team - Sysadmin user",
|
||||
team: openTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: openTeamOpenChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: openTeamOpenPost,
|
||||
client: sysadminClient,
|
||||
hasAccess: true,
|
||||
},
|
||||
|
||||
// Open channel - Alt Open Team
|
||||
{
|
||||
name: "Open post - Alt open team - Basic user",
|
||||
team: altOpenTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: altOpenTeamOpenChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: altOpenTeamOpenPost,
|
||||
client: client,
|
||||
hasAccess: true,
|
||||
},
|
||||
{
|
||||
name: "Open post - Alt open team - Sysadmin user",
|
||||
team: altOpenTeam,
|
||||
hasJoinedTeam: false,
|
||||
channel: altOpenTeamOpenChannel,
|
||||
hasJoinedChannel: false,
|
||||
post: altOpenTeamOpenPost,
|
||||
client: sysadminClient,
|
||||
hasAccess: true,
|
||||
},
|
||||
|
||||
// Open channel - Invite Team
|
||||
{
|
||||
name: "Open post - Invite team - Basic user",
|
||||
team: inviteTeam,
|
||||
channel: inviteTeamOpenChannel,
|
||||
post: inviteTeamOpenPost,
|
||||
client: client,
|
||||
hasAccess: false,
|
||||
},
|
||||
{
|
||||
name: "Open post - Invite team - Sysadmin user",
|
||||
team: inviteTeam,
|
||||
hasJoinedTeam: true,
|
||||
channel: inviteTeamOpenChannel,
|
||||
hasJoinedChannel: true,
|
||||
post: inviteTeamOpenPost,
|
||||
client: sysadminClient,
|
||||
hasAccess: true,
|
||||
},
|
||||
}
|
||||
|
||||
for _, tc := range testCases {
|
||||
t.Run(tc.name, func(t *testing.T) {
|
||||
info, resp, err := tc.client.GetPostInfo(tc.post.Id)
|
||||
if !tc.hasAccess {
|
||||
require.Error(t, err)
|
||||
CheckNotFoundStatus(t, resp)
|
||||
return
|
||||
}
|
||||
|
||||
require.NoError(t, err)
|
||||
CheckOKStatus(t, resp)
|
||||
require.Equal(t, tc.channel.Id, info.ChannelId)
|
||||
require.Equal(t, tc.channel.Type, info.ChannelType)
|
||||
require.Equal(t, tc.channel.DisplayName, info.ChannelDisplayName)
|
||||
require.Equal(t, tc.hasJoinedChannel, info.HasJoinedChannel)
|
||||
if tc.team != nil {
|
||||
require.Equal(t, tc.team.Id, info.TeamId)
|
||||
require.Equal(t, tc.team.Type, info.TeamType)
|
||||
require.Equal(t, tc.team.DisplayName, info.TeamDisplayName)
|
||||
require.Equal(t, tc.hasJoinedTeam, info.HasJoinedTeam)
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
|
||||
func TestAcknowledgePost(t *testing.T) {
|
||||
th := Setup(t).InitBasic()
|
||||
defer th.TearDown()
|
||||
|
||||
Ссылка в новой задаче
Block a user