Skip to content

Commit 7da417e

Browse files
Restore macos/windows
Signed-off-by: Peter Zhu <[email protected]>
1 parent 72f7b3f commit 7da417e

File tree

1 file changed

+49
-52
lines changed

1 file changed

+49
-52
lines changed

.github/workflows/unit-tests-workflow.yml

Lines changed: 49 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -46,55 +46,52 @@ jobs:
4646
- name: Uploads coverage
4747
uses: codecov/codecov-action@v1
4848

49-
# tests-macos-windows:
50-
# name: Run unit tests
51-
# strategy:
52-
# matrix:
53-
# os: [ macos-latest, windows-latest ]
54-
# runs-on: ${{ matrix.os }}
55-
# steps:
56-
# # Enable longer filenames for windows
57-
# - name: Enable longer filenames
58-
# if: ${{ matrix.os == 'windows-latest' }}
59-
# run: git config --system core.longpaths true
60-
# - name: Checkout OpenSearch Dashboards
61-
# uses: actions/checkout@v2
62-
# with:
63-
# repository: opensearch-project/OpenSearch-Dashboards
64-
# ref: ${{ env.OPENSEARCH_DASHBOARDS_VERSION }}
65-
# path: OpenSearch-Dashboards
66-
# - name: Setup Node
67-
# uses: actions/setup-node@v3
68-
# with:
69-
# node-version-file: './OpenSearch-Dashboards/.nvmrc'
70-
# registry-url: 'https://registry.npmjs.org'
71-
# - name: Install Yarn
72-
# # Need to use bash to avoid having a windows/linux specific step
73-
# shell: bash
74-
# run: |
75-
# YARN_VERSION=$(node -p "require('./OpenSearch-Dashboards/package.json').engines.yarn")
76-
# echo "Installing yarn@$YARN_VERSION"
77-
# npm i -g yarn@$YARN_VERSION
78-
# - run: node -v
79-
# - run: yarn -v
80-
# - name: Checkout Alerting OpenSearch Dashboards plugin
81-
# uses: actions/checkout@v2
82-
# with:
83-
# path: OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
84-
# - name: Bootstrap plugin/opensearch-dashboards
85-
# run: |
86-
# cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
87-
# yarn osd bootstrap
88-
# - name: Run window tests
89-
# if: ${{ matrix.os == 'windows-latest' }}
90-
# run: |
91-
# cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
92-
# yarn run test:jest:windows --coverage
93-
# - name: Run non-window tests
94-
# if: ${{ matrix.os != 'windows-latest' }}
95-
# run: |
96-
# cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
97-
# yarn run test:jest --coverage
98-
# - name: Uploads coverage
99-
# if: ${{ matrix.os == 'ubuntu-latest' }}
100-
# uses: codecov/codecov-action@v1
49+
tests-macos-windows:
50+
name: Run unit tests
51+
strategy:
52+
matrix:
53+
os: [ macos-latest, windows-latest ]
54+
runs-on: ${{ matrix.os }}
55+
steps:
56+
# Enable longer filenames for windows
57+
- name: Enable longer filenames
58+
if: ${{ matrix.os == 'windows-latest' }}
59+
run: git config --system core.longpaths true
60+
- name: Checkout OpenSearch Dashboards
61+
uses: actions/checkout@v2
62+
with:
63+
repository: opensearch-project/OpenSearch-Dashboards
64+
ref: ${{ env.OPENSEARCH_DASHBOARDS_VERSION }}
65+
path: OpenSearch-Dashboards
66+
- name: Setup Node
67+
uses: actions/setup-node@v3
68+
with:
69+
node-version-file: './OpenSearch-Dashboards/.nvmrc'
70+
registry-url: 'https://registry.npmjs.org'
71+
- name: Install Yarn
72+
# Need to use bash to avoid having a windows/linux specific step
73+
shell: bash
74+
run: |
75+
YARN_VERSION=$(node -p "require('./OpenSearch-Dashboards/package.json').engines.yarn")
76+
echo "Installing yarn@$YARN_VERSION"
77+
npm i -g yarn@$YARN_VERSION
78+
- run: node -v
79+
- run: yarn -v
80+
- name: Checkout Alerting OpenSearch Dashboards plugin
81+
uses: actions/checkout@v2
82+
with:
83+
path: OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
84+
- name: Bootstrap plugin/opensearch-dashboards
85+
run: |
86+
cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
87+
yarn osd bootstrap
88+
- name: Run window tests
89+
if: ${{ matrix.os == 'windows-latest' }}
90+
run: |
91+
cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
92+
yarn run test:jest:windows --coverage
93+
- name: Run non-window tests
94+
if: ${{ matrix.os != 'windows-latest' }}
95+
run: |
96+
cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin
97+
yarn run test:jest --coverage

0 commit comments

Comments
 (0)