Skip to content

Conversation

@jamesross03
Copy link
Contributor

@jamesross03 jamesross03 commented Sep 24, 2025

Overview

Primarily a patch to the 1.0.2-SNAPSHOT but also updates common-pom to 4.1.0-SNAPSHOT and adds additional unit-tests for RecordRepository.

Closes #8, closes #11, closes #12

Changelist

Fixes:

  • Reinstates long ids instead of String ids, removing remnants of rolled-back functionality in neo-storr.

Testing:

  • Adds additional unit-testing for RecordRepository to test Neo4J backing using a test harness
  • Adds additional unit-testing for RecordTypes

Dependencies:

## Overview
Adds unit-testing for RecordRepository, reinstates long ids, and updates to common-pom 4.1.0.

## Commits included
- Updates Hashcode functionality for record_types
- Adds unit-testing for record types
- Implementation of some tests for RecordRepository
     - Basic setup for RecordRepository tests, overriding Neo4J using Neo4J
test harness.
- Adds more tests and asserts successful teardown
- Updates `common-pom` to 4.1.0-SNAPSHOT
Updates [`common-pom`](https://github.com/stacs-srg/common-pom/) to
[`4.1.0-SNAPSHOT`](stacs-srg/common-pom#5) to
take advantage of updated JDK and JUnit dependencies. See stacs-srg#13

* Reinstates long ids instead of String ids

Removes remnants of rolled-back updates to neo-storr.

* Implemented more RecordRepository tests

Implemented more thorough testing of the Neo4J backing of
RecordRepository using a test-harness.

---------

Co-authored-by: jar35 <[email protected]>
@grahamkirby grahamkirby merged commit 175b14b into stacs-srg:master Sep 24, 2025
1 check failed
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.

Update common-pom to 4.1.0-SNAPSHOT Add proper unit-testing Add more thorough unit-testing

2 participants