Mostly finish pytest to Bats conversion (#1200)

* Mostly finish pytest to Bats conversion

* Add Bats version note to testing docs

* Move Bats CI check to separate job

Author: hyperupcall

.github/workflows/ci.yml

@@ -65,9 +65,9 @@ jobs:
             grep -v -e 'History\.md' -e 'AUTHORS' -e 'man/.*\.1' -e 'man/.*\.html' | \
             xargs poetry run codespell --ignore-words=../.github/.ignore_words
 
-  test:
+  test-pytest:
+    name: 'Test with Pytest'
     runs-on: ubuntu-latest
-
     steps:
       - uses: actions/checkout@v4
         with:
@@ -85,13 +85,23 @@ jobs:
         run: |
           cd tests || exit
           poetry install --only test
-      - name: Setup Bats
-        id: setup-bats
-        uses: bats-core/bats-action@3.0.0
       - name: Test with Pytest
         run: |
           cd tests
           poetry run pytest
+
+  test-bats:
+    name: 'Test with Bats'
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+           submodules: recursive
+      - name: Setup Bats
+        id: setup-bats
+        uses: bats-core/bats-action@3.0.0
+        with:
+          bats-version: 'v1.8.1'
       - name: Test with Bats
         env:
          BATS_LIB_PATH: ${{ steps.setup-bats.outputs.lib-path }}

.gitignore

@@ -1,2 +1,3 @@
 .venv/
 __pycache__/
+coverage/

tests/README.md

@@ -1,7 +1,28 @@
-# Test for git-extras
-The git-extras has its own testcases now, and the more is on the way! So let's introduce it.
+# Testing
 
-We choose python to help us to reach to other shore cause **python is life saver**.
+Originally, the tests were written in pytest. However, tests are in the process of being converted to Bats so coverage can be calculated.
+
+## Bats Testing
+
+We require a somewhat recent version of Bats. Version v1.8.1 is tested in CI. Once it is installed, the tests can be executed like so:
+
+```sh
+bats ./tests
+```
+
+We highly recommend adding tests for new features and fixes.
+
+### Code Coverage
+
+Coverage can be calculated with [bashcov](https://github.com/infertux/bashcov) like so:
+
+```sh
+bashcov -- bats ./tests
+```
+
+By default, the report will be generated in `./coverage/index.html`.
+
+## Python Testing
 
 The test part depends on:
 
@@ -12,7 +33,8 @@ The test part depends on:
 
 So the versions are higher than above is recommended.
 
-# How to run the tests
+### How to run the tests
+
 1. Install `poetry`
 2. Install the dependencies via `poetry install`
 3. Run `poetry run pytest`
@@ -27,7 +49,8 @@ It is done or go without `poetry`,
 
 The second way maybe blocked the some missing dependencies at someday, so the first one is recommended.
 
-# What and how to create a unit test
+### What and how to create a unit test
+
 One command has a unit test, because one `git-*` command is just do one thing, so we can eat a piece of `git-*` command in one time.
 
 For example,
@@ -39,7 +62,8 @@ For example,
     * `named_temp_repo` is just same as `temp_repo` except the custom directory renaming.
 4. Loop the third step until the 100% coverage of the function of the `git-alias`
 
-# References
+### References
+
 * [poetry](https://github.com/python-poetry/poetry)
 * [pytest](https://github.com/pytest-dev/pytest/)
 * [git python](https://github.com/gitpython-developers/GitPython)

tests/bin/open

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+printf '%s\n' "open $*"

tests/bin/powershell.exe

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+printf '%s\n' "powershell.exe $*"

tests/bin/start

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+printf '%s\n' "start $*"

tests/bin/xdg-open

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+printf '%s\n' "xdg-open $*"

tests/git-abort.bats

@@ -9,7 +9,7 @@ setup_file() {
 setup() {
 	test_util.cd_test
 
-	git init
+	test_util.git_init
 	git commit --allow-empty -m "Initial commit"
 	git branch A
 	git branch B
@@ -24,7 +24,7 @@ setup() {
 	git status
 }
 
-@test "cherry pick" {
+@test "works with cherry pick" {
 	run git cherry-pick A
 	assert_failure
 
@@ -41,7 +41,7 @@ setup() {
 	assert_success
 }
 
-@test "merge" {
+@test "works with merge" {
 	run git merge A
 	assert_failure
 
@@ -58,7 +58,7 @@ setup() {
 	assert_success
 }
 
-@test "rebase" {
+@test "works with rebase" {
 	run git rebase A
 	assert_failure
 
@@ -75,7 +75,7 @@ setup() {
 	assert_success
 }
 
-@test "revert" {
+@test "works with revert" {
 	run git revert A
 	assert_failure
 

tests/git-alias.bats

@@ -9,7 +9,7 @@ setup_file() {
 setup() {
 	test_util.cd_test
 
-	git init
+	test_util.git_init
 	git config --global alias.globalalias status
 	git config --global alias.x status
 	git config --local alias.localalias status

tests/git-archive-file.bats

@@ -9,7 +9,7 @@ setup_file() {
 setup() {
 	test_util.cd_test
 
-	git init
+	test_util.git_init
 	printf '%s\n' 'data' > tmpfile
 	git add .
 	git commit -m 'test: add data'