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.

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

[Feature]: Add JUnit Support V2 #183

Merged
merged 12 commits into from
May 26, 2024
Merged

[Feature]: Add JUnit Support V2 #183

merged 12 commits into from
May 26, 2024

Conversation

marchermans
Copy link
Contributor

Changes:

  • Create the possibility to handle unit test based runs.
  • Modify the task generation behaviour for runs
  • Create test tasks for unit test runs instead of exec tasks
  • Create a convention to reuse the test task by default, if the junit test is configured following the normal gradle conventions
  • Add support for multiple different unit test set ups
  • Add support for running with test source sets.
  • Only read test sourcesets when the run is setup for unit test modes.
  • Have platform create a unit test capable run by default

Contributions from:

  • Sharte
  • Maty

Required PRs in other changes:

  • FML needs to have its counterpart merged
  • Neo will need a bump to NG as well as its features merged.

Tests:

  • None: The above mentioned infrastructural components are missing, and preventing automated tests from being created.

Supersedes:

Closes: #107

@neoforged-pr-publishing
Copy link

neoforged-pr-publishing bot commented May 19, 2024

  • Publish PR to GitHub Packages

Last commit published: 6fe6e4c77a8d275650aaf7bcca27cd0efac3f92f.

PR Publishing

The artifacts published by this PR:

Repository Declaration

In order to use the artifacts published by the PR, add the following repository to your buildscript:

repositories {
    maven {
        name 'Maven for PR #183' // https://github.com/neoforged/NeoGradle/pull/183
        url 'https://prmaven.neoforged.net/NeoGradle/pr183'
        content {
            includeModule('net.neoforged.gradle', 'dsl-common')
            includeModule('net.neoforged.gradle', 'neoform')
            includeModule('net.neoforged.gradle.common', 'net.neoforged.gradle.common.gradle.plugin')
            includeModule('net.neoforged.gradle.userdev', 'net.neoforged.gradle.userdev.gradle.plugin')
            includeModule('net.neoforged.gradle', 'dsl-neoform')
            includeModule('net.neoforged.gradle', 'dsl-platform')
            includeModule('net.neoforged.gradle', 'vanilla')
            includeModule('net.neoforged.gradle', 'dsl-mixin')
            includeModule('net.neoforged.gradle', 'dsl-vanilla')
            includeModule('net.neoforged.gradle.mixin', 'net.neoforged.gradle.mixin.gradle.plugin')
            includeModule('net.neoforged.gradle', 'dsl-userdev')
            includeModule('net.neoforged.gradle.platform', 'net.neoforged.gradle.platform.gradle.plugin')
            includeModule('net.neoforged.gradle', 'common')
            includeModule('net.neoforged.gradle', 'userdev')
            includeModule('net.neoforged.gradle.neoform', 'net.neoforged.gradle.neoform.gradle.plugin')
            includeModule('net.neoforged.gradle.vanilla', 'net.neoforged.gradle.vanilla.gradle.plugin')
            includeModule('net.neoforged.gradle', 'mixin')
            includeModule('net.neoforged.gradle', 'platform')
            includeModule('net.neoforged.gradle', 'utils')
        }
    }
}

# Conflicts:
#	common/src/main/java/net/neoforged/gradle/common/util/run/RunsUtil.java
@marchermans marchermans mentioned this pull request May 19, 2024
@marchermans marchermans enabled auto-merge (squash) May 26, 2024 13:32
@marchermans marchermans merged commit a678f0d into NG_7.0 May 26, 2024
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants