Skip to content

Commit

Permalink
Provide env variable using Github Actions directly
Browse files Browse the repository at this point in the history
  • Loading branch information
garrettmflynn committed May 30, 2024
1 parent f2f3eea commit 6376765
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 21 deletions.
25 changes: 5 additions & 20 deletions .github/workflows/testing_pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,34 +94,19 @@ jobs:
with:
path: ./behavior_testing_data
key: behavior-datasets-${{ matrix.os }}-${{ steps.behavior.outputs.HASH_behavior_DATASET }}

- name: Save working directory to environment file
run: echo "GIN_DATA_DIR=$(pwd)" >> .env
if: runner.os != 'Windows'

- name: Save working directory to environment file (Windows)
run: echo GIN_DATA_DIR=%cd% >> .env
shell: bash
if: runner.os == 'Windows'

# Display environment file for debugging
- name: Print environment file
run: cat .env
if: runner.os != 'Windows'

- name: Print environment file
run: type .env
shell: bash
if: runner.os == 'Windows'


# Run pipeline tests
- if: matrix.os != 'ubuntu-latest'
name: Run tests
run: npm run test:pipelines
env:
GIN_DATA_DIRECTORY: ${{ github.workspace }}

- if: matrix.os == 'ubuntu-latest'
name: Run tests with xvfb
run: xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- npm run test:pipelines
env:
GIN_DATA_DIRECTORY: ${{ github.workspace }}

- name: Archive Pipeline Test Screenshots
if: always()
Expand Down
2 changes: 1 addition & 1 deletion tests/e2e/pipelines.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ import { sleep } from '../puppeteer';


// NOTE: We assume the user has put the GIN data in ~/NWB_GUIDE/test-data/GIN
const testGINPath = process.env.GIN_DATA_DIR ?? join(homedir(), paths.root, 'test-data', 'GIN')
const testGINPath = process.env.GIN_DATA_DIRECTORY ?? join(homedir(), paths.root, 'test-data', 'GIN')
console.log('Using test GIN data at:', testGINPath)

const pipelineDescribeFn = existsSync(testGINPath) ? describe : describe.skip
Expand Down

0 comments on commit 6376765

Please sign in to comment.