@@ -112,21 +112,17 @@ public void HashCode()
112
112
[ Test ]
113
113
public void NotEqualHashCode ( )
114
114
{
115
- // This test is debatable, since GetHashCode semantic does not guarantee no collision may ever occur.
115
+ // GetHashCode semantic does not guarantee no collision may ever occur, but the algorithm should
116
+ // generates different hashcodes for similar but inequal cases. These tests check that cache keys
117
+ // for a query generated for different parameters values are no more equal.
116
118
FilterDescLikeToCompare ( out var fk , out var fk1 , false ) ;
117
- // Unfortunately due to https://stackoverflow.com/a/5927853/1178314 and our implementation resulting in a call
118
- // to GetHashCode on a KeyValuePair<string, TypedValue>, following assert always fails.
119
- //Assert.That(fk.GetHashCode(), Is.Not.EqualTo(fk1.GetHashCode()), "fk & fk1");
119
+ Assert . That ( fk . GetHashCode ( ) , Is . Not . EqualTo ( fk1 . GetHashCode ( ) ) , "fk & fk1" ) ;
120
120
121
121
FilterDescValueToCompare ( out var fvk , out var fvk1 , false ) ;
122
- // Unfortunately due to https://stackoverflow.com/a/5927853/1178314 and our implementation resulting in a call
123
- // to GetHashCode on a KeyValuePair<string, TypedValue>, following assert always fails.
124
- //Assert.That(fvk.GetHashCode(), Is.Not.EqualTo(fvk1.GetHashCode()), "fvk & fvk1");
122
+ Assert . That ( fvk . GetHashCode ( ) , Is . Not . EqualTo ( fvk1 . GetHashCode ( ) ) , "fvk & fvk1" ) ;
125
123
126
124
FilterValueInToCompare ( out var fik , out var fik1 , false ) ;
127
- // Unfortunately due to https://stackoverflow.com/a/5927853/1178314 and our implementation resulting in a call
128
- // to GetHashCode on a KeyValuePair<string, TypedValue>, following assert always fails.
129
- //Assert.That(fik.GetHashCode(), Is.Not.EqualTo(fik1.GetHashCode()), "fik & fik1");
125
+ Assert . That ( fik . GetHashCode ( ) , Is . Not . EqualTo ( fik1 . GetHashCode ( ) ) , "fik & fik1" ) ;
130
126
131
127
Assert . That ( fk . GetHashCode ( ) , Is . Not . EqualTo ( fvk . GetHashCode ( ) ) , "fk & fvk" ) ;
132
128
Assert . That ( fk1 . GetHashCode ( ) , Is . Not . EqualTo ( fvk1 . GetHashCode ( ) ) , "fk1 & fvk1" ) ;
0 commit comments