Skip to content

Commit

Permalink
Remove diktat-test-framework
Browse files Browse the repository at this point in the history
### What's done:
- removed `diktat-test-framework`
- added `diktat-test-common` with common util methods for tests
- removed usage of diff multiplatform tool

It closes #1768
  • Loading branch information
nulls committed Nov 8, 2023
1 parent dc0f747 commit d36f8dd
Show file tree
Hide file tree
Showing 31 changed files with 8 additions and 471 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/dependencies.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,12 +39,12 @@ jobs:
gradle-build-module: |-
:diktat-api
:diktat-common
:diktat-common-test
:diktat-ktlint-engine
:diktat-gradle-plugin
:diktat-maven-plugin
:diktat-rules
:diktat-ruleset
:diktat-test-framework
:diktat-dev-ksp
:diktat-cli
gradle-build-configuration: |-
Expand Down
6 changes: 3 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ Rules are very simple:

Main components are:
1) diktat-rules — number of rules that are supported by diKTat;
2) diktat-test-framework — functional/unit test framework that can be used for running your code fixer on the initial code and compare it with the expected result;
2) diktat-common-testutil methods for functional/unit tests that can be used for running your code fixer on the initial code and compare it with the expected result;
3) also see our demo: diktat-demo in a separate repository.

Mainly we wanted to create a common configurable mechanism that
Expand All @@ -31,7 +31,7 @@ Before you make a pull request, make sure the build is clean as we have lot of t
$ mvn clean install
```

# Hooks
# Hooks

We have some hooks to a commit messages:
1) your commit message should have the following format:
Expand All @@ -40,7 +40,7 @@ Brief Description
### What's done:
1) Long description
2) Long description
2) Long description
```

2) Please also do not forget to update documentation on Wiki after the merge approval and before merge.
2 changes: 1 addition & 1 deletion diktat-cli/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ dependencies {
implementation(libs.log4j2.core)
implementation(libs.log4j2.slf4j2)

testImplementation(projects.diktatTestFramework)
testImplementation(projects.diktatCommonTest)
testImplementation(libs.junit.jupiter)
testImplementation(libs.junit.platform.suite)
testImplementation(libs.assertj.core)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ plugins {
id("com.saveourtool.diktat.buildutils.publishing-default-configuration")
}

project.description = "Test framework for diktat"
project.description = "Diktat coommon for tests"

dependencies {
api(projects.diktatCommon)
Expand Down
2 changes: 1 addition & 1 deletion diktat-rules/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ dependencies {
// guava is used for string case utils
implementation(libs.guava)
implementation(libs.kotlin.logging)
testImplementation(projects.diktatTestFramework)
testImplementation(projects.diktatCommonTest)
testImplementation(projects.diktatKtlintEngine)
testImplementation(libs.log4j2.slf4j2)
testImplementation(libs.junit.jupiter)
Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Loading

0 comments on commit d36f8dd

Please sign in to comment.