[ 
https://issues.apache.org/jira/browse/TINKERPOP-3020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17793694#comment-17793694
 ] 

Stephen Mallette commented on TINKERPOP-3020:
---------------------------------------------

for the {{PartitionStrategy}} and the {{mergeE()}} tests, "weight" is 
traditionally float/double in the static test graphs but these tests use an 
empty graph and add the weight as {{property("weight",2)}} which is int. i 
think it's fine if "weight" in this context is made to be consistent for sake 
of graphs with a schema to adhere to.

{{g_VX1X_formatXstrX_byXconstantXhelloXX_byXvaluesXnameXX}} definitely needs to 
be fixed.

> Incorrect tests
> ---------------
>
>                 Key: TINKERPOP-3020
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-3020
>             Project: TinkerPop
>          Issue Type: Test
>          Components: test-suite
>    Affects Versions: 3.7.1
>            Reporter: pieter martin
>            Priority: Major
>
> The following tests are incorrectly specified.
> {code:java}
> OptOut...
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_aXX_V_bothE_weight",
>  "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_bXX_V_bothE_weight",
>  "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_a_bXX_V_bothE_dedup_weight",
>  "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_cXX_V_bothE_weight",
>  "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_aXX_V_both_name", 
> "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_bXX_V_both_name", 
> "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_a_bXX_V_both_dedup_name",
>  "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_cXX_V_both_name", 
> "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_aXX_V_out_name", 
> "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_bXX_V_in_name", 
> "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_a_bXX_V_out_name", 
> "int should be double"),
> Map.entry("g_withStrategiesXPartitionStrategyXwrite_a_read_cXX_V_out_name", 
> "int should be double"),
> Map.entry("g_VX1X_formatXstrX_byXconstantXhelloXX_byXvaluesXnameXX", "id can 
> not be an int"),
> Map.entry("g_withSideEffectXc_created_YX_withSideEffectXm_matchedX_mergeEXlabel_knows_out_marko_in_vadasX_optionXonCreate_selectXcXX_optionXonMatch_sideEffectXpropertiesXweightX_dropX_selectXmXX_exists",
>  "int should be double"),
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to