Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add GitHub action for verifying tests #94

Merged
merged 14 commits into from
Sep 29, 2023

Conversation

aSemy
Copy link
Contributor

@aSemy aSemy commented Aug 19, 2022

This adds a GitHub action that will run the tests on every pull request or change on master. This can be really helpful to quickly validate that an incoming pull request passes

Demo here: aSemy#1

WIP - I'm seeing if I can get a better test report generator I think I've found the best test report generator. I couldn't find a good Jacoco one though.

@aSemy aSemy marked this pull request as ready for review August 19, 2022 09:26
@hajk1 hajk1 merged commit 6ca84c2 into arturmkrtchyan:master Sep 29, 2023
@aSemy aSemy deleted the feat/github-action-mvn-verify branch October 1, 2023 00:50
hajk1 pushed a commit to hajk1/iban4j that referenced this pull request Oct 2, 2023
* add GHA for maven verify

* add GHA for maven verify

* add test report step

* experimenting with some other reporting actions

* tweaking GHA

* tweaking GHA

* tweaking GHA

* tweaking GHA

* rm broken/buggy test report actions

* try fixing jacoco report

* rm jacoco report upload

* rm jacoco report upload

* rm jacoco report upload
hajk1 added a commit that referenced this pull request Oct 2, 2023
* enable Java 8

* checks for national check digit be present in BBAN

* enable skipTests

* dependency update

* Bump junit from 4.11 to 4.13.1

Bumps [junit](https://github.com/junit-team/junit4) from 4.11 to 4.13.1.
- [Release notes](https://github.com/junit-team/junit4/releases)
- [Changelog](https://github.com/junit-team/junit4/blob/main/doc/ReleaseNotes4.11.md)
- [Commits](junit-team/junit4@r4.11...r4.13.1)

Signed-off-by: dependabot[bot] <[email protected]>

* #67: fix VG struncture

* java 6 replaced with java 7

* Belarus BBAN bug fixed

* update oss plugin

* release 3.2.2-RELEASE

* Add French subterritories support

* release 3.2.3-RELEASE

* add 3.2.3-RELEASE to the README.md

* #67: add VA iban structure

* upgrade jacoco maven plugin inorder to support Java versions higher than 8

* update Maven config

- remove deprecated parent pom
- bump java version to 1.8
- add maven wrapper
- bump versions
- change broken project URL to GitHub project URL
- add maven wrapper

* adding Iraq IBAN support

* new release updated

* added Egypt IBAN structure

* Adding Gabon BbanStructure

Added correct BbanStructure for Gabon and included Test case

* Release 3.2.5

* Fix typo in Guernsey

* adding GB and FI sub-territories support (#100)

Co-authored-by: Remy Duval <[email protected]>

* Refactor tests (#108)

* update junit version to 5

* update junit version to 5

* test cases refactored

* test cases refactored

* test cases refactored

* test cases refactored to new class

* test cases refactored

* test cases refactored

* test cases refactored

* Parametrized test cases refactored

* Hamcrest test cases refactored

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* fixing test case

* Create CODE_OF_CONDUCT.md

* GitHub action (#109)

* testing github action

* testing github action java

* testing github action java

* testing github action

* testing github action

* testing github action

* GitHub action (#110)

* testing github action

* testing github action java

* testing github action java

* testing github action

* testing github action

* testing github action

* testing github action

* GitHub action (#111)

* testing github action

* testing github action java

* testing github action java

* testing github action

* testing github action

* testing github action

* testing github action

* update java adopt jdk badge

* Issue 33 (#112)

* update java adopt jdk badge

* adding non default padded character

* adding branch code to use padding left

* change the README.md

* release 3.2.6-RELEASE

* fixed formatting

* fixed formatting

* Issue 101 (#113)

* adding two new methods

* adding test for the isValid false results

* adding missing tests

* adding valid tests for isValid

* making static utility methods public (#118)

* Add GitHub action for verifying tests (#94)

* add GHA for maven verify

* add GHA for maven verify

* add test report step

* experimenting with some other reporting actions

* tweaking GHA

* tweaking GHA

* tweaking GHA

* tweaking GHA

* rm broken/buggy test report actions

* try fixing jacoco report

* rm jacoco report upload

* rm jacoco report upload

* rm jacoco report upload

* Delete .github/workflows/build-github-actions.yml

deleted because of new ci yml

* exception changed to iban4jException (#119)

* bank code validation changed to accept alphanumeric values

* 3.2.7 released

* readme
updated to show 3.2.7 release

* changing the urls to make renovatebot happy (#121)

* Adding Cap Verde structure (#102)

* Adding Cap Verde structure

* Adding tests for CV

---------

Co-authored-by: RHoog <[email protected]>

* Allow setting a custom Random (#93)

* fix spelling mistake, ibanContruction -> ibanConstruction

* add overload to allow passing a custom Random

- add toString() to BbanStructureEntry.java
- tests to check if randomness is deterministic
- add additional tests for validity of BbanStructureEntry.java
- TODO figure out if 'c' can generate a-z

* remove TODO - a-z is not required

* fix code suggestions/warnings

* update tests to JUnit5

* remove duplicated `ibanConstructionRandom()` test

* update assertions

* fix test name after merge

* update expected random values

* re-add accidentally removed test

---------

Co-authored-by: aSemy <[email protected]>

* checks for national check digit be present in BBAN

* enable skipTests

* Adding Gabon BbanStructure

Added correct BbanStructure for Gabon and included Test case

* GitHub action (#109)

* testing github action

* testing github action java

* testing github action java

* testing github action

* testing github action

* testing github action

* Delete .github/workflows/build-github-actions.yml

deleted because of new ci yml

* Adding Gabon BbanStructure

Added correct BbanStructure for Gabon and included Test case

* GitHub action (#109)

* testing github action

* testing github action java

* testing github action java

* testing github action

* testing github action

* testing github action

* Delete .github/workflows/build-github-actions.yml

deleted because of new ci yml

* making java 8 release compatible

* Adding Gabon BbanStructure

Added correct BbanStructure for Gabon and included Test case

* GitHub action (#109)

* testing github action

* testing github action java

* testing github action java

* testing github action

* testing github action

* testing github action

* Delete .github/workflows/build-github-actions.yml

deleted because of new ci yml

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: hajk1 <BlackHeart59>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Constantin Costescu <[email protected]>
Co-authored-by: Kevin Aldebert <[email protected]>
Co-authored-by: Adam <[email protected]>
Co-authored-by: Kayvan Tehrani <[email protected]>
Co-authored-by: Bogac Aslanyurek <[email protected]>
Co-authored-by: dvg2207x <[email protected]>
Co-authored-by: Becky Fulton <[email protected]>
Co-authored-by: Rémy Duval <[email protected]>
Co-authored-by: Remy Duval <[email protected]>
Co-authored-by: hoorom <[email protected]>
Co-authored-by: RHoog <[email protected]>
Co-authored-by: aSemy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants