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

Further improvements for nightly integration-tests #868

Closed
hohwille opened this issue Sep 8, 2022 · 1 comment · Fixed by #899
Closed

Further improvements for nightly integration-tests #868

hohwille opened this issue Sep 8, 2022 · 1 comment · Fixed by #899
Assignees
Labels
commandlet related to commandlets (scripts/command/*) enhancement New feature or request test related to testing and QA

Comments

@hohwille
Copy link
Member

hohwille commented Sep 8, 2022

From review of PR #844:
A new file functions was added in test:

maybe this file should have a different name (e.g. test-functions, ``)? Otherwise we might end up in confusions if someone talks about functions and it is unclear if he means the productive functions or the test functions. We also have a functions documentation and some places that just talk about functions what is currently a unique file and so it is clear what we are talking about.

Further we currently have tests for

  • terraform
  • vscode

But how about other tools and features?

  • eclipse
  • intellij
  • node
  • npm
  • az
  • ng
  • python
  • pip
  • yarn
  • oc
  • graalvm
  • helm
  • dotnet
  • cobigen
  • ...
@hohwille hohwille added the enhancement New feature or request label Sep 8, 2022
@hohwille hohwille added commandlet related to commandlets (scripts/command/*) test related to testing and QA Team_IDE labels Sep 9, 2022
@alfeilex
Copy link
Member

To add new commands for the integration test, you can create a new file named integration-test-<command> and follow this example.

#!/bin/bash
source "$(dirname "${0}")"/functions
doCommandTest terraform

Windows users must be careful when pushing the new test-scripts. Because Windows does not have a permission system like Linux, the following command must be run beforehand, otherwise a permission denied error will occur in github action run.

git update-index --chmod=+x <script-path>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commandlet related to commandlets (scripts/command/*) enhancement New feature or request test related to testing and QA
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants