Remove --no-rollback from library tests; remove sudo usage from oauth…

…2 bootstrap script
jefflester · Nov 22, 2024 · 7a95acf · 7a95acf
1 parent 602868a
commit 7a95acf
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,7 +4,6 @@ repos:
     rev: 24.10.0
     hooks:
       - id: black
-        always_run: true
 
   - repo: local
     hooks:

diff --git a/src/lib/modules/security/oauth2/resources/bootstrap/bootstrap-trino.sh b/src/lib/modules/security/oauth2/resources/bootstrap/bootstrap-trino.sh
@@ -8,7 +8,7 @@ openssl s_client -connect oauth2-server:8100 2>/dev/null </dev/null \
     > /tmp/oauth2-server.pem
 
 echo "Adding OAuth2 server certificate to truststore..."
-sudo keytool -import -noprompt -trustcacerts \
+keytool -import -noprompt -trustcacerts \
     -storepass changeit \
     -alias oauth2-server \
     -file /tmp/oauth2-server.pem \

diff --git a/src/test/src/lib/runner.py b/src/test/src/lib/runner.py
@@ -42,12 +42,10 @@ def run_tests(self, tests=[], workers=False):
         """Runs module tests."""
 
         if not workers:
-            common.execute_command(
-                f"minitrino -v provision -m {self.module} --no-rollback"
-            )
+            common.execute_command(f"minitrino -v provision -m {self.module}")
         else:
             common.execute_command(
-                f"minitrino -v provision -m {self.module} --workers 1 --no-rollback"
+                f"minitrino -v provision -m {self.module} --workers 1"
             )
 
         for t in tests: