-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RI007_Stationing-property #64
base: main
Are you sure you want to change the base?
Changes from 6 commits
93327fa
d2bdc79
cdd6b8e
fa1ffa3
8305628
e93e18c
81c7d59
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,39 @@ | ||||||
@implementer-agreement | ||||||
@RI | ||||||
Feature: RI007 - Stationing property | ||||||
The rule verifies, that station information can be exported in IFC using Pset_Stationing. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
Scenario: Agreement on alignment stationing | ||||||
|
||||||
Given A file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3" | ||||||
And An IfcReferent | ||||||
And Its attribute Name | ||||||
Then The value must exist | ||||||
|
||||||
Scenario: Agreement on alignment stationing | ||||||
|
||||||
Given A file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3" | ||||||
And An IfcReferent | ||||||
And Its attribute IsDefinedBy | ||||||
And Its attribute RelatingPropertyDefinition | ||||||
And Its attribute Name | ||||||
Comment on lines
+18
to
+19
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This would allow a |
||||||
Then The value must exist | ||||||
|
||||||
Scenario: Agreement on alignment stationing | ||||||
|
||||||
Given A file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3" | ||||||
And An IfcReferent | ||||||
And Its attribute IsDefinedBy | ||||||
And Its attribute RelatingPropertyDefinition | ||||||
And Its attribute HasProperties | ||||||
Then The value of attribute Name must be Station | ||||||
|
||||||
Scenario: Agreement on alignment stationing | ||||||
|
||||||
Given A file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3" | ||||||
And An IfcReferent | ||||||
And Its attribute IsDefinedBy | ||||||
And Its attribute RelatingPropertyDefinition | ||||||
And Its attribute HasProperties | ||||||
And Its attribute NominalValue | ||||||
Then The value must exist | ||||||
Comment on lines
+22
to
+39
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I do not agree with splitting this up in independent scenarios. For example, any attached property with NominalValue non empty would pass the bottom test. The requirement was that the property named Station would have a non empty value. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -118,6 +118,13 @@ class InvalidValueError: | |
def __str__(self): | ||
return f"On instance {misc.fmt(self.related)} the following invalid value for {self.attribute} has been found: {self.value}" | ||
|
||
@dataclass | ||
class MissingValueError: | ||
related: ifcopenshell.entity_instance | ||
|
||
def __str__(self): | ||
return f"On instance {misc.fmt(self.related)} the value doesn't exist" | ||
Comment on lines
+121
to
+126
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The value of what? |
||
|
||
|
||
@dataclass | ||
class PolyobjectDuplicatePointsError: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
| File name | Expected result | Error log | Description | | ||
|-----------------------------------------------------------------------------------|-----------------|-----------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------| | ||
| pass-ri007-stationing-property | success | n.a. | | | ||
| fail-ri007-scenario01-no-name-attribute-of-ifcreferent-default | fail | On instance #532=IfcReferent('13nJTlK...ATION.) the value doesn't exist | A value was expected for Name attribute of #532=IfcReferent(), but $ was found | | ||
| fail-ri007-scenario01-no-name-attribute-of-ifcreferent-empty | fail | On instance #532=IfcReferent('13nJTlK...ION.,$) the value doesn't exist | A value was expected for Name attribute of #532=IfcReferent(), but '' was found | | ||
| fail-ri007-scenario02-no-name-attribute-of-ifcpropertyset-default | fail | On instance (#536=IfcPropertySet('0oZ...537)),) the value doesn't exist | A value was expected for Name attribute of #536=IfcPropertySet(), but $ was found | | ||
| fail-ri007-scenario02-no-name-attribute-of-ifcpropertyset-empty | fail | On instance (#536=IfcPropertySet('0oZ...537)),) the value doesn't exist | A value was expected for Name attribute of #536=IfcPropertySet(), but '' was found | | ||
| fail-ri007-scenario03-wrong-name-value | fail | On instance #537=IfcPropertySingleVal...3.1),$) the following invalid value for Name has been found: Body | The expected IfcPropertySingleValue value was Station. | | ||
| fail-ri007-scenario04-no-nominalvalue-attribute-of-ifcpropertysinglevalue-default | fail | On instance ((#537=IfcPropertySingleV...,$),),) the value doesn't exist | A value was expected for NominalValue attribute of #537=IFCPROPERTYSINGLEVALUE(), but $ was found | | ||
| fail-ri007-scenario04-no-nominalvalue-attribute-of-ifcpropertysinglevalue-empty | fail | On instance ((#537=IfcPropertySingleV...,$),),) the value doesn't exist | A value was expected for NominalValue attribute of #537=IFCPROPERTYSINGLEVALUE(), but '' was found | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All scenarios should not have the same name.