Skip to content

Refactoring harness and template test files to remove 1 TODO#6440

Open
Amemeda wants to merge 4 commits into
apache:mainfrom
Amemeda:todo-file-refactoring
Open

Refactoring harness and template test files to remove 1 TODO#6440
Amemeda wants to merge 4 commits into
apache:mainfrom
Amemeda:todo-file-refactoring

Conversation

@Amemeda

@Amemeda Amemeda commented Jun 22, 2026

Copy link
Copy Markdown
Contributor
  • Removing "# TODO: these test classes should be moved into the correct package for the module" from line 42 of check-module-packaging-conventions.sh
  • created new folder src/main/java/org/apache/accumulo/test/harness
  • deleted old folder src/main/java/org/apache/accumulo/harness/conf
  • refactored all 12 integration test files into new test/harness folder
  • Fixed many many import errors resulting from refactoring
  • moved 3 template test files out of start/src/test/java/test and into new start/src/test/java/test/java/org/apache/accumulo/start/test directory
  • deleted test folder the template files were in

@Amemeda

Amemeda commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

This pr resolves 1 TODO from #2699

@Amemeda Amemeda marked this pull request as ready for review June 23, 2026 16:13
@Amemeda Amemeda changed the title Refactoring harness integeration test files Refactoring harness integration test files and template test files Jun 23, 2026
@Amemeda Amemeda changed the title Refactoring harness integration test files and template test files Refactoring harness and template test files to remove 1 TODO Jun 24, 2026
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.

1 participant