Skip to content

Commit 7b6ebdb

Browse files
lfarrellbbpennel
andauthored
Draft: Add vue files to code climate (#1469)
* 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 <[email protected]>
1 parent 29fe722 commit 7b6ebdb

File tree

7 files changed

+3835
-3934
lines changed

7 files changed

+3835
-3934
lines changed

.github/workflows/build.yml

+15-13
Original file line numberDiff line numberDiff line change
@@ -37,21 +37,10 @@ jobs:
3737
- name: Package integration module with Maven
3838
run: mvn -B -U -pl integration clean package -DskipTests
3939

40-
- name: Report to CodeClimate
41-
uses: paambaati/[email protected]
42-
env:
43-
CC_TEST_REPORTER_ID: ${{secrets.CC_TEST_REPORTER_ID}}
44-
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"
45-
with:
46-
coverageCommand: mvn -pl !clamav-java verify
47-
coverageLocations: |
48-
${{github.workspace}}/**/target/site/jacoco/jacoco.xml:jacoco
49-
${{github.workspace}}/**/target/site/jacoco-it/jacoco.xml:jacoco
50-
5140
- name: Set up nodejs
5241
uses: actions/setup-node@v2
5342
with:
54-
node-version: '12'
43+
node-version: '16'
5544
- name: Cache npm modules
5645
uses: actions/cache@v2
5746
with:
@@ -62,6 +51,19 @@ jobs:
6251
- run: npm --prefix static/js/vue-cdr-access install
6352
- run: npm --prefix static/js/admin/vue-permissions-editor install
6453

65-
- run: npm install -g jest-cli@28.1.2
54+
- run: npm install -g jest-cli@29.0.3
6655
- run: npm --prefix static/js/vue-cdr-access run test
6756
- run: npm --prefix static/js/admin/vue-permissions-editor run test
57+
58+
- name: Report to CodeClimate
59+
uses: paambaati/[email protected]
60+
env:
61+
CC_TEST_REPORTER_ID: ${{secrets.CC_TEST_REPORTER_ID}}
62+
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"
63+
with:
64+
coverageCommand: mvn -pl !clamav-java verify
65+
coverageLocations: |
66+
${{github.workspace}}/**/target/site/jacoco/jacoco.xml:jacoco
67+
${{github.workspace}}/**/target/site/jacoco-it/jacoco.xml:jacoco
68+
${{github.workspace}}/static/js/vue-cdr-access/coverage/lcov.info:lcov
69+
${{github.workspace}}/static/js/admin/vue-permissions-editor/coverage/lcov.info:lcov

static/js/admin/vue-permissions-editor/jest.config.js

+4
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
11
module.exports = {
2+
collectCoverage: true,
3+
collectCoverageFrom: ["**/src/components/**", "**/src/mixins/**"],
4+
coverageProvider: 'v8',
5+
coverageReporters: [["lcov", {"projectRoot": "../../../../"}], "json", "text"],
26
moduleFileExtensions: ['vue', 'js', 'json'],
37
testEnvironment: 'jsdom',
48
testEnvironmentOptions: {

0 commit comments

Comments
 (0)