feat(CLOUDDEV-432): added new format for test #132
Conversation
Test Results8 tests 7 ✅ 6m 42s ⏱️ For more details on these failures, see this check. Results for commit 2247cc6. ♻️ This comment has been updated with latest results. |
Uiopas
force-pushed
the
feat/CLOUDDEV-432-added-test-instance
branch
2 times, most recently
from
eda72ed to
f6075ed
Compare
Uiopas
force-pushed
the
feat/CLOUDDEV-432-added-test-instance
branch
from
f6075ed to
c8c54a9
Compare
|
|
||
| - name: Tests data_source |
There was a problem hiding this comment.
Старую версию сразу убирать не надо. Будем постепенно убирать то, что добавили в новой версии тестов
There was a problem hiding this comment.
Так же прошу сейчас добавить динамическое создание/удаление проектов для старых тестов
| tfSecGOpts terraform.Options | ||
| ) | ||
|
|
||
| func TestMain(m *testing.M) { |
There was a problem hiding this comment.
Сейчас подумал, что при таком подходе мы лишаемся возможности локально запускать отдельные тесты. Попробуй реализовать через подход once.Do (https://golang.hotexamples.com/examples/sync/Once/Do/golang-once-do-method-examples.html)Добавив вызов функции подготовки окружения в каждый тест. Тогда функция подготовки окружения будет вызвана один раз (что сэкономит время), при этом будет возможность локального запуска отдельных тестов. В таком случае функциюTestCreatePrepareResourcesForTerraformEdgeCenterInstance нужно будет переименовать в PrepareResourcesForTerraformEdgeCenterInstance
| // Инициализация и применение конфигурации сетевого модуля. | ||
| initializeNetworkModule(&tfNetOpts) | ||
| applyModule(t, &tfNetOpts, "network") | ||
| // Получение и сохранение идентификаторов сети и подсети. |
There was a problem hiding this comment.
Комменты в коде лучше все-таки на английском сделать
| defer os.Exit(code) | ||
| } | ||
|
|
||
| func TestCreatePrepareResourcesForTerraformEdgeCenterInstance(t *testing.T) { |
There was a problem hiding this comment.
Кажется, что слово Create в названии лишнее
|
|
||
| func TestCreatePrepareResourcesForTerraformEdgeCenterInstance(t *testing.T) { | ||
| // Инициализация и применение конфигурации сетевого модуля. | ||
| initializeNetworkModule(&tfNetOpts) |
There was a problem hiding this comment.
В данных функциях, на мой взгляд, больше подходят названия типа "initializeNetworkTfOpts"
| if err != nil { | ||
| t.Fatalf("failed to get output %s: %v", key, err) | ||
| } | ||
| require.Equal(t, expectedValue, output, fmt.Sprintf("%s should be updated to the new value", key)) |
There was a problem hiding this comment.
не обязательно updated
| return | ||
| } | ||
|
|
||
| fmt.Println(project.ID) |
There was a problem hiding this comment.
Чуть дополнить вывод fmt.Sprintf("Created project ID: %d", project.ID)
|
|
||
| fmt.Printf("Статус ответа: %s\n", resp.Status) | ||
| fmt.Printf("Тело ответа: %s\n", string(body)) | ||
| } |
There was a problem hiding this comment.
Хорошо бы вывести сообщение что проект с таким-то ID успешно удален
| return | ||
| } | ||
|
|
||
| baseURL := fmt.Sprintf("https://api.edgecenter.online/cloud/v1/projects/%s", projectID) |
There was a problem hiding this comment.
Хостнейм лучше бы вывести в переменную среды, чтобы можно было запускать в разных окружениях
| } | ||
|
|
||
| func main() { | ||
| baseURL := "https://api.edgecenter.online/cloud/v1/projects" |
There was a problem hiding this comment.
Хостнейм лучше бы вывести в переменную среды, чтобы можно было запускать в разных окружениях
No description provided.