Skip to content

Utility endpoints refactored and moved#239

Open
Nefarious46 wants to merge 4 commits into
teragrep:mainfrom
Nefarious46:utilityEndpointRefactoring
Open

Utility endpoints refactored and moved#239
Nefarious46 wants to merge 4 commits into
teragrep:mainfrom
Nefarious46:utilityEndpointRefactoring

Conversation

@Nefarious46
Copy link
Copy Markdown
Contributor

Utility endpoints refactored and moved accordingly. Attempted to find a way to test out Api version endpoint but no clear solution was found. Version number changes every time tests were ran. Junit does testing alphabetically but version number changed during every test execution.

Version numbers received during testing:

25873
54517
83161
111789

@Nefarious46 Nefarious46 requested a review from kortemik March 25, 2026 14:39
@Nefarious46 Nefarious46 self-assigned this Mar 25, 2026
@Nefarious46 Nefarious46 removed the request for review from kortemik March 27, 2026 08:40
@Nefarious46
Copy link
Copy Markdown
Contributor Author

Nefarious46 commented Mar 30, 2026

Should still create unit tests to make sure value returned is a number and that number can be used to fetch values from the database.
This is relevant. #101

@Nefarious46
Copy link
Copy Markdown
Contributor Author

Now tests out if Api version is numeric. Required inserting something to the database in order to start the version flowing (no pun intended). Also implemented a test that retrieves All Flows with the version number included. This is implemented in a way that user would utilize the version endpoint.

@Nefarious46 Nefarious46 force-pushed the utilityEndpointRefactoring branch from 73be6c3 to 069c5d6 Compare May 15, 2026 06:34
@Nefarious46
Copy link
Copy Markdown
Contributor Author

Rebased. Refactoring to TestApiClient.


int version = Integer.parseInt(responseStringGet);

HttpGet requestFlow = new HttpGet("http://localhost:" + port + "/flow?version=" + version);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

delete the flow right after and then test with the version number that was received upon the insert

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.

Rectified here f4fa331

@Nefarious46 Nefarious46 force-pushed the utilityEndpointRefactoring branch from 34f3f7d to ac63cae Compare May 15, 2026 11:55
@Nefarious46 Nefarious46 requested a review from kortemik May 15, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants