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.

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

Conversation

gentee
Copy link

@gentee gentee commented Aug 3, 2023

No description provided.

README.md Outdated
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.

Переделал.

README.md Outdated
C:\temp\metricstest -test.v -test.run=^TestIteration1$ -agent-binary-path=cmd/agent/agent
```

Если вы не хотите указывать всё время полный путь, то можно сохранить файл автотеста в директорию, которая добавлена в переменную окружения `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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants