Skip to content
This repository was archived by the owner on Jan 7, 2025. It is now read-only.

Feature/automated theme tests #107

Merged
paulpopus merged 14 commits into from
Oct 26, 2020
Merged

Feature/automated theme tests #107

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
3c40747
Add Mocha and Chai to run automated tests for the theme
Oct 20, 2020
753fac4
Add new gulp task for generate that doesn't include browsersync or wa…
Oct 20, 2020
7910366
Add github workflow test for NPM and CSS compilation test with Mocha
Oct 21, 2020
c181803
Fix workflows node actions version
Oct 21, 2020
a8a1f56
Add shell and working directory for github workflows
Oct 21, 2020
8b3392f
Fix indents in yml
Oct 21, 2020
af5fbb7
Remove github shell declarations
Oct 21, 2020
86f2c4a
Use npm ci instead of npm install
Oct 21, 2020
e85f404
Add actions/checkout for mocha tests
Oct 21, 2020
4b1d4f6
Remove other node versions
Oct 21, 2020
703dace
Fix script running and move it to npm test as per standard
Oct 21, 2020
db0b4a7
Updated tests to be synchronous and more predictable
Oct 21, 2020
5100021
Simplify some helper functions in some places
Oct 22, 2020
09bb2d3
Remove unused import and add helper function for removing files
Oct 22, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 21 additions & 1 deletion .github/workflows/localgov-drupal-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
branches: [ master ]

jobs:
test:
DrupalUnitTests:

runs-on: ubuntu-latest

Expand Down Expand Up @@ -43,3 +43,23 @@ jobs:
- name: Run tests
run: ./run-tests.sh
shell: bash

MochaThemeTests:

runs-on: ubuntu-latest

strategy:
matrix:
node-version: [10.x]
Copy link
Contributor

@Adnan-cds Adnan-cds Oct 22, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1. I have never tried this "matrix" feature :)

Copy link
Collaborator Author

@paulpopus paulpopus Oct 22, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah it's quite nice, should allow us to just add other node version we want to test in that list


steps:
- uses: actions/checkout@v2
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node-version }}

- name: Install Node dependencies
run: npm install

- run: npm run health-check
6 changes: 6 additions & 0 deletions gulpfile.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -130,8 +130,14 @@ const dev = gulp.series(
gulp.parallel(browserSyncStart, watchStyle, serve)
);

/** Generates style without any listeners or browserSync */
const generate = gulp.series(
generateStyle,
);

// Expose the task by exporting it, this allows you to run it from the commandline
exports.dev = dev;
exports.generate = generate;


/*
Expand Down
Loading