[TINKERPOP-990] Mixed types in VertexPropertyTest

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.0.2-incubating
• Fix Version/s: 3.1.1-incubating
• Component/s: test-suite
• Assignee: Stephen Mallette
• Reporter: Matthias Broecheler
• Created: 24/Nov/15 01:41
• Updated: 30/Nov/15 15:06
• Resolved: 30/Nov/15 15:06

Description

shouldRemoveMultiPropertiesWhenVerticesAreRemoved of VertexPropertyTest.VertexPropertyRemoval mixes types. The test uses String data types for "name" but then also adds integers. In systems with a strict typesystem, those integers would get converted to Strings which makes the subsequent removal test fail which checks for integer.class.

Such mixing of types should be avoided in TinkerPop tests.

This can furthermore be found in the following test cases:

• AddEdgeTest#g_VX1X_asXaX_outXcreatedX_addOutEXcreatedBy_a_weight_2X (mixed double with integer for "weight")
• AddEdgeTest#g_VX1X_asXaX_outXcreatedX_addEXcreatedByX_toXaX_propertyXweight_2X (mixed double with integer for "weight")