Develop#18
Merged
Merged
Conversation
Add rudimentary cache reload tests which verify that basic db entity caches initialize correctly from database: - rems.db.attachments - rems.db.blacklist - rems.db.catalogue - rems.db.form - rems.db.licenses - rems.db.organizations - rems.db.resource - rems.db.roles - rems.db.user-mappings - rems.db.user-settings (pre-existing) - rems.db.users - rems.db.workflow
Change CircleCI test splitting to use test names instead of filenames (documentation was buried deep on this one). This should improve parallel test distribution, especially for browser tests that are all in one file.
- Renamed kaocha plugins - Write cache statistics to files in target/cache-statistics-plugin instead of stdout, and store as CircleCI artifacts - Write test-ids and split-test-ids to files in target/circleci-parallel-plugin and store as CircleCI artifacts
This was unnecessary part from the beginning, and mainly just interesting while debugging early version of the cache. There is a subtle "bug" regarding config startup: mount is not started before the namespace is loaded. It's not an easy issue to solve, because caches are defined in many different namespaces and initialized using function call.
rems.main was removed at some point as unnecessary require, but it is now used by hooks test, and sometimes Kaocha test runner goes haywire about the missing require.
We should use `ok` helper like elsewhere.
We should use `ok` helper like elsewhere.
This can be removed once cprop fix is released. See tolitius/cprop#60
In these places since JDK 19+ reflection does not find the method without explicitly using long.
Fix markdown syntax in a few links
The recent refactoring wrt. the catalogue tree accessibility had a small regression in that the row commands are visible even when not logged in. The check remains in the catalogue tree and we can easily add that back to the catalogue table too.
Fix catalogue table command
Upstream update
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Checklist for author
Remove items that aren't applicable, check items that are done.
Reviewability
Backwards compatibility
Documentation
Testing
Follow-up