Skip to content

Commit

Permalink
Merge pull request #1 from OpenSmock/fix_should_serialize
Browse files Browse the repository at this point in the history
fix should serialized
  • Loading branch information
labordep authored Mar 8, 2024
2 parents 34a5882 + 5523bb6 commit c6185c8
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/Toplo-Serialization/ToElement.extension.st
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ Extension { #name : #ToElement }
ToElement >> shouldSerializedChildren [
"Toplo elements should not serialize children by default. Manage exeptions in subclasses."

^ false
^ (self class == ToElement)
]

{ #category : #'*Toplo-Serialization' }
Expand Down
6 changes: 3 additions & 3 deletions src/Toplo-Serialization/ToSerializerTest.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -107,14 +107,14 @@ ToSerializerTest >> testIsLeaf4 [
yourself.
b1 addChild: b2.

self deny: b1 shouldSerializedChildren.
self assert: b1 shouldSerializedChildren.
self assert: b1 id equals: #b1.
self assert: b1 childrenCount equals: 1.

element := b1 serializeThenMaterialize.
self deny: element shouldSerializedChildren.
self assert: element shouldSerializedChildren.
self assert: element id equals: #b1.
self assert: element childrenCount equals: 0.
self assert: element childrenCount equals: 1.
]

{ #category : #tests }
Expand Down

0 comments on commit c6185c8

Please sign in to comment.