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/establish ci pipeline #23

Closed
wants to merge 17 commits into from
Closed

Conversation

ibardos
Copy link
Owner

@ibardos ibardos commented Dec 9, 2023

No description provided.

- Runs only when '[CI]' in commit message
- Set up JDK 21
- Build and test with Maven
  - Exclude testing of application context in CI
- Runs only when '[CI]' in commit message
- Mark out "working-directory" of front-end app
- Set up Node versions: 16, 18, 21
- Cache package-lock.json file
- Clean and install node dependencies
- Install React scripts for React app
- Build
- Test with ESLint
In the CI pipeline, front-end code is tested by a linter, therefore related warnings should have solved, as they can crash a build process.
…ment

- Follow end-to-end testing principles
- Use H2 in-memory "mock" relational-database
- Perform automatic server start during test runs
- Perform database initialization before each test
…tests

Tests now work with database initialization before each test, therefore getAll API methods should not expect the added record to be present, by previously run add() API method.
Implement a more sophisticated test for Application context load.
As tests now use a "mock" database, there is no further need for database initialization code.
- Add H2 dependency to use during end-to-end testing with refactored test methods.
- Resolve a known vulnerability of Spring Web.
- Follow up the changes made for end-to-end testing.
Aim is to express previously implemented end-to-end test upgrade.
…covered [CI]

- Refactor useEffect() dependencies.
- Refactor setCurrentRecord() state manipulation inside useEffect()s, as "undefined" could have assigned after deletion, which caused the UI to crash.
@ibardos
Copy link
Owner Author

ibardos commented Dec 9, 2023

Should run CI pipeline automatic build and tests.

@ibardos ibardos closed this Dec 9, 2023
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.

None yet

1 participant