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

local integration tests #389

Closed
p4u1 wants to merge 6 commits from (deleted):integration-tests into main
pull from: (deleted):integration-tests
merge into: :main
Conversation 5 Commits 6 Files Changed 2
+8 -4
p4u1 commented 2023-11-13 14:41:08 +00:00
Owner
Copy Link
Copy Source

Setup local server:

docker swarm init
docker network create -d overlay proxy

Then run tests with the following env variables:

TEST_SERVER=default ABRA_DIR=tests/dir bats -Tp tests/integration
Setup local server: ``` docker swarm init docker network create -d overlay proxy ``` Then run tests with the following env variables: ``` TEST_SERVER=default ABRA_DIR=tests/dir bats -Tp tests/integration ```
p4u1 added 2 commits 2023-11-13 14:41:09 +00:00
fix app_cmd.bats 8065f29f68
test/integration local server
continuous-integration/drone/pr Build is failing
Details
f57b45888b
p4u1 added 1 commit 2023-11-14 11:03:37 +00:00
app_deploy.bats
continuous-integration/drone/pr Build is failing
Details
ab3518ad98
p4u1 added 1 commit 2023-11-14 17:56:20 +00:00
fix build error
continuous-integration/drone/pr Build is failing
Details
5aad497ed0
p4u1 force-pushed integration-tests from 27fb892208 to 5aad497ed0 2023-11-20 20:37:42 +00:00 Compare
p4u1 added 1 commit 2023-11-20 21:41:59 +00:00
os independent
continuous-integration/drone/pr Build is failing
Details
0142814d34
p4u1 force-pushed integration-tests from 7139948e81 to a275202e0b 2023-11-20 21:47:09 +00:00 Compare
p4u1 marked the pull request as ready for review 2023-11-20 21:47:24 +00:00
p4u1 commented 2023-11-20 21:49:16 +00:00
Author
Owner
Copy Link
Copy Source

@decentral1se this is ready for review. I removed all changes to actual test files to not increase the size of the pull request.

@decentral1se this is ready for review. I removed all changes to actual test files to not increase the size of the pull request.
3wordchant commented 2023-11-23 11:20:14 +00:00
Owner
Copy Link
Copy Source

I'll try and take a look today, unless you beat me to it @decentral1se ?

I'll try and take a look today, unless you beat me to it @decentral1se ?
decentral1se approved these changes 2023-11-27 09:32:45 +00:00
decentral1se left a comment
Owner
Copy Link
Copy Source

LGTM, YOLO mergin' thanks a bunch @p4u1 👏 🥇

LGTM, YOLO mergin' thanks a bunch @p4u1 👏 🥇
decentral1se commented 2023-11-27 09:43:57 +00:00
Owner
Copy Link
Copy Source

Oh strange, I don't seem to be able to merge it. Also I'm a bit confused about the 6 commits on the branch but then the changes showing only a limited selection of those changes? So, from the changes listed on https://git.coopcloud.tech/coop-cloud/abra/pulls/389/files I merged them in 316b59b465. I hope that's the intended change set. Lemme know if not and we can sort it out. Thanks!

Oh strange, I don't seem to be able to merge it. Also I'm a bit confused about the 6 commits on the branch but then the changes showing only a limited selection of those changes? So, from the changes listed on https://git.coopcloud.tech/coop-cloud/abra/pulls/389/files I merged them in 316b59b465b7cc6e0ab09edfc25760d81fb9a424. I hope that's the intended change set. Lemme know if not and we can sort it out. Thanks!
decentral1se closed this pull request 2023-11-27 09:43:57 +00:00
p4u1 commented 2023-11-27 14:47:24 +00:00
Author
Owner
Copy Link
Copy Source

@decentral1se Thanks for merging :) This makes it easier for me to work in different branches.

And sorry for the mess with the commits. I didn't clean them up, because I'm used to a git workflow, where the commits get squashed before merging. Will do that by myself next time.

@decentral1se Thanks for merging :) This makes it easier for me to work in different branches. And sorry for the mess with the commits. I didn't clean them up, because I'm used to a git workflow, where the commits get squashed before merging. Will do that by myself next time.
❤️ 1

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
decentral1se
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
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#389