container should be idempotent

miRoox · Nov 20, 2023 · 818d7bb · 818d7bb
1 parent 3c195f2
commit 818d7bb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/container.jl b/src/container.jl
@@ -62,6 +62,9 @@ end
 setcontainerproperty!(obj::AnyContainer, name::Symbol, value::Any) = getfield(obj, 1)[name] = value
 setcontainerproperty!(obj::ShadowContainer, name::Symbol, value) = setfield!(obj, name, value)
 
+Base.:(==)(x::AnyContainer{T}, y::AnyContainer{T}) where {T} = all(p -> getproperty(x, p) == getproperty(y, p), fieldnames(T))
+Base.:(!=)(x::AnyContainer{T}, y::AnyContainer{T}) where {T} = any(p -> getproperty(x, p) != getproperty(y, p), fieldnames(T))
+
 Base.summary(io::IO, ::Container{T}) where {T} = show(io, Container{T})
 
 function Base.show(io::IO, obj::Container)
@@ -115,3 +118,4 @@ function Container(obj::T) where {T}
     end
     res
 end
+Container(c::Container) = c
diff --git a/test/runtests.jl b/test/runtests.jl
@@ -85,6 +85,8 @@ end
         @test repr(UndefProperty()) == "#undef"
         @test_throws ArgumentError Container(1)
         @test Container(im).im
+        @test Container(Container(1//2)) == Container(1//2)
+        @test Container(Container(2//1)) != Container(1//2)
         @test Container{Complex{Bool}}().im == UndefProperty{Bool}()
         @test Container{Complex{Rational{Int}}}().im.num == UndefProperty{Int}()
         @test_throws ErrorException Container{Complex{Bool}}().i