Skip to content

Commit

Permalink
Add version verification to tests (#17)
Browse files Browse the repository at this point in the history
<!-- markdownlint-disable-file MD041 -->

## Pull request checklist

Please check if your PR fulfills the following requirements:

- [ ] I have read the
[CONTRIBUTING](https://github.com/fabasoad/setup-jolie-action/blob/main/CONTRIBUTING.md)
      doc.
- [ ] Tests for the changes have been added (for bug fixes / features).
- [ ] Docs have been reviewed and added / updated if needed (for bug
fixes / features).

## Pull request type

<!-- Please do not submit updates to dependencies unless it fixes an
issue. -->

<!-- Please try to limit your pull request to one type, submit multiple
pull
requests if needed. -->

Please check the type of change your PR introduces:

- [ ] Bugfix
- [ ] Feature
- [ ] Code style update (formatting, renaming)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] Documentation content changes
- [ ] Other (please describe):

## What is the current behavior

<!-- Please describe the current behavior that you are modifying, or
link to a
relevant issue. -->

## What is the new behavior

<!-- Please describe the behavior or changes that are being added by
this PR. -->

-
-
-

## Does this introduce a breaking change

- [ ] Yes
- [ ] No

<!-- If this introduces a breaking change, please describe the impact
and
migration path for existing applications below. -->

## Other information

<!-- Any other information that is important to this PR such as
screenshots of
how the component looks before and after the change. -->
<!-- This document was adapted from the open-source
[appium/appium](https://github.com/appium/appium/blob/master/.github/PULL_REQUEST_TEMPLATE.md)
repository. -->

---

Closes #{IssueNumber}
  • Loading branch information
fabasoad authored Jun 16, 2024
1 parent 2fccfd4 commit 772776c
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 7 deletions.
22 changes: 18 additions & 4 deletions .github/workflows/functional-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,15 @@ jobs:
uses: ./
with:
version: ${{ matrix.version }}
- name: Print version
run: jolie --version
- name: Verify installed version
run: |
actual_version=$(jolie --version 2>&1 | cut -d ' ' -f 2)
expected_version="${{ matrix.version }}"
if [ "${expected_version}" != "${actual_version}" ]; then
msg="Expected - ${expected_version}. Received - ${actual_version}."
echo "::error title=Unexpected jolie version::${msg}"
exit 1
fi
setup-jolie-alpine:
name: Setup jolie
timeout-minutes: 15
Expand All @@ -57,5 +64,12 @@ jobs:
uses: ./
with:
version: ${{ matrix.version }}
- name: Print version
run: jolie --version
- name: Verify installed version
run: |
actual_version=$(jolie --version 2>&1 | cut -d ' ' -f 2)
expected_version="${{ matrix.version }}"
if [ "${expected_version}" != "${actual_version}" ]; then
msg="Expected - ${expected_version}. Received - ${actual_version}."
echo "::error title=Unexpected jolie version::${msg}"
exit 1
fi
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ jobs:
distribution: zulu
- uses: fabasoad/setup-jolie-action@v0
with:
version: 1.10.13
version: "1.12.0"
- name: Print version
run: jolie --version
```
Expand All @@ -74,7 +74,7 @@ jobs:
run: apk add --update maven
- uses: fabasoad/setup-jolie-action@v0
with:
version: 1.10.13
version: "1.12.0"
- name: Print version
run: jolie --version
```
Expand All @@ -83,5 +83,5 @@ jobs:
```shell
Run jolie --version
Jolie 1.10.13 (C) 2006-2021 the Jolie developers
Jolie 1.12.0 (C) 2006-2022 the Jolie developers
```

0 comments on commit 772776c

Please sign in to comment.