Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Configure parallelism for Kotest #44

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Configure parallelism for Kotest #44

wants to merge 6 commits into from

Conversation

JordanLongstaff
Copy link
Owner

Latest attempt to speed up the build action.

@JordanLongstaff
Configure parallelism for Kotest
00fee5d
@JordanLongstaff
Merge branch 'main' into kotest-parallelism
2c53be1 
# Conflicts:
#	IAN/enums/src/test/resources/kotest.properties
#	IAN/packets/src/test/resources/kotest.properties
#	IAN/src/test/resources/kotest.properties
#	IAN/udp/src/test/resources/kotest.properties
#	IAN/util/src/test/resources/kotest.properties
#	IAN/vesseldata/src/test/resources/kotest.properties
#	IAN/world/src/test/resources/kotest.properties
@JordanLongstaff
Add parallelism to app tests also
d8c47bf
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 9, 2025 •
edited
Loading

Instrumented Test Results

 2 files   -  13   2 suites   - 13   4m 41s ⏱️ - 53m 17s
21 tests ±  0  18 ✅  -   3  2 💤 + 2  1 ❌ +1 
42 runs   - 273  37 ✅  - 254  4 💤  - 20  1 ❌ +1 

For more details on these failures, see this check.

Results for commit f4e7fd3. ± Comparison against base commit 86536ab.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 9, 2025 •
edited
Loading

Unit Test Results

   94 files  ±  0     94 suites  ±0   16m 24s ⏱️ - 1m 3s
2 267 tests +618  2 256 ✅ +607  0 💤 ±0  11 ❌ +11 
3 090 runs  ±  0  3 079 ✅  -  11  0 💤 ±0  11 ❌ +11 

For more details on these failures, see this check.

Results for commit f4e7fd3. ± Comparison against base commit 86536ab.

This pull request removes 205 and adds 823 tests. Note that renamed tests count towards both. 
artemis.agent.cpu.HailResponseEffectTest ‑ AMBASSADOR
artemis.agent.cpu.HailResponseEffectTest ‑ COMMANDEERED
artemis.agent.cpu.HailResponseEffectTest ‑ CONTRABAND
artemis.agent.cpu.HailResponseEffectTest ‑ FIGHTER_TRAP
artemis.agent.cpu.HailResponseEffectTest ‑ FLYING_BLIND
artemis.agent.cpu.HailResponseEffectTest ‑ HOSTAGE
artemis.agent.cpu.HailResponseEffectTest ‑ Hail, Bold Privateer...
artemis.agent.cpu.HailResponseEffectTest ‑ Hail, Bold Privateer.....need some.: false
artemis.agent.cpu.HailResponseEffectTest ‑ Hail, Bold Privateer.....need some.: true
artemis.agent.cpu.HailResponseEffectTest ‑ Hail, Bold Privateer...: false
…

artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status AMBASSADOR We're dead...our ambassador...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status COMMANDEERED We have commandeered...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status CONTRABAND We are...our way.
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status FIGHTER_TRAP We're broken down...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status FLYING_BLIND Our sensors are...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status HOSTAGE We are holding...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status MALFUNCTION Our shipboard computer...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status MINE_TRAP We're just...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status NEED_DAMCON Our engines...
artemis.agent.cpu.HailResponseEffectTest ‑ HailResponseEffect Ally status NEED_ENERGY We're out of...
…

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 9, 2025 •
edited
Loading

Konsist Test Results

 12 files  ±  0   12 suites  ±0   9s ⏱️ -1s
476 tests +150  476 ✅ +150  0 💤 ±0  0 ❌ ±0 
655 runs  ±  0  655 ✅ ±  0  0 💤 ±0  0 ❌ ±0 

Results for commit f4e7fd3. ± Comparison against base commit 86536ab.

This pull request removes 25 and adds 175 tests. Note that renamed tests count towards both. 
artemis.agent.FragmentKonsistTest ‑ AlliesFragment
artemis.agent.FragmentKonsistTest ‑ AllySettingsFragment
artemis.agent.FragmentKonsistTest ‑ BiomechSettingsFragment
artemis.agent.FragmentKonsistTest ‑ BiomechsFragment
artemis.agent.FragmentKonsistTest ‑ ClientSettingsFragment
artemis.agent.FragmentKonsistTest ‑ ConnectFragment
artemis.agent.FragmentKonsistTest ‑ ConnectionSettingsFragment
artemis.agent.FragmentKonsistTest ‑ DestroyedStationsFragment
artemis.agent.FragmentKonsistTest ‑ EnemiesFragment
artemis.agent.FragmentKonsistTest ‑ EnemySettingsFragment
…

artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment AlliesFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment AllySettingsFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment BiomechSettingsFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment BiomechsFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment ClientSettingsFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment ConnectFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment ConnectionSettingsFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment DestroyedStationsFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment EnemiesFragment
artemis.agent.FragmentKonsistTest ‑ Classes with names ending in Fragment are subclasses of Fragment EnemySettingsFragment
…

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JordanLongstaff