[PR #59] [MERGED] Fix tests and add a CI job that runs tests #60

New issue

Closed

opened 2026-03-02 04:08:05 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 04:08:05 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/gadievron/raptor/pull/59
Author: @joshbouncesecurity
Created: 1/22/2026
Status: ✅ Merged
Merged: 1/23/2026
Merged by: @danielcuthbert

Base: main ← Head: addtests

📝 Commits (10+)

2012968 Add testing in CI
32e8e4d Refactor for efficiency
bdc93ad Try and fix venv issue
fc2a6dc Fix hidden files
16b65c5 Fix upload dir
711f159 Refactor to reduce repetition
4e08e6a Name changing
f2c8b9d Fix permissions
2c973b7 Fix test and cleanup artifacts
8207920 Change name

📊 Changes

6 files changed (+372 additions, -89 deletions)

View changed files

➕ .github/workflows/bash-test.yml (+43 -0)
➕ .github/workflows/tests.yml (+120 -0)
📝 packages/llm_analysis/tests/test_llm_callbacks.py (+17 -0)
📝 packages/llm_analysis/tests/test_llm_callbacks_instructor.py (+9 -0)
📝 test/integration_tests.sh (+179 -85)
📝 test/real_tests.sh (+4 -4)

📄 Description

I was thinking of proposing some refactoring so I thought I would make sure tests were operating correctly before I did plus build them into CI

This does a few things:

Creates a Github Action which will run the existing bash and python tests on new pull requests and pushes to main
Changes some tests which were failing in python due to missing API keys to skip instead.
Makes some small fixes to test/real_tests.sh where there were a couple of bugs.
Fixes up integration tests which were previously failing and adds a debug mode.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/gadievron/raptor/pull/59 **Author:** [@joshbouncesecurity](https://github.com/joshbouncesecurity) **Created:** 1/22/2026 **Status:** ✅ Merged **Merged:** 1/23/2026 **Merged by:** [@danielcuthbert](https://github.com/danielcuthbert) **Base:** `main` ← **Head:** `addtests` --- ### 📝 Commits (10+) - [`2012968`](https://github.com/gadievron/raptor/commit/2012968326a85b207a58bf291a13671d0886ad1f) Add testing in CI - [`32e8e4d`](https://github.com/gadievron/raptor/commit/32e8e4dec61f7a94c94d613112281440e1d4788d) Refactor for efficiency - [`bdc93ad`](https://github.com/gadievron/raptor/commit/bdc93adee4295fcabbf7a5bc811abaf4d54e958e) Try and fix venv issue - [`fc2a6dc`](https://github.com/gadievron/raptor/commit/fc2a6dcc74a9357ea0e65c384e08d9250944cc62) Fix hidden files - [`16b65c5`](https://github.com/gadievron/raptor/commit/16b65c53b760ff384e3052166ff9fa945b9e71bc) Fix upload dir - [`711f159`](https://github.com/gadievron/raptor/commit/711f159f341d6cf64d3f099beba89bceecd1cc73) Refactor to reduce repetition - [`4e08e6a`](https://github.com/gadievron/raptor/commit/4e08e6a62c5dfd205126589c66dabab44fa6ad1e) Name changing - [`f2c8b9d`](https://github.com/gadievron/raptor/commit/f2c8b9dde3f961a9d7a7508187a78691b0d91b38) Fix permissions - [`2c973b7`](https://github.com/gadievron/raptor/commit/2c973b7fc6c6a8db46bcfe86df3b87c0fd5949ad) Fix test and cleanup artifacts - [`8207920`](https://github.com/gadievron/raptor/commit/820792087e9f2be059763948d5dbb4be4cf02183) Change name ### 📊 Changes **6 files changed** (+372 additions, -89 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/bash-test.yml` (+43 -0) ➕ `.github/workflows/tests.yml` (+120 -0) 📝 `packages/llm_analysis/tests/test_llm_callbacks.py` (+17 -0) 📝 `packages/llm_analysis/tests/test_llm_callbacks_instructor.py` (+9 -0) 📝 `test/integration_tests.sh` (+179 -85) 📝 `test/real_tests.sh` (+4 -4) </details> ### 📄 Description I was thinking of proposing some refactoring so I thought I would make sure tests were operating correctly before I did plus build them into CI This does a few things: 1) Creates a Github Action which will run the existing bash and python tests on new pull requests and pushes to main 2) Changes some tests which were failing in python due to missing API keys to skip instead. 3) Makes some small fixes to `test/real_tests.sh` where there were a couple of bugs. 3) Fixes up integration tests which were previously failing and adds a debug mode. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>