toolshed/abra
4
4
Fork 23
Code Issues 82 Pull Requests 11 Packages Releases 45 Activity

WIP: automated integration tests #338

Closed
decentral1se wants to merge 1 commits from automated-integration-tests into main
pull from: automated-integration-tests
merge into: :main
Conversation 1 Commits 1 Files Changed 5
+160 -65
decentral1se commented 2023-08-02 08:50:54 +00:00
Owner
Copy Link
Copy Source

Attempting to be somewhat certain we actually fix the regressions of:

Idea is to run a new job under a debian image with bats, docker, git etc. installed. Then it runs the commands and asserts output. I'm hoping to have the experience of running bats locally and via the CI be the same, so it's easy to work with.

This is passing for me locally atm:

$ apt install bats bats-support bats-assert
$ bats tests/integration/integration.bats

CI is struggling. I built a custom image (docker build -t decentral1se/abra-test-int .).

Attempting to be somewhat certain we actually fix the regressions of: - https://git.coopcloud.tech/coop-cloud/organising/issues/478 - https://git.coopcloud.tech/coop-cloud/organising/issues/477 - https://git.coopcloud.tech/coop-cloud/organising/issues/476 Idea is to run a new job under a debian image with [bats](https://bats-core.readthedocs.io/en/stable/), docker, git etc. installed. Then it runs the commands and asserts output. I'm hoping to have the experience of running `bats` locally and via the CI be the same, so it's easy to work with. This is passing for me locally atm: ``` $ apt install bats bats-support bats-assert $ bats tests/integration/integration.bats ``` CI is struggling. I built a custom image (`docker build -t decentral1se/abra-test-int .`).
decentral1se force-pushed automated-integration-tests from 59db7f13bd to 68963a6b11 2023-08-02 08:57:13 +00:00 Compare
decentral1se force-pushed automated-integration-tests from 68963a6b11 to a340922f94 2023-08-02 09:18:07 +00:00 Compare
decentral1se force-pushed automated-integration-tests from a340922f94 to 0c91441d36 2023-08-02 09:36:01 +00:00 Compare
decentral1se changed title from wip: automated integrationt tests to WIP: automated integration tests 2023-08-02 09:37:27 +00:00
decentral1se force-pushed automated-integration-tests from 0c91441d36 to 9283593095 2023-08-02 09:38:24 +00:00 Compare
decentral1se force-pushed automated-integration-tests from 9283593095 to 2f10fb80a8 2023-08-02 10:00:01 +00:00 Compare
decentral1se commented 2023-08-26 08:06:45 +00:00
Author
Owner
Copy Link
Copy Source

Closing off for now and re-grouping for how best to approach this. It's a large piece of work.

Closing off for now and re-grouping for how best to approach this. It's a large piece of work.
decentral1se closed this pull request 2023-08-26 08:07:35 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
This pull request is marked as a work in progress.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
 build
go build related issues
 ci/cd
Building things with CI/CD
 critical fix
https://docs.coopcloud.tech/federation/resolutions/passed/010/
 design
UI/UX
 documentation
Documenting all the things
 duplicate
This issue or pull request already exists
 easy-first-issue
Something for new people to get stuck into. We hope it's easy!
 enhancement
New feature
 help wanted
Need some help
 i10n
Everything to do with localisation
 i18n
Everything to do with internationalisation
 installer
Everything to do with the install script.
 invalid
Something is wrong
 question
More information is needed
 release
Release management
 release-candidate
Related to the new release candidate
 security
Security related
 tech-debt
test
Unit/integration testing
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) Apfelwurm BornDeleuze Brooke carla cas (Cassowary) coopcloud cyrnel decentral1se (d1) dede devydave fauno (fauno) iexos jade (Jade Ambrose) jjsfunhouse jmakdah2 (Jackie Makdah) joe-irving (Joe Irving) kawaiipunk (KawaiiPunk) knoflook kolaente lambdabundesverband linnealovespie (April) moosemower moritz notplants oxaliq (sorrel) p4u1 pharaohgraphy (Andrew 🐦‍🔥❤️‍🔥✴️) renovate-bot (Comrade Renovate Bot) ripclap simon sixsmith (Sixsmith) stevensting trav (Trav Fryer) val (val (he/him)) yksflip
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#338