Skip to content

manual refactoring, element reordering, static imports, this, final, …#3246

Merged
lbownik merged 2 commits intodevelopfrom
harvesting_client_refactoring
Apr 7, 2026
Merged

manual refactoring, element reordering, static imports, this, final, …#3246
lbownik merged 2 commits intodevelopfrom
harvesting_client_refactoring

Conversation

@lbownik
Copy link
Copy Markdown
Contributor

@lbownik lbownik commented Mar 25, 2026

…methods extraction, optimizations


import org.junit.jupiter.api.Test;

public class HervestingClientTest {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in class name HervestingClientTest -> HarvestingClientTest

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed


ClientHarvestRun run2 = new ClientHarvestRun();
run2.setResult(ClientHarvestRun.RunResultType.SUCCESS);
run1.setStartTime(new Date());
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be run2 insted of run1?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@rscipien rscipien assigned lbownik and unassigned rscipien Apr 7, 2026
@lbownik lbownik requested a review from rscipien April 7, 2026 06:01
@lbownik lbownik assigned rscipien and unassigned lbownik Apr 7, 2026
@rscipien rscipien assigned lbownik and unassigned rscipien Apr 7, 2026
@lbownik lbownik force-pushed the harvesting_client_refactoring branch from 0d73514 to 82b3f05 Compare April 7, 2026 06:21
@lbownik lbownik merged commit 1dc8fb5 into develop Apr 7, 2026
1 check passed
@lbownik lbownik deleted the harvesting_client_refactoring branch April 7, 2026 06:25
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