Simplify test groups

[wojtek-coreum]

Description

Reviewers checklist:

• Try to write more meaningful comments with clear actions to be taken.
• Nit-picking should be unblocking. Focus on core issues.

Authors checklist

• Provide a concise and meaningful description
• Review the code yourself first, before making the PR.
• Annotate your PR in places that require explanation.
• Think and try to split the PR to smaller PR if it is big.

---

This change is 

[dzmitryhil]

Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on @miladz68, @wojtek-coreum, and @ysv)

---

infra/testing/run.go line 32 at r1 (raw file):

	TestGroupCoreumIBC                 = "coreum-ibc"
	TestGroupFaucet                    = "faucet"
	TestGroupCoreumBridgeXRPLContract  = "coreumbridge-xrpl-contract"

IMO we don't need the test goups for the bridge at all. The bridge runs all tests himself, so no need for crust to build and run it.

[wojtek-coreum]

Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on @dzmitryhil, @miladz68, and @ysv)

---

infra/testing/run.go line 32 at r1 (raw file):

Previously, dzmitryhil (Dzmitry Hil) wrote…

IMO we don't need the test goups for the bridge at all. The bridge runs all tests himself, so no need for crust to build and run it.

This will be the new command to run bridge tests:

crust znet --test-groups=coreumbridge-xrpl-contract

[wojtek-coreum]

Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on @dzmitryhil, @miladz68, and @ysv)

---

infra/testing/run.go line 32 at r1 (raw file):

Previously, wojtek-coreum (Wojtek) wrote…

This will be the new command to run bridge tests:

crust znet --test-groups=coreumbridge-xrpl-contract

This task is about bringing our standard building system to bridge: https://app.clickup.com/t/8687dy4q2

[dzmitryhil]

Reviewable status: 0 of 4 files reviewed, 1 unresolved discussion (waiting on @miladz68, @wojtek-coreum, and @ysv)

---

infra/testing/run.go line 32 at r1 (raw file):

Previously, wojtek-coreum (Wojtek) wrote…

This task is about bringing our standard building system to bridge: https://app.clickup.com/t/8687dy4q2

IMO we should stop doint that here as well :-)
That's the repo specific thing

[miladz68]

Reviewed 1 of 4 files at r1, 3 of 3 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dzmitryhil and @ysv)

---

infra/testing/run.go line 32 at r1 (raw file):

Previously, dzmitryhil (Dzmitry Hil) wrote…

IMO we should stop doint that here as well :-)
That's the repo specific thing

I agree we should not run bridge tests with znet

[ysv]

Reviewed 3 of 4 files at r1, 3 of 3 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dzmitryhil)

---

infra/testing/run.go line 32 at r1 (raw file):

Previously, miladz68 (milad) wrote…

I agree we should not run bridge tests with znet

also discussed this on call and agreed to not run these tests with znet

[miladz68]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dzmitryhil and @wojtek-coreum)

[wojtek-coreum]

Reviewable status: 3 of 4 files reviewed, 1 unresolved discussion (waiting on @dzmitryhil, @miladz68, and @ysv)

---

infra/testing/run.go line 32 at r1 (raw file):

Previously, ysv (Yaroslav Savchuk) wrote…

also discussed this on call and agreed to not run these tests with znet

Done.

[dzmitryhil]

Reviewable status: 3 of 4 files reviewed, all discussions resolved (waiting on @miladz68)

[miladz68]

Reviewed 1 of 1 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @wojtek-coreum)

[ysv]

Reviewed 1 of 1 files at r3, 1 of 1 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @wojtek-coreum)