mirrors/mostlymatter
1
0
Форкнуть 0
Код Задачи Запросы на слияние Действия Пакеты Проекты Релизы Вики Активность

Fixing unit tests for Go 1.10 (#8568)

Просмотр исходного кода
Этот коммит содержится в:
Christopher Speller
2018-04-03 12:41:02 -07:00
коммит произвёл Joram Wilander
родитель 1476ca5b2d
Коммит 80925f7cfd
3 изменённых файлов: 16 добавлений и 2 удалений
Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

4
Makefile
Просмотреть файл

@@ -336,6 +336,7 @@ ifeq ($(BUILD_ENTERPRISE_READY),true)
endif
test-server-race: test-te-race test-ee-race ## Checks for race conditions.
find . -type d -name data -not -path './vendor/*' | xargs rm -rf
do-cover-file: ## Creates the test coverage report file.
@echo "mode: count" > cover.out
@@ -360,6 +361,7 @@ ifeq ($(BUILD_ENTERPRISE_READY),true)
endif
test-server: test-te test-ee ## Runs tests.
find . -type d -name data -not -path './vendor/*' | xargs rm -rf
internal-test-web-client: ## Runs web client tests.
$(GO) run $(GOFLAGS) $(PLATFORM_FILES) test web_client_tests
@@ -477,7 +479,7 @@ clean: stop-docker ## Clean up everything except persistant server data.
cd $(BUILD_WEBAPP_DIR) && $(MAKE) clean
find . -type d -name data -not -path './vendor/*' | xargs rm -r
find . -type d -name data -not -path './vendor/*' | xargs rm -rf
rm -rf logs
rm -f mattermost.log

11
api4/command_test.go
Просмотреть файл

@@ -214,6 +214,17 @@ func TestListCommands(t *testing.T) {
defer th.TearDown()
Client := th.Client
enableCommands := *th.App.Config().ServiceSettings.EnableCommands
enableOnlyAdminIntegrations := *th.App.Config().ServiceSettings.EnableOnlyAdminIntegrations
defer func() {
th.App.UpdateConfig(func(cfg *model.Config) { cfg.ServiceSettings.EnableCommands = &enableCommands })
th.App.UpdateConfig(func(cfg *model.Config) {
cfg.ServiceSettings.EnableOnlyAdminIntegrations = &enableOnlyAdminIntegrations
})
}()
th.App.UpdateConfig(func(cfg *model.Config) { *cfg.ServiceSettings.EnableCommands = true })
th.App.UpdateConfig(func(cfg *model.Config) { *cfg.ServiceSettings.EnableOnlyAdminIntegrations = true })
newCmd := &model.Command{
CreatorId: th.BasicUser.Id,
TeamId: th.BasicTeam.Id,

3
api4/file_test.go
Просмотреть файл

@@ -359,7 +359,8 @@ func TestGetFileHeaders(t *testing.T) {
t.Run("js", testHeaders(data, "test.js", "text/plain", false))
t.Run("go", testHeaders(data, "test.go", "application/octet-stream", false))
t.Run("zip", testHeaders(data, "test.zip", "application/zip", false))
t.Run("exe", testHeaders(data, "test.exe", "application/x-ms", false))
// Not every platform can recognize these
//t.Run("exe", testHeaders(data, "test.exe", "application/x-ms", false))
t.Run("no extension", testHeaders(data, "test", "application/octet-stream", false))
t.Run("no extension 2", testHeaders([]byte("<html></html>"), "test", "application/octet-stream", false))
}