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

Introduce shell test suite 🧪 #42

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

jamacku
Copy link
Member

@jamacku jamacku commented Jun 29, 2022

Test suite:

  • e2e tests
  • unit tests
    • clean_array()
    • file_to_array()
    • has_shebang()
    • is_debug()
    • is_shell_extension()
    • is_script_listed()
    • is_shell_extension()
    • join_by()
  • test coverage
  • Super Linter

Other changes:

@jamacku jamacku self-assigned this Jun 29, 2022
@jamacku jamacku linked an issue Jun 29, 2022 that may be closed by this pull request
@jamacku jamacku added this to the v2.2.0 milestone Jun 29, 2022
@jamacku jamacku marked this pull request as ready for review June 29, 2022 14:41
@jamacku jamacku added the test label Jun 29, 2022
@jamacku jamacku force-pushed the ci_br_dest branch 5 times, most recently from 83ec70d to cbcea2e Compare June 30, 2022 10:32
@mergify mergify bot added the source label Jun 30, 2022
@jamacku jamacku force-pushed the ci_br_dest branch 8 times, most recently from c6afdf4 to 836d6a4 Compare June 30, 2022 15:20
@jamacku jamacku changed the title Introduce shell test suite Introduce shell test suite 🧪 Jun 30, 2022
@jamacku jamacku force-pushed the ci_br_dest branch 7 times, most recently from 17d5480 to e7339ee Compare July 1, 2022 07:17
@jamacku jamacku force-pushed the ci_br_dest branch 20 times, most recently from 2b2f0c8 to 5b43641 Compare July 7, 2022 14:23
@jamacku jamacku force-pushed the ci_br_dest branch 4 times, most recently from bf2a966 to ec5c67e Compare July 11, 2022 11:23
@jamacku
Copy link
Member Author

jamacku commented Jul 11, 2022

Let's merge this PR and resolve workflow issues in follow-up PR.

@jamacku jamacku merged commit 081f3da into redhat-plumbers-in-action:main Jul 11, 2022
@jamacku jamacku deleted the ci_br_dest branch July 11, 2022 11:30
@jamacku
Copy link
Member Author

jamacku commented Jul 11, 2022

Follow-up: #50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add testsuite to avoid regressions 🧪
1 participant