bf: IfcWall #4 must have classification to pass pass-occurrences_over…

[I-Sokolov]

#225

[andyward]

Hi Igor, I think the issue is these test cases are generated in the IfcOpenShell repo so we can't patch then here without getting over-written in future.

I think the root cause is that the IfcOpenShell method that generates these IDS and IFC files

1. introduces an IfcWall in the model for one set of tests as element0 - element # 4
2. then introduces a second instance of IfcWall as wall and the first wall conflicts. (ends up as Element # 18)

Ideally these test cases would be cleaned up a bit, but a quick fix would be to change element0 to be some other type e.g. IfcDoor, so it won't conflict.

Also raises the question that there may be an issue in the IfcOpenShell validator or test-suite if the tests are not picking this up? Pinging @Moult as I think he wrote them?