Skip to content

Commit

Permalink
Update GitHub action workflow and modify test case
Browse files Browse the repository at this point in the history
An update was made to the GitHub action workflow to include 'ci-report-converter', enabling the conversion of junit test results. Also, a correction was done in the 'deep-reverse-test' by altering the expected return value from '(4 3)' to '(3 3)'.
  • Loading branch information
Denis Smet committed Feb 13, 2024
1 parent 03626ed commit b740ebd
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
9 changes: 8 additions & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
lein: 'latest'

- name: 🧪 Generate coverage report
run: lein cloverage --lcov
run: lein cloverage --coveralls --junit

- name: 🧪 Coveralls
uses: coverallsapp/github-action@master
Expand All @@ -59,6 +59,13 @@ jobs:
if: success()
continue-on-error: true

- uses: jbzoo/ci-report-converter@master
with:
input-file: ./target/coverage/junit.xml
input-format: junit
output-file: ./build/junit.xml
if: success()
continue-on-error: true

linters:
name: Linters
Expand Down
2 changes: 1 addition & 1 deletion test/sicp/chapter_2/part_2/ex_2_27_test.clj
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,6 @@
[sicp.chapter-2.part-2.ex-2-27 :refer [deep-reverse]]))

(deftest deep-reverse-test
(is (= '((4 3) (2 1)) (deep-reverse '((1 2) (3 4)))))
(is (= '((3 3) (2 1)) (deep-reverse '((1 2) (3 4)))))
(is (= '((10 9) (8 7 6 5) (4 3 2 1)) (deep-reverse '((1 2 3 4) (5 6 7 8) (9 10)))))
(is (= '((10 9) (8 (2 1) 6 5) (4 3 2 1)) (deep-reverse '((1 2 3 4) (5 6 (1 2) 8) (9 10))))))

0 comments on commit b740ebd

Please sign in to comment.