c4ab0010d1
.idea: upgrade findbugs and assertion verification
2016-05-24 23:30:46 +01:00
e06feddec3
Merge pull request #9 from kemitix/demo-of-full-path-node-name
...
NodeItemTest: demonstrate using dynamic name supplier to generate nod…
2016-05-24 23:03:11 +01:00
aa4be5ebf1
Merge pull request #8 from kemitix/insertinpath-uses-setdata
...
insertInPath() uses setData() to merge nodes
2016-05-24 23:02:35 +01:00
d82b0290a1
NodeItemTest: demonstrate using dynamic name supplier to generate node paths
2016-05-24 22:35:24 +01:00
94f7b5fb95
Merge pull request #7 from kemitix/dynamic-names
...
Dynamic names
2016-05-24 22:31:59 +01:00
eebd293633
NodeItem.insertInPath(): use setData() to when target is already in tree
...
Where an empty node is already found in the tree where we want to insert
a node, use the setData() method to place the data. This replaces the
previous process of attempting to swapping in the new node.
2016-05-24 22:16:35 +01:00
96ac0f720f
Node.setName(): ensure that setting name to null enables the name supplier
2016-05-24 22:04:00 +01:00
7de6c5ad49
Node.getName(): generated each time called when using a name supplier
...
Previously the name was set when the node was created and only a call to
setName() would have changed it.
isNamed() uses the generated name, so this may change if the generated name
is even null or the empty string.
2016-05-24 22:03:14 +01:00
7ccef4e011
Merge pull request #6 from kemitix/mutation-testing
...
Additional tests after PIT Mutation testing
2016-05-24 16:55:44 +01:00
080d995ba0
NodeItemTest: more tests
2016-05-24 16:36:10 +01:00
c0bbadf119
Merge pull request #5 from kemitix/tidy-interface
...
Tidy interface
2016-05-24 15:22:29 +01:00
a31c4501ac
Fix typos
2016-05-24 15:05:53 +01:00
f95029d348
Node.getChild(): method to get a child or throw an exception if not found
2016-05-24 14:44:03 +01:00
04599861c0
Node.createChild(data, name): can create named child nodes
...
# Conflicts:
# src/test/java/net/kemitix/node/NodeItemTest.java
2016-05-24 14:44:03 +01:00
fc55bacb04
Node.setData(): add ability to change node's data after instantiation
2016-05-24 14:44:03 +01:00
2ed0024bc0
Node: clean up method names
...
Deprecate:
* findOrCreateChild()
Rename:
* getChild() => findChild()
* isChildOf() => isDescendantOf()
* walkTree() => findInPath()
* placeNodeIn() => insertInPath()
* findChildNamed() => findChildByName()
* getChildNamed() => getChildByName()
2016-05-24 14:25:55 +01:00
e9c6300ad4
Node: remove pointless final in parameters in interface
2016-05-24 14:10:46 +01:00
a5c1fc72fd
Merge pull request #4 from kemitix/named-nodes
...
Named nodes
2016-05-24 14:06:07 +01:00
3053eab9b0
NodeItem.drawTree(): fix typo
2016-05-24 13:50:38 +01:00
6f81a62162
NodeItem.removeParent(): use same name supplier in the new root node
2016-05-24 13:33:53 +01:00
d82a7d6e9f
NoteItem: replace empty target node properly
2016-05-24 13:13:55 +01:00
aaab7bbe67
Reorganise code
2016-05-24 11:56:14 +01:00
180f325f79
NodeItemTest: overhaul of test suite
2016-05-24 11:56:14 +01:00
b456e18316
NodeItem: fix up
2016-05-24 11:56:14 +01:00
45bd77bbca
Node.placeNodeIn(): add node to tree under the path of named elements
...
Any intervening nodes that don't exist will be created.
e.g. placeNodeIn(node, "alpha", "beta")
Will add node:
[root]
"alpha"
"beta"
node
2016-05-24 11:56:14 +01:00
9fca56b4c6
Node: remove{Child,Parent}(): split trees branches apart
...
removeChild(node): removes the node from children of the current node,
making the child node into a new root node.
removeParent(): removes the current from from it's parent's children,
making itself into a new root node.
2016-05-24 11:56:14 +01:00
b2c3032ec0
Node:drawTree(): creates a String representing the node in the tree
...
The NodeItem implementation on includes nodes with names, or where they
have child nodes. In which case the are shown as '(unnamed)'.
2016-05-24 11:56:14 +01:00
b278fc0f98
Node: add {get,find}ChildNamed() methods
...
Both methods look for a child with the given name as an immediate child of
the current node.
findChildNamed:
Will return an Optional containing the found node or empty.
getChildNames:
Is more insistent and will return the found node itself. If a node by that
name is not found, then a NodeException will be thrown.
2016-05-24 11:56:14 +01:00
b18020708b
Node: may have names and a functional name supplier
...
Names, where present, must be unique for each parent.
Node:
* String getName()
* void setName(String name)
* boolean isNamed()
NodeItem - replaces all constructor:
* (data)
* (data, name)
* (data, nameSupplier)
* (data, parent)
* (data, nameSupplier, parent)
The name supplier takes a node and generates a string at the time the node
is constructed. The root node has a default name supplier that returns null
which means that the node is considered unnamed. Other nodes may provide
their own name supplier that would be used be new nodes created within
their subtree.
2016-05-24 11:56:14 +01:00
36efe5d83a
Node: may be empty, having no data
...
* isEmpty()
2016-05-24 11:56:14 +01:00
921cf98b13
Add assertj-core dependency for testing
2016-05-24 11:56:14 +01:00
c6cc3ec9d9
Merge pull request #2 from kemitix/update-code-style
...
Update code style
2016-03-14 15:56:47 +00:00
8187a28795
NodeItemTest: use val and Assert.assertThat()
...
* Lombok's val simplfies the local variable declarations
* Assert.assertThat() gives better error messages
* javadoc tidy
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:51:44 +00:00
24ad03ccab
NodeExceptionTest: use lombok's val
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:43:56 +00:00
58363e474d
NodeItem: apply code style
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:43:21 +00:00
9daebcea46
{Node,NodeItem}: javadoc tidy
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:42:53 +00:00
d9d19a849e
NodeItem.findOrCreateChild(): use Optional.orElseGet() to simplify
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:42:07 +00:00
d60f6ae0bb
NodeItem.isChildOf(): simplify if statements
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:41:23 +00:00
79170b898a
NodeItem: initialise children set in field and make final
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:40:51 +00:00
1aa2daf14b
Add lombok back for use in testing only
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 15:39:47 +00:00
d0c6b07b66
.idea - fixup
2016-03-14 15:39:16 +00:00
fee38dc929
.idea: add IntelliJ configuration
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 14:35:57 +00:00
7ce9d9d336
Merge pull request #1 from kemitix/drop-lombok
...
Drop lombok dependency
2016-03-14 14:31:07 +00:00
635af8089d
Drop lombok dependency
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-03-14 14:17:14 +00:00
85bd763e09
NodeItem.walkTree(): refactor using existing getChild()
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-01-09 18:09:38 +00:00
bff6b5be5e
Merge branch release/0.1.0 into develop
...
Initial Release
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-01-09 18:00:33 +00:00
9479ae6c14
pom.xml: version set to 0.2.0-SNAPSHOT
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-01-09 18:00:33 +00:00
06a2890325
CHANGELOG
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-01-09 18:00:33 +00:00
d74d44ba9d
pom.xml: version set to 0.1.0
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-01-09 18:00:33 +00:00
1f508bc3d2
README.md: fix typo
...
Signed-off-by: Paul Campbell <pcampbell@kemitix.net>
2016-01-09 17:55:24 +00:00
