MM-43341: Fix flaky TestImportImportUser (#20140)

We use the specifically created NewTestId
to circumvent any clashes with reserved words.

https://mattermost.atlassian.net/browse/MM-43341

```release-note
NONE
```
This commit is contained in:
Agniva De Sarker 2022-05-05 20:03:37 +05:30 committed by GitHub
parent 8358f92786
commit 2c5f816e02
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -1384,7 +1384,7 @@ func TestImportImportUser(t *testing.T) {
require.NoError(t, nErr, "Failed to import scheme")
teamData := &TeamImportData{
Name: ptrStr(model.NewId()),
Name: ptrStr(NewTestId()),
DisplayName: ptrStr("Display Name"),
Type: ptrStr("O"),
Description: ptrStr("The team description."),
@ -1398,7 +1398,7 @@ func TestImportImportUser(t *testing.T) {
channelData := &ChannelImportData{
Team: &teamName,
Name: ptrStr(model.NewId()),
Name: ptrStr(NewTestId()),
DisplayName: ptrStr("Display Name"),
Type: &chanTypeOpen,
Header: ptrStr("Channel Header"),
@ -1943,7 +1943,7 @@ func TestImportimportMultiplePostLines(t *testing.T) {
require.Nil(t, err, "Failed to get team from database.")
// Create a Channel.
channelName := model.NewId()
channelName := NewTestId()
chanTypeOpen := model.ChannelTypeOpen
th.App.importChannel(th.Context, &ChannelImportData{
Team: &teamName,
@ -2022,7 +2022,7 @@ func TestImportimportMultiplePostLines(t *testing.T) {
data = LineImportWorkerData{
LineImportData{
Post: &PostImportData{
Team: ptrStr(model.NewId()),
Team: ptrStr(NewTestId()),
Channel: &channelName,
User: &username,
Message: ptrStr("Message"),
@ -2043,7 +2043,7 @@ func TestImportimportMultiplePostLines(t *testing.T) {
LineImportData{
Post: &PostImportData{
Team: &teamName,
Channel: ptrStr(model.NewId()),
Channel: ptrStr(NewTestId()),
User: &username,
Message: ptrStr("Message"),
CreateAt: ptrInt64(model.GetMillis()),
@ -2547,7 +2547,7 @@ func TestImportImportPost(t *testing.T) {
require.Nil(t, appErr, "Failed to get team from database.")
// Create a Channel.
channelName := model.NewId()
channelName := NewTestId()
chanTypeOpen := model.ChannelTypeOpen
th.App.importChannel(th.Context, &ChannelImportData{
Team: &teamName,
@ -2645,7 +2645,7 @@ func TestImportImportPost(t *testing.T) {
data := LineImportWorkerData{
LineImportData{
Post: &PostImportData{
Team: ptrStr(model.NewId()),
Team: ptrStr(NewTestId()),
Channel: &channelName,
User: &username,
Message: ptrStr("Message"),
@ -2665,7 +2665,7 @@ func TestImportImportPost(t *testing.T) {
LineImportData{
Post: &PostImportData{
Team: &teamName,
Channel: ptrStr(model.NewId()),
Channel: ptrStr(NewTestId()),
User: &username,
Message: ptrStr("Message"),
CreateAt: ptrInt64(model.GetMillis()),
@ -4175,7 +4175,7 @@ func TestImportPostAndRepliesWithAttachments(t *testing.T) {
require.Nil(t, appErr, "Failed to get team from database.")
// Create a Channel.
channelName := model.NewId()
channelName := NewTestId()
chanTypeOpen := model.ChannelTypeOpen
th.App.importChannel(th.Context, &ChannelImportData{
Team: &teamName,
@ -4451,7 +4451,7 @@ func TestZippedImportPostAndRepliesWithAttachments(t *testing.T) {
require.Nil(t, appErr, "Failed to get team from database.")
// Create a Channel.
channelName := model.NewId()
channelName := NewTestId()
chanTypeOpen := model.ChannelTypeOpen
th.App.importChannel(th.Context, &ChannelImportData{
Team: &teamName,