Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Уточнение для запуска автотестов на Windows #66

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
+20 −0
Open

Уточнение для запуска автотестов на Windows #66

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
bb14642
Уточнение для запуска автотестов на Windows
gentee Aug 3, 2023
2f7f10d
Изменения по ревью
gentee Aug 4, 2023
ed00cf7
Изменения по ревью
gentee Aug 8, 2023
600246a
Убрал PATH
gentee Aug 8, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,20 @@ shortenertest -test.v -test.run=^TestIteration1$ -binary-path=cmd/shortener/shor
metricstest -test.v -test.run=^TestIteration1$ -agent-binary-path=cmd/agent/agent
```

### Запуск на Windows

Ниже приведены замечания и уточнения при запуске автотестов на Windows.

1. В команде `go build` следует указывать расширение `.exe` для результирующего файла. Например, `go build -o shortener.exe *.go`, `go build -o server.exe *.go`.
2. Скачивать файл нужно с суффиксом `-windows-amd64`. Например, `metricstest-windows-amd64.exe`, `shortenertest-windows-amd64.exe`. Лучще сразу переименовать его, убрав этот суффикс - `metricstest.exe`, `shortenertest.exe`.
3. Так как запуск должен происходить в корневой директории проекта (вашего репозитария), то при запуске нужно указывать полный путь к файлу автотеста. Программа может запускаться и без указания расширения `exe`. Например:

```shell
C:\temp\metricstest -test.v -test.run=^TestIteration1$ -agent-binary-path=cmd/agent/agent
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут непонятно откуда взялась папка c:\temp, скачивание по-умолчанию идёт в какой-нибудь <UserFolder/Downloads> (название условное).

Предлагаю поменять инструкцию и сказать что-то вроде:
2. Скачивать файл нужно с суффиксом -windows-amd64. Например, metricstest-windows-amd64.exe, shortenertest-windows-amd64.exe.
3. После скачивание переместите файл в папку тестируемого проекта и удалите у него суффикс -windows-adm64, т.е. файл metricstest-windows-amd64.exe должен называться metricstest.exe и находиться в корневой папке вашего проекта.
4. Запускать тесты нужно из корневой папки вашего проекта:

cd <project-folder>
metricstest.exe -test.v -test.run=^TestIteration1$ -agent-binary-path=cmd/agent/agent

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Переделал.

```

Если вы не хотите указывать всё время полный путь, то можно сохранить файл автотеста в директорию, которая добавлена в переменную окружения `PATH`. Также вы можете самостоятельно добавить туда нужную директорию нажав **Windows + R** и запустив там `sysdm.cpl`. В открывшемся окне **Свойства системы** перейти на закладку **Дополнительно** и нажать кнопку **Переменные среды** внизу закладки. Выберите переменную **Path** в открывшемся списке и нажмите на кнопку **Изменить**. Вам останется добавить нужную директорию и сохранить изменения. После этого вы можете запускать автотесты без указания полного пути к программе.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот эту часть я бы вообще убрал - добавление Downloads в Path может добавить неожиданного поведения если туда скачается что-то вроде cd.exe или setup.exe и потом по setup будет всегда вызваться что-то непонятное.

Лучше положить экзешник в понятное место (папка проекта) и дальше работать с ним из той же папки проекта. Удобство как-будто он в path, а побочных эффектов нет.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хорошо, убрал про PATH.


### Запуск на Mac с процессором Apple Silicon

Если у вас возникают трудности с локальным запуском автотестов на компьютере Mac на базе процессора Apple Silicon (M1 и старше), убедитесь, что:
Expand Down