Skip to content

Commit

Permalink
Draft: Add vue files to code climate (#1469)
Browse files Browse the repository at this point in the history
* Add vue files to code climate

* Add vue files to code climate

* Remove unneeded flag

Coverage is handled by npm run test in package.json

* Add vuejs applications to codeclimate coverage

* Move to after js testing

* * Switch to v8 for test coverage. See vuejs/vue-jest#422.
* Update dependencies

* Bump node versions so vue tests can run via v8

* Set project root

Co-authored-by: Ben Pennell <bbpennel@email.unc.edu>
  • Loading branch information
lfarrell and bbpennel committed Sep 16, 2022
1 parent 29fe722 commit 7b6ebdb
Show file tree
Hide file tree
Showing 7 changed files with 3,835 additions and 3,934 deletions.
28 changes: 15 additions & 13 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,21 +37,10 @@ jobs:
- name: Package integration module with Maven
run: mvn -B -U -pl integration clean package -DskipTests

- name: Report to CodeClimate
uses: paambaati/codeclimate-action@v3.0.0
env:
CC_TEST_REPORTER_ID: ${{secrets.CC_TEST_REPORTER_ID}}
JACOCO_SOURCE_PATH: "${{github.workspace}}/auth-api/src/main/java ${{github.workspace}}/auth-fcrepo/src/main/java ${{github.workspace}}/common-utils/src/main/java ${{github.workspace}}/deposit-app/src/main/java ${{github.workspace}}/deposit-utils/src/main/java ${{github.workspace}}/fcrepo-utils/src/main/java ${{github.workspace}}/indexing-solr/src/main/java ${{github.workspace}}/integration/src/main/java ${{github.workspace}}/model-api/src/main/java ${{github.workspace}}/model-fcrepo/src/main/java ${{github.workspace}}/operations/src/main/java ${{github.workspace}}/operations-jms/src/main/java ${{github.workspace}}/persistence/src/main/java ${{github.workspace}}/persistence-api/src/main/java ${{github.workspace}}/search-api/src/main/java ${{github.workspace}}/search-solr/src/main/java ${{github.workspace}}/services-camel-app/src/main/java ${{github.workspace}}/web-access-app/src/main/java ${{github.workspace}}/web-access-war/src/main/java ${{github.workspace}}/web-admin-app/src/main/java ${{github.workspace}}/web-common/src/main/java ${{github.workspace}}/web-services-app/src/main/java ${{github.workspace}}/web-sword/src/main/java ${{github.workspace}}/clamav-java/src/main/java"
with:
coverageCommand: mvn -pl !clamav-java verify
coverageLocations: |
${{github.workspace}}/**/target/site/jacoco/jacoco.xml:jacoco
${{github.workspace}}/**/target/site/jacoco-it/jacoco.xml:jacoco
- name: Set up nodejs
uses: actions/setup-node@v2
with:
node-version: '12'
node-version: '16'
- name: Cache npm modules
uses: actions/cache@v2
with:
Expand All @@ -62,6 +51,19 @@ jobs:
- run: npm --prefix static/js/vue-cdr-access install
- run: npm --prefix static/js/admin/vue-permissions-editor install

- run: npm install -g jest-cli@28.1.2
- run: npm install -g jest-cli@29.0.3
- run: npm --prefix static/js/vue-cdr-access run test
- run: npm --prefix static/js/admin/vue-permissions-editor run test

- name: Report to CodeClimate
uses: paambaati/codeclimate-action@v3.0.0
env:
CC_TEST_REPORTER_ID: ${{secrets.CC_TEST_REPORTER_ID}}
JACOCO_SOURCE_PATH: "${{github.workspace}}/auth-api/src/main/java ${{github.workspace}}/auth-fcrepo/src/main/java ${{github.workspace}}/common-utils/src/main/java ${{github.workspace}}/deposit-app/src/main/java ${{github.workspace}}/deposit-utils/src/main/java ${{github.workspace}}/fcrepo-utils/src/main/java ${{github.workspace}}/indexing-solr/src/main/java ${{github.workspace}}/integration/src/main/java ${{github.workspace}}/model-api/src/main/java ${{github.workspace}}/model-fcrepo/src/main/java ${{github.workspace}}/operations/src/main/java ${{github.workspace}}/operations-jms/src/main/java ${{github.workspace}}/persistence/src/main/java ${{github.workspace}}/persistence-api/src/main/java ${{github.workspace}}/search-api/src/main/java ${{github.workspace}}/search-solr/src/main/java ${{github.workspace}}/services-camel-app/src/main/java ${{github.workspace}}/web-access-app/src/main/java ${{github.workspace}}/web-access-war/src/main/java ${{github.workspace}}/web-admin-app/src/main/java ${{github.workspace}}/web-common/src/main/java ${{github.workspace}}/web-services-app/src/main/java ${{github.workspace}}/web-sword/src/main/java ${{github.workspace}}/clamav-java/src/main/java"
with:
coverageCommand: mvn -pl !clamav-java verify
coverageLocations: |
${{github.workspace}}/**/target/site/jacoco/jacoco.xml:jacoco
${{github.workspace}}/**/target/site/jacoco-it/jacoco.xml:jacoco
${{github.workspace}}/static/js/vue-cdr-access/coverage/lcov.info:lcov
${{github.workspace}}/static/js/admin/vue-permissions-editor/coverage/lcov.info:lcov
4 changes: 4 additions & 0 deletions static/js/admin/vue-permissions-editor/jest.config.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,8 @@
module.exports = {
collectCoverage: true,
collectCoverageFrom: ["**/src/components/**", "**/src/mixins/**"],
coverageProvider: 'v8',
coverageReporters: [["lcov", {"projectRoot": "../../../../"}], "json", "text"],
moduleFileExtensions: ['vue', 'js', 'json'],
testEnvironment: 'jsdom',
testEnvironmentOptions: {
Expand Down
Loading

0 comments on commit 7b6ebdb

Please sign in to comment.