aboutsummaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorEgor Tensin <egor@tensin.name>2024-06-23 20:13:05 +0200
committerEgor Tensin <egor@tensin.name>2024-06-23 20:13:29 +0200
commit093c4abac75fdbf0f4efa6a3f80556aeb8e85671 (patch)
tree7a442e02e3110a3fb7805b87486a826e545d73ae
parentinclude error.html in the frontend to signal errors (diff)
downloadcgitize-093c4abac75fdbf0f4efa6a3f80556aeb8e85671.tar.gz
cgitize-093c4abac75fdbf0f4efa6a3f80556aeb8e85671.zip
drop workdir logic
I don't think it ever worked. For example, the test_workdir test case was disfunctional. Additionally, "master" was hardcoded as the default branch name, which is obviously a bug.
-rw-r--r--cgitize/cgit.py8
-rwxr-xr-xtest/integration/local/test.sh30
2 files changed, 1 insertions, 37 deletions
diff --git a/cgitize/cgit.py b/cgitize/cgit.py
index 1fbddb7..c525214 100644
--- a/cgitize/cgit.py
+++ b/cgitize/cgit.py
@@ -166,10 +166,4 @@ class CGitRepositories:
repo_dir = self.get_repo_dir(repo)
with chdir(repo_dir):
with Git.setup_auth(repo):
- if not Git.check('remote', 'update', '--prune'):
- return False
- # In case the local repository is not a bare repository, but a
- # full-fledged working copy:
- if Git.check('rev-parse', '--verify', '--quiet', 'origin/master'):
- return Git.check('reset', '--soft', 'origin/master')
- return True
+ return Git.check('remote', 'update', '--prune')
diff --git a/test/integration/local/test.sh b/test/integration/local/test.sh
index dbed2e1..04208ab 100755
--- a/test/integration/local/test.sh
+++ b/test/integration/local/test.sh
@@ -107,18 +107,6 @@ setup_bare() {
setup_cgitize
}
-setup_workdir() {
- setup_bare
-
- echo
- echo ----------------------------------------------------------------------
- echo Setting up local repository clone
- echo ----------------------------------------------------------------------
-
- mkdir -p -- "$output_dir"
- _git clone --quiet -- "$upstream_repo_dir" "$output_dir/test_repo"
-}
-
cgitize() {
echo
echo ----------------------------------------------------------------------
@@ -216,23 +204,6 @@ test_bare() {
success
}
-test_workdir() {
- echo
- echo ======================================================================
- echo "${FUNCNAME[0]}"
- echo ======================================================================
-
- setup_workdir
- cgitize
- verify_initial_commits
- add_commits
- cgitize
- verify_added_commits
- verify_error_header_empty
- cleanup
- success
-}
-
test_failure() {
echo
echo ======================================================================
@@ -267,7 +238,6 @@ test_failure() {
main() {
trap cleanup EXIT
test_bare
- test_workdir
test_failure
}