Skip to content

Commit

Permalink
Idnoring scripts in npm. Adding dockerfile to sonar tests
Browse files Browse the repository at this point in the history
  • Loading branch information
dothomson authored and sandy-may committed Aug 15, 2024
1 parent 0e931f1 commit d80cae1
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
5 changes: 4 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
node_modules

# Jest test report
report.xml
report.xml

# Jest code coverage output
coverage/

# AWS SAM
.aws-sam/
samconfig.toml

cf-output.txt
docker-vars.env
2 changes: 1 addition & 1 deletion backend-api/Dockerfile.test
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ RUN adduser --disabled-password test
RUN chown test .

COPY package.json package-lock.json ./
RUN npm install
RUN npm install --ignore-scripts


## Update container and install awscli
Expand Down
2 changes: 1 addition & 1 deletion sonar-project.properties
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ sonar.projectKey=mobile-id-check-async
sonar.organization=govuk-one-login

# Path is relative to the sonar-project.properties file. Replace "\" by "/" on Windows.
sonar.sources=backend-api/src
sonar.sources=backend-api/src,backend-api/Dockerfile.test
sonar.javascript.lcov.reportPaths=backend-api/coverage/lcov.info
sonar.language=ts
sonar.exclusions=**/*.test.ts,**/*.test.js,**/testUtils/**,**/tests/mocks.ts
Expand Down

0 comments on commit d80cae1

Please sign in to comment.