Redo testing #28

Merged
k.marinus merged 8 commits from refactor/remove-spade-redo-tests into refactor/remove-spade-update-tests 2025-11-21 17:16:43 +00:00
k.marinus commented 2025-11-21 16:04:44 +00:00 (Migrated from git.science.uu.nl)

Redid testing structure, added tests and changed some tests.

ref: N25B-301

Redid testing structure, added tests and changed some tests. ref: N25B-301
k.marinus commented 2025-11-21 16:06:07 +00:00 (Migrated from git.science.uu.nl)

added 1 commit

  • 24863cb6 - chore: update CI/CD testing command

Compare with previous version

added 1 commit <ul><li>24863cb6 - chore: update CI/CD testing command</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134030&start_sha=5fb923e20d7b4cb210dd7612896dd1aed5ab5895)
k.marinus commented 2025-11-21 16:11:10 +00:00 (Migrated from git.science.uu.nl)

added 1 commit

Compare with previous version

added 1 commit <ul><li>9fdcacc3 - chore: update gitlab ci file</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134032&start_sha=24863cb6afca4daebde419d296ea3e04e3a32881)
k.marinus commented 2025-11-21 16:18:55 +00:00 (Migrated from git.science.uu.nl)

added 1 commit

Compare with previous version

added 1 commit <ul><li>359633ef - chore: update dependencies</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134034&start_sha=9fdcacc342a22fcc6e7e891616fccd94c3c219ed)
k.marinus commented 2025-11-21 16:19:41 +00:00 (Migrated from git.science.uu.nl)

added 1 commit

Compare with previous version

added 1 commit <ul><li>027b88ad - chore: update pipeline</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134036&start_sha=359633effc2530069fae3c3674aeaec4e5035df7)
k.marinus commented 2025-11-21 16:29:35 +00:00 (Migrated from git.science.uu.nl)

added 1 commit

Compare with previous version

added 1 commit <ul><li>5a6ff12d - chore: update dependencies</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134038&start_sha=027b88adf396c7bff22e493631bd9a40d633ced7)
k.marinus commented 2025-11-21 16:38:33 +00:00 (Migrated from git.science.uu.nl)

changed title from refactor: testing to Redo testing

<p>changed title from <code class="idiff"><span class="idiff left right deletion">refactor:</span> testing</code> to <code class="idiff"><span class="idiff left right addition">Redo</span> testing</code></p>
0950726 commented 2025-11-21 16:46:55 +00:00 (Migrated from git.science.uu.nl)

I don't see this one recovered in the unit tests, correct?

I don't see this one recovered in the unit tests, correct?
k.marinus commented 2025-11-21 16:53:14 +00:00 (Migrated from git.science.uu.nl)

Oops yes you're right, fixed this now.

Oops yes you're right, fixed this now.
k.marinus commented 2025-11-21 16:55:21 +00:00 (Migrated from git.science.uu.nl)

added 1 commit

Compare with previous version

added 1 commit <ul><li>b488effd - chore: add back missing test</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134041&start_sha=5a6ff12d8df441db431810fd68cef1e3a358f5c4)
k.marinus commented 2025-11-21 16:55:44 +00:00 (Migrated from git.science.uu.nl)

resolved all threads

resolved all threads
0950726 commented 2025-11-21 17:04:33 +00:00 (Migrated from git.science.uu.nl)

I think we can now do integration tests in the pipeline as well

    - uv run --only-group test pytest test
I think we can now do integration tests in the pipeline as well ```suggestion:-0+0 - uv run --only-group test pytest test ```
k.marinus commented 2025-11-21 17:05:02 +00:00 (Migrated from git.science.uu.nl)

resolved all threads

resolved all threads
k.marinus commented 2025-11-21 17:11:15 +00:00 (Migrated from git.science.uu.nl)

changed this line in version 8 of the diff

changed this line in [version 8 of the diff](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134047&start_sha=b488effddc3d9ad6f19a7e7526b2255cb877d471#587d266bb27a4dc3022bbed44dfa19849df3044c_25_25)
k.marinus commented 2025-11-21 17:11:18 +00:00 (Migrated from git.science.uu.nl)

added 1 commit

  • 1d6781c5 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added 1 commit <ul><li>1d6781c5 - Apply 1 suggestion(s) to 1 file(s)</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/28/diffs?diff_id=134047&start_sha=b488effddc3d9ad6f19a7e7526b2255cb877d471)
0950726 commented 2025-11-21 17:16:43 +00:00 (Migrated from git.science.uu.nl)

mentioned in commit f8f833df64

mentioned in commit f8f833df640c84a5b26d919e5050702bb7a6034b
0950726 (Migrated from git.science.uu.nl) merged commit f8f833df64 into refactor/remove-spade-update-tests 2025-11-21 17:16:43 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pepperplus/pepperplus-cb#28